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

Incorrect monitoring in 26MHz targets #582

Closed
SergioGasquez opened this issue Feb 14, 2024 · 0 comments · Fixed by #584
Closed

Incorrect monitoring in 26MHz targets #582

SergioGasquez opened this issue Feb 14, 2024 · 0 comments · Fixed by #584
Labels
bug Something isn't working
Milestone

Comments

@SergioGasquez
Copy link
Member

SergioGasquez commented Feb 14, 2024

After merging esp-rs/esp-hal#1165, I've regenerated C2 stub, with dprint feature enabled, stub is attached. Using this stub, I can flash both a 26MHz C2 and a 40MHz C2, but there are still a few issues:

  • I can't properly monitor a 26MHz C2 (baudrate is not correct, hence I only see noise)
    • Tested using different baudrates in espflash (115_200)
    • Also tried using screen, same result.
  • espflash reports 26MHz C2 as 40MHz

If I use a C stub, I cant flash a 26MHz C2 as espflash does not detect the OHAI message, which results in a timeout, this probably happens because we are using an incorrect baudrate and we cant decode the message.

Using --no-stub, everything works fine.

This is the branch where I am conducting the tests: https://github.com/SergioGasquez/espflash/tree/fix/flash-26mhz

stub_flasher_32c2.zip

Update: I've been debugging this morning with @bjoernQ and we found the part of issue, we were using APB clock as source for UART, we changed it, regenerated the stub, and now espflash detects the xtal frequency properly. Here is the updated stub:

stub_flasher_32c2.zip

@MabezDev MabezDev added this to the v3 milestone Feb 14, 2024
@MabezDev MabezDev added the bug Something isn't working label Feb 14, 2024
@SergioGasquez SergioGasquez linked a pull request Feb 15, 2024 that will close this issue
@github-project-automation github-project-automation bot moved this from Todo to Done in esp-rs Feb 16, 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

Successfully merging a pull request may close this issue.

2 participants