Re: [PATCH] Revert "xhci: Limit USB2 port wake support for AMD Promontory hosts"

[Mathias Nyman]

On 16.09.2017 07:49, Kai-Heng Feng wrote:
> On Mon, Aug 28, 2017 at 9:56 PM, Kai-Heng Feng
> <kai.heng.feng@xxxxxxxxxxxxx> wrote:
> > On Mon, Aug 28, 2017 at 6:14 PM, Mathias Nyman
> > <mathias.nyman@xxxxxxxxxxxxxxx> wrote:
> > > On 28.08.2017 12:29, Greg KH wrote:
> > >
> > > Adding more people who were involved in the original patch.
> > >
> > > Users are now seeing the unresponsive USB2 ports with Promontory hosts.
> > > Is there any update on a better way to solve the original issue.
> > >
> > > To me a "dead" USB2 port seems like a much worse issue for a user
> > > than a BIOS disabled port waking up on plug/unplug (wake on
> > > connect/disconnect),
> > > so I'm myself in favor of doing this revert.
> >
> > At least I can't find "Disable USB2" on my ASUS PRIME B350M-A, so the
> > new behavior is quite surprising.
> >
> > > But there was a strong push from Promontory developers to get the original
> > > fix in,
> > > and I would like to get some comment from them before I do anything about
> > > it.
> >
> > You looped them to the mail thread which I reported the regression two
> > weeks ago, and there is no response since then...
>
> Still no response from AMD and ASMedia guys.
>
> I don't like to use out-of-tree patches for myself, but probably it's
> the only way here?

I'll revert it and send it forward now since rc1 is out

-Mathias