Re: [PATCH] iio: adc: Kconfig: Make RZG2L_ADC depend on ARCH_RZG2L

[Geert Uytterhoeven]

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