From: Seung-Woo Kim Date: Tue, 10 Mar 2020 06:06:07 +0000 (+0900) Subject: Revert "usb: thor: fix possible alignment issues" X-Git-Tag: accepted/tizen/unified/20200311.130826~1 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;ds=sidebyside;h=942eea7147ee8dee3e0701ddcb591ee5a0e0315e;p=profile%2Fmobile%2Fplatform%2Fkernel%2Fu-boot-tm1.git Revert "usb: thor: fix possible alignment issues" This reverts commit d1cf1d9dddf3d53d8b1fb08033bd6504f7374349. Windows THOR tool fails to communucate with this change, so revert it and build warning with gcc9 will be fixed with ohter way. Change-Id: Id2d3fd38951bf9c00d7f040605afc12a31e20ecf Reported-by: Jeongmo Yang Suggested-by: Jaehoon Chung --- diff --git a/property/usb/cdc_descriptor.c b/property/usb/cdc_descriptor.c index 6dba80f..bf27148 100644 --- a/property/usb/cdc_descriptor.c +++ b/property/usb/cdc_descriptor.c @@ -8,8 +8,6 @@ #define THOR_VENDOR_NUM 0x04E8 #define THOR_PRODUCT_NUM 0x685D -typedef struct { __le16 val; } __attribute__((aligned(16))) __le16_packed; - static struct usb_device_descriptor thor_device_desc_high __align (32)= { .bLength = sizeof(thor_device_desc_high), .bDescriptorType = USB_DT_DEVICE, @@ -274,15 +272,14 @@ unsigned char *thor_get_config_desc(unsigned int speed) return (unsigned char *) &function_desc_buf; } -static void str2wide (char *str, void *wide) +static void str2wide (char *str, u16 * wide) { int i; - __le16_packed *tmp = wide; for (i = 0; i < strlen (str) && str[i]; i++){ #if defined(__LITTLE_ENDIAN) - tmp[i].val = (u16) str[i]; + wide[i] = (u16) str[i]; #elif defined(__BIG_ENDIAN) - tmp[i].val = ((u16)(str[i])<<8); + wide[i] = ((u16)(str[i])<<8); #else #error "__LITTLE_ENDIAN or __BIG_ENDIAN undefined" #endif