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

Add OpenOCD support for onboard ESP32-S2-Kaluga-1 debug tool #1185

Closed
wants to merge 3 commits into from

Conversation

thorrak
Copy link

@thorrak thorrak commented Sep 6, 2023

This PR updates the default OpenOCD version from v0.11 to v0.12 (which adds support for the Kaluga dev board amongst other things) and changes the board definition to refer to the Espressif-provided configuration file.

I've tested this with a Kaluga board locally and can confirm that - although requiring a bit of driver-juggling, similar to other Espressif dev boards - it works.

@thorrak
Copy link
Author

thorrak commented Sep 6, 2023

Separately, I've also submitted a PR for the docs repo to reflect that this board does in fact have an on-board debugger that is compatible with Platformio.

@valeros valeros closed this in a9b9d09 Sep 6, 2023
@valeros
Copy link
Member

valeros commented Sep 6, 2023

The doc page for this board will be automatically updated once the next release of this dev-platform is published.

@thorrak
Copy link
Author

thorrak commented Sep 6, 2023

One note about the code that was merged - will that alsoadd FTDI to the flash protocols? That was a requirement to get it to work when I was testing on my end.

@valeros
Copy link
Member

valeros commented Sep 6, 2023

One note about the code that was merged - will that alsoadd FTDI to the flash protocols?

Yes, FTDI is added, but only for ESP32-S2-Kaluga-1.

@thorrak
Copy link
Author

thorrak commented Sep 6, 2023

Cool. I’ll give the updated version a shot this afternoon and report back.

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