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

Wire.readBytes #617

Closed
TheFidax opened this issue Nov 1, 2021 · 5 comments
Closed

Wire.readBytes #617

TheFidax opened this issue Nov 1, 2021 · 5 comments

Comments

@TheFidax
Copy link

TheFidax commented Nov 1, 2021

Is possible add readBytes function in Wire library?
Code tested and working correctly with Arduino Core (UNO, Nano), Mini Core(ATmega328), Mega Core(ATmega128), with ATtiny841 has this error:

error: 'class TwoWire' has no member named 'readBytes'

@SpenceKonde SpenceKonde added this to the 2.0.0 release milestone Nov 5, 2021
@SpenceKonde
Copy link
Owner

stream looks like it needs to be updated. (Wire inherits this from Stream)

@TheFidax
Copy link
Author

In 2.0.0 release, is possible add another function not implemented: Wire.setWireTimeout() ?
Documentations are available here: arduino/reference-en#895

@SpenceKonde
Copy link
Owner

All parts except the 441, 841, and 828 will get that i 2,.0.0 when #618 is merged in. I2C master on these parts is hideous kludge becuse there;sno support whatsioever for I2C master in hardware.

@SpenceKonde
Copy link
Owner

Actually, it turrns out that arduino/ArduinoCore-avr#226 predates extensive changes to the wire library and I cannot merge it into 2.0.0. Sorry.

@SpenceKonde
Copy link
Owner

Status: Wire.readBytes should work in 2.0.0-dev, Wire.setTimeout requires attention from an expert to implement in the 2.0.0 codebase since arduino/ArduinoCore-avr#226 could not be merged. Closing this issue as the title issue is solved and the second issue is not something I intend to take on. I already have refactored that monstrosity twice for 2.0.0-dev. (which is why the PR which was submitted against the dead 1.5.x branch instead of the current development branch couldn't be merged)

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

No branches or pull requests

2 participants