[riot-notifications] [RIOT-OS/RIOT] esptool: Allow to pass the partition table CSV. (#16307)

iosabi notifications at github.com
Sun Apr 11 02:46:28 CEST 2021


### Contribution description

The partition table of the device in the esp8266 and esp32 based boards
was set to a default table with one "factory" partition with exactly
the size of the compiled firmware. This is problematic if we want to
update the device on the field.

This patch allows to set the `PARTITION_TABLE_CSV` variable from the
Makefile to a .csv file with a custom partition table, for example this
could be set to a partition table with two ota entries, or with a single
factory entry but of a known fixed size.

As a side effect of the make cleanup in this patch we now support
passing `-j` to the `make flash` command so we can compile in parallel
and still run the flash commands only once at the end. Before this
patch the conversion from .elf to .elf.bin was happening before the
code was recompiled when running in parallel.

### Testing procedure

`make BOARD=esp8266-esp-12x Q= -C tests/lwip flash -j term`

Verified that the compile commands at the end (after the link step) appear in the right order.

`make BOARD=esp8266-esp-12x PARTITION_TABLE_CSV=mytable.csv -C tests/lwip flash -j term`

Verified that the `mytable.csv` file is used instead.

### Issues/PRs references

None
You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/16307

-- Commit Summary --

  * esptool: Allow to pass the partition table CSV.

-- File Changes --

    M makefiles/tools/esptool.inc.mk (32)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/16307.patch
https://github.com/RIOT-OS/RIOT/pull/16307.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/16307
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210410/f6f42de3/attachment.htm>


More information about the notifications mailing list