Skip to content

Conversation

@ESP32DE
Copy link

see #1061
here a tested version and an optional suggestion

see espressif#1061 here a tested version and an optional suggestion
@ESP32DE
Copy link
Author

note:
i tested the patch with an earlier version successful
if you use the actually version you must use the option
to make the pin HIGH before you setup the pin as OPENDRAIN
( see the comment in the code )

so perhabs would be better, if we use the optional version then,
cause i think, there is nobody using an earlier version ?

just in time tested with the actually version too now
i did use the optional and set Pin Level HIGH before i set it as OPENDRAIN
successful too.

@ESP32DE
Copy link
Author

i will close this PR cause this is only work in a prev version
we must use the optional in the actually version to kick the glitch on sda and scl on wire.begin().
so you will get asap the optional for better merging :)

@ESP32DEESP32DE closed this Feb 4, 2018
@ESP32DE
Copy link
Author

see PR #1065
i added the optional to this.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@ESP32DE