2019-11-13 05:23:25

by Radhey Shyam Pandey

Subject: [PATCH v3 net-next] net: axienet: In kconfig remove arch dependency for axi_emac

To enable xilinx axi_emac driver support on zynqmp ultrascale platform
(ARCH64) there are two choices, mention ARCH64 as a dependency list
and other is to check if this ARCH dependency list is really needed.
Later approach seems more reasonable, so remove the obsolete ARCH
dependency list for the axi_emac driver.

Sanity test done for microblaze, zynq and zynqmp ultrascale platform.

Signed-off-by: Radhey Shyam Pandey <[email protected]>
---
Changes for v3:
Remove obsolete dependency list. Suggested by Michal and Jakub.
Modified commit description.
---
drivers/net/ethernet/xilinx/Kconfig | 4 +---
1 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/drivers/net/ethernet/xilinx/Kconfig b/drivers/net/ethernet/xilinx/Kconfig
index 8d994ce..6304ebd 100644
--- a/drivers/net/ethernet/xilinx/Kconfig
+++ b/drivers/net/ethernet/xilinx/Kconfig
@@ -6,7 +6,6 @@
config NET_VENDOR_XILINX
bool "Xilinx devices"
default y
- depends on PPC || PPC32 || MICROBLAZE || ARCH_ZYNQ || MIPS || X86 || ARM || COMPILE_TEST
---help---
If you have a network (Ethernet) card belonging to this class, say Y.

@@ -26,11 +25,10 @@ config XILINX_EMACLITE

config XILINX_AXI_EMAC
tristate "Xilinx 10/100/1000 AXI Ethernet support"
- depends on MICROBLAZE || X86 || ARM || COMPILE_TEST
select PHYLINK
---help---
This driver supports the 10/100/1000 Ethernet from Xilinx for the
- AXI bus interface used in Xilinx Virtex FPGAs.
+ AXI bus interface used in Xilinx Virtex FPGAs and Soc's.

config XILINX_LL_TEMAC
tristate "Xilinx LL TEMAC (LocalLink Tri-mode Ethernet MAC) driver"
--
1.7.1

2019-11-13 19:45:51

by David Miller

Subject: Re: [PATCH v3 net-next] net: axienet: In kconfig remove arch dependency for axi_emac

From: Radhey Shyam Pandey <[email protected]>
Date: Wed, 13 Nov 2019 10:51:23 +0530

> To enable xilinx axi_emac driver support on zynqmp ultrascale platform
> (ARCH64) there are two choices, mention ARCH64 as a dependency list
> and other is to check if this ARCH dependency list is really needed.
> Later approach seems more reasonable, so remove the obsolete ARCH
> dependency list for the axi_emac driver.
>
> Sanity test done for microblaze, zynq and zynqmp ultrascale platform.
>
> Signed-off-by: Radhey Shyam Pandey <[email protected]>

Applied.