Dear Heiko, In message <4a069233.2080...@denx.de> you wrote: > > > -#define immr ((immap_t *)CONFIG_SYS_IMMR) > > - > > /* by default set I2C bus 0 active */ > > static unsigned int bus_num = 0; > > If no other bus then the first needed, when running from Flash, > its okay for me.
Actually no I2C is needed while running from flash. In most cases, I2C will not be touched at all in U-Boot (i. e. only when a user runs a I2C command). > > @@ -222,9 +226,9 @@ void i2c_init (int speed, int saddr) > > } > > > > /* Disable interrupts */ > > - immr->i2c.icr = 0; > > + im->i2c.icr = 0; > > shouldn;t we use in/out accessors for this? > > > /* Turn off filters */ > > - immr->i2c.mifr = 0; > > + im->i2c.mifr = 0; > > here too (and so on) ... Yes - as you saw later, I decided to do this in a separate commit. Would you prefer if I squashed these commits into one? I can do that, of course. But I found debugging easier as is now... 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: w...@denx.de Perfection is reached, not when there is no longer anything to add, but when there is no longer anything to take away. - Antoine de Saint-Exupery _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot