[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