summaryrefslogtreecommitdiffstats
path: root/drivers/usb/gadget/ether.c
diff options
context:
space:
mode:
authorDavid Brownell <dbrownell@users.sourceforge.net>2008-06-19 18:19:16 -0700
committerGreg Kroah-Hartman <gregkh@suse.de>2008-07-21 15:16:10 -0700
commit15b2d2b529d11449910ac86f6093124bce8f6103 (patch)
treef8c153f86a19354e789c16af277dff2b68779f94 /drivers/usb/gadget/ether.c
parent7bb5ea54be47584869b9a748696e06788c55e28f (diff)
downloadblackbird-op-linux-15b2d2b529d11449910ac86f6093124bce8f6103.tar.gz
blackbird-op-linux-15b2d2b529d11449910ac86f6093124bce8f6103.zip
usb gadget: RNDIS cleanups
Some cleanup to the RNDIS code: - Minor bugfix: rndis_unit() is supposed to put the link into the RNDIS_UNINITIALIZED state, which does not mean "unused". There's a separate method to stop using the link. (Bug doesn't affect anything right now because of how the code is used.) - Reduce coupling between RNDIS code and its user(s), in preparation for updates in that code: * Decouple RNDIS_RESPONSE_AVAILABLE notifications from net_device by passing just a void* handle. (Also, remove the unused return value of the notification callback.) * When it needs a copy of net_device stats, just ask for it - Remove unused/untested code backing various never-used OIDs: * RNDIS_PM, RNDIS_WAKEUP ... "should" get implemented, but the relevant docs were unclear, ambguous, and incomplete. Someone with access to the Hidden Gospels (maybe in the EU?) might be able to figure out what this should do. * RNDIS_OPTIONAL_STATS ... as the name suggests, optional. Never implemented in part because not all the semantics were clear. * OID_GEN_RNDIS_CONFIG_PARAMETER, which has been #if 0 forever. - A few small whitespace fixes Plus switch the VERBOSE symbol over to the newer VERBOSE_DEBUG style. There should be no functional changes because of this patch; it's a net source code shrink (because of the dead/unused code removal) and a small object code shrink (a couple hundred bytes on ARMv5). Signed-off-by: David Brownell <dbrownell@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/gadget/ether.c')
-rw-r--r--drivers/usb/gadget/ether.c24
1 files changed, 11 insertions, 13 deletions
diff --git a/drivers/usb/gadget/ether.c b/drivers/usb/gadget/ether.c
index d3891b2fc36e..dee1f081165d 100644
--- a/drivers/usb/gadget/ether.c
+++ b/drivers/usb/gadget/ether.c
@@ -1106,6 +1106,8 @@ static void eth_reset_config (struct eth_dev *dev)
netif_stop_queue (dev->net);
netif_carrier_off (dev->net);
+
+ /* RNDIS enters RNDIS_UNINITIALIZED state */
rndis_uninit(dev->rndis_config);
/* disable endpoints, forcing (synchronous) completion of
@@ -1604,8 +1606,6 @@ eth_disconnect (struct usb_gadget *gadget)
eth_reset_config (dev);
spin_unlock_irqrestore (&dev->lock, flags);
- /* FIXME RNDIS should enter RNDIS_UNINITIALIZED */
-
/* next we may get setup() calls to enumerate new connections;
* or an unbind() during shutdown (including removing module).
*/
@@ -2067,23 +2067,23 @@ rndis_control_ack_complete (struct usb_ep *ep, struct usb_request *req)
eth_req_free(ep, req);
}
-static int rndis_control_ack (struct net_device *net)
+static void rndis_resp_avail(void *_dev)
{
- struct eth_dev *dev = netdev_priv(net);
+ struct eth_dev *dev = _dev;
int length;
struct usb_request *resp = dev->stat_req;
/* in case RNDIS calls this after disconnect */
if (!dev->status) {
DEBUG (dev, "status ENODEV\n");
- return -ENODEV;
+ return;
}
/* in case queue length > 1 */
if (resp->context) {
resp = eth_req_alloc (dev->status_ep, 8, GFP_ATOMIC);
if (!resp)
- return -ENOMEM;
+ return;
}
/* Send RNDIS RESPONSE_AVAILABLE notification;
@@ -2101,13 +2101,11 @@ static int rndis_control_ack (struct net_device *net)
resp->status = 0;
rndis_control_ack_complete (dev->status_ep, resp);
}
-
- return 0;
}
#else
-#define rndis_control_ack NULL
+#define rndis_resp_avail NULL
#endif /* RNDIS */
@@ -2566,18 +2564,18 @@ autoconf_fail:
/* FIXME RNDIS vendor id == "vendor NIC code" == ? */
- dev->rndis_config = rndis_register (rndis_control_ack);
- if (dev->rndis_config < 0) {
+ status = rndis_register(rndis_resp_avail, dev);
+ if (status < 0) {
fail0:
unregister_netdev (dev->net);
- status = -ENODEV;
goto fail;
}
+ dev->rndis_config = status;
/* these set up a lot of the OIDs that RNDIS needs */
rndis_set_host_mac (dev->rndis_config, dev->host_mac);
if (rndis_set_param_dev (dev->rndis_config, dev->net,
- &dev->stats, &dev->cdc_filter))
+ &dev->cdc_filter))
goto fail0;
if (rndis_set_param_vendor(dev->rndis_config, vendorID,
manufacturer))
OpenPOWER on IntegriCloud