PX4-Autopilot icon indicating copy to clipboard operation
PX4-Autopilot copied to clipboard

ist8310: Add I2C communication error handling

Open muramura opened this issue 1 year ago • 3 comments

Solved Problem

There is no I2C communication error handling. I2C communication is vulnerable to external noise.

Solution

Add I2X communication error processing.

Changelog Entry

None

Alternatives

None

Test coverage

None

Context

Screenshot from 2024-03-31 14-02-14 Screenshot from 2024-03-31 14-01-28 Screenshot from 2024-03-31 14-00-41

muramura avatar Mar 31 '24 05:03 muramura

This is already implemented lower level in our I2C class, take a look for _retries. https://github.com/PX4/PX4-Autopilot/blob/main/src/lib/drivers/device/nuttx/I2C.cpp

dagar avatar Apr 01 '24 16:04 dagar

@dagar san. I have added a retransmission count setting in the constructor.

muramura avatar Apr 03 '24 12:04 muramura

@dagar do you think this is necessary?

ischollETH avatar May 23 '24 14:05 ischollETH