diff options
author | Bjørn Mork <bjorn@mork.no> | 2013-11-01 06:16:47 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-11-02 02:02:02 -0400 |
commit | 832922362e1308aaef95a43383577d56f51fbc3c (patch) | |
tree | 23801dae1712dad3d00bdc052781cd43c955fc15 /drivers/net/usb | |
parent | 1b5287a7abec1177257a988b523c78dab7056b41 (diff) |
net: cdc_ncm: remove descriptor pointers
header_desc was completely unused and union_desc was never used
outside cdc_ncm_bind_common.
Cc: Alexey Orishko <alexey.orishko@gmail.com>
Signed-off-by: Bjørn Mork <bjorn@mork.no>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/usb')
-rw-r--r-- | drivers/net/usb/cdc_ncm.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/drivers/net/usb/cdc_ncm.c b/drivers/net/usb/cdc_ncm.c index b8de8ddd6d98..435fcc75d706 100644 --- a/drivers/net/usb/cdc_ncm.c +++ b/drivers/net/usb/cdc_ncm.c | |||
@@ -372,6 +372,7 @@ static const struct ethtool_ops cdc_ncm_ethtool_ops = { | |||
372 | 372 | ||
373 | int cdc_ncm_bind_common(struct usbnet *dev, struct usb_interface *intf, u8 data_altsetting) | 373 | int cdc_ncm_bind_common(struct usbnet *dev, struct usb_interface *intf, u8 data_altsetting) |
374 | { | 374 | { |
375 | const struct usb_cdc_union_desc *union_desc = NULL; | ||
375 | struct cdc_ncm_ctx *ctx; | 376 | struct cdc_ncm_ctx *ctx; |
376 | struct usb_driver *driver; | 377 | struct usb_driver *driver; |
377 | u8 *buf; | 378 | u8 *buf; |
@@ -406,16 +407,15 @@ int cdc_ncm_bind_common(struct usbnet *dev, struct usb_interface *intf, u8 data_ | |||
406 | 407 | ||
407 | switch (buf[2]) { | 408 | switch (buf[2]) { |
408 | case USB_CDC_UNION_TYPE: | 409 | case USB_CDC_UNION_TYPE: |
409 | if (buf[0] < sizeof(*(ctx->union_desc))) | 410 | if (buf[0] < sizeof(*union_desc)) |
410 | break; | 411 | break; |
411 | 412 | ||
412 | ctx->union_desc = | 413 | union_desc = (const struct usb_cdc_union_desc *)buf; |
413 | (const struct usb_cdc_union_desc *)buf; | ||
414 | 414 | ||
415 | ctx->control = usb_ifnum_to_if(dev->udev, | 415 | ctx->control = usb_ifnum_to_if(dev->udev, |
416 | ctx->union_desc->bMasterInterface0); | 416 | union_desc->bMasterInterface0); |
417 | ctx->data = usb_ifnum_to_if(dev->udev, | 417 | ctx->data = usb_ifnum_to_if(dev->udev, |
418 | ctx->union_desc->bSlaveInterface0); | 418 | union_desc->bSlaveInterface0); |
419 | break; | 419 | break; |
420 | 420 | ||
421 | case USB_CDC_ETHERNET_TYPE: | 421 | case USB_CDC_ETHERNET_TYPE: |
@@ -458,7 +458,7 @@ advance: | |||
458 | } | 458 | } |
459 | 459 | ||
460 | /* some buggy devices have an IAD but no CDC Union */ | 460 | /* some buggy devices have an IAD but no CDC Union */ |
461 | if (!ctx->union_desc && intf->intf_assoc && intf->intf_assoc->bInterfaceCount == 2) { | 461 | if (!union_desc && intf->intf_assoc && intf->intf_assoc->bInterfaceCount == 2) { |
462 | ctx->control = intf; | 462 | ctx->control = intf; |
463 | ctx->data = usb_ifnum_to_if(dev->udev, intf->cur_altsetting->desc.bInterfaceNumber + 1); | 463 | ctx->data = usb_ifnum_to_if(dev->udev, intf->cur_altsetting->desc.bInterfaceNumber + 1); |
464 | dev_dbg(&intf->dev, "CDC Union missing - got slave from IAD\n"); | 464 | dev_dbg(&intf->dev, "CDC Union missing - got slave from IAD\n"); |