On Wed, Jun 20, 2012 at 11:55:20PM +0100, Russell King - ARM Linux wrote:

On Tue, Jun 19, 2012 at 01:26:56PM +0800, Shawn Guo wrote:

quoted

On Mon, Jun 18, 2012 at 05:44:02PM +0100, Russell King - ARM Linux wrote:

quoted

FIQs should be an entirely separate number space from IRQs, as we
may want to totally decouple them from the IRQ stuff (we probably
should have already done this when genirq came along.)

About the only stuff FIQs use is the enable/disable_irq as a short
cut to dealing with the mask registers.

I do not quite understand what you are asking for, but I'm guessing it
with the patch below.  Please elaborate it a little bit more if that's
not what you are asking for.

I was thinking about moving entirely away from any bits of genirq for
this.  We shouldn't really be mixing these two things together by
nabbing some of the IRQ numberspace for this.

Looking at this code, I'm thinking about taking this further, and
santising the whole thing... though that's not going to be a quick
change.

What I'm looking for is a quick and reasonable fixing for a compile
error when SPARSE_IRQ is enabled for imx.  Can we take the original
patch as it is and defer the whole cleanup to another series?

-- 
Regards,
Shawn