All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] usb:udc: Remove duplicated USB definitions from include/linux/usb/ch9.h file
@ 2012-04-12  7:10 Lukasz Majewski
  2012-04-12  8:08 ` Marek Vasut
  2012-04-14  9:42 ` Marek Vasut
  0 siblings, 2 replies; 4+ messages in thread
From: Lukasz Majewski @ 2012-04-12  7:10 UTC (permalink / raw)
  To: u-boot

Remove the repeated USB descriptor code and use usbdescriptors.h file.
ch9.h file has been copied from linux and is needed for USB gadget
related work.
Now usbdescriptors.h and ch9.h shall be used together.

Signed-off-by: Lukasz Majewski <l.majewski@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Cc: Marek Vasut <marex@denx.de>
---
 drivers/usb/gadget/config.c      |    1 +
 drivers/usb/gadget/epautoconf.c  |    1 +
 drivers/usb/gadget/ether.c       |    1 +
 drivers/usb/gadget/s3c_udc_otg.c |    1 +
 drivers/usb/gadget/usbstring.c   |    1 +
 include/linux/usb/ch9.h          |   67 --------------------------------------
 include/usb/s3c_udc.h            |    1 +
 7 files changed, 6 insertions(+), 67 deletions(-)

diff --git a/drivers/usb/gadget/config.c b/drivers/usb/gadget/config.c
index f9163a8..f88d0c1 100644
--- a/drivers/usb/gadget/config.c
+++ b/drivers/usb/gadget/config.c
@@ -27,6 +27,7 @@
 #include <linux/string.h>
 
 #include <linux/usb/ch9.h>
+#include <usbdescriptors.h>
 #include <linux/usb/gadget.h>
 
 
diff --git a/drivers/usb/gadget/epautoconf.c b/drivers/usb/gadget/epautoconf.c
index 5b8776e..b656c8b 100644
--- a/drivers/usb/gadget/epautoconf.c
+++ b/drivers/usb/gadget/epautoconf.c
@@ -23,6 +23,7 @@
 
 #include <common.h>
 #include <linux/usb/ch9.h>
+#include <usbdescriptors.h>
 #include <asm/errno.h>
 #include <linux/usb/gadget.h>
 #include <asm/unaligned.h>
diff --git a/drivers/usb/gadget/ether.c b/drivers/usb/gadget/ether.c
index 9fb0e80..f693fea 100644
--- a/drivers/usb/gadget/ether.c
+++ b/drivers/usb/gadget/ether.c
@@ -24,6 +24,7 @@
 #include <asm/errno.h>
 #include <linux/netdevice.h>
 #include <linux/usb/ch9.h>
+#include <usbdescriptors.h>
 #include <linux/usb/cdc.h>
 #include <linux/usb/gadget.h>
 #include <net.h>
diff --git a/drivers/usb/gadget/s3c_udc_otg.c b/drivers/usb/gadget/s3c_udc_otg.c
index 1050a98..f7f7b54 100644
--- a/drivers/usb/gadget/s3c_udc_otg.c
+++ b/drivers/usb/gadget/s3c_udc_otg.c
@@ -37,6 +37,7 @@
 #include <malloc.h>
 
 #include <linux/usb/ch9.h>
+#include <usbdescriptors.h>
 #include <linux/usb/gadget.h>
 
 #include <asm/byteorder.h>
diff --git a/drivers/usb/gadget/usbstring.c b/drivers/usb/gadget/usbstring.c
index 95555cf..4dbe060 100644
--- a/drivers/usb/gadget/usbstring.c
+++ b/drivers/usb/gadget/usbstring.c
@@ -13,6 +13,7 @@
 #include <common.h>
 #include <asm/errno.h>
 #include <linux/usb/ch9.h>
+#include <usbdescriptors.h>
 #include <linux/usb/gadget.h>
 
 #include <asm/unaligned.h>
diff --git a/include/linux/usb/ch9.h b/include/linux/usb/ch9.h
index 49b7483..ce1d1e1 100644
--- a/include/linux/usb/ch9.h
+++ b/include/linux/usb/ch9.h
@@ -204,28 +204,6 @@ struct usb_descriptor_header {
 	__u8  bDescriptorType;
 } __attribute__ ((packed));
 
