rndis_host: allow rndis_wlan to see all indications

Allow rndis_wlan to see all indications. Currently rndis_host lets rndis_wlan to
know about link state changes only, but there is whole set of other
802.11-specific indications that rndis_wlan should handle properly. So rename
link_change() to indication() and convert rndis_wlan to use it.

Signed-off-by: Jussi Kivilinna <jussi.kivilinna@mbnet.fi>
Cc: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>

1 files changed, 29 insertions, 21 deletions

diff --git a/drivers/net/usb/rndis_host.c b/drivers/net/usb/rndis_host.c
index 2232232b7989..d032bba9bc4c 100644
--- a/drivers/net/usb/rndis_host.c
+++ b/drivers/net/usb/rndis_host.c
@@ -65,6 +65,32 @@ void rndis_status(struct usbnet *dev, struct urb *urb)
 EXPORT_SYMBOL_GPL(rndis_status);
 
 /*
+ * RNDIS indicate messages.
+ */
+static void rndis_msg_indicate(struct usbnet *dev, struct rndis_indicate *msg,
+                                int buflen)
+{
+        struct cdc_state *info = (void *)&dev->data;
+        struct device *udev = &info->control->dev;
+
+        if (dev->driver_info->indication) {
+                dev->driver_info->indication(dev, msg, buflen);
+        } else {
+                switch (msg->status) {
+                case RNDIS_STATUS_MEDIA_CONNECT:
+                        dev_info(udev, "rndis media connect\n");
+                        break;
+                case RNDIS_STATUS_MEDIA_DISCONNECT:
+                        dev_info(udev, "rndis media disconnect\n");
+                        break;
+                default:
+                        dev_info(udev, "rndis indication: 0x%08x\n",
+                                        le32_to_cpu(msg->status));
+                }
+        }
+}
+
+/*
  * RPC done RNDIS-style.  Caller guarantees:
  * - message is properly byteswapped
  * - there's no other request pending
@@ -143,27 +169,9 @@ int rndis_command(struct usbnet *dev, struct rndis_msg_hdr *buf, int buflen)
                                         request_id, xid);
                                 /* then likely retry */
                         } else switch (buf->msg_type) {
-                        case RNDIS_MSG_INDICATE: {      /* fault/event */
-                                struct rndis_indicate *msg = (void *)buf;
-                                int state = 0;
-
-                                switch (msg->status) {
-                                case RNDIS_STATUS_MEDIA_CONNECT:
-                                        state = 1;
-                                case RNDIS_STATUS_MEDIA_DISCONNECT:
-                                        dev_info(&info->control->dev,
-                                                "rndis media %sconnect\n",
-                                                !state?"dis":"");
-                                        if (dev->driver_info->link_change)
-                                                dev->driver_info->link_change(
-                                                        dev, state);
-                                        break;
-                                default:
-                                        dev_info(&info->control->dev,
-                                                "rndis indication: 0x%08x\n",
-                                                le32_to_cpu(msg->status));
-                                }
-                                }
+                        case RNDIS_MSG_INDICATE:        /* fault/event */
+                                rndis_msg_indicate(dev, (void *)buf, buflen);
+
                                 break;
                         case RNDIS_MSG_KEEPALIVE: {     /* ping */
                                 struct rndis_keepalive_c *msg = (void *)buf;