[PATCH] pcmcia: at91_cf: depend on !ARCH_MULTIPLATFORM

[PATCH] pcmcia: at91_cf: depend on !ARCH_MULTIPLATFORM

[Alexandre Belloni]

Until the driver is corrected to stop including mach/at91_ramc.h and using
mach/io.h, it won't compile in a ARCH_MULTIPLATFORM configuration.

Suggested-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
 drivers/pcmcia/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/pcmcia/Kconfig b/drivers/pcmcia/Kconfig
index 910e90bf16c6..45f67c63d385 100644
--- a/drivers/pcmcia/Kconfig
+++ b/drivers/pcmcia/Kconfig
@@ -276,6 +276,7 @@ config BFIN_CFPCMCIA
 config AT91_CF
 	tristate "AT91 CompactFlash Controller"
 	depends on PCMCIA && ARCH_AT91
+	depends on !ARCH_MULTIPLATFORM
 	help
 	  Say Y here to support the CompactFlash controller on AT91 chips.
 	  Or choose M to compile the driver as a module named "at91_cf".
-- 
2.1.0

Re: [PATCH] pcmcia: at91_cf: depend on !ARCH_MULTIPLATFORM

[Nicolas Ferre]

Le 13/01/2015 19:28, Alexandre Belloni a écrit :
> Until the driver is corrected to stop including mach/at91_ramc.h and using
> mach/io.h, it won't compile in a ARCH_MULTIPLATFORM configuration.
> 
> Suggested-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>

> ---
>  drivers/pcmcia/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/pcmcia/Kconfig b/drivers/pcmcia/Kconfig
> index 910e90bf16c6..45f67c63d385 100644
> --- a/drivers/pcmcia/Kconfig
> +++ b/drivers/pcmcia/Kconfig
> @@ -276,6 +276,7 @@ config BFIN_CFPCMCIA
>  config AT91_CF
>  	tristate "AT91 CompactFlash Controller"
>  	depends on PCMCIA && ARCH_AT91
> +	depends on !ARCH_MULTIPLATFORM
>  	help
>  	  Say Y here to support the CompactFlash controller on AT91 chips.
>  	  Or choose M to compile the driver as a module named "at91_cf".
> 


-- 
Nicolas Ferre

Re: [PATCH] pcmcia: at91_cf: depend on !ARCH_MULTIPLATFORM

[Nicolas Ferre]

Le 14/01/2015 10:21, Nicolas Ferre a écrit :
> Le 13/01/2015 19:28, Alexandre Belloni a écrit :
>> Until the driver is corrected to stop including mach/at91_ramc.h and using
>> mach/io.h, it won't compile in a ARCH_MULTIPLATFORM configuration.
>>
>> Suggested-by: Arnd Bergmann <arnd@arndb.de>
>> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> 
> Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>

Arnd, Olof, Kevin,

It seems that nobody is taking this lonely patch. It can be interesting
to have it for 3.20.
So, can you please add it to the "drivers" branch so that the switch to
multi-platform is possible soon.

Here is the reference of it in patchwork:

https://patchwork.kernel.org/patch/5623391/

Tell me if it's better to ask Greg or Andrew.

Bye,

>> ---
>>  drivers/pcmcia/Kconfig | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/pcmcia/Kconfig b/drivers/pcmcia/Kconfig
>> index 910e90bf16c6..45f67c63d385 100644
>> --- a/drivers/pcmcia/Kconfig
>> +++ b/drivers/pcmcia/Kconfig
>> @@ -276,6 +276,7 @@ config BFIN_CFPCMCIA
>>  config AT91_CF
>>  	tristate "AT91 CompactFlash Controller"
>>  	depends on PCMCIA && ARCH_AT91
>> +	depends on !ARCH_MULTIPLATFORM
>>  	help
>>  	  Say Y here to support the CompactFlash controller on AT91 chips.
>>  	  Or choose M to compile the driver as a module named "at91_cf".
>>
> 
> 


-- 
Nicolas Ferre

Re: [PATCH] pcmcia: at91_cf: depend on !ARCH_MULTIPLATFORM

[Olof Johansson]

On Thu, Jan 29, 2015 at 10:40:49AM +0100, Nicolas Ferre wrote:
> Le 14/01/2015 10:21, Nicolas Ferre a écrit :
> > Le 13/01/2015 19:28, Alexandre Belloni a écrit :
> >> Until the driver is corrected to stop including mach/at91_ramc.h and using
> >> mach/io.h, it won't compile in a ARCH_MULTIPLATFORM configuration.
> >>
> >> Suggested-by: Arnd Bergmann <arnd@arndb.de>
> >> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> > 
> > Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
> 
> Arnd, Olof, Kevin,
> 
> It seems that nobody is taking this lonely patch. It can be interesting
> to have it for 3.20.
> So, can you please add it to the "drivers" branch so that the switch to
> multi-platform is possible soon.
> 
> Here is the reference of it in patchwork:
> 
> https://patchwork.kernel.org/patch/5623391/
> 
> Tell me if it's better to ask Greg or Andrew.

This is small and trivial enough that I think we can just take it. Applied to
next/drivers.


-Olof