[U-Boot-Users] PATCH: Add command support for secondI2Ccontroller
x0nishan at ti.com
Tue May 16 19:15:51 CEST 2006
> One sticky part is, how do we deal with the 'CFG_I2C_NOPROBE' stuff?
In my board, I had ignored this.
> structure that includes all bus information:
> typedef struct _i2c_bus
> int baseAddress
> int busSpeed;
> uchar noProbeList;
> ... /* whatever else might be useful */
> } i2c_bus;
If we keep the following requirements:
1. We don't want to modify implementations of NOPROBE for all previous
2. as less intrusive change as possible to cmd_i2c.c.
Two approaches appear to me:
1. use additional set_i2c_ignore cmd to select the ignore list/even make
CFG_I2C_NOPROBES indexed on busses.
+ve, minimal change to cmd_i2c.c,
-ve - will be patch on..
2. The structure defined as you gave it
+ve - cleaner
-ve - multiple changes to config files+ additional implementation to all
My view: I'd go with option 1. Primarily coz I suspect half our
community might be too busy to go back and change and test all previous
More information about the U-Boot