Hi Jonathan, On Sun, Apr 10, 2022 at 7:52 PM Jonathan Cameron <jic23@xxxxxxxxxx> wrote: > On Wed, 6 Apr 2022 08:03:15 +0100 > Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> wrote: > > > ADC block is common on Renesas RZ/G2L and RZ/V2L SoC's, so instead of > > adding dependency for each SoC's add dependency on ARCH_RZG2L. The > > ARCH_RZG2L config option is already selected by ARCH_R9A07G044 and > > ARCH_R9A07G054. > > > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > Sounds like a fix? > > If so, please supply a Fixes tag. > no need to resend, just reply with one to this email. This is not really a fix, as the original dependency was correct at that time. This is a change to add support for the RZ/V2L (r9a07g054) SoC, and avoiding the need to update the dependencies when support is added for more SoCs of the RZ/G2L family later. > > --- a/drivers/iio/adc/Kconfig > > +++ b/drivers/iio/adc/Kconfig > > @@ -910,7 +910,7 @@ config ROCKCHIP_SARADC > > > > config RZG2L_ADC > > tristate "Renesas RZ/G2L ADC driver" > > - depends on ARCH_R9A07G044 || COMPILE_TEST > > + depends on ARCH_RZG2L || COMPILE_TEST > > help > > Say yes here to build support for the ADC found in Renesas > > RZ/G2L family. 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