Heiko Schocher wrote: > Hello Peter, > > Peter Tyser wrote: >> This patch series removes the "individual" I2C commands (and the >> CONFIG_I2C_CMD_TREE define) and migrates all boards to the newer >> "tree style" I2C commands. >> >> A small amount of cleanup was performed before and after removing >> the individual commands. > > Thanks for your work, looks good to me. I make some tests > with your patches, and if they are OK, I apply your patches to > u-boot-i2c.git. > > Hmm.. maybe you can have a look at my posting: > > http://lists.denx.de/pipermail/u-boot/2009-March/049506.html > > There is a new i2c_core.c file which holds all "common" i2c > functions, for example the i2c_[set|get]_bus_speed(). I > think such a file is a better place for it. > > I wonder I get no response for cleaning up the i2c subsystem ... > is here no interest in doing such a cleanup? (I think it is > a necessary step ...) > > comments are welcome ;-) > > bye > Heiko
I am in favor of an i2c cleanup, and also Peter's changeover to the i2c "tree style" fix. I would take the lack of opposition as tacit approval, rather than apathy. Thankfully, the u-boot list isn't made up of a flock of dittoheads (def'n #2): <http://www.urbandictionary.com/define.php?term=dittohead> Thanks, gvb _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot