[U-Boot] [PATCH v3 04/23] i2c: Add TPS6586X driver
Stephen Warren
swarren at wwwdotorg.org
Mon Apr 9 23:01:41 CEST 2012
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 <sjg at 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.
More information about the U-Boot
mailing list