mbox series

[SRU,Bionic,OEM-B,0/2] Fix "xhci_hcd 0000:00:14.0: Root hub is not suspended"

Message ID 20180703083808.15858-1-kai.heng.feng@canonical.com
Headers show
Series Fix "xhci_hcd 0000:00:14.0: Root hub is not suspended" | expand

Message

Kai-Heng Feng July 3, 2018, 8:38 a.m. UTC
BugLink: https://bugs.launchpad.net/bugs/1779823

[Impact]
Runtime suspended xHC keeps spewing out this message:
xhci_hcd 0000:00:14.0: Root hub is not suspended

[Test]
With the fix, the message stops.

[Fix]
Only decrement PM counter when DBC was started.

[Regression Potential]
Low. It fixes the overlooked logic.

Kai-Heng Feng (1):
  usb: xhci: dbc: Don't decrement runtime PM counter if DBC is not
    started

Lu Baolu (1):
  usb: xhci: dbc: Fix lockdep warning

 drivers/usb/host/xhci-dbgcap.c | 32 ++++++++++++++++++++------------
 drivers/usb/host/xhci-dbgtty.c | 20 ++++++++++++--------
 2 files changed, 32 insertions(+), 20 deletions(-)

Comments

Timo Aaltonen July 5, 2018, 10:32 a.m. UTC | #1
On 03.07.2018 11:38, Kai-Heng Feng wrote:
> BugLink: https://bugs.launchpad.net/bugs/1779823
> 
> [Impact]
> Runtime suspended xHC keeps spewing out this message:
> xhci_hcd 0000:00:14.0: Root hub is not suspended
> 
> [Test]
> With the fix, the message stops.
> 
> [Fix]
> Only decrement PM counter when DBC was started.
> 
> [Regression Potential]
> Low. It fixes the overlooked logic.
> 
> Kai-Heng Feng (1):
>   usb: xhci: dbc: Don't decrement runtime PM counter if DBC is not
>     started
> 
> Lu Baolu (1):
>   usb: xhci: dbc: Fix lockdep warning
> 
>  drivers/usb/host/xhci-dbgcap.c | 32 ++++++++++++++++++++------------
>  drivers/usb/host/xhci-dbgtty.c | 20 ++++++++++++--------
>  2 files changed, 32 insertions(+), 20 deletions(-)

applied to oem-b oem-next
Po-Hsu Lin July 9, 2018, 3:28 a.m. UTC | #2
Clean cherry-pick.
Acked-by: Po-Hsu Lin <po-hsu.lin@canonical.com>
Seth Forshee July 9, 2018, 10:14 p.m. UTC | #3
On Tue, Jul 03, 2018 at 04:38:06PM +0800, Kai-Heng Feng wrote:
> BugLink: https://bugs.launchpad.net/bugs/1779823
> 
> [Impact]
> Runtime suspended xHC keeps spewing out this message:
> xhci_hcd 0000:00:14.0: Root hub is not suspended
> 
> [Test]
> With the fix, the message stops.
> 
> [Fix]
> Only decrement PM counter when DBC was started.
> 
> [Regression Potential]
> Low. It fixes the overlooked logic.

Clean cherry picks, looks reasonable.

Acked-by: Seth Forshee <seth.forshee@canonical.com>

Patch 1 was in 4.16. Applied patch 2 to cosmic/master-next and
unstable/master, thanks!
Kleber Sacilotto de Souza July 27, 2018, 12:31 p.m. UTC | #4
On 07/03/18 10:38, Kai-Heng Feng wrote:
> BugLink: https://bugs.launchpad.net/bugs/1779823
> 
> [Impact]
> Runtime suspended xHC keeps spewing out this message:
> xhci_hcd 0000:00:14.0: Root hub is not suspended
> 
> [Test]
> With the fix, the message stops.
> 
> [Fix]
> Only decrement PM counter when DBC was started.
> 
> [Regression Potential]
> Low. It fixes the overlooked logic.
> 
> Kai-Heng Feng (1):
>   usb: xhci: dbc: Don't decrement runtime PM counter if DBC is not
>     started
> 
> Lu Baolu (1):
>   usb: xhci: dbc: Fix lockdep warning
> 
>  drivers/usb/host/xhci-dbgcap.c | 32 ++++++++++++++++++++------------
>  drivers/usb/host/xhci-dbgtty.c | 20 ++++++++++++--------
>  2 files changed, 32 insertions(+), 20 deletions(-)
> 

Applied to bionic/master-next branch.

Thanks,
Kleber