-
-/*-------------------------------------------------------------------------*/
-
-/* USB_DT_DEVICE: Device descriptor */
-struct usb_device_descriptor {
-	__u8  bLength;
-	__u8  bDescriptorType;
-
-	__le16 bcdUSB;
-	__u8  bDeviceClass;
-	__u8  bDeviceSubClass;
-	__u8  bDeviceProtocol;
-	__u8  bMaxPacketSize0;
-	__le16 idVendor;
-	__le16 idProduct;
-	__le16 bcdDevice;
-	__u8  iManufacturer;
-	__u8  iProduct;
-	__u8  iSerialNumber;
-	__u8  bNumConfigurations;
-} __attribute__ ((packed));
-
 #define USB_DT_DEVICE_SIZE		18
 
 
@@ -282,56 +260,11 @@ struct usb_config_descriptor {
 #define USB_CONFIG_ATT_WAKEUP		(1 << 5)	/* can wakeup */
 #define USB_CONFIG_ATT_BATTERY		(1 << 4)	/* battery powered */
 
-/*-------------------------------------------------------------------------*/
-
-/* USB_DT_STRING: String descriptor */
-struct usb_string_descriptor {
-	__u8  bLength;
-	__u8  bDescriptorType;
-
-	__le16 wData[1];		/* UTF-16LE encoded */
-} __attribute__ ((packed));
-
 /* note that "string" zero is special, it holds language codes that
  * the device supports, not Unicode characters.
  */
 
-/*-------------------------------------------------------------------------*/
-
-/* USB_DT_INTERFACE: Interface descriptor */
-struct usb_interface_descriptor {
-	__u8  bLength;
-	__u8  bDescriptorType;
-
-	__u8  bInterfaceNumber;
-	__u8  bAlternateSetting;
-	__u8  bNumEndpoints;
-	__u8  bInterfaceClass;
-	__u8  bInterfaceSubClass;
-	__u8  bInterfaceProtocol;
-	__u8  iInterface;
-} __attribute__ ((packed));
-
 #define USB_DT_INTERFACE_SIZE		9
-
-/*-------------------------------------------------------------------------*/
-
-/* USB_DT_ENDPOINT: Endpoint descriptor */
-struct usb_endpoint_descriptor {
-	__u8  bLength;
-	__u8  bDescriptorType;
-
-	__u8  bEndpointAddress;
-	__u8  bmAttributes;
-	__le16 wMaxPacketSize;
-	__u8  bInterval;
-
-	/* NOTE:  these two are _only_ in audio endpoints. */
-	/* use USB_DT_ENDPOINT*_SIZE in bLength, not sizeof. */
-	__u8  bRefresh;
-	__u8  bSynchAddress;
-} __attribute__ ((packed));
-
 #define USB_DT_ENDPOINT_SIZE		7
 #define USB_DT_ENDPOINT_AUDIO_SIZE	9	/* Audio extension */
 
diff --git a/include/usb/s3c_udc.h b/include/usb/s3c_udc.h
index 6c973b6..3e6135a 100644
--- a/include/usb/s3c_udc.h
+++ b/include/usb/s3c_udc.h
@@ -24,6 +24,7 @@
 
 #include <asm/errno.h>
 #include <linux/usb/ch9.h>
+#include <usbdescriptors.h>
 #include <linux/usb/gadget.h>
 #include <linux/list.h>
 #include <usb/lin_gadget_compat.h>
-- 
1.7.2.3

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH] usb:udc: Remove duplicated USB definitions from include/linux/usb/ch9.h file
  2012-04-12  7:10 [U-Boot] [PATCH] usb:udc: Remove duplicated USB definitions from include/linux/usb/ch9.h file Lukasz Majewski
