Add alternate bitfield padding option

Adds configuration option CFG_TUSB_ALT_BIT_PACKING_ALIGNMENT, which
substitutes bitfield variable " : 0" padding syntax with an unused
variable of size equal to the remaining number of bits.

This change resolves aligned access issues for some platforms.

Default behavior is original if the option is not explicitly enabled.

1 files changed, 10 insertions, 3 deletions

diff --git a/src/host/ehci/ehci.h b/src/host/ehci/ehci.h
index a6342b2d2..acaf545c1 100644
--- a/src/host/ehci/ehci.h
+++ b/src/host/ehci/ehci.h
@@ -1,4 +1,4 @@
-/* 
+/*
  * The MIT License (MIT)
  *
  * Copyright (c) 2019 Ha Thach (tinyusb.org)
@@ -231,7 +231,11 @@ typedef struct TU_ATTR_ALIGNED(32)
 	uint32_t             : 1; ///< reserved
 	uint32_t port_number : 7; ///< This field is the port number of the recipient transaction translator.
 	uint32_t direction   : 1; ///<  0 = OUT; 1 = IN. This field encodes whether the full-speed transaction should be an IN or OUT.
-	uint32_t             : 0; // padding to the end of current storage unit
+#if CFG_TUSB_ALT_BIT_PACKING_ALIGNMENT
+                              ///< All 32 bits are used
+#else
+    uint32_t             : 0; // padding to the end of current storage unit
+#endif /* CFG_TUSB_ALT_BIT_PACKING_ALIGNMENT */
 
 	// Word 2: Micro-frame Schedule Control
 	uint8_t int_smask   ; ///< This field (along with the Activeand SplitX-statefields in the Statusbyte) are used to determine during which micro-frames the host controller should execute complete-split transactions
@@ -423,7 +427,11 @@ typedef volatile struct
       uint32_t nxp_port_force_fullspeed    : 1; ///< NXP customized: Writing this bit to a 1 will force the port to only connect at Full Speed. It disables the chirp sequence that allowsthe port to identify itself as High Speed. This is useful for testing FS configurations with a HS host, hub or device.
       uint32_t                             : 1;
       uint32_t nxp_port_speed              : 2; ///< NXP customized: This register field indicates the speed atwhich the port is operating. For HS mode operation in the host controllerand HS/FS operation in the device controller the port routing steers data to the Protocol engine. For FS and LS mode operation in the host controller, the port routing steers data to the Protocol Engine w/ Embedded Transaction Translator. 0x0: Fullspeed, 0x1: Lowspeed, 0x2: Highspeed
+#if CFG_TUSB_ALT_BIT_PACKING_ALIGNMENT
+      uint32_t unused                      : 4; ///< padding
+#else
       uint32_t                             : 0; // padding to the boundary of storage unit
+#endif /* CFG_TUSB_ALT_BIT_PACKING_ALIGNMENT */
     }portsc_bm;
   };
 }ehci_registers_t;
@@ -461,4 +469,3 @@ typedef struct
 
 /** @} */
 /** @} */
-