[v2] stmmac: dwmac-loongson: Add architecture dependency

Message ID 20231115115331.19eb0c24@endymion.delvare
State New
Headers
Series [v2] stmmac: dwmac-loongson: Add architecture dependency |

Commit Message

Jean Delvare Nov. 15, 2023, 10:53 a.m. UTC
  Only present the DWMAC_LOONGSON option on architectures where it can
actually be used.

This follows the same logic as the DWMAC_INTEL option.

Signed-off-by: Jean Delvare <jdelvare@suse.de>
Cc: Keguang Zhang <keguang.zhang@gmail.com>
---
Changes since v1:
* Enable build testing (suggested by Simon Horman)

 drivers/net/ethernet/stmicro/stmmac/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Simon Horman Nov. 15, 2023, 5:11 p.m. UTC | #1
On Wed, Nov 15, 2023 at 11:53:31AM +0100, Jean Delvare wrote:
> Only present the DWMAC_LOONGSON option on architectures where it can
> actually be used.
> 
> This follows the same logic as the DWMAC_INTEL option.
> 
> Signed-off-by: Jean Delvare <jdelvare@suse.de>
> Cc: Keguang Zhang <keguang.zhang@gmail.com>
> ---
> Changes since v1:
> * Enable build testing (suggested by Simon Horman)

Thanks for the update, this version looks good to me.

Reviewed-by: Simon Horman <horms@kernel.org>
  
patchwork-bot+netdevbpf@kernel.org Nov. 16, 2023, 11:50 p.m. UTC | #2
Hello:

This patch was applied to netdev/net.git (main)
by David S. Miller <davem@davemloft.net>:

On Wed, 15 Nov 2023 11:53:31 +0100 you wrote:
> Only present the DWMAC_LOONGSON option on architectures where it can
> actually be used.
> 
> This follows the same logic as the DWMAC_INTEL option.
> 
> Signed-off-by: Jean Delvare <jdelvare@suse.de>
> Cc: Keguang Zhang <keguang.zhang@gmail.com>
> 
> [...]

Here is the summary with links:
  - [v2] stmmac: dwmac-loongson: Add architecture dependency
    https://git.kernel.org/netdev/net/c/7fbd5fc2b35a

You are awesome, thank you!
  

Patch

--- linux-6.6.orig/drivers/net/ethernet/stmicro/stmmac/Kconfig
+++ linux-6.6/drivers/net/ethernet/stmicro/stmmac/Kconfig
@@ -269,7 +269,7 @@  config DWMAC_INTEL
 config DWMAC_LOONGSON
 	tristate "Loongson PCI DWMAC support"
 	default MACH_LOONGSON64
-	depends on STMMAC_ETH && PCI
+	depends on (MACH_LOONGSON64 || COMPILE_TEST) && STMMAC_ETH && PCI
 	depends on COMMON_CLK
 	help
 	  This selects the LOONGSON PCI bus support for the stmmac driver,