@ 2012-04-12  8:08 ` Marek Vasut
  2012-04-12  9:06   ` Lukasz Majewski
  2012-04-14  9:42 ` Marek Vasut
  1 sibling, 1 reply; 4+ messages in thread
From: Marek Vasut @ 2012-04-12  8:08 UTC (permalink / raw)
  To: u-boot

Dear Lukasz Majewski,

> Remove the repeated USB descriptor code and use usbdescriptors.h file.
> ch9.h file has been copied from linux and is needed for USB gadget
> related work.
> Now usbdescriptors.h and ch9.h shall be used together.

Please don't expect my response on these patches earlier than in 2 days time -- 
I have a slow build-test machine. Also, I'll be eventually scheduling these for 
next release, ok?

Though I'll likely review these today.

btw. this patch looks reasonable, but I'll need to build-test it .

> 
> Signed-off-by: Lukasz Majewski <l.majewski@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> Cc: Marek Vasut <marex@denx.de>
> ---
>  drivers/usb/gadget/config.c      |    1 +
>  drivers/usb/gadget/epautoconf.c  |    1 +
>  drivers/usb/gadget/ether.c       |    1 +
>  drivers/usb/gadget/s3c_udc_otg.c |    1 +
>  drivers/usb/gadget/usbstring.c   |    1 +
>  include/linux/usb/ch9.h          |   67
> -------------------------------------- include/usb/s3c_udc.h            | 
>   1 +
>  7 files changed, 6 insertions(+), 67 deletions(-)
> 
> diff --git a/drivers/usb/gadget/config.c b/drivers/usb/gadget/config.c
> index f9163a8..f88d0c1 100644
> --- a/drivers/usb/gadget/config.c
> +++ b/drivers/usb/gadget/config.c
> @@ -27,6 +27,7 @@
>  #include <linux/string.h>
> 
>  #include <linux/usb/ch9.h>
> +#include <usbdescriptors.h>
>  #include <linux/usb/gadget.h>
> 
> 
> diff --git a/drivers/usb/gadget/epautoconf.c
> b/drivers/usb/gadget/epautoconf.c index 5b8776e..b656c8b 100644
> --- a/drivers/usb/gadget/epautoconf.c
> +++ b/drivers/usb/gadget/epautoconf.c
> @@ -23,6 +23,7 @@
> 
>  #include <common.h>
>  #include <linux/usb/ch9.h>
> +#include <usbdescriptors.h>
>  #include <asm/errno.h>
>  #include <linux/usb/gadget.h>
>  #include <asm/unaligned.h>
> diff --git a/drivers/usb/gadget/ether.c b/drivers/usb/gadget/ether.c
> index 9fb0e80..f693fea 100644
> --- a/drivers/usb/gadget/ether.c
> +++ b/drivers/usb/gadget/ether.c
> @@ -24,6 +24,7 @@
>  #include <asm/errno.h>
>  #include <linux/netdevice.h>
>  #include <linux/usb/ch9.h>
> +#include <usbdescriptors.h>
>  #include <linux/usb/cdc.h>
>  #include <linux/usb/gadget.h>
>  #include <net.h>
> diff --git a/drivers/usb/gadget/s3c_udc_otg.c
> b/drivers/usb/gadget/s3c_udc_otg.c index 1050a98..f7f7b54 100644
> --- a/drivers/usb/gadget/s3c_udc_otg.c
> +++ b/drivers/usb/gadget/s3c_udc_otg.c
> @@ -37,6 +37,7 @@
>  #include <malloc.h>
> 
>  #include <linux/usb/ch9.h>
> +#include <usbdescriptors.h>
>  #include <linux/usb/gadget.h>
> 
>  #include <asm/byteorder.h>
> diff --git a/drivers/usb/gadget/usbstring.c
> b/drivers/usb/gadget/usbstring.c index 95555cf..4dbe060 100644
> --- a/drivers/usb/gadget/usbstring.c
> +++ b/drivers/usb/gadget/usbstring.c
> @@ -13,6 +13,7 @@
>  #include <common.h>
>  #include <asm/errno.h>
>  #include <linux/usb/ch9.h>
> +#include <usbdescriptors.h>
>  #include <linux/usb/gadget.h>
> 
>  #include <asm/unaligned.h>
> diff --git a/include/linux/usb/ch9.h b/include/linux/usb/ch9.h
> index 49b7483..ce1d1e1 100644
> --- a/include/linux/usb/ch9.h
> +++ b/include/linux/usb/ch9.h
> @@ -204,28 +204,6 @@ struct usb_descriptor_header {
>  	__u8  bDescriptorType;
>  } __attribute__ ((packed));
> 
> -
> -/*------------------------------------------------------------------------
> -*/ -
> -/* USB_DT_DEVICE: Device descriptor */
> -struct usb_device_descriptor {
> -	__u8  bLength;
> -	__u8  bDescriptorType;
> -
> -	__le16 bcdUSB;
> -	__u8  bDeviceClass;
> -	__u8  bDeviceSubClass;
> -	__u8  bDeviceProtocol;
> -	__u8  bMaxPacketSize0;
> -	__le16 idVendor;
> -	__le16 idProduct;
> -	__le16 bcdDevice;
> -	__u8  iManufacturer;
> -	__u8  iProduct;
> -	__u8  iSerialNumber;
> -	__u8  bNumConfigurations;
> -} __attribute__ ((packed));
> -
>  #define USB_DT_DEVICE_SIZE		18
> 
> 
> @@ -282,56 +260,11 @@ struct usb_config_descriptor {
>  #define USB_CONFIG_ATT_WAKEUP		(1 << 5)	/* can wakeup */
>  #define USB_CONFIG_ATT_BATTERY		(1 << 4)	/* battery 
powered */
> 
> -/*------------------------------------------------------------------------
> -*/ -
> -/* USB_DT_STRING: String descriptor */
> -struct usb_string_descriptor {
> -	__u8  bLength;
> -	__u8  bDescriptorType;
> -
> -	__le16 wData[1];		/* UTF-16LE encoded */
> -} __attribute__ ((packed));
> -
>  /* note that "string" zero is special, it holds language codes that
>   * the device supports, not Unicode characters.
>   */
> 
> -/*------------------------------------------------------------------------
> -*/ -
> -/* USB_DT_INTERFACE: Interface descriptor */
> -struct usb_interface_descriptor {
> -	__u8  bLength;
> -	__u8  bDescriptorType;
> -
> -	__u8  bInterfaceNumber;
> -	__u8  bAlternateSetting;
> -	__u8  bNumEndpoints;
> -	__u8  bInterfaceClass;
> -	__u8  bInterfaceSubClass;
> -	__u8  bInterfaceProtocol;
> -	__u8  iInterface;
> -} __attribute__ ((packed));
> -
>  #define USB_DT_INTERFACE_SIZE		9
> -
> -/*------------------------------------------------------------------------
> -*/ -
> -/* USB_DT_ENDPOINT: Endpoint descriptor */
> -struct usb_endpoint_descriptor {
> -	__u8  bLength;
> -	__u8  bDescriptorType;
> -
> -	__u8  bEndpointAddress;
> -	__u8  bmAttributes;
> -	__le16 wMaxPacketSize;
> -	__u8  bInterval;
> -
> -	/* NOTE:  these two are _only_ in audio endpoints. */
> -	/* use USB_DT_ENDPOINT*_SIZE in bLength, not sizeof. */
> -	__u8  bRefresh;
> -	__u8  bSynchAddress;
> -} __attribute__ ((packed));
> -
>  #define USB_DT_ENDPOINT_SIZE		7
>  #define USB_DT_ENDPOINT_AUDIO_SIZE	9	/* Audio extension */
> 
> diff --git a/include/usb/s3c_udc.h b/include/usb/s3c_udc.h
> index 6c973b6..3e6135a 100644
> --- a/include/usb/s3c_udc.h
> +++ b/include/usb/s3c_udc.h
> @@ -24,6 +24,7 @@
> 
>  #include <asm/errno.h>
>  #include <linux/usb/ch9.h>
> +#include <usbdescriptors.h>
>  #include <linux/usb/gadget.h>
>  #include <linux/list.h>
>  #include <usb/lin_gadget_compat.h>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH] usb:udc: Remove duplicated USB definitions from include/linux/usb/ch9.h file
  2012-04-12  8:08 ` Marek Vasut
