tnt submitted this change.

View Change


Approvals: tnt: Looks good to me, approved Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve
icE1usb fw: Ensure alignement of the structure for the usb_data_write

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>
Change-Id: Id80468ef95bb908cf34b6eeca04c83f1285fd2f5
---
M firmware/ice40-riscv/icE1usb/usb_gps.c
1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/firmware/ice40-riscv/icE1usb/usb_gps.c b/firmware/ice40-riscv/icE1usb/usb_gps.c
index ad893a5..0fe5c98 100644
--- a/firmware/ice40-riscv/icE1usb/usb_gps.c
+++ b/firmware/ice40-riscv/icE1usb/usb_gps.c
@@ -138,7 +138,7 @@
{
/* Default request */
/* Put as static to work around gcc aliasing bug ... */
- static struct usb_cdc_notif_serial_state notif = {
+ static struct usb_cdc_notif_serial_state __attribute__((aligned(4))) notif = {
.hdr = {
.bmRequestType = USB_REQ_READ | USB_REQ_TYPE_CLASS | USB_REQ_RCPT_INTF,
.bRequest = USB_NOTIF_CDC_SERIAL_STATE,

To view, visit change 36186. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-e1-hardware
Gerrit-Branch: master
Gerrit-Change-Id: Id80468ef95bb908cf34b6eeca04c83f1285fd2f5
Gerrit-Change-Number: 36186
Gerrit-PatchSet: 2
Gerrit-Owner: tnt <tnt@246tNt.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: tnt <tnt@246tNt.com>
Gerrit-MessageType: merged