diff mbox

staging: fsl-dpaa2/eth: add ETHERNET dependency

Message ID 20170516113016.4166462-1-arnd@arndb.de
State Accepted
Commit b72d7451209a0bad4264f5f4cb389e7f71cc5ad4
Headers show

Commit Message

Arnd Bergmann May 16, 2017, 11:30 a.m. UTC
The new driver cannot link correctly when the netdevice infrastructure
is disabled:

ERROR: "netdev_info" [drivers/staging/fsl-dpaa2/ethernet/fsl-dpaa2-eth.ko] undefined!
ERROR: "skb_to_sgvec" [drivers/staging/fsl-dpaa2/ethernet/fsl-dpaa2-eth.ko] undefined!
ERROR: "napi_disable" [drivers/staging/fsl-dpaa2/ethernet/fsl-dpaa2-eth.ko] undefined!
ERROR: "napi_schedule_prep" [drivers/staging/fsl-dpaa2/ethernet/fsl-dpaa2-eth.ko] undefined!
ERROR: "__napi_schedule_irqoff" [drivers/staging/fsl-dpaa2/ethernet/fsl-dpaa2-eth.ko] undefined!
ERROR: "netif_carrier_on" [drivers/staging/fsl-dpaa2/ethernet/fsl-dpaa2-eth.ko] undefined!

This adds a dependency on NETDEVICES and ETHERNET.

Fixes: 0352d1d85201 ("staging: fsl-dpaa2/eth: Add APIs for DPNI objects")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>

---
 drivers/staging/fsl-dpaa2/Kconfig | 1 +
 1 file changed, 1 insertion(+)

-- 
2.9.0

Comments

Ruxandra Ioana Radulescu May 17, 2017, 12:42 p.m. UTC | #1
> -----Original Message-----

> From: Arnd Bergmann [mailto:arnd@arndb.de]

> Sent: Tuesday, May 16, 2017 2:30 PM

> To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>; Arnd Bergmann

> <arnd@arndb.de>

> Cc: Ruxandra Ioana Radulescu <ruxandra.radulescu@nxp.com>;

> devel@driverdev.osuosl.org; linux-kernel@vger.kernel.org

> Subject: [PATCH] staging: fsl-dpaa2/eth: add ETHERNET dependency

> 

> The new driver cannot link correctly when the netdevice infrastructure

> is disabled:

> 

> ERROR: "netdev_info" [drivers/staging/fsl-dpaa2/ethernet/fsl-dpaa2-eth.ko]

> undefined!

> ERROR: "skb_to_sgvec" [drivers/staging/fsl-dpaa2/ethernet/fsl-dpaa2-

> eth.ko] undefined!

> ERROR: "napi_disable" [drivers/staging/fsl-dpaa2/ethernet/fsl-dpaa2-

> eth.ko] undefined!

> ERROR: "napi_schedule_prep" [drivers/staging/fsl-dpaa2/ethernet/fsl-

> dpaa2-eth.ko] undefined!

> ERROR: "__napi_schedule_irqoff" [drivers/staging/fsl-dpaa2/ethernet/fsl-

> dpaa2-eth.ko] undefined!

> ERROR: "netif_carrier_on" [drivers/staging/fsl-dpaa2/ethernet/fsl-dpaa2-

> eth.ko] undefined!

> 

> This adds a dependency on NETDEVICES and ETHERNET.

> 

> Fixes: 0352d1d85201 ("staging: fsl-dpaa2/eth: Add APIs for DPNI objects")

> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

> ---

>  drivers/staging/fsl-dpaa2/Kconfig | 1 +

>  1 file changed, 1 insertion(+)

> 

> diff --git a/drivers/staging/fsl-dpaa2/Kconfig b/drivers/staging/fsl-

> dpaa2/Kconfig

> index 2e325cb747ae..730fd6d4db33 100644

> --- a/drivers/staging/fsl-dpaa2/Kconfig

> +++ b/drivers/staging/fsl-dpaa2/Kconfig

> @@ -12,6 +12,7 @@ config FSL_DPAA2

>  config FSL_DPAA2_ETH

>  	tristate "Freescale DPAA2 Ethernet"

>  	depends on FSL_DPAA2 && FSL_MC_DPIO

> +	depends on NETDEVICES && ETHERNET

>  	---help---

>  	  Ethernet driver for Freescale DPAA2 SoCs, using the

>  	  Freescale MC bus driver

> --

> 2.9.0


Thanks!
Acked-by: Ioana Radulescu <ruxandra.radulescu@nxp.com>
diff mbox

Patch

diff --git a/drivers/staging/fsl-dpaa2/Kconfig b/drivers/staging/fsl-dpaa2/Kconfig
index 2e325cb747ae..730fd6d4db33 100644
--- a/drivers/staging/fsl-dpaa2/Kconfig
+++ b/drivers/staging/fsl-dpaa2/Kconfig
@@ -12,6 +12,7 @@  config FSL_DPAA2
 config FSL_DPAA2_ETH
 	tristate "Freescale DPAA2 Ethernet"
 	depends on FSL_DPAA2 && FSL_MC_DPIO
+	depends on NETDEVICES && ETHERNET
 	---help---
 	  Ethernet driver for Freescale DPAA2 SoCs, using the
 	  Freescale MC bus driver