Merge #694

694: const alternate register accessors r=therealprof a=burrbull



Co-authored-by: Andrey Zgarbul <[email protected]>

Author: bors[bot]

.github/workflows/ci.yml

@@ -72,7 +72,7 @@ jobs:
           - { rust: stable, vendor: Toshiba, options: all }
           - { rust: stable, vendor: Toshiba, options: "" }
           # Test MSRV
-          - { rust: 1.60.0, vendor: Nordic, options: "" }
+          - { rust: 1.61.0, vendor: Nordic, options: "" }
           # Use nightly for architectures which don't support stable
           - { rust: nightly, vendor: MSP430, options: "--nightly" }
           - { rust: nightly, vendor: MSP430, options: "" }

CHANGELOG.md

@@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [Unreleased]
 
+- mark alternate register accessors with `const`, bump `pac` MSRV to 1.61
 - `fields` fn refactoring
 - Test patched STM32
 - simplify ci strategy

README.md

@@ -13,9 +13,9 @@ This project is developed and maintained by the [Tools team][team].
 
 ## Minimum Supported Rust Version (MSRV)
 
-The **generated code** is guaranteed to compile on stable Rust 1.60.0 and up.
+The **generated code** is guaranteed to compile on stable Rust 1.61.0 and up.
 
-If you encounter compilation errors on any stable version newer than 1.60.0, please open an issue.
+If you encounter compilation errors on any stable version newer than 1.61.0, please open an issue.
 
 # Testing Locally
 

src/generate/peripheral.rs

@@ -535,10 +535,8 @@ fn register_or_cluster_block(
                 accessors.extend(quote! {
                     #[doc = #comment]
                     #[inline(always)]
-                    pub fn #name(&self) -> &#ty {
-                        unsafe {
-                            &*(((self as *const Self) as *const u8).add(#offset) as *const #ty)
-                        }
+                    pub const fn #name(&self) -> &#ty {
+                        unsafe { &*(self as *const Self).cast::<u8>().add(#offset).cast() }
                     }
                 });
             } else {