On Sat, Apr 2, 2011 at 2:34 AM, Prafulla Wadaskar <prafu...@marvell.com> wrote: > > >> -----Original Message----- >> From: Lei Wen [mailto:lei...@marvell.com] >> Sent: Thursday, March 31, 2011 2:07 PM >> To: Heiko Schocher; Prafulla Wadaskar; Wolfgang Denk; u- >> b...@lists.denx.de; Marek Vasut; Ashish Karkare; Prabhanjan Sarnaik; Yu >> Tang; adrian.w...@gmail.com >> Subject: [PATCH V6 3/5] I2C: add i2c support for Pantheon platform >> >> Add i2c support to dkb board with pantheon soc. >> >> Signed-off-by: Lei Wen <lei...@marvell.com> >> --- >> Changelog: >> V2: >> NO CHANGE >> >> V3: >> clean code sytle issue >> Add i2c clock enable code include in I2C configure define block >> >> V4: >> make i2c definition included in the ifdef >> >> V5: >> NO CHANGE >> >> V6: >> Move the CONFIG_CMD_I2C define place >> >> arch/arm/cpu/arm926ejs/pantheon/cpu.c | 12 ++++++++++++ >> arch/arm/include/asm/arch-pantheon/cpu.h | 4 +++- >> arch/arm/include/asm/arch-pantheon/mfp.h | 6 ++++-- >> board/Marvell/dkb/dkb.c | 4 ++++ >> include/configs/dkb.h | 13 +++++++++++++ >> 5 files changed, 36 insertions(+), 3 deletions(-) >> >> diff --git a/arch/arm/cpu/arm926ejs/pantheon/cpu.c >> b/arch/arm/cpu/arm926ejs/pantheon/cpu.c >> index 9ddc77c..8b2eafa 100644 >> --- a/arch/arm/cpu/arm926ejs/pantheon/cpu.c >> +++ b/arch/arm/cpu/arm926ejs/pantheon/cpu.c >> @@ -59,6 +59,12 @@ int arch_cpu_init(void) >> /* Enable GPIO clock */ >> writel(APBC_APBCLK, &apbclkres->gpio); >> >> +#ifdef CONFIG_I2C_MV >> + /* Enable I2C clock */ >> + writel(APBC_RST | APBC_FNCLK | APBC_APBCLK, &apbclkres->twsi); >> + writel(APBC_FNCLK | APBC_APBCLK, &apbclkres->twsi); >> +#endif >> + >> icache_enable(); >> >> return 0; >> @@ -76,3 +82,9 @@ int print_cpuinfo(void) >> return 0; >> } >> #endif >> + >> +#ifdef CONFIG_I2C_MV >> +void i2c_clk_enable(void) >> +{ >> +} >> +#endif >> diff --git a/arch/arm/include/asm/arch-pantheon/cpu.h >> b/arch/arm/include/asm/arch-pantheon/cpu.h >> index 30f4393..60955c5 100644 >> --- a/arch/arm/include/asm/arch-pantheon/cpu.h >> +++ b/arch/arm/include/asm/arch-pantheon/cpu.h >> @@ -50,7 +50,9 @@ struct panthapb_registers { >> u32 uart0; /*0x000*/ >> u32 uart1; /*0x004*/ >> u32 gpio; /*0x008*/ >> - u8 pad0[0x034 - 0x08 - 4]; >> + u8 pad0[0x02c - 0x08 - 4]; >> + u32 twsi; /*0x02c*/ >> + u8 pad1[0x034 - 0x2c - 4]; >> u32 timers; /*0x034*/ >> }; >> >> diff --git a/arch/arm/include/asm/arch-pantheon/mfp.h >> b/arch/arm/include/asm/arch-pantheon/mfp.h >> index fb291cf..e939196 100644 >> --- a/arch/arm/include/asm/arch-pantheon/mfp.h >> +++ b/arch/arm/include/asm/arch-pantheon/mfp.h >> @@ -32,8 +32,10 @@ >> * offset, pull,pF, drv,dF, edge,eF ,afn,aF >> */ >> /* UART2 */ >> -#define MFP47_UART2_RXD MFP_REG(0x198) | MFP_AF6 | >> MFP_DRIVE_MEDIUM >> -#define MFP48_UART2_TXD MFP_REG(0x19c) | MFP_AF6 | >> MFP_DRIVE_MEDIUM >> +#define MFP47_UART2_RXD (MFP_REG(0x198) | MFP_AF6 | >> MFP_DRIVE_MEDIUM) >> +#define MFP48_UART2_TXD (MFP_REG(0x19c) | MFP_AF6 | >> MFP_DRIVE_MEDIUM) >> +#define MFP53_CI2C_SCL (MFP_REG(0x1b0) | MFP_AF2 | >> MFP_DRIVE_MEDIUM) >> +#define MFP54_CI2C_SDA (MFP_REG(0x1b4) | MFP_AF2 | >> MFP_DRIVE_MEDIUM) >> >> /* More macros can be defined here... */ >> >> diff --git a/board/Marvell/dkb/dkb.c b/board/Marvell/dkb/dkb.c >> index 72a2d2a..00f73e7 100644 >> --- a/board/Marvell/dkb/dkb.c >> +++ b/board/Marvell/dkb/dkb.c >> @@ -36,6 +36,10 @@ int board_early_init_f(void) >> MFP47_UART2_RXD, >> MFP48_UART2_TXD, >> >> + /* I2C */ >> + MFP53_CI2C_SCL, >> + MFP54_CI2C_SDA, >> + >> MFP_EOC /*End of configureation*/ >> }; >> /* configure MFP's */ >> diff --git a/include/configs/dkb.h b/include/configs/dkb.h >> index 638af5e..c893a9d 100644 >> --- a/include/configs/dkb.h >> +++ b/include/configs/dkb.h >> @@ -47,6 +47,7 @@ >> #define CONFIG_SYS_NO_FLASH /* Declare no flash (NOR/SPI) */ >> #include <config_cmd_default.h> >> #define CONFIG_CMD_AUTOSCRIPT >> +#define CONFIG_CMD_I2C >> #undef CONFIG_CMD_NET >> #undef CONFIG_CMD_NFS >> /* >> @@ -56,6 +57,18 @@ >> #include "mv-common.h" >> >> #undef CONFIG_ARCH_MISC_INIT > > >> + >> +/* >> + * I2C definition >> + */ >> +#ifdef CONFIG_CMD_I2C >> +#define CONFIG_I2C_MV 1 >> +#define CONFIG_PXA_I2C_REG 0xd4011000 > > PXA looks invalid reference here, should be MV. > >> +#define CONFIG_HARD_I2C 1 >> +#define CONFIG_SYS_I2C_SPEED 0 >> +#define CONFIG_SYS_I2C_SLAVE 0xfe >> +#endif > > Sorry being not reported earlier. > But this #ifdefed code should be in mv-common.h or <arch/config.h> so that it > can be shared across all the boards supported by the same SoC or IP.
I'd prefer to put it into <arch/config.h>. Would do this change for next post. Best regards, Lei _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
