* [PATCH] iio: adc: xilinx: support all platforms
@ 2019-06-10 16:23 Robert Hancock
2019-06-11 6:28 ` Ardelean, Alexandru
2019-06-16 15:20 ` Jonathan Cameron
0 siblings, 2 replies; 5+ messages in thread
From: Robert Hancock @ 2019-06-10 16:23 UTC (permalink / raw)
To: linux-iio
Cc: jic23, ardeleanalex, michal.simek, knaack.h, pmeerw, lars,
Robert Hancock
Since the XADC logic can be used with standalone Xilinx FPGAs, this
driver can potentially be used with many different platforms, not just
the Zynq and MicroBlaze platforms this driver was allowed to be built
for. There should be no platform-specific code in this driver, so just
delete the platform dependency.
Signed-off-by: Robert Hancock <hancock@sedsystems.ca>
---
This is an updated version of an earlier patch "iio: adc: xilinx: support all
ARM platforms", but this one just deletes the platform restriction entirely
as it seems unnecessary.
drivers/iio/adc/Kconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig
index f96a770..7e32862 100644
--- a/drivers/iio/adc/Kconfig
+++ b/drivers/iio/adc/Kconfig
@@ -1085,7 +1085,6 @@ config VIPERBOARD_ADC
config XILINX_XADC
tristate "Xilinx XADC driver"
- depends on ARCH_ZYNQ || MICROBLAZE || COMPILE_TEST
depends on HAS_IOMEM
select IIO_BUFFER
select IIO_TRIGGERED_BUFFER
--
1.8.3.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] iio: adc: xilinx: support all platforms
2019-06-10 16:23 [PATCH] iio: adc: xilinx: support all platforms Robert Hancock
@ 2019-06-11 6:28 ` Ardelean, Alexandru
2019-06-11 6:52 ` Lars-Peter Clausen
2019-06-16 15:20 ` Jonathan Cameron
1 sibling, 1 reply; 5+ messages in thread
From: Ardelean, Alexandru @ 2019-06-11 6:28 UTC (permalink / raw)
To: hancock, linux-iio
Cc: ardeleanalex, michal.simek, jic23, knaack.h, pmeerw, lars
On Mon, 2019-06-10 at 10:23 -0600, Robert Hancock wrote:
> [External]
>
>
> Since the XADC logic can be used with standalone Xilinx FPGAs, this
> driver can potentially be used with many different platforms, not just
> the Zynq and MicroBlaze platforms this driver was allowed to be built
> for. There should be no platform-specific code in this driver, so just
> delete the platform dependency.
>
> Signed-off-by: Robert Hancock <hancock@sedsystems.ca>
> ---
>
> This is an updated version of an earlier patch "iio: adc: xilinx: support all
> ARM platforms", but this one just deletes the platform restriction entirely
> as it seems unnecessary.
>
> drivers/iio/adc/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig
> index f96a770..7e32862 100644
> --- a/drivers/iio/adc/Kconfig
> +++ b/drivers/iio/adc/Kconfig
> @@ -1085,7 +1085,6 @@ config VIPERBOARD_ADC
>
> config XILINX_XADC
> tristate "Xilinx XADC driver"
> - depends on ARCH_ZYNQ || MICROBLAZE || COMPILE_TEST
Just curios here: is `depends COMPILE_TEST` needed so that this gets included in a compile-test build ?
I'm somewhat vague on Linux's makefile logic/structure.
Thanks
Alex
> depends on HAS_IOMEM
> select IIO_BUFFER
> select IIO_TRIGGERED_BUFFER
> --
> 1.8.3.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] iio: adc: xilinx: support all platforms
2019-06-11 6:28 ` Ardelean, Alexandru
@ 2019-06-11 6:52 ` Lars-Peter Clausen
2019-06-11 6:57 ` Ardelean, Alexandru
0 siblings, 1 reply; 5+ messages in thread
From: Lars-Peter Clausen @ 2019-06-11 6:52 UTC (permalink / raw)
To: Ardelean, Alexandru, hancock, linux-iio
Cc: ardeleanalex, michal.simek, jic23, knaack.h, pmeerw
On 6/11/19 8:28 AM, Ardelean, Alexandru wrote:
> On Mon, 2019-06-10 at 10:23 -0600, Robert Hancock wrote:
>> [External]
>>
>>
>> Since the XADC logic can be used with standalone Xilinx FPGAs, this
>> driver can potentially be used with many different platforms, not just
>> the Zynq and MicroBlaze platforms this driver was allowed to be built
>> for. There should be no platform-specific code in this driver, so just
>> delete the platform dependency.
>>
>> Signed-off-by: Robert Hancock <hancock@sedsystems.ca>
>> ---
>>
>> This is an updated version of an earlier patch "iio: adc: xilinx: support all
>> ARM platforms", but this one just deletes the platform restriction entirely
>> as it seems unnecessary.
>>
>> drivers/iio/adc/Kconfig | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig
>> index f96a770..7e32862 100644
>> --- a/drivers/iio/adc/Kconfig
>> +++ b/drivers/iio/adc/Kconfig
>> @@ -1085,7 +1085,6 @@ config VIPERBOARD_ADC
>>
>> config XILINX_XADC
>> tristate "Xilinx XADC driver"
>> - depends on ARCH_ZYNQ || MICROBLAZE || COMPILE_TEST
>
> Just curios here: is `depends COMPILE_TEST` needed so that this gets included in a compile-test build ?
> I'm somewhat vague on Linux's makefile logic/structure.
COMPILE_TEST enables the driver for compile testing if it is not already
enabled otherwise. Now that the driver has no dependencies at all it
will always be enabled, which includes compile testing.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] iio: adc: xilinx: support all platforms
2019-06-11 6:52 ` Lars-Peter Clausen
@ 2019-06-11 6:57 ` Ardelean, Alexandru
0 siblings, 0 replies; 5+ messages in thread
From: Ardelean, Alexandru @ 2019-06-11 6:57 UTC (permalink / raw)
To: hancock, lars, linux-iio
Cc: ardeleanalex, michal.simek, jic23, knaack.h, pmeerw
On Tue, 2019-06-11 at 08:52 +0200, Lars-Peter Clausen wrote:
> [External]
>
>
> On 6/11/19 8:28 AM, Ardelean, Alexandru wrote:
> > On Mon, 2019-06-10 at 10:23 -0600, Robert Hancock wrote:
> > > [External]
> > >
> > >
> > > Since the XADC logic can be used with standalone Xilinx FPGAs, this
> > > driver can potentially be used with many different platforms, not just
> > > the Zynq and MicroBlaze platforms this driver was allowed to be built
> > > for. There should be no platform-specific code in this driver, so just
> > > delete the platform dependency.
> > >
> > > Signed-off-by: Robert Hancock <hancock@sedsystems.ca>
> > > ---
> > >
> > > This is an updated version of an earlier patch "iio: adc: xilinx: support all
> > > ARM platforms", but this one just deletes the platform restriction entirely
> > > as it seems unnecessary.
> > >
> > > drivers/iio/adc/Kconfig | 1 -
> > > 1 file changed, 1 deletion(-)
> > >
> > > diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig
> > > index f96a770..7e32862 100644
> > > --- a/drivers/iio/adc/Kconfig
> > > +++ b/drivers/iio/adc/Kconfig
> > > @@ -1085,7 +1085,6 @@ config VIPERBOARD_ADC
> > >
> > > config XILINX_XADC
> > > tristate "Xilinx XADC driver"
> > > - depends on ARCH_ZYNQ || MICROBLAZE || COMPILE_TEST
> >
> > Just curios here: is `depends COMPILE_TEST` needed so that this gets included in a compile-test build ?
> > I'm somewhat vague on Linux's makefile logic/structure.
>
> COMPILE_TEST enables the driver for compile testing if it is not already
> enabled otherwise. Now that the driver has no dependencies at all it
> will always be enabled, which includes compile testing.
I see.
Thanks for the info
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] iio: adc: xilinx: support all platforms
2019-06-10 16:23 [PATCH] iio: adc: xilinx: support all platforms Robert Hancock
2019-06-11 6:28 ` Ardelean, Alexandru
@ 2019-06-16 15:20 ` Jonathan Cameron
1 sibling, 0 replies; 5+ messages in thread
From: Jonathan Cameron @ 2019-06-16 15:20 UTC (permalink / raw)
To: Robert Hancock
Cc: linux-iio, ardeleanalex, michal.simek, knaack.h, pmeerw, lars
On Mon, 10 Jun 2019 10:23:44 -0600
Robert Hancock <hancock@sedsystems.ca> wrote:
> Since the XADC logic can be used with standalone Xilinx FPGAs, this
> driver can potentially be used with many different platforms, not just
> the Zynq and MicroBlaze platforms this driver was allowed to be built
> for. There should be no platform-specific code in this driver, so just
> delete the platform dependency.
>
> Signed-off-by: Robert Hancock <hancock@sedsystems.ca>
Applied to the togreg branch of iio.git and pushed out as testing
to see if any of us missed anything. Hopefully the many compiles 0-day
will do on this will highlight if there is anything hiding we haven't thought
of to make this a bad idea (more likely needing a little fix ;)
Jonathan
> ---
>
> This is an updated version of an earlier patch "iio: adc: xilinx: support all
> ARM platforms", but this one just deletes the platform restriction entirely
> as it seems unnecessary.
>
> drivers/iio/adc/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig
> index f96a770..7e32862 100644
> --- a/drivers/iio/adc/Kconfig
> +++ b/drivers/iio/adc/Kconfig
> @@ -1085,7 +1085,6 @@ config VIPERBOARD_ADC
>
> config XILINX_XADC
> tristate "Xilinx XADC driver"
> - depends on ARCH_ZYNQ || MICROBLAZE || COMPILE_TEST
> depends on HAS_IOMEM
> select IIO_BUFFER
> select IIO_TRIGGERED_BUFFER
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-06-16 15:20 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-10 16:23 [PATCH] iio: adc: xilinx: support all platforms Robert Hancock
2019-06-11 6:28 ` Ardelean, Alexandru
2019-06-11 6:52 ` Lars-Peter Clausen
2019-06-11 6:57 ` Ardelean, Alexandru
2019-06-16 15:20 ` Jonathan Cameron
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).