-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[BUG] ESP32: esptool below 4.8 silently produces broken firmware. #13824
Comments
@fdcavalcanti please take a look ^ |
Thanks @acassis, @fdcavalcanti this is quite important issue as it needs to be fixed in 12.7.0 release (it was found during 12.7.0-RC0 testing). |
I believe the proper way to proceed is using the most up-to-date esptool. |
|
@cederom any update ? |
Waiting for a fix proposition from @fdcavalcanti and @almir-okato of Espressif. |
For sure we may leave current state as-is "using the most up-to-date esptool" then people with esptool <4.8 will get broken firmware without warning. Two impacts possible:
The simplest fix is just to enforce latest esptool and error build when older tool is in use. Not elegant nor compatible. The best fix would be to adapt ESP32 bootloader to what esptool version is available (i.e up to -3 minor, or build error). |
@cederom what do you think about adding a config option which is set to 4.8 but can be changed by the user? |
This part should be done automatically based on the available tool version with some log message on what happened (something like: esptool version X.Y implies bootloader version P.Q). Adding version to config will not change anything because that version needs to be checked anyway which is not done right now what causes the problem. Do we know why exactly 4.8 esptool needs to be used for current NuttX master? Are there any specific features in the new bootloader that were not there before and we need them for some reason? Why older esptool creates invalid firmware image silently? @fdcavalcanti @almir-okato I think we should bump this issue on esptool repo too maybe we get more details that way :-) |
Hi @cederom sorry about the delay. We'll set it to latest version and later add a warning for unsupported |
As reported in espressif/esptool#1018 by @radimkarnis:
Looking at the provided information
Previous releases of I have asked about possible alternative solutions for older Can you please prepare this kind of patch @fdcavalcanti @almir-okato ? :-)
Thank you :-) |
@cederom thanks for posting the team's response here on this thread. Its good to keep it all documented. |
backport #14268 |
@cederom please test releases/12.7 and close the bug |
WORKS LIKE A CHARM! THANK YOU @fdcavalcanti @acassis @jerpelea @radimkarnis :-)
Also we have updated esptool system package to 4.8.1 on FreeBSD :-) |
Description / Steps to reproduce the issue
Lets try with the CoreMark:
I found this commit 457f9d3 to update the CI in order to build for ESP32 but manual builds should return error on older esptool versions or the older bootloader should be used? :-)
On which OS does this issue occur?
[OS: BSD]
What is the version of your OS?
13.3-RELEASE-p7 GENERIC amd64
NuttX Version
12.7.0-RC0
Issue Architecture
[Arch: xtensa]
Issue Area
[Area: Build System]
Verification
The text was updated successfully, but these errors were encountered: