Get rid of unnecessary .ok().unwrap() statements and bubble any ProtocolErrors upward instead

Author: jhodapp

esp32-wroom-rp/src/spi.rs

@@ -113,17 +113,17 @@ where
             .param(NinaSmallArrayParam::new(ssid))
             .param(NinaSmallArrayParam::new(passphrase));
 
-        self.execute(&operation).ok().unwrap();
+        self.execute(&operation)?;
 
-        self.receive(&operation).ok().unwrap();
+        self.receive(&operation)?;
         Ok(())
     }
 
     fn get_conn_status(&mut self) -> Result<u8, ProtocolError> {
         let operation =
             Operation::new(NinaCommand::GetConnStatus, 1).with_no_params(NinaNoParams::new(""));
 
-        self.execute(&operation).ok().unwrap();
+        self.execute(&operation)?;
 
         let result = self.receive(&operation)?;
 
@@ -134,9 +134,9 @@ where
         let dummy_param = NinaByteParam::from_bytes(&[ControlByte::Dummy as u8]);
         let operation = Operation::new(NinaCommand::Disconnect, 1).param(dummy_param);
 
-        self.execute(&operation).ok().unwrap();
+        self.execute(&operation)?;
 
-        self.receive(&operation).ok().unwrap();
+        self.receive(&operation)?;
 
         Ok(())
     }