-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
changed SOC_I2C_NUM to SOC_HP_I2C_NUM #10453
Conversation
👋 Hello sivar2311, we appreciate your contribution to this project! Click to see more instructions ...
Review and merge process you can expect ...
|
…_NUM (#10408) The ESP32, ESP32-S and ESP32-H series have two I2C interfaces, while the ESP32-C series has only one.
Memory usage test (comparing PR against master branch)The table below shows the summary of memory usage change (decrease - increase) in bytes and percentage for each target.
Click to expand the detailed deltas report [usage change in BYTES]
|
@sivar2311 for master based on IDF 5.1.x it is fine as it is. For branch release/v3.1 it is already fixed. |
This fix is not needed for master and it is already applied in the |
Description of Change
Changed
SOC_I2C_NUM
toSOC_HP_I2C_NUM
as suggested by Jason2866 and commented by igrrTests scenarios
none
Related links
5d873c0#commitcomment-147823189