diff options
| author | Bjørn Mork <bjorn@mork.no> | 2013-11-01 06:16:42 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-11-02 02:02:02 -0400 |
| commit | bed6f762123fc53c63efef386531dd877cba2468 (patch) | |
| tree | 3233a7b19c055a0fdbc4424666f0f793cb00d269 /include/linux/usb | |
| parent | ff1632aa8581b7103ac2af1ea3cb4a415eb9d6ad (diff) | |
net: cdc_ncm: remove redundant netdev field
Too many pointers back and forth are likely to confuse developers,
creating subtle bugs whenever we forget to syncronize them all.
As a usbnet driver, we should stick with the standard struct
usbnet fields as much as possible. The netdevice is one such
field.
Cc: Greg Suarez <gsuarez@smithmicro.com>
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 'include/linux/usb')
| -rw-r--r-- | include/linux/usb/cdc_ncm.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/include/linux/usb/cdc_ncm.h b/include/linux/usb/cdc_ncm.h index 36e1e153ca2d..5c47bd9620d5 100644 --- a/include/linux/usb/cdc_ncm.h +++ b/include/linux/usb/cdc_ncm.h | |||
| @@ -98,7 +98,6 @@ struct cdc_ncm_ctx { | |||
| 98 | const struct usb_cdc_union_desc *union_desc; | 98 | const struct usb_cdc_union_desc *union_desc; |
| 99 | const struct usb_cdc_ether_desc *ether_desc; | 99 | const struct usb_cdc_ether_desc *ether_desc; |
| 100 | 100 | ||
| 101 | struct net_device *netdev; | ||
| 102 | struct usb_device *udev; | 101 | struct usb_device *udev; |
| 103 | struct usb_interface *control; | 102 | struct usb_interface *control; |
| 104 | struct usb_interface *data; | 103 | struct usb_interface *data; |
| @@ -129,7 +128,7 @@ struct cdc_ncm_ctx { | |||
| 129 | extern u8 cdc_ncm_select_altsetting(struct usbnet *dev, struct usb_interface *intf); | 128 | extern u8 cdc_ncm_select_altsetting(struct usbnet *dev, struct usb_interface *intf); |
| 130 | extern int cdc_ncm_bind_common(struct usbnet *dev, struct usb_interface *intf, u8 data_altsetting); | 129 | extern int cdc_ncm_bind_common(struct usbnet *dev, struct usb_interface *intf, u8 data_altsetting); |
| 131 | extern void cdc_ncm_unbind(struct usbnet *dev, struct usb_interface *intf); | 130 | extern void cdc_ncm_unbind(struct usbnet *dev, struct usb_interface *intf); |
| 132 | extern struct sk_buff *cdc_ncm_fill_tx_frame(struct cdc_ncm_ctx *ctx, struct sk_buff *skb, __le32 sign); | 131 | extern struct sk_buff *cdc_ncm_fill_tx_frame(struct usbnet *dev, struct sk_buff *skb, __le32 sign); |
| 133 | extern int cdc_ncm_rx_verify_nth16(struct cdc_ncm_ctx *ctx, struct sk_buff *skb_in); | 132 | extern int cdc_ncm_rx_verify_nth16(struct cdc_ncm_ctx *ctx, struct sk_buff *skb_in); |
| 134 | extern int cdc_ncm_rx_verify_ndp16(struct sk_buff *skb_in, int ndpoffset); | 133 | extern int cdc_ncm_rx_verify_ndp16(struct sk_buff *skb_in, int ndpoffset); |
| 135 | 134 | ||
