Move device init to command level

src/cmd/bench.rs

@@ -17,6 +17,7 @@ pub struct Cmd {
 
 impl Cmd {
     pub fn run(&self, device: &Device) -> Result {
+        device.init()?;
         let keypair = device.get_keypair(false)?;
         let duration = bench_sign(&keypair, self.iterations)?;
         let rate = self.iterations as f64 / duration.as_secs_f64();

src/cmd/config.rs

@@ -6,6 +6,7 @@ pub struct Cmd {}
 
 impl Cmd {
     pub fn run(&self, device: &Device) -> Result {
+        device.init()?;
         let config = device.get_config()?;
         print_json(&config)
     }

src/cmd/info.rs

@@ -6,6 +6,7 @@ pub struct Cmd {}
 
 impl Cmd {
     pub fn run(&self, device: &Device) -> Result {
+        device.init()?;
         let info = device.get_info()?;
         print_json(&info)
     }

src/cmd/key.rs

@@ -14,6 +14,7 @@ pub struct Cmd {
 
 impl Cmd {
     pub fn run(&self, device: &Device) -> Result {
+        device.init()?;
         let keypair = device.get_keypair(self.generate)?;
         print_keypair(&keypair)
     }

src/cmd/provision.rs

@@ -6,6 +6,7 @@ pub struct Cmd {}
 
 impl Cmd {
     pub fn run(&self, device: &Device) -> Result {
+        device.init()?;
         let keypair = device.provision()?;
         print_keypair(&keypair)
     }

src/cmd/test.rs

@@ -15,6 +15,7 @@ pub struct Cmd {}
 
 impl Cmd {
     pub fn run(&self, device: &Device) -> Result {
+        device.init()?;
         let tests = device.get_tests();
         let results: Vec<(String, TestResult)> = tests
             .iter()

src/device/ecc.rs

@@ -62,7 +62,7 @@ impl Device {
         })
     }
 
-    fn init_device(&self) -> Result {
+    pub fn init(&self) -> Result {
         // Initialize the global instance if not already initialized
         Ok(ecc608::init(
             &self.path.to_string_lossy(),
@@ -72,7 +72,6 @@ impl Device {
     }
 
     pub fn get_info(&self) -> Result<Info> {
-        self.init_device()?;
         let info = with_ecc(|ecc: &mut Ecc| {
             ecc.get_info()
                 .and_then(|info| ecc.get_serial().map(|serial| Info { info, serial }))
@@ -81,7 +80,6 @@ impl Device {
     }
 
     pub fn get_keypair(&self, create: bool) -> Result<Keypair> {
-        self.init_device()?;
         let keypair: Keypair = with_ecc(|ecc| {
             if create {
                 generate_compact_key_in_slot(ecc, self.slot)
@@ -93,7 +91,6 @@ impl Device {
     }
 
     pub fn provision(&self) -> Result<Keypair> {
-        self.init_device()?;
         let slot_config = ecc608::SlotConfig::default();
         let key_config = ecc608::KeyConfig::default();
         for slot in 0..=ecc608::MAX_SLOT {
@@ -107,7 +104,6 @@ impl Device {
     }
 
     pub fn get_config(&self) -> Result<Config> {
-        self.init_device()?;
         let slot_config = with_ecc(|ecc| ecc.get_slot_config(self.slot))?;
         let key_config = with_ecc(|ecc| ecc.get_key_config(self.slot))?;
         let zones = [ecc608::Zone::Config, ecc608::Zone::Data]

src/device/mod.rs

@@ -212,6 +212,14 @@ impl DeviceArgs {
 }
 
 impl Device {
+    pub fn init(&self) -> Result {
+        match self {
+            #[cfg(feature = "ecc608")]
+            Self::Ecc(device) => device.init(),
+            _ => Ok(()),
+        }
+    }
+
     pub fn get_info(&self) -> Result<Info> {
         let info = match self {
             #[cfg(feature = "ecc608")]