diff mbox series

net: cdc_ncm: Fix build error

Message ID 20200826065231.14344-1-yuehaibing@huawei.com
State Accepted
Delegated to: David Miller
Headers show
Series net: cdc_ncm: Fix build error | expand

Commit Message

Yue Haibing Aug. 26, 2020, 6:52 a.m. UTC
If USB_NET_CDC_NCM is y and USB_NET_CDCETHER is m, build fails:

drivers/net/usb/cdc_ncm.o:(.rodata+0x1d8): undefined reference to `usbnet_cdc_update_filter'

Select USB_NET_CDCETHER for USB_NET_CDC_NCM to fix this.

Reported-by: Hulk Robot <hulkci@huawei.com>
Fixes: e10dcb1b6ba7 ("net: cdc_ncm: hook into set_rx_mode to admit multicast traffic")
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 drivers/net/usb/Kconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Bjørn Mork Aug. 26, 2020, 7:03 a.m. UTC | #1
YueHaibing <yuehaibing@huawei.com> writes:

> If USB_NET_CDC_NCM is y and USB_NET_CDCETHER is m, build fails:
>
> drivers/net/usb/cdc_ncm.o:(.rodata+0x1d8): undefined reference to `usbnet_cdc_update_filter'
>
> Select USB_NET_CDCETHER for USB_NET_CDC_NCM to fix this.

Ouch.  For some reason I assumed that was always selected with usbnet.
Thanks for fixing.



Bjørn
David Miller Aug. 26, 2020, 2:29 p.m. UTC | #2
From: YueHaibing <yuehaibing@huawei.com>
Date: Wed, 26 Aug 2020 14:52:31 +0800

> If USB_NET_CDC_NCM is y and USB_NET_CDCETHER is m, build fails:
> 
> drivers/net/usb/cdc_ncm.o:(.rodata+0x1d8): undefined reference to `usbnet_cdc_update_filter'
> 
> Select USB_NET_CDCETHER for USB_NET_CDC_NCM to fix this.
> 
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Fixes: e10dcb1b6ba7 ("net: cdc_ncm: hook into set_rx_mode to admit multicast traffic")
> Signed-off-by: YueHaibing <yuehaibing@huawei.com>

Applied, thank you.
diff mbox series

Patch

diff --git a/drivers/net/usb/Kconfig b/drivers/net/usb/Kconfig
index a7fbc3ccd29e..c7bcfca7d70b 100644
--- a/drivers/net/usb/Kconfig
+++ b/drivers/net/usb/Kconfig
@@ -252,6 +252,7 @@  config USB_NET_CDC_EEM
 config USB_NET_CDC_NCM
 	tristate "CDC NCM support"
 	depends on USB_USBNET
+	select USB_NET_CDCETHER
 	default y
 	help
 	  This driver provides support for CDC NCM (Network Control Model