Re: [RFC PATCH net-next] drivers/net Documentation/networking: Createdirectory intel_wired_lan

[Jeff Kirsher]

On Sun, Oct 10, 2010 at 13:42, Joe Perches <joe@xxxxxxxxxxx> wrote:
> Perhaps it's better to move drivers from the very populated
> drivers/net directory into vendor specific directories similar
> to the Atheros approach used for drivers/net/wireless/ath/
>
> Move intel drivers and Documentation to separate directories
> Create drivers/net/intel_wired_lan/Kconfig.<speed> and Makefile
> Modify drivers/net/Kconfig and Makefile
> Update MAINTAINERS
>
> Signed-off-by: Joe Perches <joe@xxxxxxxxxxx>
> ---
> Â.../networking/{ => intel_wired_lan}/e100.txt   Â|  Â0
> Â.../networking/{ => intel_wired_lan}/e1000.txt   |  Â0
> Â.../networking/{ => intel_wired_lan}/igb.txt    |  Â0
> Â.../networking/{ => intel_wired_lan}/igbvf.txt   |  Â0
> Â.../networking/{ => intel_wired_lan}/ixgb.txt   Â|  Â0
> Â.../networking/{ => intel_wired_lan}/ixgbe.txt   |  Â0
> Â.../networking/{ => intel_wired_lan}/ixgbevf.txt  |  Â0
> ÂMAINTAINERS Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â| Â 18 +--
> Âdrivers/net/Kconfig                Â| Â214 +-------------------
> Âdrivers/net/Makefile                |  Â8 -
> Âdrivers/net/intel_wired_lan/Kconfig.100 Â Â Â Â Â Â| Â 25 +++
> Âdrivers/net/intel_wired_lan/Kconfig.1000 Â Â Â Â Â | Â102 ++++++++++
> Âdrivers/net/intel_wired_lan/Kconfig.10000 Â Â Â Â Â| Â 81 ++++++++
> Âdrivers/net/intel_wired_lan/Makefile        |  Â9 +
> Âdrivers/net/{ => intel_wired_lan}/e100.c      |  Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000/Makefile  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000/e1000.h  Â|  Â0
> Â.../{ => intel_wired_lan}/e1000/e1000_ethtool.c  Â|  Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000/e1000_hw.c | Â Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000/e1000_hw.h | Â Â0
> Â.../net/{ => intel_wired_lan}/e1000/e1000_main.c  |  Â0
> Â.../net/{ => intel_wired_lan}/e1000/e1000_osdep.h Â| Â Â0
> Â.../net/{ => intel_wired_lan}/e1000/e1000_param.c Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000e/82571.c  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000e/Makefile Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000e/defines.h | Â Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000e/e1000.h  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000e/es2lan.c Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000e/ethtool.c | Â Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000e/hw.h   Â|  Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000e/ich8lan.c | Â Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000e/lib.c   |  Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000e/netdev.c Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000e/param.c  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/e1000e/phy.c   |  Â0
> Âdrivers/net/{ => intel_wired_lan}/igb/Makefile   |  Â0
> Â.../net/{ => intel_wired_lan}/igb/e1000_82575.c  Â|  Â0
> Â.../net/{ => intel_wired_lan}/igb/e1000_82575.h  Â|  Â0
> Â.../net/{ => intel_wired_lan}/igb/e1000_defines.h Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/igb/e1000_hw.h  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/igb/e1000_mac.c Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/igb/e1000_mac.h Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/igb/e1000_mbx.c Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/igb/e1000_mbx.h Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/igb/e1000_nvm.c Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/igb/e1000_nvm.h Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/igb/e1000_phy.c Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/igb/e1000_phy.h Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/igb/e1000_regs.h | Â Â0
> Âdrivers/net/{ => intel_wired_lan}/igb/igb.h    Â|  Â0
> Â.../net/{ => intel_wired_lan}/igb/igb_ethtool.c  Â|  Â0
> Âdrivers/net/{ => intel_wired_lan}/igb/igb_main.c  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/igbvf/Makefile  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/igbvf/defines.h Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/igbvf/ethtool.c Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/igbvf/igbvf.h  Â|  Â0
> Âdrivers/net/{ => intel_wired_lan}/igbvf/mbx.c   Â|  Â0
> Âdrivers/net/{ => intel_wired_lan}/igbvf/mbx.h   Â|  Â0
> Âdrivers/net/{ => intel_wired_lan}/igbvf/netdev.c  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/igbvf/regs.h   |  Â0
> Âdrivers/net/{ => intel_wired_lan}/igbvf/vf.c    |  Â0
> Âdrivers/net/{ => intel_wired_lan}/igbvf/vf.h    |  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgb/Makefile  Â|  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgb/ixgb.h   Â|  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgb/ixgb_ee.c  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgb/ixgb_ee.h  |  Â0
> Â.../net/{ => intel_wired_lan}/ixgb/ixgb_ethtool.c Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgb/ixgb_hw.c  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgb/ixgb_hw.h  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgb/ixgb_ids.h Â| Â Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgb/ixgb_main.c | Â Â0
> Â.../net/{ => intel_wired_lan}/ixgb/ixgb_osdep.h  Â|  Â0
> Â.../net/{ => intel_wired_lan}/ixgb/ixgb_param.c  Â|  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgbe/Makefile  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgbe/ixgbe.h  Â|  Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_82598.c Â| Â Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_82599.c Â| Â Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_common.c | Â Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_common.h | Â Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_dcb.c  Â|  Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_dcb.h  Â|  Â0
> Â.../{ => intel_wired_lan}/ixgbe/ixgbe_dcb_82598.c Â| Â Â0
> Â.../{ => intel_wired_lan}/ixgbe/ixgbe_dcb_82598.h Â| Â Â0
> Â.../{ => intel_wired_lan}/ixgbe/ixgbe_dcb_82599.c Â| Â Â0
> Â.../{ => intel_wired_lan}/ixgbe/ixgbe_dcb_82599.h Â| Â Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_dcb_nl.c | Â Â0
> Â.../{ => intel_wired_lan}/ixgbe/ixgbe_ethtool.c  Â|  Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_fcoe.c  |  Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_fcoe.h  |  Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_main.c  |  Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_mbx.c  Â|  Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_mbx.h  Â|  Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_phy.c  Â|  Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_phy.h  Â|  Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_sriov.c Â| Â Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_sriov.h Â| Â Â0
> Â.../net/{ => intel_wired_lan}/ixgbe/ixgbe_type.h  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgbevf/Makefile | Â Â0
> Â.../net/{ => intel_wired_lan}/ixgbevf/defines.h  Â|  Â0
> Â.../net/{ => intel_wired_lan}/ixgbevf/ethtool.c  Â|  Â0
> Â.../net/{ => intel_wired_lan}/ixgbevf/ixgbevf.h  Â|  Â0
> Â.../{ => intel_wired_lan}/ixgbevf/ixgbevf_main.c  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgbevf/mbx.c  Â|  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgbevf/mbx.h  Â|  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgbevf/regs.h  |  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgbevf/vf.c   |  Â0
> Âdrivers/net/{ => intel_wired_lan}/ixgbevf/vf.h   |  Â0
> Â107 files changed, 224 insertions(+), 233 deletions(-)
>

NAK
I agree with Stephen that this will generate a lot of confusion and....

First, I think we need to keep the documentation in /Documentation/networking.
Second, the changes are extensive and would create a lot of regression testing.

We have been looking at solutions like this for future
drivers/hardware and is on the list of items we are currently working
on, but feel it should not be made retroactively due to the regression
testing and massive changes that would need to be made.

-- 
Cheers,
Jeff
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/