Re: [PATCH] usb: gadget: fsl: Add FSL USB Gadget entry in platform device id

From: Felipe Balbi
Date: Fri Mar 14 2014 - 21:42:09 EST


On Fri, Mar 14, 2014 at 08:52:19PM +0000, suresh.gupta@xxxxxxxxxxxxx wrote:
> Hi,
> Thanks for reviewing my patches.
> Please find my comments inline
>
> -----Original Message-----
> From: Felipe Balbi [mailto:balbi@xxxxxx]
> Sent: Thursday, March 13, 2014 8:56 PM
> To: Gupta Suresh-B42813
> Cc: balbi@xxxxxx; gregkh@xxxxxxxxxxxxxxxxxxx; linux-usb@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Gupta Suresh-B42813
> Subject: Re: [PATCH] usb: gadget: fsl: Add FSL USB Gadget entry in platform device id
>
> On Thu, Mar 13, 2014 at 07:35:31PM +0530, Suresh Gupta wrote:
> > From: Suresh Gupta <B42813@xxxxxxxxxxxxx>
> >
> > Add FSL USB Gadget entry in platform device id table
>
> why this tab ?
> [SuresH] I will remove it in next version.
>
> > Signed-off-by: Suresh Gupta <B42813@xxxxxxxxxxxxx>
> > ---
> > drivers/usb/gadget/fsl_udc_core.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/usb/gadget/fsl_udc_core.c b/drivers/usb/gadget/fsl_udc_core.c
> > index b7dea4e..35b20e6 100644
> > --- a/drivers/usb/gadget/fsl_udc_core.c
> > +++ b/drivers/usb/gadget/fsl_udc_core.c
> > @@ -2654,6 +2654,8 @@ static const struct platform_device_id fsl_udc_devtype[] = {
> > }, {
> > .name = "imx-udc-mx51",
> > }, {
> > + .name = "fsl-usb2-udc",
>
> why aren't you just using chipidea ?
> [SuresH] This is our legacy driver for all previous and existing ppc
> socs. Many of our customers are already using this, and we need to
> support them on this driver. We do have plans to shift to chipidea,
> but after some time.

cool, you already have plans, so we will see a new glue layer for v3.16
right ? Which means I don't need to take this patch either.

--
balbi

Attachment: signature.asc
Description: Digital signature