[PATCH] versatile: don't generate a duplicate IRQ domain
Jamie Iles
jamie at jamieiles.com
Thu Jan 12 16:58:01 EST 2012
Hi Russell,
On Thu, Jan 12, 2012 at 08:37:56PM +0000, Russell King - ARM Linux wrote:
> On Fri, Jan 06, 2012 at 04:39:57PM +0000, Jamie Iles wrote:
> > Now that the VIC driver handles the irqdomain natively we don't need to
> > generate one in the versatile core code. Longer term we should move the
> > initialisation of both IRQ controllers to using of_irq_init() but
> > that'll need a little more work.
> >
> > Cc: Russell King <linux at arm.linux.org.uk>
> > Cc: Grant Likely <grant.likely at secretlab.ca>
> > Signed-off-by: Jamie Iles <jamie at jamieiles.com>
>
> Please confirm that this patch can be applied ontop of commit
> 356b95424cfb456e14a59eaa579422ce014c424b
> (Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/geert/linux-m68k)
OK, this does apply, but it doesn't fail in the way I had expected.
Actually, it doesn't barf at all because the generated domain that my
patch removes has 0 IRQ's so it never gets attached to the irq_desc's.
However, this patch is still valid as we end up with two domains
associated with the VIC but we get lucky as the real domain is
registered first. I can update the patch description to reflect this if
you still want to apply it.
Incidentally, I've managed to get the versatile kernel booting under
qemu (using an experimental build with DT support, up to trying to mount
a rootfs), which seems to work nicely. I just need to find a suitable
rootfs and that'll be a nice test system.
Jamie
More information about the linux-arm-kernel
mailing list