generic register rebase on master

Author: burrbull

CHANGELOG.md

@@ -11,7 +11,11 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 - `variant()` method for field reader and `Variant` enum for fields with reserved values
 
-- Field readers and writers use one enum where it is possible
+### Changed
+
+  - Field readers and writers use one enum where it is possible
+
+  - Replace register and its reader/writer by generic types
 
 ## [v0.15.2] - 2019-07-29
 

src/generate/generic.rs

@@ -10,6 +10,18 @@ pub fn render() -> Result<Vec<Tokens>> {
     generic_items.push(quote! {
         use core::marker;
 
+        ///Marker trait for readable register/field
+        pub trait Readable {}
+
+        ///Marker trait for writable register/field
+        pub trait Writable {}
+
+        ///Reset value of the register
+        pub trait ResetValue<U> {
+            ///Reset value of the register
+            fn reset_value() -> U;
+        }
+
         ///Converting enumerated values to bits
         pub trait ToBits<N> {
             ///Conversion method
@@ -18,23 +30,97 @@ pub fn render() -> Result<Vec<Tokens>> {
     });
 
     generic_items.push(quote! {
-        ///Value read from the register
-        pub struct FR<U, T> {
-            pub(crate) bits: U,
-            _reg: marker::PhantomData<T>,
+        ///Wrapper for registers
+        pub struct Reg<U, REG> {
+            register: vcell::VolatileCell<U>,
+            _marker: marker::PhantomData<REG>,
         }
 
-        impl<U, T, FI> PartialEq<FI> for FR<U, T>
+        impl<U, REG> Reg<U, REG>
         where
-            U: PartialEq,
-            FI: ToBits<U>
+            Self: Readable,
+            U: Copy
         {
-            fn eq(&self, other: &FI) -> bool {
-                self.bits.eq(&other._bits())
+            ///Reads the contents of the register
+            #[inline(always)]
+            pub fn read(&self) -> R<U, Self> {
+                R {bits: self.register.get(), _reg: marker::PhantomData}
+            }
+        }
+
+        impl<U, REG> Reg<U, REG>
+        where
+            Self: ResetValue<U> + Writable,
+            U: Copy,
+        {
+            ///Writes the reset value to the register
+            #[inline(always)]
+            pub fn reset(&self) {
+                self.register.set(Self::reset_value())
             }
         }
+    });
 
-        impl<U, T> FR<U, T>
+    generic_items.push(quote! {
+        impl<U, REG> Reg<U, REG>
+        where
+            Self: ResetValue<U> + Writable,
+            U: Copy
+        {
+            ///Writes to the register
+            #[inline(always)]
+            pub fn write<F>(&self, f: F)
+            where
+                F: FnOnce(&mut W<U, Self>) -> &mut W<U, Self>
+            {
+                self.register.set(f(&mut W {bits: Self::reset_value(), _reg: marker::PhantomData}).bits);
+            }
+        }
+    });
+
+    generic_items.push(quote! {
+        impl<U, REG> Reg<U, REG>
+        where
+            Self: Writable,
+            U: Copy + Default
+        {
+            ///Writes Zero to the register
+            #[inline(always)]
+            pub fn write_with_zero<F>(&self, f: F)
+            where
+                F: FnOnce(&mut W<U, Self>) -> &mut W<U, Self>
+            {
+                self.register.set(f(&mut W {bits: U::default(), _reg: marker::PhantomData }).bits);
+            }
+        }
+    });
+
+    generic_items.push(quote! {
+        impl<U, REG> Reg<U, REG>
+        where
+            Self: Readable + Writable,
+            U: Copy,
+        {
+            ///Modifies the contents of the register
+            #[inline(always)]
+            pub fn modify<F>(&self, f: F)
+            where
+                for<'w> F: FnOnce(&R<U, Self>, &'w mut W<U, Self>) -> &'w mut W<U, Self>
+            {
+                let bits = self.register.get();
+                self.register.set(f(&R {bits, _reg: marker::PhantomData}, &mut W {bits, _reg: marker::PhantomData}).bits);
+            }
+        }
+    });
+
+    generic_items.push(quote! {
+        ///Register/field reader
+        pub struct R<U, T> {
+            pub(crate) bits: U,
+            _reg: marker::PhantomData<T>,
+        }
+
+        impl<U, T> R<U, T>
         where
             U: Copy
         {
@@ -46,7 +132,7 @@ pub fn render() -> Result<Vec<Tokens>> {
                     _reg: marker::PhantomData,
                 }
             }
-            ///Read raw bits from field
+            ///Read raw bits from register/field
             #[inline(always)]
             pub fn bits(&self) -> U {
                 self.bits
@@ -55,7 +141,19 @@ pub fn render() -> Result<Vec<Tokens>> {
     });
 
     generic_items.push(quote! {
-        impl<FI> FR<bool, FI> {
+        impl<U, T, FI> PartialEq<FI> for R<U, T>
+        where
+            U: PartialEq,
+            FI: ToBits<U>
+        {
+            fn eq(&self, other: &FI) -> bool {
+                self.bits.eq(&other._bits())
+            }
+        }
+    });
+
+    generic_items.push(quote! {
+        impl<FI> R<bool, FI> {
             ///Value of the field as raw bits
             #[inline(always)]
             pub fn bit(&self) -> bool {
@@ -74,6 +172,27 @@ pub fn render() -> Result<Vec<Tokens>> {
         }
     });
 
+    generic_items.push(quote! {
+        ///Register writer
+        pub struct W<U, REG> {
+            ///Writable bits
+            pub bits: U,
+            _reg: marker::PhantomData<REG>,
+        }
+    });
+
+    generic_items.push(quote! {
+        impl<U, REG> W<U, REG> {
+            ///Writes raw bits to the register
+            #[inline(always)]
+            pub fn bits(&mut self, bits: U) -> &mut Self {
+                self.bits = bits;
+                self
+            }
+        }
+    });
+
+
     generic_items.push(quote! {
         ///Used if enumerated values cover not the whole range
         #[derive(Clone,Copy,PartialEq)]
@@ -88,7 +207,7 @@ pub fn render() -> Result<Vec<Tokens>> {
     code.push(quote! {
         #[allow(unused_imports)]
         use generic::*;
-        /// Common register and bit access and modify traits
+        ///Common register and bit access and modify traits
         pub mod generic {
             #(#generic_items)*
         }