From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Mon, 18 Aug 2014 16:49:17 +0200 Subject: [U-Boot] [UBOOT RFC PATCH 07/13] usb: gadget: g_dnl: Explicitly set the max packet size in descriptor In-Reply-To: <20140818143856.GF9662@saruman.home> References: <1408372115-4570-1-git-send-email-kishon@ti.com> <1408372115-4570-8-git-send-email-kishon@ti.com> <20140818143856.GF9662@saruman.home> Message-ID: <20140818164917.3406bd30@amdc2363> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Kishon, > 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. It seems like this patch has slipped without my attention. Hence I'd like to ask why do you need to hardcode the 0x40 value? > > > > Signed-off-by: Kishon Vijay Abraham I > > --- > > 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 ? > -- Best regards, Lukasz Majewski Samsung R&D Institute Poland (SRPOL) | Linux Platform Group