From mboxrd@z Thu Jan  1 00:00:00 1970
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
Date: Thu, 27 Jun 2013 12:11:36 +0200
Subject: [U-Boot] [PATCH] usb: fix unaligned access in device_qual()
In-Reply-To: <1372320297-16275-1-git-send-email-hs@denx.de>
References: <1372320297-16275-1-git-send-email-hs@denx.de>
Message-ID: <20130627121136.0411b3f1@lilith>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Heiko,

On Thu, 27 Jun 2013 10:04:57 +0200, Heiko Schocher <hs@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 <hs@denx.de>
> Cc: Marek Vasut <marek.vasut@gmail.com>
> Cc: Samuel Egli <samuel.egli@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.