[PATCH][F][SRU B/D/E] UBUNTU: SAUCE: Revert "usb: handle warm-reset port requests on hub resume"

[dann frazier]

On Wed, Jan 8, 2020 at 8:18 AM Stefan Bader <stefan.bader at canonical.com> wrote:
>
> On 07.01.20 23:11, dann frazier wrote:
> > BugLink: https://bugs.launchpad.net/bugs/1856608
> >
> > This reverts upstream commit 4fdc1790e6a9ef22399c6bc6e63b80f4609f3b7e
> > due to a regression.
> >
> > Signed-off-by: dann frazier <dann.frazier at canonical.com>
> > ---
>
> I guess for Bionic its confirmed but are we positive this is the same for D/E?
> Could be a missing pre-req in 4.15 only...

Can't say 100% because we didn't explicitly test those (and doing so
is difficult, as it requires remote hands). But we did explicitly test
upstream, and found that this revert was necessary as of 5.5-rc2. So,
I'd be surprised if D/E were somehow immune.

  -dann

> -Stefan
>
> >  drivers/usb/core/hub.c | 7 -------
> >  1 file changed, 7 deletions(-)
> >
> > diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c
> > index 59b708ffc019f..4ddac9bd68e3c 100644
> > --- a/drivers/usb/core/hub.c
> > +++ b/drivers/usb/core/hub.c
> > @@ -104,8 +104,6 @@ EXPORT_SYMBOL_GPL(ehci_cf_port_reset_rwsem);
> >  static void hub_release(struct kref *kref);
> >  static int usb_reset_and_verify_device(struct usb_device *udev);
> >  static int hub_port_disable(struct usb_hub *hub, int port1, int set_state);
> > -static bool hub_port_warm_reset_required(struct usb_hub *hub, int port1,
> > -             u16 portstatus);
> >
> >  static inline char *portspeed(struct usb_hub *hub, int portstatus)
> >  {
> > @@ -1112,11 +1110,6 @@ static void hub_activate(struct usb_hub *hub, enum hub_activation_type type)
> >                                                  USB_PORT_FEAT_ENABLE);
> >               }
> >
> > -             /* Make sure a warm-reset request is handled by port_event */
> > -             if (type == HUB_RESUME &&
> > -                 hub_port_warm_reset_required(hub, port1, portstatus))
> > -                     set_bit(port1, hub->event_bits);
> > -
> >               /*
> >                * Add debounce if USB3 link is in polling/link training state.
> >                * Link will automatically transition to Enabled state after
> >
>
>