Skip to content

Commit

Permalink
Makefile: Remove semihosting targets
Browse files Browse the repository at this point in the history
Also remove the documentation that mentions those targets
  • Loading branch information
NickeZ committed Nov 11, 2024
1 parent 735b280 commit 5c5644d
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 100 deletions.
4 changes: 0 additions & 4 deletions .ci/ci
Original file line number Diff line number Diff line change
Expand Up @@ -60,10 +60,6 @@ make -j8 firmware
make -j8 firmware-btc
make -j8 factory-setup

# Semihosting
make -j8 bootloader-semihosting
make -j8 firmware-semihosting

# Disallow some symbols in the final binary that we don't want.
if arm-none-eabi-nm build/bin/firmware.elf | grep -q "float_to_decimal_common_shortest"; then
echo "Rust fmt float formatting like {.1} adds significant binary bloat."
Expand Down
18 changes: 1 addition & 17 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -38,11 +38,6 @@ build-build-rust-unit-tests/Makefile:
cd build-build-rust-unit-tests && cmake .. -DCOVERAGE=OFF -DSANITIZE_ADDRESS=OFF -DSANITIZE_UNDEFINED=OFF
$(MAKE) -C py/bitbox02

build-semihosting/Makefile:
mkdir -p build-semihosting
cd build-semihosting && cmake .. -DCMAKE_TOOLCHAIN_FILE=arm.cmake -DSEMIHOSTING=ON
${MAKE} -C py/bitbox02

# Directory for building for "host" machine according to gcc convention
build: build/Makefile

Expand All @@ -54,20 +49,13 @@ build-build: build-build/Makefile
# address santizers when they link code compiled with gcc.
build-build-rust-unit-tests: build-build-rust-unit-tests/Makefile

# Directory for building for "host" machine but with semihosting enbled
build-semihosting: build-semihosting/Makefile

firmware: | build
# Generate python bindings for protobuf for test scripts
$(MAKE) -C build firmware.elf
firmware-semihosting: | build-semihosting
$(MAKE) -C build-semihosting firmware.elf
firmware-btc: | build
$(MAKE) -C build firmware-btc.elf
bootloader: | build
$(MAKE) -C build bootloader.elf
bootloader-semihosting: | build-semihosting
$(MAKE) -C build-semihosting bootloader-development.elf
bootloader-development: | build
$(MAKE) -C build bootloader-development.elf
bootloader-development-locked: | build
Expand Down Expand Up @@ -110,8 +98,6 @@ flash-dev-firmware:
./py/load_firmware.py build/bin/firmware.bin --debug
jlink-flash-bootloader-development: | build
JLinkExe -if SWD -device ATSAMD51J20 -speed 4000 -autoconnect 1 -CommanderScript ./build/scripts/bootloader-development.jlink
jlink-flash-bootloader-semihosting: | build-semihosting
JLinkExe -if SWD -device ATSAMD51J20 -speed 4000 -autoconnect 1 -CommanderScript ./build-semihosting/scripts/bootloader-development.jlink
jlink-flash-bootloader-development-locked: | build
JLinkExe -if SWD -device ATSAMD51J20 -speed 4000 -autoconnect 1 -CommanderScript ./build/scripts/bootloader-development-locked.jlink
jlink-flash-bootloader: | build
Expand All @@ -126,8 +112,6 @@ jlink-flash-firmware-btc: | build
JLinkExe -if SWD -device ATSAMD51J20 -speed 4000 -autoconnect 1 -CommanderScript ./build/scripts/firmware-btc.jlink
jlink-flash-factory-setup: | build
JLinkExe -if SWD -device ATSAMD51J20 -speed 4000 -autoconnect 1 -CommanderScript ./build/scripts/factory-setup.jlink
jlink-flash-firmware-semihosting: | build-semihosting
JLinkExe -if SWD -device ATSAMD51J20 -speed 4000 -autoconnect 1 -CommanderScript ./build-semihosting/scripts/firmware.jlink
dockerinit:
./scripts/container.sh build --pull --force-rm --no-cache -t shiftcrypto/firmware_v2:$(shell cat .containerversion) .
dockerpull:
Expand All @@ -144,4 +128,4 @@ prepare-tidy: | build build-build
make -C build rust-cbindgen
make -C build-build rust-cbindgen
clean:
rm -rf build build-build build-semihosting build-build-rust-unit-tests
rm -rf build build-build build-build-rust-unit-tests
8 changes: 0 additions & 8 deletions scripts/jlink-gdb-debug/.gdbinit

This file was deleted.

71 changes: 0 additions & 71 deletions scripts/jlink-gdb-debug/README.md

This file was deleted.

0 comments on commit 5c5644d

Please sign in to comment.