This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git
The following commit(s) were added to refs/heads/master by this push: new 1645f4ac10 arch/mpfs/usb: Align usb_ctrlreq_s properly to 32-bit boundary 1645f4ac10 is described below commit 1645f4ac10512bdb911da49bb25daa0fb7fc4780 Author: Jukka Laitinen <jukka.laiti...@tii.ae> AuthorDate: Wed Apr 9 11:53:57 2025 +0300 arch/mpfs/usb: Align usb_ctrlreq_s properly to 32-bit boundary The alignment of the ctrlreq was correct by luck, and was broken when the spinlock was added to the structure. In non-smp configurations spinlock_t is currently 8-bits, causing wrong alignment of the ctrlreq structure. Signed-off-by: Jukka Laitinen <jukka.laiti...@tii.ae> --- arch/risc-v/src/mpfs/hardware/mpfs_usb.h | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/risc-v/src/mpfs/hardware/mpfs_usb.h b/arch/risc-v/src/mpfs/hardware/mpfs_usb.h index bb375c85ef..b34616e576 100644 --- a/arch/risc-v/src/mpfs/hardware/mpfs_usb.h +++ b/arch/risc-v/src/mpfs/hardware/mpfs_usb.h @@ -455,6 +455,7 @@ struct mpfs_usbdev_s /* USB-specific fields */ + aligned_data(4) struct usb_ctrlreq_s ctrl; /* Last EP0 request */ uint8_t devstate; /* State of the device (see enum mpfs_devstate_e) */ uint8_t prevstate; /* Previous state of the device before SUSPEND */