[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