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

Ben Warren bwarren at qstreams.com
Thu May 18 16:22:39 CEST 2006


Thanks Wolfgang.  

I have no idea how you got this again - strange things are afoot with
the sourceforge mailing list.  Anyway, I'll be on the lookout for
traling white space in future submissions.

regards,
Ben

On Thu, 2006-05-18 at 00:24 +0200, Wolfgang Denk wrote:

> 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
> 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.denx.de/pipermail/u-boot/attachments/20060518/f9e499e6/attachment.htm 


More information about the U-Boot mailing list