improvements for SD card module

Author: robamu

CHANGELOG.md

@@ -17,6 +17,7 @@ The format is based on [Keep a Changelog] and this project adheres to [Semantic
 - Added `open_long_name_file_in_dir` API, to open files using their Long File Name
 - Updated directory iterator callback, to allow bailing out early (breaking change)
 - `crc7` function now returns the actual CRC7 value without left-shifting by and XOR-ing by/with 1.
+- `CardType::SDHC` renamed to `CardType::SdhcSdcx`
 
 ### Added
 

src/sdcard/cid.rs

@@ -5,7 +5,7 @@ use arbitrary_int::{u7, u12, u40};
 use crate::sdcard::crc7;
 
 /// Checksum is invalid.
-#[derive(Debug, thiserror::Error)]
+#[derive(Debug, Copy, Clone, thiserror::Error)]
 #[error("checksum is invalid")]
 pub struct ChecksumInvalidError;
 

src/sdcard/csd.rs

@@ -15,7 +15,7 @@ pub enum Csd {
 }
 
 /// The CSD structure field is invalid.
-#[derive(Debug, PartialEq, Eq, thiserror::Error)]
+#[derive(Debug, PartialEq, Eq, Copy, Clone, thiserror::Error)]
 pub enum CsdCreationError {
     /// Invalid CSD structure field.
     #[error("invalid CSD structure field")]

src/sdcard/spi.rs

@@ -203,7 +203,7 @@ where
     fn read(&mut self, blocks: &mut [Block], start_block_idx: BlockIdx) -> Result<(), Error> {
         let start_idx = match self.card_type {
             Some(CardType::SD1 | CardType::SD2) => start_block_idx.0 * 512,
-            Some(CardType::SDHC) => start_block_idx.0,
+            Some(CardType::SdhcSdxc) => start_block_idx.0,
             None => return Err(Error::CardNotFound),
         };
 
@@ -227,7 +227,7 @@ where
     fn write(&mut self, blocks: &[Block], start_block_idx: BlockIdx) -> Result<(), Error> {
         let start_idx = match self.card_type {
             Some(CardType::SD1 | CardType::SD2) => start_block_idx.0 * 512,
-            Some(CardType::SDHC) => start_block_idx.0,
+            Some(CardType::SdhcSdxc) => start_block_idx.0,
             None => return Err(Error::CardNotFound),
         };
         if blocks.len() == 1 {
@@ -293,7 +293,7 @@ where
                 self.read_data(&mut csd_raw)?;
                 Ok(csd::Csd::V1(csd::CsdV1::from_be_bytes(&csd_raw)))
             }
-            Some(CardType::SD2 | CardType::SDHC) => {
+            Some(CardType::SD2 | CardType::SdhcSdxc) => {
                 if self.card_command(CMD9, 0)? != 0 {
                     return Err(Error::RegisterReadError);
                 }
@@ -443,7 +443,7 @@ where
                 let mut buffer = [0xFF; 4];
                 s.transfer_bytes(&mut buffer)?;
                 if (buffer[0] & 0xC0) == 0xC0 {
-                    card_type = CardType::SDHC;
+                    card_type = CardType::SdhcSdxc;
                 }
                 // Ignore the other three bytes
             }
@@ -642,10 +642,10 @@ pub enum CardType {
     ///
     /// Uses byte-addressing internally, so limited to 2GiB in size.
     SD2,
-    /// An high-capacity 'SDHC' Card.
+    /// An high-capacity 'SDHC' Card or an extended-capacity 'SDXC' card.
     ///
     /// Uses block-addressing internally to support capacities above 2GiB.
-    SDHC,
+    SdhcSdxc,
 }
 
 /// This an object you can use to busy-wait with a timeout.