On 08/21/13 13:58, Mateusz Zalega wrote: > Structure exynos_usb_phy lacked __attribute__ ((packed)), which might > have led to broken EHCI functionality in builds based on more recent > compilers. > > Signed-off-by: Mateusz Zalega <[email protected]> > Signed-off-by: Kyungmin Park <[email protected]> > Cc: Minkyu Kang <[email protected]> > Cc: Rajeshwari Shinde <[email protected]> > --- > Changes since v1: > - changed linux/compiler-gcc.h to linux/compiler.h > > --- > arch/arm/include/asm/arch-exynos/ehci.h | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/include/asm/arch-exynos/ehci.h > b/arch/arm/include/asm/arch-exynos/ehci.h > index d79f25c..c965b2c 100644 > --- a/arch/arm/include/asm/arch-exynos/ehci.h > +++ b/arch/arm/include/asm/arch-exynos/ehci.h > @@ -9,6 +9,7 @@ > > #ifndef __ASM_ARM_ARCH_EHCI_H__ > #define __ASM_ARM_ARCH_EHCI_H__ > +#include <linux/compiler.h> > > #define CLK_24MHZ 5 > > @@ -45,7 +46,7 @@ struct exynos_usb_phy { > unsigned int usbotgsys; > unsigned int reserved4; > unsigned int usbotgtune; > -}; > +} __packed; > > /* Switch on the VBUS power. */ > int board_usb_vbus_init(void); >
I may have reacted too fast. I assumed that we should keep __packed attribute here, because we're referring to platform register and we'd better underline our need to keep compiler from tampering with our layout. It turned out this code will work just fine, because ARM ABI mandates correct layout of structure members in such cases: http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.dui0491c/Babjddhe.html It wouldn't work if there were any bitfields or unaligned members in Exynos register structures, but after more careful examination I can tell there are none. Regards, -- Mateusz Zalega Samsung R&D Institute Poland _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
