Hi Marek, On Tue, 25 Oct 2011 14:09:11 +0200 Marek Vasut <[email protected]> wrote:
> The problem was caused by a global variable being used early in the boot > process. > > This patch also contains the multibyte address swapping fix by: > Anatolij Gustschin <[email protected]> > > Signed-off-by: Marek Vasut <[email protected]> > Cc: Anatolij Gustschin <[email protected]> > Cc: Wolfgang Denk <[email protected]> > Cc: Albert ARIBAUD <[email protected]> > Cc: Heiko Schocher <[email protected]> > Cc: Stefano Babic <[email protected]> > --- > drivers/i2c/mxc_i2c.c | 29 ++++++++++++++++------------- > 1 files changed, 16 insertions(+), 13 deletions(-) Great, this patch fixes the issue! Thanks! Tested-by: Anatolij Gustschin <[email protected]> But, I think we should slightly fix the commit title and commit message to describe the observed problem better. The board is named as "imx31_phycore" in the code, config file uses this name, too. So we should use "imx31_phycore" name in U-Boot commits instead of pcm037 to avoid confusion for other users (pcm037 name is used in Linux code for this board). In the commit message we should also mention what the problem was: on imx31_phycore board reading the environment from I2C EEPROM didn't work correctly. Thanks, Anatolij -- 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: [email protected] _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
