Repository navigation
Harden LPC55S69 TrustZone isolation and add tests, CI and docs - #950
aidangarske wants to merge 1 commit into
Conversation
wolfSSL-Fenrir-bot
left a comment
There was a problem hiding this comment.
Fenrir Automated Review — PR #950
Scan targets checked: wolfboot-src, wolfboot-bugs
Not reviewed (no scan target covers them): .github/workflows/lpc55s69-emulator-tests.yml, .github/workflows/test-configs.yml, .github/workflows/trustzone-emulator-tests.yml, .gitignore, arch.mk, config/examples/lpc55s69-benchmark.config, config/examples/lpc55s69-tz-psa.config, config/examples/lpc55s69-tz.config, config/examples/lpc55s69.config, docs/Targets.md, hal/lpc55s69-ns.ld, hal/lpc55s69.h, hal/lpc55s69.ld, options.mk, test-app/ARM-lpc55s69-ns.ld, test-app/ARM-lpc55s69.ld, test-app/Makefile, test-app/emu-test-apps/Makefile, test-app/emu-test-apps/lpc55s69/Makefile, test-app/emu-test-apps/lpc55s69/target.ld.in and 7 more
Coverage: 7 of 9 in-scope changed file(s) opened by the reviewer; not opened: test-app/emu-test-apps/common/emu_hal.c, test-app/emu-test-apps/lpc55s69/uart.c
Fenrir result: Approved ✅
No new issues found in the changed files.
Advisory only — this automated result does not count as a GitHub approval.
Review tier: Lite
hal/lpc55s69.h, test-app linker fixes (RAM length, heap), TARGET_lpc55s69 ifdefs removed from startup_arm.c,
wolfcrypt_support.c and user_settings.h.
lpc55s69 and lpc55s69-tz; trustzone-emulator-tests.yml boots PKCS11 and HWPUF twice with --persist and checks the objects and the UEID survive; five new build jobs including non-TZ HWPUF=1. The --persist