Add remaining GPIO pins for ESP32-S2/S3 ULP

Dominaezzz · Jun 19, 2024 · 5444b29 · 5444b29
1 parent 2d90fa3
commit 5444b29
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/esp-lp-hal/CHANGELOG.md b/esp-lp-hal/CHANGELOG.md
@@ -15,6 +15,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Add the `ulp-riscv-hal` package (#840)
 - Add LP_UART basic driver (#1113)
 - Added basic `LP-I2C` driver for C6 (#1185)
+- Add remaining GPIO pins for ESP32-S2/S3 (#1695)
 
 ### Changed
 

diff --git a/esp-lp-hal/src/gpio.rs b/esp-lp-hal/src/gpio.rs
@@ -7,6 +7,11 @@ type LpIo = crate::pac::LP_IO;
 #[cfg(any(feature = "esp32s2", feature = "esp32s3"))]
 type LpIo = crate::pac::RTC_IO;
 
+#[cfg(feature = "esp32c6")]
+const MAX_GPIO_PIN: u8 = 7;
+#[cfg(any(feature = "esp32s2", feature = "esp32s3"))]
+const MAX_GPIO_PIN: u8 = 21;
+
 #[non_exhaustive]
 pub struct Input<const PIN: u8> {}
 
@@ -40,7 +45,7 @@ impl<const PIN: u8> Output<PIN> {
 // Used by the `entry` procmacro:
 #[doc(hidden)]
 pub unsafe fn conjure_output<const PIN: u8>() -> Option<Output<PIN>> {
-    if PIN > 7 {
+    if PIN > MAX_GPIO_PIN {
         None
     } else {
         Some(Output {})
@@ -50,7 +55,7 @@ pub unsafe fn conjure_output<const PIN: u8>() -> Option<Output<PIN>> {
 // Used by the `entry` procmacro:
 #[doc(hidden)]
 pub unsafe fn conjure_input<const PIN: u8>() -> Option<Input<PIN>> {
-    if PIN > 7 {
+    if PIN > MAX_GPIO_PIN {
         None
     } else {
         Some(Input {})