On Mon, Apr 19, 2021 at 5:18 PM Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote: > On Mon, Apr 19, 2021 at 4:14 PM Andy Shevchenko > <andy.shevchenko@xxxxxxxxx> wrote: > > On Mon, Apr 19, 2021 at 4:58 PM Andy Shevchenko > > <andy.shevchenko@xxxxxxxxx> wrote: > > > On Mon, Apr 19, 2021 at 4:54 PM Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote: > > > > > > > In any case it's not true. We have the platform drivers w/o legacy > > > > > users that are not dependent on OF. > > > > > > > > Example? ;-) > > > > > > i2c-owl.c > > > > In case you want more > > sound/sparc/amd7930.c > > SND_SUN_AMD7930 depends on SND_SPARC && SBUS > SND_SPARC depends on SPARC > SPARC selects OF > > Hence, SND_SUN_AMD7930 depends on OF. Exactly my point. Read back what I wrote. TL;DR: It's *fine* to have _indirect_ dependency like this. -- With Best Regards, Andy Shevchenko