Hi Mark, On Thu, Jul 6, 2023 at 2:18 PM Mark Brown <broonie@xxxxxxxxxx> wrote: > On Thu, Jul 06, 2023 at 01:42:03PM +0200, Geert Uytterhoeven wrote: > > > Fix this by making REGMAP_SLIMBUS select REGMAP. > > Why is this being done as a separate patch? Because this is a new select, which was not handled through the "default y" before. [PATCH 1/3] converted only the existing ones. > > Drop the selection of REGMAP by MFD_WCD934X, as this is not needed > > (now both REGMAP_SLIMBUS and REGMAP_IRQ select REGMAP). > > This has always been redudnant, why is it mixed into this patch? Because it is related to REGMAP_SLIMBUS selecting REGMAP. I agree it was redundant before, as REGMAP_IRQ already selected REGMAP. 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