From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mailout1.hostsharing.net ([83.223.95.204]:46109 "EHLO mailout1.hostsharing.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751273AbdBDXc4 (ORCPT ); Sat, 4 Feb 2017 18:32:56 -0500 Date: Sun, 5 Feb 2017 00:34:43 +0100 From: Lukas Wunner To: Yinghai Lu Cc: Bjorn Helgaas , "Rafael J. Wysocki" , "linux-pci@vger.kernel.org" , Mika Westerberg Subject: Re: pciehp is broken from 4.10-rc1 Message-ID: <20170204233443.GA234@wunner.de> References: <20170203055200.GA29413@wunner.de> <20170204081254.GA29595@wunner.de> <20170204185607.GA29957@wunner.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: Sender: linux-pci-owner@vger.kernel.org List-ID: On Sat, Feb 04, 2017 at 01:44:34PM -0800, Yinghai Lu wrote: > On Sat, Feb 4, 2017 at 10:56 AM, Lukas Wunner wrote: > > On Sat, Feb 04, 2017 at 09:12:54AM +0100, Lukas Wunner wrote: > > Section 6.7.3.4 of the PCIe Base spec seems to support the theory above, > > so here's a tentative patch. > > > > > > -- >8 -- > > Subject: [PATCH] PCI: pciehp: Don't enable PME on runtime suspend > > it works: Thanks a lot for the report and for testing the patch! @Rafael: Bjorn usually wants you to ack PM-related patches. Looking at the patch again I'm now thinking it might be cleaner to move the check for is_hotplug_bridge into pci_dev_run_wake(). Thoughts? Thanks, Lukas