RE: [PATCH v2 REGRESSION RESEND] usb: ohci-at91: use descriptor-based gpio APIs correctly

From	<Wenyou.Yang@microchip.com>
Newsgroups	linux.kernel
Subject	RE: [PATCH v2 REGRESSION RESEND] usb: ohci-at91: use descriptor-based gpio APIs correctly
Date	2016-12-26 06:20 +0100
Message-ID	<sSvRv-hq-3@gated-at.bofh.it> (permalink)
References	<sRflL-3Vq-5@gated-at.bofh.it> <sRflL-3Vq-3@gated-at.bofh.it> <sRCs1-1T6-3@gated-at.bofh.it> <sRCs1-1T6-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw


> -----Original Message-----
> From: Greg Kroah-Hartman [mailto:gregkh@linuxfoundation.org]
> Sent: 2016年12月24日 2:05
> To: Peter Rosin <peda@axentia.se>
> Cc: linux-kernel@vger.kernel.org; Alan Stern <stern@rowland.harvard.edu>;
> linux-usb@vger.kernel.org; Wenyou Yang - A41535
> <Wenyou.Yang@microchip.com>
> Subject: Re: [PATCH v2 REGRESSION RESEND] usb: ohci-at91: use descriptor-
> based gpio APIs correctly
> 
> On Thu, Dec 22, 2016 at 09:38:08PM +0100, Peter Rosin wrote:
> > On 2016-12-22 18:27, Greg Kroah-Hartman wrote:
> > > On Thu, Dec 22, 2016 at 08:43:55AM +0100, Peter Rosin wrote:
> > >> The gpiod_get* function family does not want the -gpio suffix.
> > >> Use devm_gpiod_get_index_optional instead of devm_gpiod_get_optional.
> > >> The descriptor based APIs handle active high/low automatically.
> > >> The vbus-gpios are output, request enable while getting the gpio.
> > >> Don't try to get any vbus-gpios for ports outside num-ports.
> > >>
> > >> WTF? Big sigh.
> > >>
> > >> Fixes: 054d4b7b577d ("usb: ohci-at91: Use descriptor-based gpio
> > >> APIs")
> > >> Signed-off-by: Peter Rosin <peda@axentia.se>
> > >> ---
> > >>
> > >> Hi!
> > >>
> > >> Resending this, since the only response I've got is that the merge
> > >> window is open and that this patch has been put on hold due to that.
> > >> But I think this regression (which happend between v4.9 and current
> > >> master) should be fixed before the merge window closes.
> > >
> > > I don't merge patches before -rc1 comes out, sorry, people should
> > > have tested linux-next better :)
> >
> > Neat, shift the blame for the shit patch over to the messenger :)
> 
> Not at all, I blame the original developer :)

I am very very sorry. It is my ignorance. Sorry.

I tested this patch on linux-next branch on the SAMA5D3 and SAMA5D4 Xplained board.

> 
> > > I'll catch up the first week of January, relax.
> >
> > As we all know, unrelated regressions are painful when trying to
> > locate other problems. It's seems silly to have a few extra for no good reason.
> 
> I am supposed to be on vacation and not reading email until the 3rd of January,
> relax, we will catch up on stuff like this, and other minor things, soon enough, in
> plenty of time for 4.10-final.
> 
> thanks,
> 
> greg k-h


Best Regards,
Wenyou Yang

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH v2 REGRESSION RESEND] usb: ohci-at91: use  descriptor-based gpio APIs correctly Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-12-22 18:30 +0100
  Re: [PATCH v2 REGRESSION RESEND] usb: ohci-at91: use  descriptor-based gpio APIs correctly Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-12-23 19:10 +0100
    RE: [PATCH v2 REGRESSION RESEND] usb: ohci-at91: use  descriptor-based gpio APIs correctly <Wenyou.Yang@microchip.com> - 2016-12-26 06:20 +0100

csiph-web