@ 2012-04-12  9:06   ` Lukasz Majewski
  0 siblings, 0 replies; 4+ messages in thread
From: Lukasz Majewski @ 2012-04-12  9:06 UTC (permalink / raw)
  To: u-boot


Hi Marek,

> Dear Lukasz Majewski,
> 
> > Remove the repeated USB descriptor code and use usbdescriptors.h
> > file. ch9.h file has been copied from linux and is needed for USB
> > gadget related work.
> > Now usbdescriptors.h and ch9.h shall be used together.


> 
> Please don't expect my response on these patches earlier than in 2
> days time -- I have a slow build-test machine. Also, I'll be
> eventually scheduling these for next release, ok?

I think that good testing is welcome, since <usbstrings.h> and <ch9.h>
are used widely (up to now separately) in USB. It can be submitted for
-rc or next release. It depends if we manage to test it thoroughly.

BTW: Can you share information how could I get toolchains for mips,
powerPC to test more architectures from MAKEALL script?

Do you use ELDK 5.1 or any "customized" toolchain?
I'm planning to setup some build server.

> 
> Though I'll likely review these today.

Great.
> 
> btw. this patch looks reasonable, but I'll need to build-test it .
> 


-- 
Best regards,

Lukasz Majewski

Samsung Poland R&D Center
Platform Group

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH] usb:udc: Remove duplicated USB definitions from include/linux/usb/ch9.h file
  2012-04-12  7:10 [U-Boot] [PATCH] usb:udc: Remove duplicated USB definitions from include/linux/usb/ch9.h file Lukasz Majewski
  2012-04-12  8:08 ` Marek Vasut
