Fix clippy warnings, update READMEs (#284)

* Fix a couple clippy warnings

* Create features for architecture-specific depdendencies

* README updates

README.md

@@ -62,11 +62,13 @@ There are a number of other crates within the [esp-rs organization] which can be
 | :-------------: | :----------------------------------------------------------------------------: |
 |   [esp-alloc]   |                        A simple `no_std` heap allocator                        |
 | [esp-backtrace] |                 Backtrace support for bare-metal applications                  |
+|  [esp-println]  |                Provides `print!` and `println!` implementations                |
 |  [esp-storage]  | Implementation of [embedded-storage] traits to access unencrypted flash memory |
 
 [esp-rs organization]: https://github.com/esp-rs
 [esp-alloc]: https://github.com/esp-rs/esp-alloc
 [esp-backtrace]: https://github.com/esp-rs/esp-backtrace
+[esp-println]: https://github.com/esp-rs/esp-println
 [esp-storage]: https://github.com/esp-rs/esp-storage
 [embedded-storage]: https://github.com/rust-embedded-community/embedded-storage
 

esp-hal-common/Cargo.toml

@@ -57,11 +57,11 @@ esp32s2 = { version = "0.6.0",  features = ["critical-section"], optional = true
 esp32s3 = { version = "0.9.0",  features = ["critical-section"], optional = true }
 
 [features]
-esp32   = ["esp32/rt"  , "procmacros/xtensa", "xtensa-lx-rt/esp32",   "xtensa-lx/esp32",   "critical-section/restore-state-u32", "lock_api"]
-esp32c2 = ["esp32c2/rt", "procmacros/riscv" , "riscv", "riscv-atomic-emulation-trap",      "critical-section/restore-state-u8"]
-esp32c3 = ["esp32c3/rt", "procmacros/riscv" , "riscv", "riscv-atomic-emulation-trap",      "critical-section/restore-state-u8"]
-esp32s2 = ["esp32s2/rt", "procmacros/xtensa", "xtensa-lx-rt/esp32s2", "xtensa-lx/esp32s2", "critical-section/restore-state-u32", "esp-synopsys-usb-otg", "usb-device"]
-esp32s3 = ["esp32s3/rt", "procmacros/xtensa", "xtensa-lx-rt/esp32s3", "xtensa-lx/esp32s3", "critical-section/restore-state-u32", "lock_api", "esp-synopsys-usb-otg", "usb-device"]
+esp32   = ["esp32/rt"  , "xtensa", "xtensa-lx/esp32",   "xtensa-lx-rt/esp32",   "lock_api"]
+esp32c2 = ["esp32c2/rt", "riscv"]
+esp32c3 = ["esp32c3/rt", "riscv"]
+esp32s2 = ["esp32s2/rt", "xtensa", "xtensa-lx/esp32s2", "xtensa-lx-rt/esp32s2",             "esp-synopsys-usb-otg", "usb-device"]
+esp32s3 = ["esp32s3/rt", "xtensa", "xtensa-lx/esp32s3", "xtensa-lx-rt/esp32s3", "lock_api", "esp-synopsys-usb-otg", "usb-device"]
 
 # Implement the `embedded-hal==1.0.0-alpha.x` traits
 eh1 = ["embedded-hal-1", "embedded-hal-nb"]
@@ -81,3 +81,7 @@ embassy = ["embassy-time"]
 
 embassy-time-systick = []
 embassy-time-timg    = []
+
+# Architecture-specific features (intended for internal use)
+riscv  = ["dep:riscv", "critical-section/restore-state-u8",  "procmacros/riscv", "riscv-atomic-emulation-trap"]
+xtensa = [             "critical-section/restore-state-u32", "procmacros/xtensa"]

esp-hal-common/README.md

@@ -10,6 +10,7 @@
 This crate should not be used directly; you should use one of the device-specific HAL crates instead:
 
 - [esp32-hal](../esp32-hal/README.md)
+- [esp32c2-hal](../esp32c2-hal/README.md)
 - [esp32c3-hal](../esp32c3-hal/README.md)
 - [esp32s2-hal](../esp32s2-hal/README.md)
 - [esp32s3-hal](../esp32s3-hal/README.md)

esp-hal-common/src/sha.rs

@@ -292,6 +292,7 @@ impl Sha {
             ShaMode::SHA1 | ShaMode::SHA256 => 64,
             #[cfg(not(esp32))]
             ShaMode::SHA224 => 64,
+            #[cfg(not(any(esp32c2, esp32c3)))]
             _ => 128,
         };
     }
@@ -493,7 +494,6 @@ impl Sha {
         }
 
         unsafe {
-
             let digest_ptr = self.output_ptr();
             let out_ptr = output.as_mut_ptr() as *mut u32;
             let digest_out = core::cmp::min(self.digest_length(), output.len()) / ALIGN_SIZE;
@@ -506,7 +506,6 @@ impl Sha {
             }
         }
 
-
         Ok(())
     }
 

esp32c3-hal/build.rs

@@ -153,8 +153,7 @@ fn check_opt_level() {
         return;
     }
 
-    match env::var_os("OPT_LEVEL") {
-        Some(ref opt_level) => {
+    if let Some(ref opt_level) = env::var_os("OPT_LEVEL") {
         if opt_level == "z" {
             println!(
                 "{}",
@@ -168,6 +167,4 @@ fn check_opt_level() {
             exit(1);
         }
     }
-        _ => {}
-    }
 }