[U-Boot] [UBOOT RFC PATCH 07/13] usb: gadget: g_dnl: Explicitly set the max packet size in descriptor

Felipe Balbi balbi at ti.com
Mon Aug 18 16:38:56 CEST 2014


On Mon, Aug 18, 2014 at 07:58:29PM +0530, Kishon Vijay Abraham I wrote:
> Explicity set the max packet size in the device descriptor to 0x40
> as specified in the device class specification for device firmware upgrade.
> Also changed debug to printf to explicitly notify the user if the
> device has been enumerated.
> 
> Signed-off-by: Kishon Vijay Abraham I <kishon at ti.com>
> ---
>  drivers/usb/gadget/composite.c | 3 +--
>  drivers/usb/gadget/g_dnl.c     | 1 +
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/usb/gadget/composite.c b/drivers/usb/gadget/composite.c
> index 7bd2562..60ca9a5 100644
> --- a/drivers/usb/gadget/composite.c
> +++ b/drivers/usb/gadget/composite.c
> @@ -331,7 +331,7 @@ static int set_config(struct usb_composite_dev *cdev,
>  	} else
>  		result = 0;
>  
> -	debug("%s: %s speed config #%d: %s\n", __func__,
> +	printf("%s: %s speed config #%d: %s\n", __func__,

this looks like it's only needed for debugging. No strong feelings
though

> @@ -987,7 +987,6 @@ static int composite_bind(struct usb_gadget *gadget)
>  
>  	memcpy(&cdev->desc, composite->dev,
>  	       sizeof(struct usb_device_descriptor));
> -	cdev->desc.bMaxPacketSize0 = gadget->ep0->maxpacket;

why are you dropping this line ?

-- 
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140818/e9b17520/attachment.pgp>


More information about the U-Boot mailing list