Hi Mark, On Tue, Nov 30, 2021 at 6:05 PM Mark Brown <broonie@xxxxxxxxxx> wrote: > On Tue, Nov 30, 2021 at 10:49:30AM -0600, Pierre-Louis Bossart wrote: > > To Geert's point, there may be an additional need to add a > > > depends on SND_SOC_AMD_ACP > > > There are also a set of > > > SND_SOC_AMD_ACPyx options, not sure if any applies as a dependency here? > > Or put them in an if block (IIRC I thought they were which was why the > dependency wasn't needed but I don't know what I was looking at if I did > check that). Probably you were looking at sound/soc/amd/acp/Kconfig, where all the other ACP options live, and where you had applied my earlier patch to wrap everything in a big "if". 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