On 04/02/2012 05:18 PM, Simon Glass wrote: > This power management chip supports battery charging and a large number > of power supplies. This initial driver only provides the ability to adjust > the two synchronous buck converters SM0 and SM1 in a stepwise manner. > > Signed-off-by: Simon Glass <s...@chromium.org>
> +#define MAX_I2C_RETRY 3 > +int tps6586x_read(int reg) ... > + for (i = 0; i < MAX_I2C_RETRY; ++i) { > + if (!i2c_read(I2C_ADDRESS, reg, 1, &data, 1)) { > + retval = (int)data; > + goto exit; > + } > + > + /* i2c access failed, retry */ > + udelay(100); > + } Why do we need this retry logic; the kernel driver doesn't appear to have this. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot