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

drivers/aip31068: initialization fails in most cases when using esp_i2c_sw #17311

Closed
HendrikVE opened this issue Dec 1, 2021 · 5 comments
Closed
Assignees
Labels
Platform: ESP Platform: This PR/issue effects ESP-based platforms Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Comments

@HendrikVE
Copy link
Contributor

HendrikVE commented Dec 1, 2021

Description

There is a 80% chance that the initialization for the driver aip31068 fails when using esp_i2c_sw instead of esp_i2c_hw.

Steps to reproduce the issue

Remove the line USEMODULE += esp_i2c_hw fromaip31068/Makefile.dep and test with real hardware.

Reason for the failing initialization is this line:

return -ENXIO;

Attachment

I used the logic analyzer software from saleae which is available here.
The attached zip-archive contains two files that can be imported to show all I2C operations for the esp_i2c_sw- and esp_i2c_hw-version of the driver. As you can see I had to reset my esp32 4 times before initialization was successful.

@HendrikVE
Copy link
Contributor Author

HendrikVE commented Dec 1, 2021

@gschorcht I thought a new issue is a better place to discuss this problem ;)

@HendrikVE HendrikVE added Platform: ESP Platform: This PR/issue effects ESP-based platforms Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) labels Dec 1, 2021
@gschorcht
Copy link
Contributor

gschorcht commented Dec 1, 2021

Where can I buy this hardware? Only at Aliexpress?

@gschorcht gschorcht self-assigned this Dec 1, 2021
@HendrikVE
Copy link
Contributor Author

I've got mine from dfrobot. This display combines the aip31068 with pca9633 in one component.

@gschorcht
Copy link
Contributor

Ordered, it takes 1 to 3 days. Then I will have a look.

@HendrikVE
Copy link
Contributor Author

Fixed with #17346

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Platform: ESP Platform: This PR/issue effects ESP-based platforms Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

No branches or pull requests

2 participants