Skip to content

Conversation

SergioGasquez
Copy link
Member

@SergioGasquez SergioGasquez commented May 15, 2023

  • Adds the read_efuse example. Example result:
MAC address [60, 55, f9, f7, 2c, 4d]
Flash Encryption false
  • Adds initial support for I2C and the related examples:

Tested the example but couldn't fully verify them as I don't have the proper HW. I will open it as a draft until we can fully verify it.

embassy_i2c

I was able to check that it tries to connect with the accelerometer, as it uses it alternate address, 0x19:
embassy_i2c

i2c_bmp180_calibration_data

I was able to check that it writes the 0x77:
i2c_bmp180_calibration_data

i2c_display

I was able to check that it tries to connect with the display, as it uses it usual address, 0x3c:
i2c_display

Copy link
Contributor

@bjoernQ bjoernQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

I was able to verify the examples (besidesembassy_i2c.rs) on hardware

@SergioGasquez SergioGasquez marked this pull request as ready for review May 15, 2023 14:18
@SergioGasquez SergioGasquez merged commit e2442f2 into esp-rs:main May 15, 2023
@SergioGasquez SergioGasquez deleted the feature/esp32h2-i2c branch May 15, 2023 14:20
@jessebraham
Copy link
Member

I'm just waiting on some ESP32-H2 devkits to be shipped to me, so once they arrive I will verify the embassy_i2c example and fix it if necessary.

SergioGasquez added a commit to SergioGasquez/esp-hal that referenced this pull request Jun 9, 2023
* feat: ✨ Enable i2c peripheral

* feat: ✨ Add I2cExt1 for H2

* feat: ✨ Initial i2c support

* feat: ✨ Add i2c examples

* ci: ✨ Add embassy_i2c check

* ci: 🐛 Fix features

* docs: 📝 Update changelog

* feat: ✨ Add read_efuse example
i404788 pushed a commit to i404788/esp-hal that referenced this pull request Jul 22, 2023
* feat: ✨ Enable i2c peripheral

* feat: ✨ Add I2cExt1 for H2

* feat: ✨ Initial i2c support

* feat: ✨ Add i2c examples

* ci: ✨ Add embassy_i2c check

* ci: 🐛 Fix features

* docs: 📝 Update changelog

* feat: ✨ Add read_efuse example
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