Re: [PATCH] pcmcia/at91_cf: don't redefine SZ_2K

2010-01-22 Thread Andrew Victor
hi,

>> I don't think the AVR32 arch uses this driver.  So the patch should be
>> fine to apply.
> I assume this is an Ack?

Sure.

Acked-by: Andrew Victor 

___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia


Re: [PATCH] pcmcia/at91_cf: don't redefine SZ_2K

2010-01-22 Thread Uwe Kleine-König
On Wed, Dec 23, 2009 at 10:26:56PM +0200, Andrew Victor wrote:
> hi,
> >> Have you checked whether everything which uses at91_cf still builds after
> >> this patch has been applied (eg, AVR32 stuff?)
> 
> > I wasn't aware that the driver is used on AVR32, too.
> 
> I don't think the AVR32 arch uses this driver.  So the patch should be
> fine to apply.
I assume this is an Ack?

Best regards
Uwe

-- 
Pengutronix e.K.  | Uwe Kleine-König|
Industrial Linux Solutions| http://www.pengutronix.de/  |

___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia


Re: [PATCH] pcmcia/at91_cf: don't redefine SZ_2K

2009-12-23 Thread Andrew Victor
hi,
>> Have you checked whether everything which uses at91_cf still builds after
>> this patch has been applied (eg, AVR32 stuff?)

> I wasn't aware that the driver is used on AVR32, too.

I don't think the AVR32 arch uses this driver.  So the patch should be
fine to apply.


Regards,
  Andrew Victor

___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia


Re: [PATCH] pcmcia/at91_cf: don't redefine SZ_2K

2009-12-22 Thread Uwe Kleine-König
On Tue, Dec 22, 2009 at 03:28:27PM +, Russell King - ARM Linux wrote:
> On Sun, Dec 20, 2009 at 10:10:34PM +0100, Uwe Kleine-König wrote:
> > This fixes:
> > 
> > drivers/pcmcia/at91_cf.c:55:1: warning: "SZ_2K" redefined
> > 
> > Since
> > 
> > c1191b0 ([ARM] Kirkwood: create a mapping for the Security Accelerator 
> > SRAM)
> 
> Have you checked whether everything which uses at91_cf still builds after
> this patch has been applied (eg, AVR32 stuff?)
I wasn't aware that the driver is used on AVR32, too.

H, drivers/pcmcia/Kconfig has:

config AT91_CF
tristate "AT91 CompactFlash Controller"
depends on PCMCIA && ARCH_AT91RM9200
...

So unless ARCH_AT91RM9200 is used on AVR32 there is no problem.
Moreover AVR32 doesn't seem to provide .

Best regards
Uwe

-- 
Pengutronix e.K.  | Uwe Kleine-König|
Industrial Linux Solutions| http://www.pengutronix.de/  |

___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia


Re: [PATCH] pcmcia/at91_cf: don't redefine SZ_2K

2009-12-22 Thread Russell King - ARM Linux
On Sun, Dec 20, 2009 at 10:10:34PM +0100, Uwe Kleine-König wrote:
> This fixes:
> 
>   drivers/pcmcia/at91_cf.c:55:1: warning: "SZ_2K" redefined
> 
> Since
> 
>   c1191b0 ([ARM] Kirkwood: create a mapping for the Security Accelerator 
> SRAM)

Have you checked whether everything which uses at91_cf still builds after
this patch has been applied (eg, AVR32 stuff?)

___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia