Re: [PATCH net-next 1/2] net: break include loop netdevice.h, dsa.h, devlink.h

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 <redacted>

Woah, thanks for doing this!

Reviewed-by: Florian Fainelli <redacted>
--
Florian