In message <[EMAIL PROTECTED]> you wrote:
> 
> > Which exact driver are you referring to? If any driver has such a bug,
> > it should be fixed.
> 
> drivers/i2c/fsl_i2c.c
> 
> The function is i2c_init().  I recently posted a patch that added this line 
> of code:
> 
>       i2c_bus_speed[0] = set_i2c_bus_speed(dev, gd->i2c1_clk, speed);
> 
> And this is what causes the problem.  i2c_bus_speed[] is a global variable, 
> and
> i2c_init() is called before relocation.  Therefore, this function writes to
> flash the first time it's called.

The driver is broken if it writes to the data segment before
relocation.

That's a bug that should be fixed.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
The only solution is ... a balance of power. We  arm  our  side  with
exactly  that  much  more.  A balance of power -- the trickiest, most
difficult, dirtiest game of them all. But the only one that preserves
both sides.
        -- Kirk, "A Private Little War", stardate 4211.8

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Reply via email to