I2C_TRANSFER description is inaccurate - Githubissues

Samsung / TizenRT

TizenRT is a lightweight RTOS-based platform to support low-end IoT devices

Apache License 2.0

567 stars 570 forks source link

I2C_TRANSFER description is inaccurate #480

Closed tadziopazur closed 4 years ago

tadziopazur commented 7 years ago

The documentation for I2C_TRANSFER (include/tinyara/i2c.h) states:

Perform a sequence of I2C transfers, each transfer is started with a
START and the final transfer is completed with a STOP. Each sequence
will be an 'atomic' operation in the sense that any other I2C actions
will be serialized and pend until this read completes. Optional.

Since S5J implements it, it should follow the specification. This one doesn't.

Start/stop depends on sequence length/message length (it should not)
try_address does not care for HW errors (hsi2c_stop/hsic_start call hsi2c_wait_xfer_done and do not check the return value), result being multiple start/stop commands get issued

rzr commented 5 years ago

May relate to: https://github.com/Samsung/TizenRT/issues/322