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

add common bootloader in main repository #48

Conversation

tridge
Copy link
Member

@tridge tridge commented Jul 26, 2024

this is an effort to move the bootloader for AM32 into the main repo, with the same bootloader sources for all MCUs.
This is a WIP PR, it is only partially functional so far. Testers would be very welcome!

MCUs:

  • F421 building and tested
  • F415 building, not tested
  • F051 building, not tested
  • F031 building, not tested
  • G071 building, not tested
  • G431 pending
  • L431 pending
  • E230 building, not tested

this is a step towards removing the recursive make, and making it
easier to add new MCUs
makes no difference to the binary
@tridge tridge force-pushed the pr-common-bootloader branch 2 times, most recently from 3f884d5 to c6b149e Compare July 27, 2024 08:54
@tridge tridge force-pushed the pr-common-bootloader branch 2 times, most recently from 4d86dcc to d71bf22 Compare July 28, 2024 02:06
tridge and others added 4 commits July 28, 2024 18:38
all bootloaders built from common source, using blutil.h for MCU
specific functions and macros
generate targets separately, avoiding the need to recursively call
make

This also fixes disabling quiet, like this:

  make FOXEER_F421 QUIET=

also see "Recursive Make Considered Harmful" paper by Peter Miller
B-G431-ESC1 target added. Compiles with warnings & notes. Not tested on hardware.
@tridge tridge force-pushed the pr-common-bootloader branch 3 times, most recently from db96e79 to e28c0a5 Compare July 28, 2024 08:47
targetting PA15 for ST G431 discovery ESC
@tridge
Copy link
Member Author

tridge commented Jul 29, 2024

closing for now so as not to overload alka - can revisit in the future

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants