[LEDE-DEV] Ethernet support: built-in or as modules?

[Felix Fietkau]

On 2017-02-07 09:15, Rafał Miłecki wrote:
> I think various targets handle it in different ways. E.g. brcm47xx has
> modules for b44, tg3 and bgmac while bcm53xx has bgmac built-in.
> 
> Is there any preference for this?
> 
> I was thinking about supporting more Ethernet-related stuff like
> switchdev, DSA, b53. It would be nice to have a clear guidance for
> handling this.
My suggestion is to be pragmatic about this. If a particular driver
affects many devices and space isn't tight, just build it into the
kernel. Use modules when it's either an exotic device, or you need to
save space through fine grained control over selected module packages.

- Felix