Skip to content

Commit 0e19286

Browse files
committed
tools/metrics.py: Change rp2 board selection to RPI_PICO_W.
This tool is used to compute size differences in the firmware (eg as part of CI), but it doesn't currently check any firmware that has bare-metal lwIP/networking, making it hard to see how firmware size changes when networking related changes are made. So, change the board selection for the rp2 port to RPI_PICO_W. Changes in size to standard RPI_PICO firmware will be very similar to other bare-metal boards like PYBV10. Signed-off-by: Damien George <[email protected]>
1 parent 932f76c commit 0e19286

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

tools/ci.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ function ci_code_size_setup {
5555
function ci_code_size_build {
5656
# check the following ports for the change in their code size
5757
PORTS_TO_CHECK=bmusxpd
58-
SUBMODULES="lib/asf4 lib/berkeley-db-1.xx lib/mbedtls lib/micropython-lib lib/nxp_driver lib/pico-sdk lib/stm32lib lib/tinyusb"
58+
SUBMODULES="lib/asf4 lib/berkeley-db-1.xx lib/btstack lib/cyw43-driver lib/lwip lib/mbedtls lib/micropython-lib lib/nxp_driver lib/pico-sdk lib/stm32lib lib/tinyusb"
5959

6060
# starts off at either the ref/pull/N/merge FETCH_HEAD, or the current branch HEAD
6161
git checkout -b pull_request # save the current location

tools/metrics.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ def __init__(self, name, dir, output, make_flags=None):
6969
"x": PortData("mimxrt", "mimxrt", "build-TEENSY40/firmware.elf"),
7070
"e": PortData("renesas-ra", "renesas-ra", "build-EK_RA6M2/firmware.elf"),
7171
"r": PortData("nrf", "nrf", "build-PCA10040/firmware.elf"),
72-
"p": PortData("rp2", "rp2", "build-RPI_PICO/firmware.elf"),
72+
"p": PortData("rp2", "rp2", "build-RPI_PICO_W/firmware.elf", "BOARD=RPI_PICO_W"),
7373
"d": PortData("samd", "samd", "build-ADAFRUIT_ITSYBITSY_M4_EXPRESS/firmware.elf"),
7474
}
7575

0 commit comments

Comments
 (0)