[PATCHv2 4/5] mtd: mxc_nand fixups

Ivo Clarysse ivo.clarysse at gmail.com
Mon Jun 21 07:47:41 EDT 2010


On Sun, Jun 20, 2010 at 11:21 AM, John Ogness <john.ogness at linutronix.de> wrote:
>
> On 2010-06-20, Ivo Clarysse <ivo.clarysse at gmail.com> wrote:
> > Yes, on i.MX21, NFC interrupts can be masked using
> > NFC_CONFIG1:NFC_INT_MSK.  But I observed that as long as NFC_INT_MSK
> > is set, NFC_CONFIG2:NFC_INT will always read out 0 on i.MX21, even if
> > the operation is completed.
> >
> > So the driver will remain stuck at:
> >
> >                  wait_event(host->irq_waitq,
> >                          readw(host->regs + NFC_CONFIG2) & NFC_INT);
>
> OK. Here is alternative patch. Do you have access to an i.MX21 to test
> this on?
>
> This patch fixes the driver so that the irq is requested as disabled. This
> prevents double irq enabling and also does not require the interrupt to
> be disabled within the interrupt handler. (Actually, I beleive this was
> the true intention of the original author.)
[...]

Yes, that was indeed my true intention :)

I tested this change on an MX21ADS board.

Acked-by: Ivo Clarysse <ivo.clarysse at gmail.com>



More information about the linux-arm-kernel mailing list