[U-Boot-Users] [DNX#2006090142000017] [PATCH 1/2] Add support for multipl [...]

DENX Support System support at denx.de
Fri Sep 1 05:50:02 CEST 2006


Hello list,

inside the automatic U-Boot patch tracking system a new ticket
[DNX#2006090142000017] was created:

<snip>
> Ben Warren stated on 8/30/2006 4:35 PM:
> 
> > Overview:
> <snip>
> > be skipped by the 'i2c probe' command. 
> Thanks Ben, this is finally in tune with Denx requirements I guess ;)
> > ------------------------------------------------------------------------
> > 
> > diff --git a/README b/README
> > index e772c1a..f674050 100644
> > --- a/README
> > +++ b/README
> > @@ -1188,7 +1188,12 @@ The following options need to be configu
> >  		clock chips. See common/cmd_i2c.c for a description of the
> >  		command line interface.
> >  
> > -		CONFIG_HARD_I2C selects the CPM hardware driver for I2C.
> > +		CONFIG_I2C_CMD_TREE is a recommended option that places
> > +		all I2C commands under a single 'i2c' root command.  The 
> > +		older 'imm', 'imd', 'iprobe' etc. commands are considered
> > +		deprecated and may disappear in the future.
> > +
> > +		CONFIG_HARD_I2C selects a hardware I2C controller.
> I guess I am slow abt this.... but !slow_i2c... is that not good enough?
> and I cant see any other part in the the common patch using it...
> > diff --git a/common/cmd_i2c.c b/common/cmd_i2c.c
> > index c543bb5..824f7c7 100644
> > --- a/common/cmd_i2c.c
> > +++ b/common/cmd_i2c.c
> 
> > +int do_i2c_bus_speed(cmd_tbl_t * cmdtp, int flag, int argc, char *argv[])
> > +{
> > +	int speed, ret=0;
> > +
> > +	if (argc == 1)  /* querying current speed */
> > +	{
> > +		printf("Current bus speed=%d\n", i2c_get_bus_speed());
> > +	}
> > +	else
> > +	{
> > +		speed = simple_strtoul(argv[1], NULL, 10);
> > +		printf("Setting bus speed to %d Hz\n", speed);
> > +		ret = i2c_set_bus_speed(speed);
> > +		if(ret)
> > +		{
> > +			printf("Failure changing bus speed (%d)\n", ret);
> > +		}
> > +	}
> > +	return ret;
> > +}
> Err... Does this mean that all other folks who can only support a single
> speed need to implement this api (including all other existing
> drivers??)...... Lotsa people are not going to like this if that is
> so...
> 
> 
> Looks good to me.. it should fit in OMAP2430 framework too :)
> 
> Some one gotta tell Denx  to update this:
> http://sourceforge.net/cvs/?group_id=65938. i thought we moved to git
> sometime back. Do we even look at cvs anymore??? :(
> 
> Regards,
> Nishanth Menon
</snip>

Your U-Boot support team




More information about the U-Boot mailing list