Hi Prafulla,
On Tue, Mar 22, 2011 at 7:22 PM, Prafulla Wadaskar <prafu...@marvell.com> wrote:
>
>
>> -----Original Message-----
>> From: Lei Wen [mailto:lei...@marvell.com]
>> Sent: Thursday, March 17, 2011 12:15 PM
>> To: Heiko Schocher; Wolfgang Denk; Prafulla Wadaskar; u-
>> b...@lists.denx.de; Marek Vasut; Ashish Karkare; Prabhanjan Sarnaik;
>> adrian.w...@gmail.com
>> Subject: [PATCH V3 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:
>> V3:
>> clean code sytle issue
>> Add i2c clock enable code include in I2C configure define block
>>
>> 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 | 11 +++++++++++
>> 5 files changed, 34 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)
>> +{
>> +}
>
> Empty function !!
As you might notice, this driver is directly taken from original cpu/pxa/i2c.c.
Original driver implent a global i2c clock enable in the reset
function call, which
is not needed by other platform, like the armada100 and pantheon.
But certainly I couldn't just rudely remove that code. So this is the reason
why I crreate the i2c_clk_enable function call and define it as void
one for the armada100
and pantheon.
>
>> +#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..1b18c44 100644
>> --- a/include/configs/dkb.h
>> +++ b/include/configs/dkb.h
>> @@ -56,6 +56,17 @@
>> #include "mv-common.h"
>>
>> #undef CONFIG_ARCH_MISC_INIT
>> +
>> +/*
>> + * I2C definition
>> + */
>> +#define CONFIG_CMD_I2C
>> +#define CONFIG_I2C_MV 1
>> +#define CONFIG_PXA_I2C_REG 0xd4011000
>> +#define CONFIG_HARD_I2C 1
>> +#define CONFIG_SYS_I2C_SPEED 0
>> +#define CONFIG_SYS_I2C_SLAVE 0xfe
>> +
> What about if I don't want to include this support
> These configuration must if ifdefed with CONFIG_CMD_I2C which will be defined
> in board config file.
Yep, that make sense. I would include this change for next post.
Thanks,
Lei
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
