On 8 February 2017 at 06:02, Simon Glass <s...@chromium.org> wrote: > On 12 January 2017 at 10:47, Moritz Fischer <moritz.fisc...@ettus.com> wrote: >> >> Replace dm_i2c_write() / dm_i2c_read() with transaction using >> struct i2c_msg[2] in order to allow for i2c controller to detect >> write/read cycle to emit a repeated start condition. >> >> Signed-off-by: Moritz Fischer <moritz.fisc...@ettus.com> >> Cc: Simon Glass <s...@chromium.org> >> Cc: u-boot@lists.denx.de >> --- >> Hi Simon, >> >> since I don't have a v2 protocol EC I couldn't test this, >> but I noticed when playing around with a v3 one, that it would simply ignore >> the packets if they don't have a repeated start condition (which from my >> understanding) >> you can *only* get if you use struct i2c_msg, since otherwise the driver has >> no way of >> knowing your intention. >> >> Do you have a v2 board around to test this? >> >> Cheers, >> >> Moritz >> --- >> drivers/misc/cros_ec_i2c.c | 26 +++++++++++++++----------- >> 1 file changed, 15 insertions(+), 11 deletions(-) > > Acked-by: Simon Glass <s...@chromium.org> > Tested on snow: > Tested-by: Simon Glass <s...@chromium.org>
Applied to u-boot-dm, thanks! _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
