Hi Mauro, On Wed, Apr 15, 2020 at 12:32 PM Mauro Carvalho Chehab <mchehab+huawei@xxxxxxxxxx> wrote: > As CEC support doesn't depend on MEDIA_SUPPORT, let's > place the platform drivers outside the media menu. Thanks for your patch! > As a side effect, instead of depends on PCI, seco driver > can select it (and DMI). I don't think that's a good idea, as it suddenly enables selecting all PCI drivers on platforms that do not have PCI. WARNING: unmet direct dependencies detected for PCI Depends on [n]: HAVE_PCI [=n] Selected by [m]: - CEC_SECO [=m] && MEDIA_CEC_SUPPORT [=y] && (X86 || IA64 || COMPILE_TEST [=y]) However, it already exposed some bugs that were hidden by PCI=n ;-) Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds