Hi Marek, On Tue, 25 Oct 2011 14:09:11 +0200 Marek Vasut <marek.va...@gmail.com> 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 <ag...@denx.de> > > Signed-off-by: Marek Vasut <marek.va...@gmail.com> > Cc: Anatolij Gustschin <ag...@denx.de> > Cc: Wolfgang Denk <w...@denx.de> > Cc: Albert ARIBAUD <albert.u.b...@aribaud.net> > Cc: Heiko Schocher <h...@denx.de> > Cc: Stefano Babic <sba...@denx.de> > --- > drivers/i2c/mxc_i2c.c | 29 ++++++++++++++++------------- > 1 files changed, 16 insertions(+), 13 deletions(-) Great, this patch fixes the issue! Thanks! Tested-by: Anatolij Gustschin <ag...@denx.de> 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: off...@denx.de _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
