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 "stcgal_protocol" field #16

Merged
merged 1 commit into from
Jan 23, 2019
Merged

Conversation

SyunSiu
Copy link
Contributor

@SyunSiu SyunSiu commented Jan 23, 2019

Add "stcgal_protocol" field in boards/stcxxxxxx.json and use this field in builder/main.py for using stcgal to upload firmware onto STC MCUs in the right way. If we don't explicitly specify which protocol stcgal will use with flag "-P", we'll get stuck in "Cycling power", because stcgal does not always select the correct protocol by autodetection.

The mapping between protocols and MCU series in stcgal is shown at:
https://github.com/grigorig/stcgal/blob/master/doc/USAGE.md#protocols

Compatibly, users can override default Generic STCxxxxxx stcgal upload_protocol using board_upload.stcgal_protocol option. For example, board_upload.stcgal_protocol = "stc12", board_upload.stcgal_protocol = "stc8", etc.

…ld in builder/main.py for using stcgal to upload firmware onto STC MCUs in the right way. If we don't explicitly specify which protocol stcgal will use with flag "-P", we'll get stuck in "Cycling power", because stcgal does not always select the correct protocol by autodetection.
@ivankravets ivankravets merged commit fe2642e into platformio:develop Jan 23, 2019
@ivankravets
Copy link
Member

Thanks!

ivankravets added a commit that referenced this pull request Jan 23, 2019
…this field in builder/main.py for using stcgal to upload firmware onto STC MCUs in the right way. If we don't explicitly specify which protocol stcgal will use with flag "-P", we'll get stuck in "Cycling power", because stcgal does not always select the correct protocol by autodetection. (#16)"

This reverts commit fe2642e.
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