Debug C SDK (PRINTF) as a feature

Author: yogh333

ledger_device_sdk/Cargo.toml

@@ -28,6 +28,7 @@ debug = []
 speculos = []
 heap = [ "ledger_secure_sdk_sys/heap" ]
 nano_nbgl = [ "ledger_secure_sdk_sys/nano_nbgl" ]
+debug_csdk = [ "ledger_secure_sdk_sys/debug_csdk" ]
 
 default = [ "heap" ]
 

ledger_secure_sdk_sys/Cargo.toml

@@ -19,6 +19,7 @@ critical-section = { version = "1.1.2", optional = true }
 [features]
 heap = ["dep:embedded-alloc", "dep:critical-section"]
 nano_nbgl = []
+debug_csdk = []
 
 [lints.rust.unexpected_cfgs]
 level = "warn"

ledger_secure_sdk_sys/build.rs

@@ -379,6 +379,12 @@ impl SDKBuilder<'_> {
             command.define(define.as_str(), value.as_deref());
         }
 
+        // If the debug_csdk feature is enabled, add PRINTF defines
+        if env::var_os("CARGO_FEATURE_DEBUG_CSDK").is_some() {
+            command.define("HAVE_PRINTF", None);
+            command.define("PRINTF", Some("mcu_usb_printf"));
+        }
+
         // Set the CFLAGS
         for cflag in &self.device.cflags {
             command.flag(cflag);

ledger_secure_sdk_sys/c_sdk_build_flex.defines

@@ -34,10 +34,4 @@
 #define SCREEN_SIZE_WALLET
 #define HAVE_FAST_HOLD_TO_APPROVE
 #define OS_IO_SEPH_BUFFER_SIZE 272
-#define HAVE_LEDGER_PKI
-
-////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-// DEBUG C SDK
-////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-//#define HAVE_PRINTF
-//#define PRINTF mcu_usb_printf
+#define HAVE_LEDGER_PKI

ledger_secure_sdk_sys/c_sdk_build_nanosplus.defines

@@ -33,9 +33,3 @@
 #define HAVE_INAPP_BLE_PAIRING
 #define HAVE_BATTERY
 #define HAVE_LEDGER_PKI
-
-////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-// DEBUG C SDK
-////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-//#define HAVE_PRINTF
-//#define PRINTF mcu_usb_printf

ledger_secure_sdk_sys/c_sdk_build_nanox.defines

@@ -36,10 +36,4 @@
 #define HAVE_FONTS
 #define HAVE_INAPP_BLE_PAIRING
 #define HAVE_BATTERY
-#define HAVE_LEDGER_PKI
-
-////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-// DEBUG C SDK
-////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-//#define HAVE_PRINTF
-//#define PRINTF mcu_usb_printf
+#define HAVE_LEDGER_PKI

ledger_secure_sdk_sys/c_sdk_build_stax.defines

@@ -33,10 +33,4 @@
 #define NBGL_USE_CASE
 #define SCREEN_SIZE_WALLET
 #define OS_IO_SEPH_BUFFER_SIZE 272
-#define HAVE_LEDGER_PKI
-
-////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-// DEBUG C SDK
-////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-//#define HAVE_PRINTF
-//#define PRINTF mcu_usb_printf
+#define HAVE_LEDGER_PKI