Use `repr(packed)` on pipe descriptors structure.

Ensure offsets and sizes are correct with tests.

1 files changed, 26 insertions, 26 deletions

diff --git a/usbh/src/pipe/status_pipe.rs b/usbh/src/pipe/status_pipe.rs
index 4f8eb41..be135c5 100644
--- a/usbh/src/pipe/status_pipe.rs
+++ b/usbh/src/pipe/status_pipe.rs
@@ -4,14 +4,14 @@
 /// Reset: 0xxxxxx
 /// Property: PAC Write-Protection, Write-Synchronized, Read-Synchronized
 #[derive(Clone, Copy, Debug)]
-#[repr(C)]
-pub(crate) struct StatusPipe(u16);
+#[repr(C, packed)]
+pub(crate) struct StatusPipe(u8);
 
 pub(crate) struct R {
-    bits: u16,
+    bits: u8,
 }
 pub(crate) struct W {
-    bits: u16,
+    bits: u8,
 }
 
 impl StatusPipe {
@@ -29,22 +29,22 @@ impl StatusPipe {
     }
 }
 
-impl From<u16> for StatusPipe {
-    fn from(v: u16) -> Self {
+impl From<u8> for StatusPipe {
+    fn from(v: u8) -> Self {
         Self(v)
     }
 }
 
 impl R {
     /// Value in raw bits.
-    pub fn bits(&self) -> u16 {
+    pub fn bits(&self) -> u8 {
         self.bits
     }
 
     pub fn ercnt(&self) -> ErCntR {
         let bits = {
             const POS: u8 = 5;
-            const MASK: u16 = 0x7;
+            const MASK: u8 = 0x7;
             ((self.bits >> POS) & MASK) as u8
         };
 
@@ -54,7 +54,7 @@ impl R {
     pub fn crc16er(&self) -> CRC16ErR {
         let bits = {
             const POS: u8 = 4;
-            const MASK: u16 = 1;
+            const MASK: u8 = 1;
             ((self.bits >> POS) & MASK) == 1
         };
 
@@ -64,7 +64,7 @@ impl R {
     pub fn touter(&self) -> TOutErrR {
         let bits = {
             const POS: u8 = 3;
-            const MASK: u16 = 1;
+            const MASK: u8 = 1;
 
             ((self.bits >> POS) & MASK) == 1
         };
@@ -75,7 +75,7 @@ impl R {
     pub fn pider(&self) -> PIDErR {
         let bits = {
             const POS: u8 = 2;
-            const MASK: u16 = 1;
+            const MASK: u8 = 1;
 
             ((self.bits >> POS) & MASK) == 1
         };
@@ -86,7 +86,7 @@ impl R {
     pub fn dapider(&self) -> DaPIDErR {
         let bits = {
             const POS: u8 = 1;
-            const MASK: u16 = 1;
+            const MASK: u8 = 1;
 
             ((self.bits >> POS) & MASK) == 1
         };
@@ -97,7 +97,7 @@ impl R {
     pub fn dtgler(&self) -> DTglErR {
         let bits = {
             const POS: u8 = 0;
-            const MASK: u16 = 1;
+            const MASK: u8 = 1;
 
             ((self.bits >> POS) & MASK) == 1
         };
@@ -222,7 +222,7 @@ impl DTglErR {
 
 impl W {
     /// Write raw bits.
-    pub unsafe fn bits(&mut self, v: u16) -> &mut Self {
+    pub unsafe fn bits(&mut self, v: u8) -> &mut Self {
         self.bits = v;
         self
     }
@@ -262,8 +262,8 @@ impl<'a> ErCntW<'a> {
     pub unsafe fn bits(self, v: u8) -> &'a mut W {
         const POS: u8 = 5;
         const MASK: u8 = 0x7;
-        self.w.bits &= !((MASK as u16) << POS);
-        self.w.bits |= ((v & MASK) as u16) << POS;
+        self.w.bits &= !((MASK as u8) << POS);
+        self.w.bits |= ((v & MASK) as u8) << POS;
         self.w
     }
 
@@ -285,8 +285,8 @@ impl<'a> CRC16ErW<'a> {
     pub fn bit(self, v: bool) -> &'a mut W {
         const POS: u8 = 4;
         const MASK: bool = true;
-        self.w.bits &= !((MASK as u16) << POS);
-        self.w.bits |= ((v & MASK) as u16) << POS;
+        self.w.bits &= !((MASK as u8) << POS);
+        self.w.bits |= ((v & MASK) as u8) << POS;
         self.w
     }
 
@@ -312,8 +312,8 @@ impl<'a> TOutErW<'a> {
     pub fn bit(self, v: bool) -> &'a mut W {
         const POS: u8 = 3;
         const MASK: bool = true;
-        self.w.bits &= !((MASK as u16) << POS);
-        self.w.bits |= ((v & MASK) as u16) << POS;
+        self.w.bits &= !((MASK as u8) << POS);
+        self.w.bits |= ((v & MASK) as u8) << POS;
         self.w
     }
 
@@ -339,8 +339,8 @@ impl<'a> PIDErW<'a> {
     pub fn bit(self, v: bool) -> &'a mut W {
         const POS: u8 = 2;
         const MASK: bool = true;
-        self.w.bits &= !((MASK as u16) << POS);
-        self.w.bits |= ((v & MASK) as u16) << POS;
+        self.w.bits &= !((MASK as u8) << POS);
+        self.w.bits |= ((v & MASK) as u8) << POS;
         self.w
     }
 
@@ -366,8 +366,8 @@ impl<'a> DaPIDErW<'a> {
     pub fn bit(self, v: bool) -> &'a mut W {
         const POS: u8 = 1;
         const MASK: bool = true;
-        self.w.bits &= !((MASK as u16) << POS);
-        self.w.bits |= ((v & MASK) as u16) << POS;
+        self.w.bits &= !((MASK as u8) << POS);
+        self.w.bits |= ((v & MASK) as u8) << POS;
         self.w
     }
 
@@ -392,8 +392,8 @@ impl<'a> DTglErW<'a> {
     pub fn bit(self, v: bool) -> &'a mut W {
         const POS: u8 = 0;
         const MASK: bool = true;
-        self.w.bits &= !((MASK as u16) << POS);
-        self.w.bits |= ((v & MASK) as u16) << POS;
+        self.w.bits &= !((MASK as u8) << POS);
+        self.w.bits |= ((v & MASK) as u8) << POS;
         self.w
     }