Hi Axel, On Sat, Jul 06, 2013 at 10:35:29PM +0800, Axel Lin wrote: > According to the datasheet[1], the Interrupt IRQ Pending Registers are > read-only. The implementation of sun4i_irq_ack() looks wrong because it writes > to these read-only registers.
Right, but the datasheet also says that these registers are "clear" register, which might indicate that it's actually R/W. I actually used one of the code dump from Allwinner for this driver, so I'm not exactly sure about wether irq_ack is needed or not. I'll test this and let you know. Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
signature.asc
Description: Digital signature
