From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0896B221FBD; Mon, 18 Aug 2025 02:01:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1755482490; cv=none; b=dGIubYA/CwiePmimVuJ8puGnwW/XCdqcrEAhMdqSKMqsrY3fhEY4WKBc9X+vyMX3tXCIEdDxgIwNrTaor2DzCMj1I7N6eUednJbiukYkJ+jhaeTfjMGygrUluLKUF0n8H/7RMhkiY5lwlnwtg1bpBq5J8PayqH5MlQcMhpGVz2c= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1755482490; c=relaxed/simple; bh=et7bJe3p9Q1yVCIYr8gWEO9Yxv/JOZsp+bMu8xhiGjw=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=IAB1iodjJ62oESzqeTK9whPJnQ2v2kVjU64bOF8K2XWxXjjJTAZk0kBZ+jwO6EgnVan+sey1xzEBGrzMRl+vZxFecdjPA0HFn3OSoBUb304Sgq+Zra4zK4kKhHL6vi6vSNKev7KdkZVITAluf0G8AlXR6dnp3nKz4PhPKhDjW5A= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=YKg7k9Hw; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="YKg7k9Hw" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 97300C116C6; Mon, 18 Aug 2025 02:01:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1755482489; bh=et7bJe3p9Q1yVCIYr8gWEO9Yxv/JOZsp+bMu8xhiGjw=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=YKg7k9HwnF2tc7oS9vdPoDn1VoIX9uTAqmaT78ZBcmTGber96XLzbpbO/p9yO4gJN Q7K5xno3LpbkzbJdGTzR7wx3CqTVqAT/uWyEEqwSDV8XDyq0mRSHeB6BaxySInH9NG 57R6FVG48CFGYvgFuo4zneO7oC7ehJwYVNuLrhTrjcTHTjEFgIu4UOvt7kyc/8h/B2 MT5RQS+be3PmLKZ3G6FQ9A3Q7IDeC09CCmD+tV9zzhcjYsfVG//J9eJPU3EgJt7ODc D0c2F+8xPdGZnOl6qoRWFOX1R94q5i1RdveYCz6pxnUmtNsxy1flOqOg2AFjtB/on5 QuaofLIxHbjKA== From: "Mario Limonciello (AMD)" To: "Rafael J . Wysocki" , Bjorn Helgaas Cc: Pavel Machek , Len Brown , Greg Kroah-Hartman , Danilo Krummrich , =?UTF-8?q?Christian=20K=C3=B6nig?= , "James E . J . Bottomley" , "Martin K . Petersen" , Steven Rostedt , linux-pm@vger.kernel.org (open list:HIBERNATION (aka Software Suspend, aka swsusp)), amd-gfx@lists.freedesktop.org (open list:RADEON and AMDGPU DRM DRIVERS), dri-devel@lists.freedesktop.org (open list:DRM DRIVERS), linux-pci@vger.kernel.org (open list:PCI SUBSYSTEM), linux-scsi@vger.kernel.org (open list:SCSI SUBSYSTEM), linux-usb@vger.kernel.org (open list:USB SUBSYSTEM), linux-trace-kernel@vger.kernel.org (open list:TRACING), AceLan Kao , Kai-Heng Feng , Mark Pearson , =?UTF-8?q?Merthan=20Karaka=C5=9F?= , Eric Naim , "Mario Limonciello (AMD)" Subject: [PATCH v6 08/11] PCI: PM: Use pci_power_manageable() in pci_pm_poweroff_noirq() Date: Sun, 17 Aug 2025 21:00:58 -0500 Message-ID: <20250818020101.3619237-9-superm1@kernel.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20250818020101.3619237-1-superm1@kernel.org> References: <20250818020101.3619237-1-superm1@kernel.org> Precedence: bulk X-Mailing-List: linux-trace-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Devices with no subordinate should be put into D3 during hibernate, but devices that have bridge_d3 set should also be put to sleep during hibernate. Adjust the check in pci_pm_poweroff_noirq() to use pci_power_manageable() to cover those as well. Tested-by: Eric Naim Signed-off-by: Mario Limonciello (AMD) --- v5: * Split out patch --- drivers/pci/pci-driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pci/pci-driver.c b/drivers/pci/pci-driver.c index fb6f1f60b2f1f..c563fd6af979d 100644 --- a/drivers/pci/pci-driver.c +++ b/drivers/pci/pci-driver.c @@ -1227,7 +1227,7 @@ static int pci_pm_poweroff_noirq(struct device *dev) return error; } - if (!pci_dev->state_saved && !pci_has_subordinate(pci_dev)) + if (!pci_dev->state_saved && pci_power_manageable(pci_dev)) pci_prepare_to_sleep(pci_dev); /* -- 2.43.0