@ 2012-04-14  9:42 ` Marek Vasut
  1 sibling, 0 replies; 4+ messages in thread
From: Marek Vasut @ 2012-04-14  9:42 UTC (permalink / raw)
  To: u-boot

Dear Lukasz Majewski,

> Remove the repeated USB descriptor code and use usbdescriptors.h file.
> ch9.h file has been copied from linux and is needed for USB gadget
> related work.
> Now usbdescriptors.h and ch9.h shall be used together.
> 
> Signed-off-by: Lukasz Majewski <l.majewski@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> Cc: Marek Vasut <marex@denx.de>

Acked-by: Marek Vasut <marek.vasut@gmail.com>

> ---
>  drivers/usb/gadget/config.c      |    1 +
>  drivers/usb/gadget/epautoconf.c  |    1 +
>  drivers/usb/gadget/ether.c       |    1 +
>  drivers/usb/gadget/s3c_udc_otg.c |    1 +
>  drivers/usb/gadget/usbstring.c   |    1 +
>  include/linux/usb/ch9.h          |   67
> -------------------------------------- include/usb/s3c_udc.h            | 
>   1 +
>  7 files changed, 6 insertions(+), 67 deletions(-)
> 
> diff --git a/drivers/usb/gadget/config.c b/drivers/usb/gadget/config.c
> index f9163a8..f88d0c1 100644
> --- a/drivers/usb/gadget/config.c
> +++ b/drivers/usb/gadget/config.c
> @@ -27,6 +27,7 @@
>  #include <linux/string.h>
> 
>  #include <linux/usb/ch9.h>
> +#include <usbdescriptors.h>
>  #include <linux/usb/gadget.h>
> 
> 
> diff --git a/drivers/usb/gadget/epautoconf.c
> b/drivers/usb/gadget/epautoconf.c index 5b8776e..b656c8b 100644
> --- a/drivers/usb/gadget/epautoconf.c
> +++ b/drivers/usb/gadget/epautoconf.c
> @@ -23,6 +23,7 @@
> 
>  #include <common.h>
>  #include <linux/usb/ch9.h>
> +#include <usbdescriptors.h>
>  #include <asm/errno.h>
>  #include <linux/usb/gadget.h>
>  #include <asm/unaligned.h>
> diff --git a/drivers/usb/gadget/ether.c b/drivers/usb/gadget/ether.c
> index 9fb0e80..f693fea 100644
> --- a/drivers/usb/gadget/ether.c
> +++ b/drivers/usb/gadget/ether.c
> @@ -24,6 +24,7 @@
>  #include <asm/errno.h>
>  #include <linux/netdevice.h>
>  #include <linux/usb/ch9.h>
> +#include <usbdescriptors.h>
>  #include <linux/usb/cdc.h>
>  #include <linux/usb/gadget.h>
>  #include <net.h>
> diff --git a/drivers/usb/gadget/s3c_udc_otg.c
> b/drivers/usb/gadget/s3c_udc_otg.c index 1050a98..f7f7b54 100644
> --- a/drivers/usb/gadget/s3c_udc_otg.c
> +++ b/drivers/usb/gadget/s3c_udc_otg.c
> @@ -37,6 +37,7 @@
>  #include <malloc.h>
> 
>  #include <linux/usb/ch9.h>
> +#include <usbdescriptors.h>
>  #include <linux/usb/gadget.h>
> 
>  #include <asm/byteorder.h>
> diff --git a/drivers/usb/gadget/usbstring.c
> b/drivers/usb/gadget/usbstring.c index 95555cf..4dbe060 100644
> --- a/drivers/usb/gadget/usbstring.c
> +++ b/drivers/usb/gadget/usbstring.c
> @@ -13,6 +13,7 @@
>  #include <common.h>
>  #include <asm/errno.h>
>  #include <linux/usb/ch9.h>
> +#include <usbdescriptors.h>
>  #include <linux/usb/gadget.h>
> 
>  #include <asm/unaligned.h>
> diff --git a/include/linux/usb/ch9.h b/include/linux/usb/ch9.h
> index 49b7483..ce1d1e1 100644
> --- a/include/linux/usb/ch9.h
> +++ b/include/linux/usb/ch9.h
> @@ -204,28 +204,6 @@ struct usb_descriptor_header {
>  	__u8  bDescriptorType;
>  } __attribute__ ((packed));
> 
> -
> -/*------------------------------------------------------------------------
> -*/ -
> -/* USB_DT_DEVICE: Device descriptor */
> -struct usb_device_descriptor {
> -	__u8  bLength;
> -	__u8  bDescriptorType;
> -
> -	__le16 bcdUSB;
> -	__u8  bDeviceClass;
> -	__u8  bDeviceSubClass;
> -	__u8  bDeviceProtocol;
> -	__u8  bMaxPacketSize0;
> -	__le16 idVendor;
> -	__le16 idProduct;
> -	__le16 bcdDevice;
> -	__u8  iManufacturer;
> -	__u8  iProduct;
> -	__u8  iSerialNumber;
> -	__u8  bNumConfigurations;
> -} __attribute__ ((packed));
> -
>  #define USB_DT_DEVICE_SIZE		18
> 
> 
> @@ -282,56 +260,11 @@ struct usb_config_descriptor {
>  #define USB_CONFIG_ATT_WAKEUP		(1 << 5)	/* can wakeup */
>  #define USB_CONFIG_ATT_BATTERY		(1 << 4)	/* battery 
powered */
> 
> -/*------------------------------------------------------------------------
> -*/ -
> -/* USB_DT_STRING: String descriptor */
> -struct usb_string_descriptor {
> -	__u8  bLength;
> -	__u8  bDescriptorType;
> -
> -	__le16 wData[1];		/* UTF-16LE encoded */
> -} __attribute__ ((packed));
> -
>  /* note that "string" zero is special, it holds language codes that
>   * the device supports, not Unicode characters.
>   */
> 
> -/*------------------------------------------------------------------------
> -*/ -
> -/* USB_DT_INTERFACE: Interface descriptor */
> -struct usb_interface_descriptor {
> -	__u8  bLength;
> -	__u8  bDescriptorType;
> -
> -	__u8  bInterfaceNumber;
> -	__u8  bAlternateSetting;
> -	__u8  bNumEndpoints;
> -	__u8  bInterfaceClass;
> -	__u8  bInterfaceSubClass;
> -	__u8  bInterfaceProtocol;
> -	__u8  iInterface;
> -} __attribute__ ((packed));
> -
>  #define USB_DT_INTERFACE_SIZE		9
> -
> -/*------------------------------------------------------------------------
> -*/ -
> -/* USB_DT_ENDPOINT: Endpoint descriptor */
> -struct usb_endpoint_descriptor {
> -	__u8  bLength;
> -	__u8  bDescriptorType;
> -
> -	__u8  bEndpointAddress;
> -	__u8  bmAttributes;
> -	__le16 wMaxPacketSize;
> -	__u8  bInterval;
> -
> -	/* NOTE:  these two are _only_ in audio endpoints. */
> -	/* use USB_DT_ENDPOINT*_SIZE in bLength, not sizeof. */
> -	__u8  bRefresh;
> -	__u8  bSynchAddress;
> -} __attribute__ ((packed));
> -
>  #define USB_DT_ENDPOINT_SIZE		7
>  #define USB_DT_ENDPOINT_AUDIO_SIZE	9	/* Audio extension */
> 
> diff --git a/include/usb/s3c_udc.h b/include/usb/s3c_udc.h
> index 6c973b6..3e6135a 100644
> --- a/include/usb/s3c_udc.h
> +++ b/include/usb/s3c_udc.h
> @@ -24,6 +24,7 @@
> 
>  #include <asm/errno.h>
>  #include <linux/usb/ch9.h>
> +#include <usbdescriptors.h>
>  #include <linux/usb/gadget.h>
>  #include <linux/list.h>
>  #include <usb/lin_gadget_compat.h>

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-04-14  9:42 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-12  7:10 [U-Boot] [PATCH] usb:udc: Remove duplicated USB definitions from include/linux/usb/ch9.h file Lukasz Majewski
2012-04-12  8:08 ` Marek Vasut
2012-04-12  9:06   ` Lukasz Majewski
2012-04-14  9:42 ` Marek Vasut

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.