search

esp-rs / esp-flasher-stub

Rust implementation of flasher stub located in esptool
Apache License 2.0
18 stars 10 forks source link

Add support for ESP8266 #54

Closed dobairoland closed 4 months ago

SergioGasquez commented 5 months ago

See https://github.com/esp-rs/esp-flasher-stub/issues/36

dobairoland commented 5 months ago

To the best of my knowledge the goal is to use esp-flasher-stub in all Espressif SW products where stub flasher is used, including esptool. However, if esp-flasher-stub won't support ESP8266 then we will have to keep maintaining two flasher stubs which kills the main idea of to have just one flasher stub.

jessebraham commented 5 months ago

To the best of my knowledge the goal is to use esp-flasher-stub in all Espressif SW products where stub flasher is used, including esptool.

This is news to me, it would be nice if this could be better communicated. I was told that it might be used in one or two flashing applications ~2 year ago, and we have received little to no feedback since, and definitely no assistance with developing the stub. I don't know that any projects other than espflash have even tried using the stub, I have not been made aware of this if it is the case.

However, if esp-flasher-stub won't support ESP8266 then we will have to keep maintaining two flasher stubs which kills the main idea of to have just one flasher stub.

I empathize with this, but I don't understand how we're expected to add ESP8266 support when we don't even have a working HAL for it. We have a small team and already we barely have the resources to keep up with the projects we already have. The ESP8266 is EOL in ~2 years and this does not feel like a good use of our time.

Maybe somebody from another team or the community can step up and make this happen if it is a priority.

dobairoland commented 4 months ago

We will make efforts to keep using the old flasher stub for ESP8266. So this can be closed...

Thank you for your comments.