Flash size as megabytes

SuperHouse / esp-open-rtos

Open source FreeRTOS-based ESP8266 software framework

BSD 3-Clause "New" or "Revised" License

1.53k stars 491 forks source link

Flash size as megabytes #536

Open ckerkhoff opened 6 years ago

ckerkhoff commented 6 years ago

When building I noticed the warning, "WARNING: Flash size arguments in megabits like '16m' are deprecated." This should take care of that.

ckerkhoff commented 6 years ago

It looks like the CI build server is using a esptool version older than Aug 28 2016, which is when they started accepting megabytes for the flash size argument.

nochkin commented 6 years ago

The esptool is a part of esp-open-sdk which is being built for esp-open-rtos: https://github.com/SuperHouse/esp-open-rtos/blob/master/utils/travis_build/install_toolchain.sh In order to support your patch, esp-open-sdk needs esptool updated.

P.S.: I'm using external esptool which supports the appropriate arguments. The same way as you do probably.