On 01/31/2017 11:17 AM, Russell King - ARM Linux wrote: > Including phy.h and phy_fixed.h into net/dsa.h causes phy*.h to be an > unnecessary dependency for quite a large amount of the kernel. There's > very little which actually requires definitions from phy.h in net/dsa.h > - the include itself only wants the declaration of a couple of > structures and IFNAMSIZ. > > Add linux/if.h for IFNAMSIZ, declarations for the structures, phy.h to > mv88e6xxx.h as it needs it for phy_interface_t, and remove both phy.h > and phy_fixed.h from net/dsa.h. > > This patch reduces from around 800 files rebuilt to around 40 - even > with ccache, the time difference is noticable. > > In order to make this change, several drivers need to be updated to > include necessary headers that they were picking up through this > include. This has resulted in a much larger patch series. > > I'm assuming the 0-day builder has had 24 hours with this series, and > hasn't reported any further issues with it - the last issue was two > weeks ago (before I became ill) which I fixed over the last weekend. > > I'm hoping this doesn't conflict with what's already in net-next... For the entire series: Acked-by: Florian Fainelli <f.fainelli@xxxxxxxxx> Thanks a lot for doing that. > > arch/mips/cavium-octeon/octeon-platform.c | 4 ---- > drivers/net/dsa/mv88e6xxx/mv88e6xxx.h | 1 + > drivers/net/ethernet/broadcom/bgmac.c | 2 ++ > drivers/net/ethernet/cadence/macb.h | 2 ++ > drivers/net/ethernet/cavium/liquidio/lio_main.c | 1 + > drivers/net/ethernet/cavium/liquidio/lio_vf_main.c | 1 + > drivers/net/ethernet/cavium/liquidio/octeon_console.c | 1 + > drivers/net/ethernet/freescale/fman/fman_memac.c | 1 + > drivers/net/ethernet/marvell/mvneta.c | 1 + > drivers/net/ethernet/qualcomm/emac/emac-sgmii.c | 1 + > drivers/net/usb/lan78xx.c | 1 + > drivers/net/wireless/ath/ath5k/ahb.c | 2 +- > drivers/target/iscsi/iscsi_target_login.c | 1 + > include/net/dsa.h | 6 ++++-- > net/core/netprio_cgroup.c | 1 + > net/sunrpc/xprtrdma/svc_rdma_backchannel.c | 1 + > 16 files changed, 20 insertions(+), 7 deletions(-) > -- Florian
