From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Thu, 30 Dec 2010 13:09:05 +0100 Subject: [U-Boot] [PATCH v3 1/8] MX5: Add initial support for MX53 processor In-Reply-To: <1293626331-18644-2-git-send-email-r64343@freescale.com> References: <1293626331-18644-1-git-send-email-r64343@freescale.com> <1293626331-18644-2-git-send-email-r64343@freescale.com> Message-ID: <4D1C7661.20802@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 12/29/2010 01:38 PM, Jason Liu wrote: > Add initial support for Freescale MX53 processor, > > - Add the iomux support and the pin definition, > - Add the regs definition, clean up some unused def from mx51, > - Add the low level init support, make use the freq input of setup_pll macro Hi Jason, > diff --git a/arch/arm/include/asm/arch-mx5/asm-offsets.h b/arch/arm/include/asm/arch-mx5/asm-offsets.h > index afd2728..2258f2f 100644 > --- a/arch/arm/include/asm/arch-mx5/asm-offsets.h > +++ b/arch/arm/include/asm/arch-mx5/asm-offsets.h > @@ -37,7 +37,12 @@ > #define CLKCTL_CCGR4 0x78 > #define CLKCTL_CCGR5 0x7C > #define CLKCTL_CCGR6 0x80 > +#if defined(CONFIG_MX53) > +#define CLKCTL_CCGR7 0x84 > #define CLKCTL_CMEOR 0x84 > +#elif defined(CONFIG_MX51) > +#define CLKCTL_CMEOR 0x84 > +#endif It seems to me that CLKCTL_CMEOR should be dropped in the CONFIG_MX53 case, as its offset is reserved for CLKCTL_CCGR7. > -/*! > * Number of GPIO port as defined in the IC Spec > */ > #define GPIO_PORT_NUM 4 I see now that GPIO_PORT_NUM seems superfluous, and it is not used in any part of code. Please drop it :-) 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-0 Fax: +49-8142-66989-80 Email: office at denx.de =====================================================================