Feature Request: Arduino bugfix v.2.5.1 is released #147

Jason2866 · 2019-05-11T14:59:18Z

Since v.2.5.1 is released could you please update to this version?
Thx!

ivankravets · 2019-05-13T16:11:04Z

@valeros could you investigate why does not work the latest source code v2.5.1 of Arduino core? it builds OK but does not work after flashing.

@d-a-v, @earlephilhower do we have critical changes in 2.5.1 build process? Do we need to upgrade to the latest ESP8266 toolchain and switch off from esptool-ck to esptool-py? Thanks!

earlephilhower · 2019-05-13T16:20:34Z

@ivankravets There was a linking change (.text into .text and .text1 segment to allow moving some specific functions to IRAM as needed for IRQ callbacks) but I think we already got you in the loop on it.

esp8266/Arduino#5922

Jason2866 · 2019-05-13T16:24:21Z

@ivankravets @valeros
I get a working build with old and new xtensa build chain. No problem with arduino core.
You can take a look here:
platform = https://github.com/Jason2866/platform-espressif8266.git#Stage-Tasmota
for new extensa build chain platformio.py has to be adapted (linked files has to be reordered)
https://github.com/Jason2866/TDM/blob/master/platformio.py
and old Xtensa build chain
platform = https://github.com/Jason2866/platform-espressif8266.git#Tasmota

Core 2.5.1 is used in Project Tasmota with Platformio. You can try online:
Tasmota via gitpod

mcspr · 2019-05-13T17:05:57Z

Just to make clear what "linked files has to be reordered" means, see
Core 2.5.0 / platform 2.0.0 issue: #128 (comment) (and further comments)
workaround for that: mcspr/platformio-core@708eff2

ivankravets · 2019-05-13T17:07:19Z

@mcspr why do you need this hook mcspr/platformio-core@708eff2 ?

Jason2866 · 2019-05-13T17:14:46Z

Without this hook, code doesnt boot when builded with new xtensa build chain.
@mcspr thx for you fix, searched the orig. issue and didnt found

mcspr · 2019-05-13T17:28:30Z

Yes, like @Jason2866 said - updated toolchain seemingly treats -Wl,-T and just -T differently.
esp8266/Arduino never encountered that, because platform.txt / arduino-builder only uses -T{user_selected_ldscript}

ivankravets · 2019-05-13T18:04:06Z

@Jason2866 @mcspr this is a bug with ESP8266 toolchain. Please file an issue here https://github.com/earlephilhower/esp-quick-toolchain and I'm sure @earlephilhower will take a look.

See GCC docs https://gcc.gnu.org/onlinedocs/gcc/Link-Options.html

-Wl,option
Pass option as an option to the linker. If option contains commas, it is split into multiple options at the commas. You can use this syntax to pass an argument to the option. For example, -Wl,-Map,output.map passes -Map output.map to the linker. When using the GNU linker, you can also get the same effect with -Wl,-Map=output.map.

PlatformIO Core is used for over 30 dev/platform which depend on the different toolchains. It would be great if we can keep PlatformIO Core compatible with GCC CLI.

earlephilhower · 2019-05-13T18:14:00Z

Guys, the toolchain got bumped in 2.5.0 and was not touched significantly for 2.5.1 AFAIR. Was 2.5.0 working?

If so, then I really think it's something else because we're building the unadulterated GNU sources for the vast majority of things (minor patches, unrelated to the linker).

Just checked, the last time the toolchain was released was Dec 14 2018. Ardui9no 8266 2.5.0 came out in Feb 2019.

Jason2866 · 2019-05-13T18:18:11Z

@earlephilhower the toolchain from 2.5.0 is working fine for 2.5.1 with this change from @mcspr

ivankravets · 2019-05-13T18:19:01Z

@earlephilhower the bug was introduced in this release https://github.com/earlephilhower/esp-quick-toolchain/releases/tag/2.5.0-3

We use a toolchain that was before "2.5.0-3". The latest "2.5.0-3" breaks ESP8266 SDKs which we support in PlatformIO.

