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

Added installation guide for recommended stable gcc version #285

Merged

Conversation

supercrazysam
Copy link
Contributor

@supercrazysam supercrazysam commented Apr 12, 2021

File changed: README.md

Added the installation guide for recommended gcc version.
Established the suggested gcc version gcc-arm-none-eabi-7-2018-q2 for firmware development, as recommended by Benjamin.

==========
Considering that there are already multiple reports of newer version of gcc not working properly. A tested stable working version of gcc should be explicitly stated in the setup section in README.md to help newcomers, alongside a guide to describe how install it.

I tested the installation method mentioned in #233(Benjamin's method) on a new machine, compiled the firmware (in both .hex and .bin) and it was successful.

==========
Reference:
#210 => People encountered problems when using newer gcc version.
#210 => The last working version that is available for download was gcc-arm-none-eabi-7-2018-q2
GNU Arm Embedded Toolchain: 7-2018-q2-update June 27, 2018

All version after this, starting with the "GNU Arm Embedded Toolchain: 8-2018-q4-major December 20, 2018" were not working. After compiling and flashing the firmware the VESC was in a rebooting loop that was noticeable when looking at the LEDs.

#233 => Instructions from Benjamin for setting up gcc-arm-none-eabi-7-2018-q2
#278 => The tested working version of gcc should be explicitly listed out for vesc fw development.

Sam

@supercrazysam supercrazysam changed the title Added the installation guide for recommended stable gcc version Added installation guide for recommended stable gcc version Apr 12, 2021
@vedderb vedderb merged commit 31a3131 into vedderb:master Apr 12, 2021
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