[U-Boot] [PATCH V2] I2C: Fix mxc_i2c.c problem on imx31_phycore
Stefano Babic
sbabic at denx.de
Wed Oct 26 10:59:16 CEST 2011
On 10/26/2011 10:25 AM, Stefano Babic wrote:
> On 10/25/2011 05:25 PM, Marek Vasut wrote:
>> The problem was caused by a global variable being used early in the boot
>> process.
>>
>> The symptoms were on imx31_phycore board, reading the environment from I2C
>> EEPROM didn't work correctly and causes default environment to be loaded.
>>
>> Signed-off-by: Marek Vasut <marek.vasut at gmail.com>
>> Cc: Wolfgang Denk <wd at denx.de>
>> Cc: Albert ARIBAUD <albert.u.boot at aribaud.net>
>> Cc: Heiko Schocher <hs at denx.de>
>> Cc: Stefano Babic <sbabic at denx.de>
>> Tested-by: Anatolij Gustschin <agust at denx.de>
>> ---
>> drivers/i2c/mxc_i2c.c | 29 ++++++++++++++++-------------
>> 1 files changed, 16 insertions(+), 13 deletions(-)
>>
>> V2: Fix commit message
>
> Tested on mx35pdk.
>
> Tested-by: Stefano Babic <sbabic at denx.de>
I tested (and it works), but there is a problem. I supposed it was a
mistake by me, but after some checks I have found the cause.
Your patch does not apply on current u-boot-imx. You sent earlier the patch:
"I2C: Add i2c_get/set_speed() to mxc_i2c.c"
that after Heiko's ACK I have applied to u-boot-imx and then it is
already merged into u-boot-arm. The patch you have sent now does not
take care of your previous patch and does not apply anymore.
Can you rebase your patch ? You can take u-boot-imx als reference, all
imx related patches are already applied.
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
=====================================================================
More information about the U-Boot
mailing list