Re: [PATCH v3 1/2] dt-bindings: iio: adc: ep93xx: Add cirrus,ep9301-adc description

From: Alexander Sverdlin
Date: Fri Dec 30 2022 - 15:42:07 EST


Hi Jonathan,

On Fri, 2022-12-30 at 18:08 +0000, Jonathan Cameron wrote:
> On Fri, 23 Dec 2022 17:26:35 +0100
> Alexander Sverdlin <alexander.sverdlin@xxxxxxxxx> wrote:
>
> > Add device tree bindings for Cirrus Logic EP9301/EP9302 internal SoCs' ADC
> > block.
> >
> > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> > Signed-off-by: Alexander Sverdlin <alexander.sverdlin@xxxxxxxxx>
>
> Applied to the togreg branch of iio.git and pushed out as testing.

Thanks!

> Whilst we are looking at this driver, Alexander, would you mind if we relaxed
> the Kconfig dependencies to:
>
> diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig
> index 46c4fc2fc534..fd1d68dce507 100644
> --- a/drivers/iio/adc/Kconfig
> +++ b/drivers/iio/adc/Kconfig
> @@ -441,7 +441,7 @@ config ENVELOPE_DETECTOR
>  
>  config EP93XX_ADC
>         tristate "Cirrus Logic EP93XX ADC driver"
> -       depends on ARCH_EP93XX
> +       depends on ARCH_EP93XX || COMPILE_TEST
>         help
>           Driver for the ADC module on the EP93XX series of SoC from Cirrus Logic.
>           It's recommended to switch on CONFIG_HIGH_RES_TIMERS option, in this
>
> I end up doing that locally to build test patches like this one and it doesn't
> seem to cause any problems.

Sure, it's fine with me!

--
Alexander Sverdlin.