[U-Boot] [PATCH 3/3] sh: ecovec: Update to new I2C framework
Nobuhiro Iwamatsu
nobuhiro.iwamatsu.yj at renesas.com
Mon Oct 21 11:40:11 CEST 2013
Hi,
Thanks for your review
2013/10/17 Heiko Schocher <hs at denx.de>:
> Hello Nobuhiro,
>
> Am 17.10.2013 09:18, schrieb Nobuhiro Iwamatsu:
>
>> Ecovec uses sh_i2c driver. sh_i2c driver updated to new I2C framwork.
>> This updates ecovec to this change.
>>
>> Signed-off-by: Nobuhiro Iwamatsu<nobuhiro.iwamatsu.yj at renesas.com>
>> ---
>> board/renesas/ecovec/ecovec.c | 4 ++--
>> include/configs/ecovec.h | 15 +++++++--------
>> 2 files changed, 9 insertions(+), 10 deletions(-)
>
>
> Could you sqash this patch into http://patchwork.ozlabs.org/patch/284106
> so we get no problems with "git bisect".
OK , I will bisect and re-send.
>
> Beside of that only one comment:
>
>> diff --git a/board/renesas/ecovec/ecovec.c b/board/renesas/ecovec/ecovec.c
>> index e2d365a..4289d2d 100644
>> --- a/board/renesas/ecovec/ecovec.c
>> +++ b/board/renesas/ecovec/ecovec.c
>> @@ -57,8 +57,8 @@ int board_late_init(void)
>>
>> outl(inl(MSTPCR2)& ~0x10000000, MSTPCR2);
>>
>>
>> - i2c_init(CONFIG_SYS_I2C_SPEED, CONFIG_SYS_I2C_SLAVE);
>> - i2c_set_bus_num(CONFIG_SYS_I2C_MODULE); /* Use I2C 1 */
>> + i2c_init(CONFIG_SYS_I2C_SH_SPEED1, CONFIG_SYS_I2C_SLAVE);
>> + i2c_set_bus_num(1); /* Use I2C 1 */
>
>
> same comment as for your patch for the kzm9g board, remove i2c_init()
> call, as it is no longer necessary, thanks!
>
I see. I will remove i2c_init().
> bye,
> Heiko
Best regards,
Nobuhiro
--
Nobuhiro Iwamatsu
More information about the U-Boot
mailing list