build(core/io): move headers to the right place
What the developer wrote
Authored by matejcik
build(core/io): move headers to the right place
after smp had been moved to io crate
What changed, and why it matters
This commit is a routine build cleanup. It moves a header file to the correct folder and updates two source files to include it from the new location. There is no security-relevant change here.
No security action required. Treat as normal maintenance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit removes a temporary private include path hack from core/embed/io/nrf/build.rs and updates nrf_uart.c and nrf_update.c to include rust_smp.h via a relative path from the parent directory. This is purely a code organization/build hygiene change following the earlier move of SMP code into the io crate. No functional logic, memory handling, or trust boundary behavior is modified.
Changed components
core/embed/io/nrf/build.rscore/embed/io/nrf/rust_smp.hcore/embed/io/nrf/stm32u5/nrf_uart.ccore/embed/io/nrf/stm32u5/nrf_update.cInspect captured patch +2 / −6
### core/embed/io/nrf/build.rs
@@ -6,10 +6,6 @@ pub fn def_module(lib: &mut CLibrary) -> Result<()> {
lib.add_define("USE_NRF", Some("1"));
- // TODO: remove this hack when nrf related code in trezor_lib is
- // moved to this crate
- lib.add_private_include("../rust");
-
if cfg!(feature = "emulator") {
lib.add_sources(["nrf/unix/nrf.c"]);
} else if cfg!(feature = "mcu_stm32u5") {
### core/embed/io/nrf/rust_smp.h
[binary or diff unavailable]
### core/embed/io/nrf/stm32u5/nrf_uart.c
@@ -26,7 +26,7 @@
#include <sys/mpu.h>
#include "../nrf_internal.h"
-#include "rust_smp.h"
+#include "../rust_smp.h"
#include "sys/systick.h"
extern nrf_driver_t g_nrf_driver;
### core/embed/io/nrf/stm32u5/nrf_update.c
@@ -27,7 +27,7 @@
#include <sys/systick.h>
#include "../nrf_internal.h"
-#include "rust_smp.h"
+#include "../rust_smp.h"
#include "sha2.h"
#define IMAGE_HASH_LEN 32Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.