Re: nt: usb: USB_RTL8153_ECM should not default to y

On Mon, Sep 20, 2021 at 9:15 PM Jakub Kicinski [off-list ref] wrote:

On Sat, 18 Sep 2021 12:53:31 +0200 Geert Uytterhoeven wrote:

quoted

quoted

Yeah.. more context here:

https://lore.kernel.org/all/7fd014f2-c9a5-e7ec-f1c6-b3e4bb0f6eb6@samsung.com/ (local)

default !USB_RTL8152 would be my favorite but that probably doesn't
compute in kconfig land. Or perhaps bring back the 'y' but more clearly
mark it as a sub-option of CDCETHER? It's hard to blame people for
expecting drivers to default to n, we should make it clearer that this
is more of a "make driver X support variation Y", 'cause now it sounds
like a completely standalone driver from the Kconfig wording. At least
to a lay person like myself.

If it can be a module (tristate), it must be a separate (sub)driver, right?

Fair point.

The problem is CDCETHER (ECM) tries to be a generic driver that just
works for USB standards compliant generic hardware...
(similarly the EEM/NCM drivers)

There shouldn't be a 'subdriver'