On Tue, Jun 6, 2023, at 12:31, Geert Uytterhoeven wrote: > On Tue, Jun 6, 2023 at 12:21 PM Arnd Bergmann <arnd@xxxxxxxx> wrote: >> On Tue, Jun 6, 2023, at 11:28, Geert Uytterhoeven wrote: >> > On Tue, Jun 6, 2023 at 11:16 AM Arnd Bergmann <arnd@xxxxxxxx> wrote: >> >> On Tue, Jun 6, 2023, at 11:01, Geert Uytterhoeven wrote: >> >> >> >> This won't work when PCS_LYNX is a loadable module and >> >> STMMAC is built-in. I think we should just select PCS_LYNX >> > >> > Oops, you're right, forgot about that case. >> > What about using IS_REACHABLE() instead? >> > No, that won't work either, as DWMAC_SOCFPGA can be modular, >> > with STMMAC builtin. >> >> It would work because of the 'select PCS_LYNX' below DWMAC_SOCFPGA, > > That was my first thought, but it won't work, as DWMAC_SOCFPGA=m > causes PCS_LYNX=m, while main STMMAC can still be builtin. Right, got it now. Arnd