earlephilhower · 2019-05-13T18:21:12Z

@ivankravets what is the exact version you have been using? Again, we are building the plain GCC code here so there's not much I expect to be able to track down. Patches are kept in a separate dir in the repo, and none relate to the g++/ld options parsing...

Jason2866 · 2019-05-13T18:21:14Z

@ivankravets anyway, with the old build chain (which is standard for Arduino ESP8266) the code compiles fine and does work.
platform = https://github.com/Jason2866/platform-espressif8266.git#Tasmota

earlephilhower · 2019-05-13T18:29:26Z

Also, does PIO use its own bootloader on-chip? Like I said, there is another segment in the generated binary now (iram.text1) which needs to be supported. The Arduino bootloader handles it seemlessly (w/o any changes), but if you've got a custom one it needs to be aware of this and support it.

PIO uses GCC in a way that causes the compiler to optimize away the IRQ vectors (since they're not used in our code itself) in the final build stage. We can avoid this by forcing all of libmain to be included in the final object. This is a small file, and almost completely related to IRQs, so space impact is minimal. Fixes esp8266#6087 and platformio/platform-espressif8266#147

octavio2895 · 2019-05-13T20:23:44Z

Just to add my few cents. I'm having some issues with OTA updates using the stable release. It appears that espota thinks its done sending the sketch before the fact and I get an Error[4]: End Failed on the serial output. The sketch gets to about 89% done uploading before it fails. I tried using the 2.5.0 release of the core in the arduino IDE and it works great, so I don't think the core is the problem. After changing the platform using #147 (comment) the uploads completes most of the times. However, when I switched to an AP with a stronger signal, the fail rate drops with stable and with #147 (comment) .

Does OTA update work differently in platformio from arduino ide?

I wasn't sure if I should open a new issue, so apologies if I should have.

ivankravets · 2019-05-13T21:15:38Z

@octavio2895 I've just switch espota to official script from Arduino core. Please re-test with upstream version http://docs.platformio.org/en/latest/platforms/espressif8266.html#upstream

ivankravets · 2019-05-13T21:18:05Z

@earlephilhower @Jason2866 @mcspr

I've just switch ElfToBin builder and upload process to esptool.py:

It works for me now. Please re-test with upstream version of dev/platform http://docs.platformio.org/en/latest/platforms/espressif8266.html#upstream

if it works for you too, we will make a final release.

Thanks!

earlephilhower · 2019-05-13T22:08:51Z

That commit uses the new elf2bin.py script which knows about the add'l .text1 section. If that really works, then the -Wl was a red herring and we can ignore it in the core.

octavio2895 · 2019-05-13T23:07:55Z

@ivankravets that fixed the issue! where can I find the version of espota.py that was being used before? I would like to take a look at it.

Also, should I make an issue for this? since its not fixed on the stable release to help people with similar problems?

mcspr · 2019-05-14T02:36:58Z

Can confirm that elf2bin change is working with platform from develop branch.

platform.json:["packages"]["toolchain-xtensa"]["version"] is still pointing to the ~1.40802.0, so toolchain problem obviously is not there.
Original issue is still reproducible with the ~2.40802.0 (toolchain package version comes from platfromio's package repo. it is the same as from esp-quick-toolchain 2.5.0-3 release, same build-id shas and everything)

Jason2866 · 2019-05-14T07:26:47Z

Can confirm develop is working. Feature request with this issue is done. Thx.

Issue with toolchain ~2.4 is still there (when used) as @mcspr mentioned.

The develop version uses ~1.4 build chain:

PlatformManager: Installing platform-espressif8266
git version 2.19.1.windows.1
Cloning into 'C:\Users\Hans\.platformio\platforms\_tmp_installing-jnoeh3-package'...
remote: Enumerating objects: 95, done.
remote: Counting objects: 100% (95/95), done.
remote: Compressing objects: 100% (74/74), done.
remote: Total 95 (delta 50), reused 34 (delta 10), pack-reused 0
Unpacking objects: 100% (95/95), done.
espressif8266 @ e7d8b22 has been successfully installed!
The platform 'https://github.com/platformio/platform-espressif8266.git#develop' has been successfully installed!
The rest of packages will be installed automatically depending on your build environment.
PackageManager: Installing toolchain-xtensa @ ~1.40802.0
Downloading  [####################################]  100%
Unpacking  [####################################]  100%
PackageManager: Installing tool-esptool @ <2
Downloading  [####################################]  100%
Unpacking  [####################################]  100%
PackageManager: Installing tool-esptoolpy @ ~1.20600.0
Downloading  [####################################]  100%
Unpacking  [####################################]  100%
PackageManager: Installing framework-arduinoespressif8266 @ ~2.20501.0
Downloading  [####################################]  100%
Unpacking  [####################################]  100%

ivankravets · 2019-05-15T11:03:45Z

Please update to the stable version of dev/platform https://github.com/platformio/platform-espressif8266/releases/tag/v2.1.0

Jason2866 · 2019-05-15T12:43:52Z

Release does install and compile results in a working firmware of tried project Tasmota

ivankravets assigned valeros May 13, 2019

ivankravets added the enhancement label May 13, 2019

Jason2866 mentioned this issue May 13, 2019

IRQ vectors not being set in PIO toolchain, causing bootloop w/2.5.x releases esp8266/Arduino#6087

Closed

earlephilhower mentioned this issue May 13, 2019

Fix PIO toolchain by ensuring IRQ vectors in link esp8266/Arduino#6088

Closed

ivankravets closed this as completed in ec1a8a1 May 13, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature Request: Arduino bugfix v.2.5.1 is released #147

Feature Request: Arduino bugfix v.2.5.1 is released #147

Jason2866 commented May 11, 2019

ivankravets commented May 13, 2019

earlephilhower commented May 13, 2019

Jason2866 commented May 13, 2019 •

edited

Loading

mcspr commented May 13, 2019

ivankravets commented May 13, 2019

Jason2866 commented May 13, 2019 •

edited

Loading

mcspr commented May 13, 2019

ivankravets commented May 13, 2019

earlephilhower commented May 13, 2019 •

edited

Loading

Jason2866 commented May 13, 2019

ivankravets commented May 13, 2019

earlephilhower commented May 13, 2019

Jason2866 commented May 13, 2019

earlephilhower commented May 13, 2019

octavio2895 commented May 13, 2019

ivankravets commented May 13, 2019

ivankravets commented May 13, 2019

earlephilhower commented May 13, 2019

octavio2895 commented May 13, 2019

mcspr commented May 14, 2019

Jason2866 commented May 14, 2019

ivankravets commented May 15, 2019

Jason2866 commented May 15, 2019

Feature Request: Arduino bugfix v.2.5.1 is released #147

Feature Request: Arduino bugfix v.2.5.1 is released #147

Comments

Jason2866 commented May 11, 2019

ivankravets commented May 13, 2019

earlephilhower commented May 13, 2019

Jason2866 commented May 13, 2019 • edited Loading

mcspr commented May 13, 2019

ivankravets commented May 13, 2019

Jason2866 commented May 13, 2019 • edited Loading

mcspr commented May 13, 2019

ivankravets commented May 13, 2019

earlephilhower commented May 13, 2019 • edited Loading

Jason2866 commented May 13, 2019

ivankravets commented May 13, 2019

earlephilhower commented May 13, 2019

Jason2866 commented May 13, 2019

earlephilhower commented May 13, 2019

octavio2895 commented May 13, 2019

ivankravets commented May 13, 2019

ivankravets commented May 13, 2019

earlephilhower commented May 13, 2019

octavio2895 commented May 13, 2019

mcspr commented May 14, 2019

Jason2866 commented May 14, 2019

ivankravets commented May 15, 2019

Jason2866 commented May 15, 2019

Jason2866 commented May 13, 2019 •

edited

Loading

Jason2866 commented May 13, 2019 •

edited

Loading

earlephilhower commented May 13, 2019 •

edited

Loading