[U-Boot-Users] PATCH: Add command support for second I2C controller

Wolfgang Denk wd at denx.de
Thu May 18 00:24:18 CEST 2006


In message <1147723625.16780.140.camel at saruman.qstreams.net> you wrote:
> 
> Attached is a patch to common/cmd_i2c.c that allows access to two I2C
> controllers on a board.  Note that this doesn't actually change any
> hardware control - it just enhances the command set and passes more
> information to whatever version of i2c_read(), i2c_write() etc. that
> you're using.  I've implemented driver changes on MPC8349 hardware, but
> they're not quite ready for review yet.  New definitions:
> CONFIG_I2C_2_CTRLS - board has two I2C controllers
> CFG_I2C2_NOPROBES {} - list of devices on bus 2 to ignore when probing
> 
> CHANGELOG: 
> 	If CONFIG_I2C_2_CTRLS is defined, the 'chip' parameter of all I2C
> commands will accept an optional controller argument.    
> e.g.  'imd 50.1 0' displays data at offset 0 of controller 1 device 50
>       'imd 50.2 0' displays data at offset 0 of controller 2 device 50
>       'iprobe 2' probes for devices on the second bus

I reject this patch.

As discussed before, I don't like the command  format.  Second,  what
happens  if  there comes a board with 3 I2C busses? Then we touch the
code gain... No, thanks.

Also note that your patch has trailing white space,  so  it  violates
the coding standard.

Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
A rolling stone gathers momentum.




More information about the U-Boot mailing list