
26 Nov
2014
26 Nov
'14
8:43 p.m.
+Heiko
Hi,
On 26 November 2014 at 12:37, Lubomir Popov lpopov@mm-sol.com wrote:
Hi Simon,
[snip]
while (length-- > 0) {
if (i2c_write(chip, devaddr++, alen, memaddr++, 1) != 0)
return i2c_report_err(-1, I2C_ERR_WRITE);
/*
- No write delay with FRAM devices.
*/
Should this be indented?
#if !defined(CONFIG_SYS_I2C_FRAM)
udelay(11000);
udelay(11000);
#endif
}
[snip]
Which indent do you mean? The udelay() is within the while loop, so it should. The comment above your note - I kept it as is, aligned with the #if/#endif.
It was hard to tell in my emailed, but yes i see, you have kept it where it was, which is fine.
I added Heiko (I2C maintainer) to cc. As mentioned, but in more detail, with driver model 'i2c flags 2' will select this behaviour for any i2c access, not just for this command. See u-boot-dm/i2c-working2 file i2c-uclass.c.
Regards, Simon