From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Mon, 18 Jan 2010 07:40:14 +0100 Subject: [U-Boot] [PATCH 3/9] MX51: Add register definitions In-Reply-To: <20100117111647.C75D4C88AE@gemini.denx.de> References: <1263212760-27272-1-git-send-email-sbabic@denx.de> <1263212760-27272-2-git-send-email-sbabic@denx.de> <1263212760-27272-3-git-send-email-sbabic@denx.de> <1263212760-27272-4-git-send-email-sbabic@denx.de> <20100117111647.C75D4C88AE@gemini.denx.de> Message-ID: <4B54024E.1070103@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 Wolfgang Denk wrote: > Dear Stefano Babic, > Hi Wolfgang, >> +#define MXC_CCM_CBCDR_EMI_PODF_OFFSET (22) Thanks, I do it. >> +/* WEIM registers */ >> +#define CSGCR1 0x00 >> +#define CSGCR2 0x04 >> +#define CSRCR1 0x08 >> +#define CSRCR2 0x0C >> +#define CSWCR1 0x10 > > This looks like something that should be converted into a C struct. You are right. >> +#define ESDCTL_ESDCDLY5 0x30 >> +#define ESDCTL_ESDCDLYGD 0x34 > > Ditto. i can say more: this defines are not used anymore and I can drop them. I have already cleanup the code removing unnecessary defines, but it seems not enough. > >> +/* CCM */ >> +#define CLKCTL_CCR 0x00 >> +#define CLKCTL_CCDR 0x04 >> +#define CLKCTL_CSR 0x08 >> +#define CLKCTL_CCSR 0x0C >> +#define CLKCTL_CACRR 0x10 >> +#define CLKCTL_CBCDR 0x14 >> +#define CLKCTL_CBCMR 0x18 >> +#define CLKCTL_CSCMR1 0x1C They are obsolete,too. There is already a C struct describing them. I drop them. >> +#define CLKCTL_CCGR0 0x68 >> +#define CLKCTL_CCGR1 0x6C >> +#define CLKCTL_CCGR2 0x70 >> +#define CLKCTL_CCGR3 0x74 >> +#define CLKCTL_CCGR4 0x78 >> +#define CLKCTL_CCGR5 0x7C >> +#define CLKCTL_CCGR6 0x80 >> +#define CLKCTL_CMEOR 0x84 >> +/* DPLL */ >> +#define PLL_DP_CTL 0x00 >> +#define PLL_DP_CONFIG 0x04 >> +#define PLL_DP_OP 0x08 >> +#define PLL_DP_MFD 0x0C >> +#define PLL_DP_MFN 0x10 >> +#define PLL_DP_MFNMINUS 0x14 >> +#define PLL_DP_MFNPLUS 0x18 >> +#define PLL_DP_HFS_OP 0x1C >> +#define PLL_DP_HFS_MFD 0x20 >> +#define PLL_DP_HFS_MFN 0x24 >> +#define PLL_DP_TOGC 0x28 >> +#define PLL_DP_DESTAT 0x2C > > And again. This is a different fall. These defines are used in the assembly lowlevel_init.S in the first initialization. I cannot remove them. However, only a part of these defines are currently used and I will drop the rest of them. 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 =====================================================================