Skip to content
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

Discussion: Drop support for espressif bootloader? #971

Closed
tius2000 opened this issue Feb 20, 2017 · 15 comments
Closed

Discussion: Drop support for espressif bootloader? #971

tius2000 opened this issue Feb 20, 2017 · 15 comments
Labels
Milestone

Comments

@tius2000
Copy link
Contributor

Are there any advantages of the espressif bootloader? Would it make sense to to support rboot only to simplify the framework?

@slaff
Copy link
Contributor

slaff commented Feb 20, 2017

If no one really insists on supporting Espressif's bootloader we can go even further:

  • If bootloader is needed then only rBoot will be used
  • We can offer also images without bootloader, thus freeing space on Flash and memory.

@ADiea
Copy link
Contributor

ADiea commented Feb 20, 2017

+1 to drop support (although not used rboot yet, but if you force me...i'll have to learn :) )

@slaff
Copy link
Contributor

slaff commented Feb 21, 2017

Moving to only one bootloader, namely the superb rBoot, will help us get rid of:

  • Different LD scripts
  • Different project makefiles
  • Different HttpFirmwareUpdate classes

Adding applications without bootloader, where no OTA or ROM switching is required, will

  • Increase the amount of free space and heap available to the application

So I can see a lot of advantages for the code quality and for the users of Sming.

Is anyone here using the Espressif Bootloader to do Over-The-Air (OTA) firmware updates?

@slaff slaff added the Ideas label Feb 21, 2017
@avr39-ripe
Copy link
Contributor

+1 to drop original bootloader
+1 to all @slaff remarks

@slaff
Copy link
Contributor

slaff commented Feb 21, 2017

Drop support for espressif bootloader

@anakod @piperpilot @raburton What is your opinion on this?

@tius2000
Copy link
Contributor Author

@slaff: great - also the option of having no bootloader at all!

@raburton
Copy link
Member

I don't think I realised we supported the espressif bootloader anyway, or if I did I'd forgotten. In what way is it currently supported?

@piperpilot
Copy link
Contributor

I use only rboot...I tried the espressif bootloader and it sucked. I +1 the switch to rboot or nothing.

@rosiste
Copy link

rosiste commented Feb 21, 2017

+1 to drop original bootloader

@slaff
Copy link
Contributor

slaff commented Feb 22, 2017

Ok, starting from version 3.2.0 (#977) we will drop support for Espressif's boolader and use only rBoot as bootloader. Plus, hopefully, add support for standalone applications.

Any final thoughts before we close the discussion?

@tius2000
Copy link
Contributor Author

I recently started to create completely refactored makefile for rboot-only projects with many additions (address calculation, automatic linker script creation, dependency detection etc.). However, it is still work in progress. It has a different name, so there are no collisions with the current makefiles.

Should I create a PR for that when it is finished?

@slaff
Copy link
Contributor

slaff commented Feb 24, 2017

Should I create a PR for that when it is finished?

Yes, please.

@slaff slaff closed this as completed Feb 24, 2017
@anakod
Copy link
Member

anakod commented Feb 25, 2017

We should drop support for old non-actual things, and moreover, we should drop support for my original OTA updater (and than remove Basic_AirUpdate example), because rboot is better and cover all cases.

@anakod anakod reopened this Feb 25, 2017
@tius2000
Copy link
Contributor Author

I submitted my refactored makefile as requested #985. However, this is still work in progress, all comments are welcome!

@riban-bw
Copy link
Contributor

riban-bw commented Mar 5, 2017

I fully support (+1) making rBoot our only second stage boot loader but please add support for standalone (no second stage boot loader) before dropping support for Espressif second stage boot loader. I am unable to get rBoot and SPIFFS working together (#1009) and we should have a method of creating images without rBoot, just in case of similar issues.

@slaff slaff added this to the 3.2.0 milestone Mar 9, 2017
@slaff slaff closed this as completed Oct 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

9 participants