Subject: [PATCH V3 2/4] drivers/i2c/busses/i2c-at91.c: add new driver

From: [email-masked] (Nicolas Ferre)
Date: Tue, 08 Nov 2011 16:15:10 +0100

On 11/08/2011 03:41 PM, Felipe Balbi :

>> +	if (cpu_is_at91rm9200()) {			/* AT91RM9200 Errata #22 */
> 
> I don't think you should be using cpu_is_* on drivers.

It is a common pattern in at91 drivers and has worked for ages.
Do you think it is related to the need to be able to compile the
driver for any SoC in the case of multi-SoC zImage support?

Best regards,
- -- 
Nicolas Ferre