Skip to content

Conversation

berkutta
Copy link
Contributor

When using the i2c_master_write function without this patch with data length of 1 like used in the Espressif example code it sends 0x00. https://github.com/espressif/ESP8266_RTOS_SDK/blob/master/examples/peripherals/i2c/main/user_main.c#L202

@CLAassistant
Copy link

CLAassistant commented Oct 16, 2018

CLA assistant check
All committers have signed the CLA.

@xiongyumail
Copy link
Collaborator

Hi @berkutta

Is there any problem with example, or is your demand not satisfied?

@berkutta
Copy link
Contributor Author

Hi @xiongyumail

Both. You can check with a logic analyzer. But the patch in this Pull Request fixes the Bug.

@xiongyumail
Copy link
Collaborator

Using NULL instead 0 will be better.

@berkutta berkutta force-pushed the bugfix/sending_single_byte branch from 46e530b to 61997f1 Compare October 17, 2018 09:06
@berkutta
Copy link
Contributor Author

Ok, thanks, good point. I have changed it now to NULL.

@xiongyumail xiongyumail merged commit c7e0d5a into espressif:master Oct 17, 2018
@xiongyumail
Copy link
Collaborator

It will be updated later.

wujiangang pushed a commit that referenced this pull request Oct 18, 2018
Signed-off-by: XiongYu <xiongyu@espressif.com>

Merges #343
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

Successfully merging this pull request may close these issues.

3 participants