diff mbox series

iio: adc: ep93xx: Enable wider build testing with COMPILE_TEST

Message ID 20221231144128.489810-1-jic23@kernel.org (mailing list archive)
State Accepted
Headers show
Series iio: adc: ep93xx: Enable wider build testing with COMPILE_TEST | expand

Commit Message

Jonathan Cameron Dec. 31, 2022, 2:41 p.m. UTC
From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

It is useful to be able to build this driver without needing
to build support for ARCH_EP93XX.

Also add an explicit dependency on HAS_IOMEM so we have
devm_platform_ioremap_resource() etc available.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Cc: Alexander Sverdlin <alexander.sverdlin@gmail.com>
Cc: Hartley Sweeten <hsweeten@visionengravers.com>
---
 drivers/iio/adc/Kconfig | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Alexander Sverdlin Jan. 4, 2023, 1:47 p.m. UTC | #1
On Sat, 2022-12-31 at 14:41 +0000, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> It is useful to be able to build this driver without needing
> to build support for ARCH_EP93XX.
> 
> Also add an explicit dependency on HAS_IOMEM so we have
> devm_platform_ioremap_resource() etc available.
> 
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

Acked-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>

> Cc: Alexander Sverdlin <alexander.sverdlin@gmail.com>
> Cc: Hartley Sweeten <hsweeten@visionengravers.com>
> ---
>  drivers/iio/adc/Kconfig | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig
> index 46c4fc2fc534..7d6e74189a2d 100644
> --- a/drivers/iio/adc/Kconfig
> +++ b/drivers/iio/adc/Kconfig
> @@ -441,7 +441,8 @@ config ENVELOPE_DETECTOR
>  
>  config EP93XX_ADC
>         tristate "Cirrus Logic EP93XX ADC driver"
> -       depends on ARCH_EP93XX
> +       depends on ARCH_EP93XX || COMPILE_TEST
> +       depends on HAS_IOMEM
>         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
Jonathan Cameron Jan. 8, 2023, 12:04 p.m. UTC | #2
On Wed, 04 Jan 2023 14:47:02 +0100
Alexander Sverdlin <alexander.sverdlin@gmail.com> wrote:

> On Sat, 2022-12-31 at 14:41 +0000, Jonathan Cameron wrote:
> > From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> > 
> > It is useful to be able to build this driver without needing
> > to build support for ARCH_EP93XX.
> > 
> > Also add an explicit dependency on HAS_IOMEM so we have
> > devm_platform_ioremap_resource() etc available.
> > 
> > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>  
> 
> Acked-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>
Applied.

Thanks,

J

> 
> > Cc: Alexander Sverdlin <alexander.sverdlin@gmail.com>
> > Cc: Hartley Sweeten <hsweeten@visionengravers.com>
> > ---
> >  drivers/iio/adc/Kconfig | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig
> > index 46c4fc2fc534..7d6e74189a2d 100644
> > --- a/drivers/iio/adc/Kconfig
> > +++ b/drivers/iio/adc/Kconfig
> > @@ -441,7 +441,8 @@ config ENVELOPE_DETECTOR
> >  
> >  config EP93XX_ADC
> >         tristate "Cirrus Logic EP93XX ADC driver"
> > -       depends on ARCH_EP93XX
> > +       depends on ARCH_EP93XX || COMPILE_TEST
> > +       depends on HAS_IOMEM
> >         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  
>
diff mbox series

Patch

diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig
index 46c4fc2fc534..7d6e74189a2d 100644
--- a/drivers/iio/adc/Kconfig
+++ b/drivers/iio/adc/Kconfig
@@ -441,7 +441,8 @@  config ENVELOPE_DETECTOR
 
 config EP93XX_ADC
 	tristate "Cirrus Logic EP93XX ADC driver"
-	depends on ARCH_EP93XX
+	depends on ARCH_EP93XX || COMPILE_TEST
+	depends on HAS_IOMEM
 	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