Skip to content

Wait for I2C power to stabilize on Heltec VME213; tidy variant folder - #4308

Merged
thebentern merged 2 commits into
masterfrom
heltec-vision-master
Jul 20, 2024
Merged

Wait for I2C power to stabilize on Heltec VME213; tidy variant folder#4308
thebentern merged 2 commits into
masterfrom
heltec-vision-master

Conversation

@todd-herbert

@todd-herbert todd-herbert commented Jul 20, 2024

Copy link
Copy Markdown
Contributor

Heltec Vision Master 213 has a QuickLink-type I2C connector
image

The 3.3V pin of connector is fed from Ve_3V3 (schematic). This supply is not enabled until partway through setup(), followed soon after by I2C device scan.

This causes issues with CardKB detection, and may also impact other I2C devices. This PR adds a one second delay for Heltec Vision Master E213 after enabling peripheral power.

@HarukiToreda found in testing that a 500ms delay was insufficient.


Note: PR also tidies the variant.h, in line with #4226. It may be helpful to view the two commits separately, to make better sense of the diff.

@todd-herbert todd-herbert changed the title Wait for I2C power to stabilize on Heltec VME213 Wait for I2C power to stabilize on Heltec VME213; tidy variant folder Jul 20, 2024
The 3.3V power for the I2C "quick link" connector is from Ve_3V3
@todd-herbert
todd-herbert force-pushed the heltec-vision-master branch from 12d4dc5 to f1a351a Compare July 20, 2024 10:51
@thebentern
thebentern merged commit 54df153 into master Jul 20, 2024
@thebentern
thebentern deleted the heltec-vision-master branch July 20, 2024 11:46
jeek pushed a commit to jeek/Meshtastic-Exploiteers-Hacker-Pager that referenced this pull request Jun 30, 2026
…meshtastic#4308)

* Tidy variant.h and pins_arduino.h (VME213)

* Wait for peripherals to stabilize after enabling I2C power
The 3.3V power for the I2C "quick link" connector is from Ve_3V3
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.

2 participants