Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

I2C_TRANSFER description is inaccurate #480

Closed
tadziopazur opened this issue Aug 30, 2017 · 1 comment
Closed

I2C_TRANSFER description is inaccurate #480

tadziopazur opened this issue Aug 30, 2017 · 1 comment

Comments

@tadziopazur
Copy link

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.

  1. Start/stop depends on sequence length/message length (it should not)
  2. 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
Copy link
Contributor

rzr commented Nov 9, 2018

May relate to:
#322

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants