On 03/28/2017 02:45 PM, Andrew Lunn wrote: > There is an include loop between netdevice.h, dsa.h, devlink.h because > of NETDEV_ALIGN, making it impossible to use devlink structures in > dsa.h. > > Break this loop by taking dsa.h out of netdevice.h, add a forward > declaration of dsa_switch_tree and netdev_set_default_ethtool_ops() > function, which is what netdevice.h requires. > > No longer having dsa.h in netdevice.h means the includes in dsa.h no > longer get included. This breaks a few other files which depend on > these includes. Add these directly in the affected file. > > Signed-off-by: Andrew Lunn <andrew@xxxxxxx> Woah, thanks for doing this! Reviewed-by: Florian Fainelli <f.fainelli@xxxxxxxxx> -- Florian -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html