Skip to content

Conversation

maidnl
Copy link
Contributor

@maidnl maidnl commented Sep 23, 2025

This PR takes care of issue #492 .
There was a default timeout of 1000 ms in I2C communication.
Timeout has been reduced to 1ms.
However (although this is not a standard Wire function) a setTimeout() function has been added to Wire class so that users can tune the value.


void setBusStatus(WireStatus_t);
/* set timeout in ms for I2C communication */
void setTimeout(unsigned int t);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@per1234 per1234 added the topic: code Related to content of the project itself label Sep 23, 2025
@maidnl
Copy link
Contributor Author

maidnl commented Sep 24, 2025

@per1234 I renamed the function according to the request, however please note that in the current implementation of Wire (at least in Core-Renesas) the Wire class is derived from HardwareI2C class, that is derived from Stream. And Stream has already setTimeout function.
void setTimeout(unsigned long timeout); // sets maximum milliseconds to wait for stream data, default is 1 second
In my opinion it is better to use setTimeout name which is in line with the current class derivation scheme.

@per1234
Copy link
Collaborator

per1234 commented Sep 25, 2025

Stream has already setTimeout function.

That function has a different purpose from what is being implemented here.

Stream::setTimeout sets the timeout for the high level blocking functions of the Stream class (e.g., readBytes).

The function implemented here is setting a timeout for the low level communication. This is the reason why the function was intentionally given a distinct name in the AVR library, rather than shadowing the Stream class member, which would have made it impossible to set the Stream timeout.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

I2C scan takes extremely long (~126 seconds) with Wire.endTransmission() on empty bus
2 participants