non_exhaustive peripheral

.github/workflows/ci.yml

@@ -83,7 +83,11 @@ jobs:
           - { rust: stable, vendor: Spansion, options: "--atomics" }
           - { rust: stable, vendor: STMicro, options: "" }
           - { rust: stable, vendor: STMicro, options: "--atomics" }
-          - { rust: stable, vendor: STM32-patched, options: "--strict -f enum_value::p: --max-cluster-size --atomics --atomics-feature atomics --impl-debug --impl-defmt defmt" }
+          - {
+              rust: stable,
+              vendor: STM32-patched,
+              options: "--strict -f enum_value::p: --max-cluster-size --atomics --atomics-feature atomics --impl-debug --impl-defmt defmt",
+            }
           - { rust: stable, vendor: Toshiba, options: all }
           - { rust: stable, vendor: Toshiba, options: "" }
           # Test MSRV
@@ -92,8 +96,16 @@ jobs:
           - { rust: nightly, vendor: MSP430, options: "--atomics" }
           - { rust: nightly, vendor: MSP430, options: "" }
           # Workaround for _1token0
-          - { rust: nightly-2024-09-25, vendor: Espressif, options: "--atomics --ident-formats-theme legacy" }
-          - { rust: nightly-2024-09-25, vendor: Espressif, options: "--ident-format register:::Reg" }
+          - {
+              rust: nightly-2024-09-25,
+              vendor: Espressif,
+              options: "--atomics --ident-formats-theme legacy",
+            }
+          - {
+              rust: nightly-2024-09-25,
+              vendor: Espressif,
+              options: "--ident-format register:::Reg",
+            }
 
     steps:
       - uses: actions/checkout@v4
@@ -107,7 +119,7 @@ jobs:
 
       - name: Self install
         run: |
-          cargo install svd2rust --path .
+          cargo install svd2rust --path . --force
 
       - name: Run CI script for `${{ matrix.vendor }}` under rust `${{ matrix.rust }}` with options=`${{ matrix.options }}`
         env:
@@ -132,7 +144,7 @@ jobs:
 
       - name: Self install
         run: |
-          cargo install svd2rust --path .
+          cargo install svd2rust --path . --force
 
       - name: Run CI script
         env:

CHANGELOG.md

@@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+- Mark peripheral with `non_exhaustive`
 - Force using rust edition 2021 in CI
 
 ## [v0.35.0] - 2024-11-12

src/generate/device.rs

@@ -64,7 +64,6 @@ pub fn render(d: &Device, config: &Config, device_x: &mut String) -> Result<Toke
 
     out.extend(quote! {
         use core::ops::Deref;
-        use core::marker::PhantomData;
     });
 
     // Retaining the previous assumption
@@ -273,7 +272,7 @@ pub fn render(d: &Device, config: &Config, device_x: &mut String) -> Result<Toke
                     #feature_attribute
                     pub #p_singleton: #p_ty,
                 });
-                exprs.extend(quote!(#feature_attribute #p_singleton: #p_ty::steal(),));
+                exprs.extend(quote!(#feature_attribute #p_singleton: #p_ty,));
             }
             Peripheral::Array(p, dim_element) => {
                 for p_name in names(p, dim_element) {
@@ -288,7 +287,7 @@ pub fn render(d: &Device, config: &Config, device_x: &mut String) -> Result<Toke
                         #feature_attribute
                         pub #p_singleton: #p_ty,
                     });
-                    exprs.extend(quote!(#feature_attribute #p_singleton: #p_ty::steal(),));
+                    exprs.extend(quote!(#feature_attribute #p_singleton: #p_ty,));
                 }
             }
         }

src/generate/peripheral.rs

@@ -77,7 +77,7 @@ pub fn render(p_original: &Peripheral, index: &Index, config: &Config) -> Result
         /// peripheral instance existing to ensure memory safety; ensure
         /// no stolen instances are passed to such software.
         pub unsafe fn steal() -> Self {
-            Self { _marker: PhantomData }
+            Self
         }
     };
 
@@ -97,7 +97,8 @@ pub fn render(p_original: &Peripheral, index: &Index, config: &Config) -> Result
             #phtml
             #doc_alias
             #feature_attribute
-            pub struct #p_ty { _marker: PhantomData<*const ()> }
+            #[non_exhaustive]
+            pub struct #p_ty;
 
             #feature_attribute
             unsafe impl Send for #p_ty {}