Hi Heiko, On Thu, 27 Jun 2013 10:04:57 +0200, Heiko Schocher <h...@denx.de> wrote:
> while playing with dfu, I tapped in an unaligned access > when doing on the host side a "lsusb -d [vendornr]: -v" > I get on the board: > > GADGET DRIVER: usb_dnl_dfu > data abort > > MAYBE you should read doc/README.arm-unaligned-accesses > > pc : [<8ff71db8>] lr : [<8ff75aec>] > sp : 8ef40d18 ip : 00000005 fp : 00000000 > r10: 00000000 r9 : 47401410 r8 : 8ef40f38 > r7 : 8ef4aae8 r6 : 0000000a r5 : 8ef4ab28 r4 : 8ef4ab80 > r3 : 0000000a r2 : 00000006 r1 : 00000006 r0 : 8ef4aae8 > Flags: Nzcv IRQs off FIQs on Mode SVC_32 > Resetting CPU ... > > reason is that in the "struct usb_composite_dev" the > "struct usb_device_descriptor desc;" is on an odd address, > and this struct gets accessed in > drivers/usb/gadget/composite.c device_qual() > > Fix it, by align this var "struct desc" fix to an aligned > address. Please keep the commit message to a minimum -- what is wrong and how it is fixed -- and move the rest (context and additional details) after the commit message separator ('---' below). > Signed-off-by: Heiko Schocher <h...@denx.de> > Cc: Marek Vasut <marek.va...@gmail.com> > Cc: Samuel Egli <samuel.e...@siemens.com> > --- > include/linux/usb/composite.h | 2 +- > 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) > > diff --git a/include/linux/usb/composite.h b/include/linux/usb/composite.h > index 53cb095..4f76f88 100644 > --- a/include/linux/usb/composite.h > +++ b/include/linux/usb/composite.h > @@ -331,7 +331,7 @@ struct usb_composite_dev { > /* private: */ > /* internals */ > unsigned int suspended:1; > - struct usb_device_descriptor desc; > + struct usb_device_descriptor __aligned(CONFIG_SYS_CACHELINE_SIZE) desc; > struct list_head configs; > struct usb_composite_driver *driver; > u8 next_string_id; Amicalement, -- Albert. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot