[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <SG2PR06MB09190A92BBFE5E530ADF1BD2D8860@SG2PR06MB0919.apcprd06.prod.outlook.com>
Date: Mon, 28 Mar 2016 10:01:18 +0000
From: Yoshihiro Shimoda <yoshihiro.shimoda.uh@...esas.com>
To: Felipe Balbi <balbi@...nel.org>,
Peter Griffin <peter.griffin@...aro.org>
CC: Gregory CLEMENT <gregory.clement@...e-electrons.com>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"mathias.nyman@...el.com" <mathias.nyman@...el.com>,
"gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
"lee.jones@...aro.org" <lee.jones@...aro.org>,
"linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
"maxime.coquelin@...com" <maxime.coquelin@...com>,
"patrice.chotard@...com" <patrice.chotard@...com>,
"stable@...r.kernel.org" <stable@...r.kernel.org>,
"felipe.balbi@...ux.intel.com" <felipe.balbi@...ux.intel.com>
Subject: RE: [PATCH] usb: host: xhci-plat: Make enum xhci_plat_type start at a
non zero value
Hi,
> Sent: Monday, March 28, 2016 5:30 PM
>
> Hi,
>
> Yoshihiro Shimoda <yoshihiro.shimoda.uh@...esas.com> writes:
> >> > ps: there might be bugs there, but it's a holiday and I really shouldn't
> >> > be spending time on this right now ;-)
> >>
> >> I'm also off on holiday now until Sunday 10th April... yay :-)
> >> >
> >> > Anyway, have fun testing. Let me know if it doesn't work.
> >>
> >> I only have access to STi platforms which were broken by this change.
> >> Not any of the platforms which rely on the functionality which
> >> was introduced (although I can't see any reason why your patch wouldn't work).
> >>
> >> Maybe Yoshihiro (on CC) could test this on the Renesas platforms and confirm?
> >
> > Thank you for sending the email to me on CC.
> >
> > I tested Felipe's patch on Renesas platfroms (R-Car Gen2 and Gen3) and
> > I fixed the patch like the following.
> >
> > However, my fixes patch might need to clean the code up more.
> >
> > Changes from Felipe's patch:
> > - Change function names of xhci_rcar_init_quirk() to xhci_rcar_setup_quirk()
>
> I'm not sure renaming that function fits on the same patch ;-) Sounds
> like it should be a separate patch altogether. I'll work on this
> tomorrow if it's okay for you guys to test on your respective platforms :-)
Thank you for the comment!
I also think it should be separate patch ;)
Also I'm okay to test your patch(es) on my platforms :)
Best regards,
Yoshihiro Shimoda
> --
> balbi
Powered by blists - more mailing lists