[U-Boot] [PATCH] USB: g_dnl: Change device class

From: Lukasz Majewski <l.majewski@majess.pl>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] USB: g_dnl: Change device class
Date: Fri, 8 Apr 2016 22:12:59 +0200	[thread overview]
Message-ID: <20160408221259.4ed4581a@jawa> (raw)
In-Reply-To: <CACUGKYO8G8ZUY-jA2w-mqoudT-76AjYwEhb54N8shaoV6X7OCg@mail.gmail.com>

On Fri, 8 Apr 2016 08:41:14 -0700
John Tobias <john.tobias.ph@gmail.com> wrote:

> Hi Lukas,
> 
> 
> On Fri, Apr 8, 2016 at 2:19 AM, Lukasz Majewski
> <l.majewski@samsung.com> wrote:
> 
> > Hi John,
> >
> > > The USB Mass Storage (ums) works in Windows, Linux and OS X (EL
> > > Capitan). But, not in OS X (Yosemite). By applying the said
> > > patch, it extends the ums support.
> > >
> > > Signed-off-by: John Tobias <john.tobias.ph@gmail.com>
> > > ---
> > >  drivers/usb/gadget/g_dnl.c | 4 ++--
> > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/usb/gadget/g_dnl.c
> > > b/drivers/usb/gadget/g_dnl.c index 2fa6da4..bd3aad9 100644
> > > --- a/drivers/usb/gadget/g_dnl.c
> > > +++ b/drivers/usb/gadget/g_dnl.c
> > > @@ -58,8 +58,8 @@ static struct usb_device_descriptor device_desc
> > > = { .bDescriptorType = USB_DT_DEVICE,
> > >
> > >       .bcdUSB = __constant_cpu_to_le16(0x0200),
> > > -     .bDeviceClass = USB_CLASS_COMM,
> > > -     .bDeviceSubClass = 0x02, /*0x02:CDC-modem ,
> > > 0x00:CDC-serial*/
> > > +     .bDeviceClass = USB_CLASS_PER_INTERFACE,
> > > +     .bDeviceSubClass = 0, /*0x02:CDC-modem , 0x00:CDC-serial*/
> >
> > Have you tested if this patch is not introducing any regression on
> > Linux and Windows?
> >
> 
> The test what I did is like this:
> 
> Linux:
>     - Run ums to expose all my eMMC partition - shows all correctly
>     - Run ums to expose only 1 partition of my eMMC - show correctly
> 
> Windows:
>     - Run ums to expose all my eMMC partition - it detects but it
> prompts, if I want to format it (due to a non windows partition)
>     - Run ums to expose only the FAT32 partition - it show the
> partition correctly.
> 

Ok, I see. Thanks for your effort on testing. I do appreciate it.

> 
> Note: I have a patch to expose only 1 partition:
> e.g: ums 0 mmc 1:8

Yes, I've just received it. I will provide feedback shortly.

> 
> 
> >
> > >
> > >       .idVendor = __constant_cpu_to_le16(CONFIG_G_DNL_VENDOR_NUM),
> > >       .idProduct =
> > > __constant_cpu_to_le16(CONFIG_G_DNL_PRODUCT_NUM),
> >
> >
> >
> > --
> > Best regards,
> >
> > Lukasz Majewski
> >
> > Samsung R&D Institute Poland (SRPOL) | Linux Platform Group
> >
> 
> Regards,
> 
> John
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160408/ab9e0bea/attachment.sig>