On 25/11/2012 01:21, Marek Vasut wrote:
> Add register definitions for the i.MX23 power control block and
> clock control block. These are essential for the basic bootstrap
> of the i.MX23. Also, properly include them in imx-regs.h .
> 
> Signed-off-by: Marek Vasut <[email protected]>
> Cc: Stefano Babic <[email protected]>
> ---

Hi Marek, Hi Otavio,

>  arch/arm/include/asm/arch-mxs/imx-regs.h          |    5 +
>  arch/arm/include/asm/arch-mxs/regs-clkctrl-mx23.h |  221 +++++++++++++
>  arch/arm/include/asm/arch-mxs/regs-power-mx23.h   |  357 
> +++++++++++++++++++++
>  3 files changed, 583 insertions(+)
>  create mode 100644 arch/arm/include/asm/arch-mxs/regs-clkctrl-mx23.h
>  create mode 100644 arch/arm/include/asm/arch-mxs/regs-power-mx23.h
> 

The main (and only issue) here is that there is not yet a board with
MX23 in u-boot to use this code. Then this is dead code, and I cannot
merge it now - I will postpone this patch.

The best way to add support for this SOC is, as usually, to get a bunch
of patches with the last ones for a board using the SOC. Else we are
unsure that the code we introduce can be really used later.

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: [email protected]
=====================================================================
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to