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

Reply via email to