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

esp32h2: default XTAL is 32mhz #1127

Closed
MabezDev opened this issue Jan 30, 2024 · 5 comments
Closed

esp32h2: default XTAL is 32mhz #1127

MabezDev opened this issue Jan 30, 2024 · 5 comments
Labels
bug Something isn't working

Comments

@MabezDev
Copy link
Member

image

This doesn't seem to be causing problems right now, but the fact we don't have a feature for it, in esp-hal or espflash means we were probably not aware of this.

@jessebraham
Copy link
Member

As a side note, which has been raised before (more than once), it's incredibly unclear how to map from the clock diagrams in the TRMs to our clock code. Would be really helpful having some documentation here, as clocks seem to be randomly selected from both the left and right edges of the diagram.

@jessebraham
Copy link
Member

Now that we've eliminated the XTAL frequency features I think we can close this? Unless I'm missing something the H2 seems to be using this frequency now.

@MabezDev please re-open if I am mistaken.

@github-project-automation github-project-automation bot moved this from Todo to Done in esp-rs Feb 28, 2024
@MabezDev
Copy link
Member Author

MabezDev commented Feb 28, 2024

The 32MHZ xtal case is not yet handled so we need to keep this open.

@MabezDev MabezDev reopened this Feb 28, 2024
@github-project-automation github-project-automation bot moved this from Done to In Progress in esp-rs Feb 28, 2024
@jessebraham
Copy link
Member

Can you elaborate please? What actually needs to be done? The XTAL frequency in boot defaults is 32MHz, where else does this need to be changed?

@MabezDev
Copy link
Member Author

I spent most of today trying to remember why I wanted to not forget about this, it was this issue I wanted to test: #766 and I think the xtal was the issue. We can close this now.

@github-project-automation github-project-automation bot moved this from In Progress to Done in esp-rs Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

2 participants