-
Notifications
You must be signed in to change notification settings - Fork 2k
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
cpu/esp*: use FLASHFILE for esp32 and esp8266 boards #11648
Conversation
Use ELFFILE as FLASHFILE as all files are created from this one using esptool.
Use ELFFILE as FLASHFILE as all files are created from this one using esptool.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on the esp32-wroom-32 and esp8266-12x. I flashed the default. Then I tried to flash test/leds with FLASHFILE set to the default elf file and it flash the default as expected.
I think it is OK, ACK.
I would not calling "Cleanup" it is updating to a new feature there. |
As long as it is something! |
*sometype |
Thank you for the review. |
Contribution description
Update to use FLASHFILE as file to be flashed on the board.
By using
FLASHFILE=path_to_file BOARD=espXXXX make flash-only
a board can be flashed from a machine without the toolchain and only pip installedesptool.py
installedTesting procedure
Both esp32 and esp8266 can still be flashed as in master.
The
flash-only
output is the same right now as in masterESPTOOL=esptool.py BOARD=esp32-wroom-32 make --no-print-directory -C examples/default/ flash-only
BOARD=esp8266-esp-12x make --no-print-directory -C examples/default/ flash-only
The file to flash can be configured from the environment using
FLASHFILE
.Remove all other files in
bin
directory than thedefault.elf
and this works:Issues/PRs references
Part of the FLASHFILE introduction #8838
Required for #11646
Real way of setting FLASHFILE that would be useful for #11449