[PATCH v2 2/6] arm/imx6q: add core definitions and low-level debug uart

[Eric Miao]

On Mon, Sep 19, 2011 at 5:55 PM, Sascha Hauer <s.hauer at pengutronix.de> wrote:
> On Mon, Sep 19, 2011 at 05:45:59PM +0800, Eric Miao wrote:
>> On Mon, Sep 19, 2011 at 5:35 PM, Sascha Hauer <s.hauer at pengutronix.de> wrote:
>> > On Thu, Sep 15, 2011 at 10:45:22PM +0800, Shawn Guo wrote:
>> >> It adds the core definitions and low-level debug uart support
>> >> for imx6q.
>> >>
>> >>
>> >>  #define UART_VADDR   IMX_IO_ADDRESS(UART_PADDR)
>> >> diff --git a/arch/arm/plat-mxc/include/mach/entry-macro.S b/arch/arm/plat-mxc/include/mach/entry-macro.S
>> >> index 066d464..341f800 100644
>> >> --- a/arch/arm/plat-mxc/include/mach/entry-macro.S
>> >> +++ b/arch/arm/plat-mxc/include/mach/entry-macro.S
>> >> @@ -1,6 +1,6 @@
>> >>  /*
>> >>   *  Copyright (C) 2007 Lennert Buytenhek <buytenh at wantstofly.org>
>> >> - *  Copyright 2004-2009 Freescale Semiconductor, Inc. All Rights Reserved.
>> >> + *  Copyright 2004-2011 Freescale Semiconductor, Inc. All Rights Reserved.
>> >>   */
>> >>
>> >>  /*
>> >> @@ -11,6 +11,17 @@
>> >>
>> >>  #include <mach/hardware.h>
>> >>
>> >> +#ifdef CONFIG_ARM_GIC
>> >> +#include <asm/hardware/entry-macro-gic.S>
>> >> +
>> >> +     .macro  disable_fiq
>> >> +     .endm
>> >> +
>> >> +     .macro  arch_ret_to_user, tmp1, tmp2
>> >> +     .endm
>> >> +
>> >> +#else
>> >
>> > That's the third interrupt entry code handled here with ifdefs. I
>> > remember there was some infrastructure to have multiple irq
>> > controllers. I can't find any pointers, but I think Eric (added him
>> > to Cc) told me about it.
>> > As this is also the last roadblock to compile i.MX3 and i.MX5 into a
>> > single kernel this might be the right moment to work on this.
>>
>> It's CONFIG_MULTIPLE_IRQ_HANDLER. And check arch/arm/mach-pxa/irq.c
>> for example. Basically to use a C function to handle the interrupts, and
>> have each machine to specify the correct handler.
>>
>> E.g. in pxa - there are pxa25x_handle_irq, pxa3xx_handle_irq ...
>
> Nice, it's already merged. I should have used grep instead of google.
> For reference, it's CONFIG_MULTI_IRQ_HANDLER.

My poor memory :-)

>
> Sascha
>
> --
> Pengutronix e.K.                           |                             |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
>