From nobody Thu Apr 9 13:23:45 2026 Received: from CH4PR04CU002.outbound.protection.outlook.com (mail-northcentralusazon11013039.outbound.protection.outlook.com [40.107.201.39]) (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 544DB186E58; Sun, 8 Mar 2026 09:29:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.201.39 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772962175; cv=fail; b=CWmXITFzi0sMdgt9/3Q+DXh2NW9fSz5lwhTmYmSEu9WRdwIrL/B3Ufdp2UOvX0ue74JU+ZLl/RCCpZUsC7B03V1bnbD+x8dhAZ8mUcreDP/2sbzPDYs6XmIssRWwUVwwbz7okJoqS8k2XkM2bKMKdpGwnjyDWu6KKFrJMI62T0o= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772962175; c=relaxed/simple; bh=qayjMXvE8takw+eh+IN8NuZpLAns0VMe+8v178bAJhY=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=mIKbTp1Rla2NN2zIFTSP0I2zBJpbWGnocxiNpD/obU7wXBdjxl8pPwyl5yQU6fvkmQ/2xBRzhnLuYBls0n5IGnrH4xxWfvdNNxDoWfPlB0XB7dSsbT+6WZnRzWlIRq+I/hEmT5Sf+A3Nf2ME5ekWyn3+DoRG37wDwcEGn7RzwZc= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=2FKJutky; arc=fail smtp.client-ip=40.107.201.39 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="2FKJutky" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=Kcd7JUDC8GyDYcF/0hkRcfkBDjJ4tVI3IUQVwyv2C84AQuiRO0WpT3C0eHgF55JVgc8CtuCZMtCip1bELbqEkCAtSN/C1hqPalHyD5dxUmQSxRTClboOi3/GpwlDvfc30xNmBoY+MoOaM0Hpk1OATYqnfFG9lbVtI/+hENTb6IFtjBcU1oD2Lm4pHqW1W83YyNCdK8iMmDIZxrPL2AIw6RPsndFeWDO+mHg3ecf2FHx8cy5ABEjCFzS9+t8gxUm7kGWWTL/7Ukzzz0ohYQJ4omttT1L0kAKozBaeyCqv1/y4gg8HilwlvIO0eQFEFoRp1FkJpp3O9YE6YcrUvl5I/w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=x1WIWGnJIXRxLg75wGiu0BuqYE8gB0xfR09nLxvGB7g=; b=MdrWLiflymNKvPTAm6M3nS07E7AeapGJY8HO0uux0ClUauuF8TSSRMJb93/0KjThahyMr+Ggn7CdUXDKfeV8vgdZKRPUnNwfAo2wPMlLgHG8LGmmkf0teEbXU7l/cB+7iD6TqwEnifDJJoNA3xY+oYR1EOlHBbXLkaTBQoEysATbxFDLnYI4n0SCElHaje1ELNnTtuEzNtdxFnLOGKVFTN7ir4kxaInC8rhAg2EeieRzSu1dZDaXGHWhqVc8Cd8BMvpIl55YEXeOKyDHEclHZny7+MrUJhT7jIBtl2cw/Vp6tRUfrcYPQQTsuKvYWIO3qx/b0nng9TI82Eeb/8BObA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=x1WIWGnJIXRxLg75wGiu0BuqYE8gB0xfR09nLxvGB7g=; b=2FKJutky97pfXK/erf4orRZeJo8rey13qu6GdcwIWXEw3NQzgFR91VW4P+PV3NfutjMZa+RSRoDfJo2tHuc5yRBCm/hAPWkUMnSAn3rt1VzuDNz9wOzPzvCF9z2848eZtoaVGX00Pl8SvwFC20Fr+wvbGVzG3gTvMpN7mqzNpiI= Received: from BL0PR02CA0003.namprd02.prod.outlook.com (2603:10b6:207:3c::16) by DS7PR12MB8324.namprd12.prod.outlook.com (2603:10b6:8:ec::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9700.9; Sun, 8 Mar 2026 09:29:27 +0000 Received: from BL6PEPF00022575.namprd02.prod.outlook.com (2603:10b6:207:3c:cafe::21) by BL0PR02CA0003.outlook.office365.com (2603:10b6:207:3c::16) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9678.22 via Frontend Transport; Sun, 8 Mar 2026 09:29:29 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=satlexmb07.amd.com; pr=C Received: from satlexmb07.amd.com (165.204.84.17) by BL6PEPF00022575.mail.protection.outlook.com (10.167.249.43) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9678.18 via Frontend Transport; Sun, 8 Mar 2026 09:29:27 +0000 Received: from airavat.amd.com (10.180.168.240) by satlexmb07.amd.com (10.181.42.216) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.17; Sun, 8 Mar 2026 04:29:24 -0500 From: Raju Rangoju To: CC: , , , , , , "Raju Rangoju" Subject: [PATCH net-next v2 1/2] amd-xgbe: Simplify powerdown/powerup paths Date: Sun, 8 Mar 2026 14:58:50 +0530 Message-ID: <20260308092851.1510214-2-Raju.Rangoju@amd.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20260308092851.1510214-1-Raju.Rangoju@amd.com> References: <20260308092851.1510214-1-Raju.Rangoju@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: satlexmb08.amd.com (10.181.42.217) To satlexmb07.amd.com (10.181.42.216) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BL6PEPF00022575:EE_|DS7PR12MB8324:EE_ X-MS-Office365-Filtering-Correlation-Id: 20f1f7a8-3ffa-4e20-c91e-08de7cf531af X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|82310400026|36860700016|1800799024; X-Microsoft-Antispam-Message-Info: 8mdWEjJLqKKwa76J6ARHLA683reN3w/KRYQ0ED0mycT2OqlUaxJmpOnwuBPLQbknEaQOC8kAQbGdRXlAH+J4fcoE8UboA2YHZMf4fAQSSkJ2lvjocPYhtKSd7l8MFsbku8KNYfUQYYMY/P3waOGWsoDNVNXvqoa4KTsRtzMoZq0131QHUWFgBAjdgtCjyI6y7/o9+tnLJeN2uS06mMy3KvlQvQGkNbrjwgIpixvv1WZRfJx1HTNp+bYjg4OflDGsmp+ox7ULDE7q62367grsoNBLTQRz1vChLFyeQdonEfhq95SgtyJuRlYb6hjhpnKA+wvTccMDo9rKSD7puTZmDvB5GrXbBBXADDiHzLqeoNPA3vSQ6GoYX8FEe1Bc26COXc8aTcYwgAZYcRDxC+aD1DZtSdyo5k0yKGCSIXjszS8Lg+OABaAPnVNVkKJG5YOEahwxWmYAcLz7TN5qyvWzD6mrnNDaJOi+rFkjov5eFHB1de20Qj4JOKVUyVwerBzyeZSbC+tAe7US6pVMlT9KoorP/NzCNWPSP+Nk1A720il0XgM56N7ld7zCpo5Lz83+hxt8EEU7woLOC8vCR8aQj9QYtUKcvpMJ3YML+l4UuUIH2pjP3XHcjSxsiGFurXsUkI9mXGzN6vWh/ze7KbSlNJMsjQyrco4eOiNMU3mWveFCgpcSdVWGu5sg5Mv91orj0grE6u/4YwMaPZQF2UofCx8h6uX1iTUt06LFKBxOc04lbQ69w6pRB+pB6CXzDAq2bkMvkVuDfGE+FFjKR2wGfA== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:satlexmb07.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(376014)(82310400026)(36860700016)(1800799024);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: Rdwbq4hi9Sdo498rhR15oSDu7BqFyQLBs/nFdvdqi4nIoSKy3LnUkfkB9sd3Mu0ztaav8bYVdUhYRNQRQqnHyKt7hKeXK0pGIUIEZKd+gP+5OUNNDXbAbB4RwhWAtNeLaGN+L95ok7NZUcXlmEgk58ULSvaJPCcdbVXgHB7AnxEUdnN5ydym2XKDR85USN1gufBc2zx3hu0oYjVBEa1hE9kPHzCS52Zfjm1RIqyYnZKd5ziscaHd81GqQrSjNw+OY10kOHe4/NxuHiWGkgMGt0eMGS82B6zVP2wg4JoUk5euNn2R8n7KiLYeF2EQXiOUWgxMQAS2DsOhDlF3lZhwib6Alefo5+DkC+jgjWz9q9I14KW0Qice5AG/Z7okBgRMsSzoRJbCQDGRHJs9Nw4bF/+lIVKgK+5skYfa/LUlVPvnSZ/jmCU4yoZc0MmIQqAe X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Mar 2026 09:29:27.5267 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 20f1f7a8-3ffa-4e20-c91e-08de7cf531af X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[satlexmb07.amd.com] X-MS-Exchange-CrossTenant-AuthSource: BL6PEPF00022575.namprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR12MB8324 Content-Type: text/plain; charset="utf-8" The caller parameter in xgbe_powerdown() and xgbe_powerup() was intended to differentiate between driver and ioctl contexts, but the only remaining usage is from the driver suspend/resume path. Simplify this by: - Removing the unused XGMAC_DRIVER_CONTEXT and XGMAC_IOCTL_CONTEXT macros - Dropping the now-unused caller parameter - Reordering operations in xgbe_powerdown() to disable NAPI before stopping TX/RX, matching the order used in xgbe_stop() This makes the powerdown/powerup paths easier to follow and keeps the ordering consistent with the rest of the driver. Signed-off-by: Raju Rangoju --- Changes since v1: - Fix compiler warnings when CONFIG_PM_SLEEP is not enabled. drivers/net/ethernet/amd/xgbe/xgbe-drv.c | 47 ++++++++----------- drivers/net/ethernet/amd/xgbe/xgbe-pci.c | 16 +++---- drivers/net/ethernet/amd/xgbe/xgbe-platform.c | 4 +- drivers/net/ethernet/amd/xgbe/xgbe.h | 8 +--- 4 files changed, 32 insertions(+), 43 deletions(-) diff --git a/drivers/net/ethernet/amd/xgbe/xgbe-drv.c b/drivers/net/etherne= t/amd/xgbe/xgbe-drv.c index 8b79d88480db..d57daf6306e1 100644 --- a/drivers/net/ethernet/amd/xgbe/xgbe-drv.c +++ b/drivers/net/ethernet/amd/xgbe/xgbe-drv.c @@ -1116,69 +1116,62 @@ static int xgbe_phy_reset(struct xgbe_prv_data *pda= ta) return pdata->phy_if.phy_reset(pdata); } =20 -int xgbe_powerdown(struct net_device *netdev, unsigned int caller) +int xgbe_powerdown(struct net_device *netdev) { struct xgbe_prv_data *pdata =3D netdev_priv(netdev); struct xgbe_hw_if *hw_if =3D &pdata->hw_if; =20 - DBGPR("-->xgbe_powerdown\n"); - - if (!netif_running(netdev) || - (caller =3D=3D XGMAC_IOCTL_CONTEXT && pdata->power_down)) { - netdev_alert(netdev, "Device is already powered down\n"); - DBGPR("<--xgbe_powerdown\n"); + if (!netif_running(netdev)) { + netdev_dbg(netdev, "Device is not running, skipping powerdown\n"); return -EINVAL; } =20 - if (caller =3D=3D XGMAC_DRIVER_CONTEXT) - netif_device_detach(netdev); + if (pdata->power_down) { + netdev_dbg(netdev, "Device is already powered down\n"); + return -EINVAL; + } =20 + netif_device_detach(netdev); netif_tx_stop_all_queues(netdev); =20 xgbe_stop_timers(pdata); flush_workqueue(pdata->dev_workqueue); =20 + xgbe_napi_disable(pdata, 0); + hw_if->powerdown_tx(pdata); hw_if->powerdown_rx(pdata); =20 - xgbe_napi_disable(pdata, 0); - pdata->power_down =3D 1; =20 - DBGPR("<--xgbe_powerdown\n"); - return 0; } =20 -int xgbe_powerup(struct net_device *netdev, unsigned int caller) +int xgbe_powerup(struct net_device *netdev) { struct xgbe_prv_data *pdata =3D netdev_priv(netdev); struct xgbe_hw_if *hw_if =3D &pdata->hw_if; =20 - DBGPR("-->xgbe_powerup\n"); - - if (!netif_running(netdev) || - (caller =3D=3D XGMAC_IOCTL_CONTEXT && !pdata->power_down)) { - netdev_alert(netdev, "Device is already powered up\n"); - DBGPR("<--xgbe_powerup\n"); + if (!netif_running(netdev)) { + netdev_dbg(netdev, "Device is not running, skipping powerup\n"); return -EINVAL; } =20 - pdata->power_down =3D 0; - - xgbe_napi_enable(pdata, 0); + if (!pdata->power_down) { + netdev_dbg(netdev, "Device is already powered up\n"); + return -EINVAL; + } =20 hw_if->powerup_tx(pdata); hw_if->powerup_rx(pdata); =20 - if (caller =3D=3D XGMAC_DRIVER_CONTEXT) - netif_device_attach(netdev); + xgbe_napi_enable(pdata, 0); =20 netif_tx_start_all_queues(netdev); - xgbe_start_timers(pdata); + netif_device_attach(netdev); =20 - DBGPR("<--xgbe_powerup\n"); + pdata->power_down =3D 0; =20 return 0; } diff --git a/drivers/net/ethernet/amd/xgbe/xgbe-pci.c b/drivers/net/etherne= t/amd/xgbe/xgbe-pci.c index f54a5040a493..9e09b269cb1d 100644 --- a/drivers/net/ethernet/amd/xgbe/xgbe-pci.c +++ b/drivers/net/ethernet/amd/xgbe/xgbe-pci.c @@ -360,14 +360,14 @@ static void xgbe_pci_remove(struct pci_dev *pdev) xgbe_free_pdata(pdata); } =20 -static int __maybe_unused xgbe_pci_suspend(struct device *dev) +static int xgbe_pci_suspend(struct device *dev) { struct xgbe_prv_data *pdata =3D dev_get_drvdata(dev); struct net_device *netdev =3D pdata->netdev; int ret =3D 0; =20 if (netif_running(netdev)) - ret =3D xgbe_powerdown(netdev, XGMAC_DRIVER_CONTEXT); + ret =3D xgbe_powerdown(netdev); =20 pdata->lpm_ctrl =3D XMDIO_READ(pdata, MDIO_MMD_PCS, MDIO_CTRL1); pdata->lpm_ctrl |=3D MDIO_CTRL1_LPOWER; @@ -376,7 +376,7 @@ static int __maybe_unused xgbe_pci_suspend(struct devic= e *dev) return ret; } =20 -static int __maybe_unused xgbe_pci_resume(struct device *dev) +static int xgbe_pci_resume(struct device *dev) { struct xgbe_prv_data *pdata =3D dev_get_drvdata(dev); struct net_device *netdev =3D pdata->netdev; @@ -388,7 +388,7 @@ static int __maybe_unused xgbe_pci_resume(struct device= *dev) XMDIO_WRITE(pdata, MDIO_MMD_PCS, MDIO_CTRL1, pdata->lpm_ctrl); =20 if (netif_running(netdev)) { - ret =3D xgbe_powerup(netdev, XGMAC_DRIVER_CONTEXT); + ret =3D xgbe_powerup(netdev); =20 /* Schedule a restart in case the link or phy state changed * while we were powered down. @@ -461,16 +461,16 @@ static const struct pci_device_id xgbe_pci_table[] = =3D { }; MODULE_DEVICE_TABLE(pci, xgbe_pci_table); =20 -static SIMPLE_DEV_PM_OPS(xgbe_pci_pm_ops, xgbe_pci_suspend, xgbe_pci_resum= e); +static DEFINE_SIMPLE_DEV_PM_OPS(xgbe_pci_pm_ops, + xgbe_pci_suspend, + xgbe_pci_resume); =20 static struct pci_driver xgbe_driver =3D { .name =3D XGBE_DRV_NAME, .id_table =3D xgbe_pci_table, .probe =3D xgbe_pci_probe, .remove =3D xgbe_pci_remove, - .driver =3D { - .pm =3D &xgbe_pci_pm_ops, - } + .driver.pm =3D pm_sleep_ptr(&xgbe_pci_pm_ops), }; =20 int xgbe_pci_init(void) diff --git a/drivers/net/ethernet/amd/xgbe/xgbe-platform.c b/drivers/net/et= hernet/amd/xgbe/xgbe-platform.c index 47d53e59ccf6..98b03a3f3a95 100644 --- a/drivers/net/ethernet/amd/xgbe/xgbe-platform.c +++ b/drivers/net/ethernet/amd/xgbe/xgbe-platform.c @@ -384,7 +384,7 @@ static int xgbe_platform_suspend(struct device *dev) DBGPR("-->xgbe_suspend\n"); =20 if (netif_running(netdev)) - ret =3D xgbe_powerdown(netdev, XGMAC_DRIVER_CONTEXT); + ret =3D xgbe_powerdown(netdev); =20 pdata->lpm_ctrl =3D XMDIO_READ(pdata, MDIO_MMD_PCS, MDIO_CTRL1); pdata->lpm_ctrl |=3D MDIO_CTRL1_LPOWER; @@ -407,7 +407,7 @@ static int xgbe_platform_resume(struct device *dev) XMDIO_WRITE(pdata, MDIO_MMD_PCS, MDIO_CTRL1, pdata->lpm_ctrl); =20 if (netif_running(netdev)) { - ret =3D xgbe_powerup(netdev, XGMAC_DRIVER_CONTEXT); + ret =3D xgbe_powerup(netdev); =20 /* Schedule a restart in case the link or phy state changed * while we were powered down. diff --git a/drivers/net/ethernet/amd/xgbe/xgbe.h b/drivers/net/ethernet/am= d/xgbe/xgbe.h index 4333d269ee84..3b9345506c2b 100644 --- a/drivers/net/ethernet/amd/xgbe/xgbe.h +++ b/drivers/net/ethernet/amd/xgbe/xgbe.h @@ -146,10 +146,6 @@ #define XGBE_MAX_PPS_OUT 4 #define XGBE_MAX_AUX_SNAP 4 =20 -/* Driver PMT macros */ -#define XGMAC_DRIVER_CONTEXT 1 -#define XGMAC_IOCTL_CONTEXT 2 - #define XGMAC_FIFO_MIN_ALLOC 2048 #define XGMAC_FIFO_UNIT 256 #define XGMAC_FIFO_ALIGN(_x) \ @@ -1309,8 +1305,8 @@ void xgbe_dump_rx_desc(struct xgbe_prv_data *, struct= xgbe_ring *, unsigned int); void xgbe_print_pkt(struct net_device *, struct sk_buff *, bool); void xgbe_get_all_hw_features(struct xgbe_prv_data *); -int xgbe_powerup(struct net_device *, unsigned int); -int xgbe_powerdown(struct net_device *, unsigned int); +int xgbe_powerup(struct net_device *netdev); +int xgbe_powerdown(struct net_device *netdev); void xgbe_init_rx_coalesce(struct xgbe_prv_data *); void xgbe_init_tx_coalesce(struct xgbe_prv_data *); void xgbe_restart_dev(struct xgbe_prv_data *pdata); --=20 2.34.1 From nobody Thu Apr 9 13:23:45 2026 Received: from SJ2PR03CU001.outbound.protection.outlook.com (mail-westusazon11012028.outbound.protection.outlook.com [52.101.43.28]) (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 1A0E12D3ED2; Sun, 8 Mar 2026 09:32:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.43.28 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772962327; cv=fail; b=YHL+nc6ya5RxOV9XWkcwx7Q1VedUASxwo/tomx6HvyC7BplUir1VTsZ2Xs/noRNP4CIzj9WAOZ4Lp0Zbu1AJwBIXGkFj1FV8Ok74JgIJFA2qtaB+6kxsoGuYm8gvWG/1hCwuXpM/YySWXssoEoSaBLvS3WOvbLDBBVBQVCmiYis= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772962327; c=relaxed/simple; bh=mPD0M5Xy9DA5n2TfEanp9FQBbc8WQp0ZLLUyk+5buuA=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=TNqnx+JMokJqye06M36myacqmYQZVYbzmfMtxAShUWUXq/GrHPAF9e2OM0pTLlzTraDw4TaQrPKQMDDsXuot1wTY7fk1cqgUA3vNcxOGfSN7NsuuaLRDZcGcESic98pcnUCowFhdKWzDd+Ld9IRyG87X2T/xhGMVwKNX+4aj/uU= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=uPJMd4l5; arc=fail smtp.client-ip=52.101.43.28 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="uPJMd4l5" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=vkmCo4xQ05BfIKIOUp9oWPcZTXEZyQmdfP1XmaEGP/jjPjacnDHJPgq47pH11zXF45vFD9ZxPXNSNQxkTCnGiTeijYIefr8Ji4MOtkW0j6Z7YQXrsUz1RktG4gwG/UxwSOHsfuFilv1DJCt0hgHHs5K4mQcnqSKDSZyG76wPRPNw6GOjdBQcWRpB4aLHe5ixPsDVVoTQjbgCTT5Oe5aqfK/jMYeTctIycMFDGn0v7L1YLqboMXjlouR6hfWShChyGg+bxz+bjBDg6vQidUUwEPq3MWDYcd+7EwnEXxP6pTEd8e0o7Kjf0vwJsFkUa+LDVMoOOIj6aNgps3Dnat+J6w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=FQZ7GvZ3YSDloFfWmU+Wl7lcVNVVg498Swlr0jqv2GA=; b=GHY5hRXPfTmqRa1RqajiLIPQkml77dM5VSnHLLccbX3Lrb1rZ3T2Volwiyq357Qlt5XoyO8HnPXFLvdrl5fENX+OzqF+dz2pfL0y8AOeUshHD+DYjdlYonvoT6g+vRPOH7W/lxMxinMuRwo1stVa0oayI+Je0TIPGLqX4L5Oom9kKiTdjaLsZLGaUALmW49xzlz9EaYoNSJOdj9mMQU+EScCuUVGxjpc1FYXLIPg5kcx51BFS/AKYdSJLvqgjxKlC8So6PI1SSnelKZuDfm8Ty9uSCisADR+/YUiYr8vzCZ0/qZV0QpJ+ZP6de4IguoD8oMC7M599V+xvch0GU1ykA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=FQZ7GvZ3YSDloFfWmU+Wl7lcVNVVg498Swlr0jqv2GA=; b=uPJMd4l5DxY4YtMp0kHElYv0ialo4mdeFAk4BHqxBhH0dQYFIlZ/XOIZKSZOWOUKxFvP1qbgjVmvRLOAm1pppqwd5Ak5/OGkN9nJFZ573/ZNMbe9j45F7PoJ6ZQHRJ/yMI1BQxa32DgedzUc5j76R130X+krn9wLv3SVXdaXPCY= Received: from BL0PR02CA0053.namprd02.prod.outlook.com (2603:10b6:207:3d::30) by PH7PR12MB5973.namprd12.prod.outlook.com (2603:10b6:510:1d8::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9654.22; Sun, 8 Mar 2026 09:29:37 +0000 Received: from BL6PEPF00022573.namprd02.prod.outlook.com (2603:10b6:207:3d:cafe::41) by BL0PR02CA0053.outlook.office365.com (2603:10b6:207:3d::30) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9678.23 via Frontend Transport; Sun, 8 Mar 2026 09:29:35 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=satlexmb07.amd.com; pr=C Received: from satlexmb07.amd.com (165.204.84.17) by BL6PEPF00022573.mail.protection.outlook.com (10.167.249.41) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9678.18 via Frontend Transport; Sun, 8 Mar 2026 09:29:37 +0000 Received: from airavat.amd.com (10.180.168.240) by satlexmb07.amd.com (10.181.42.216) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.17; Sun, 8 Mar 2026 04:29:34 -0500 From: Raju Rangoju To: CC: , , , , , , "Raju Rangoju" Subject: [PATCH net-next v2 2/2] amd-xgbe: add PCI power management for S0i3 support Date: Sun, 8 Mar 2026 14:58:51 +0530 Message-ID: <20260308092851.1510214-3-Raju.Rangoju@amd.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20260308092851.1510214-1-Raju.Rangoju@amd.com> References: <20260308092851.1510214-1-Raju.Rangoju@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: satlexmb08.amd.com (10.181.42.217) To satlexmb07.amd.com (10.181.42.216) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BL6PEPF00022573:EE_|PH7PR12MB5973:EE_ X-MS-Office365-Filtering-Correlation-Id: 0225eca4-e104-4264-a2c5-08de7cf53790 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|36860700016|376014|82310400026|1800799024; X-Microsoft-Antispam-Message-Info: AW/HgYNM36PtSaQjRXigPxLFbiZIUW+Xl3DEAb8ucIdBjSUyfLv2LRe9wKaD8kk1Tp7C6TlI1A3Jkrc2I0TjwNneOpTduDb3zV64mNTRAnNBuneBqubSzMbKdANgcX54XN7UwttZt+EMhZjyXT2VcXj/paZ/lSYxQ3/7RAo5m6f2XQTD143ZSjrNMLjD+vZ1HllYe7jE7UTgIWUY9b0tVHDIi/LTZ35SyBIhkDdVdS5vgByosReQMP1gAi6oipVRzEhlXoZvYCWcoz7ohZHTsWfgST6uc/E/JiieXpl6KDgjJ509PIFQgDE44MHnhgiaHLnC03VvMrP2649q+vcmM0N4w/RUgAwpJh67FrfuBvsybfcS29EqK9jGAKM8ObIr6kQyBF+hmyPlSpC+CTPOY0K8s9BCVsU3IA0S1YiLGXctPfpldC9T0kguaFlqQjrrXK2DoTVmEiSDnrTa/40delo+0U/eKVRYouPWRK3jp+vP+YEb99IZXXvEnxZ/nEnNMB0P8JYUEuTlDS/GE0FxvtWU+KvUO8dVOJc+ySN4PntVo32bsHZ3VjSaU74Ils7x8M2MpvZzbh9zufXXNqblmOroBYxmJp9Mg4oFJcN6kq7sM6Ij8o59fMDupeyPCUKLX/7q2l+mdyHqP9ssdUXzX/HsuqzLLviHUCzcf3hhmLKIq2HW+v1DUZ1Nr88PF3Co7EXeLkyrys47T1Plq6P1tQwEzjfAAf92pbS5Rjt8P743CQHOLNQy47940swOps55wL5AdVZyjYcGEAHeNK5+7w== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:satlexmb07.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(36860700016)(376014)(82310400026)(1800799024);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: Nx9XXMny6POmdDAgIToKS6QJXEzW3BJNPrqXTLown8viLz0LTwHkY6La5dR11CdMlmqGX/1IYXEsWgWwGs+OY5TZNJ5K+ZdRM8ECVmr9M4cmTpOGgJpUnqdC7+7PbFOOl2VRigmbR7sp1AyMzRB55lRLhSdouxWYAlqYbSzzGWdDeUrPbQWZkpvciCfmHrvaaOy2ZBhZFsB+H/Xeg2zMf73ofg6KqVPJl+pMLFkabA2ejNDB7OonuKG05e9p2Qux2tq+Dbt3ka4IBWcScwvtdPkm8wdTfIyo9CmZspJRRAeMw0jLZ4QMuiU8LwUHGON94KzjKkJqllCk9KQHDOcAeew8higuCiYXEAdPfR5nI0U9lDENNfHNqkc3SNGiTX+t7+kmiBLyGE9jeKBLJYD+r2rAYuHo1kMBhUMpJPpwZ6RQ8gLx25brGJZrmi2xqO8O X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Mar 2026 09:29:37.3805 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 0225eca4-e104-4264-a2c5-08de7cf53790 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[satlexmb07.amd.com] X-MS-Exchange-CrossTenant-AuthSource: BL6PEPF00022573.namprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR12MB5973 Content-Type: text/plain; charset="utf-8" The current suspend/resume implementation does not correctly handle PCI device power state transitions, which prevents AMD platforms from reaching the deepest suspend state (S0i3) when the amd-xgbe driver is enabled. In particular, the amd_pmc driver reports: "Last suspend didn't reach deepest state" when this device is present. Implement proper PCI power management operations following the standard PCI PM model so that the device can be cleanly powered down and resumed. Suspend path: - Power down the network interface - Put the PHY into low-power mode - Disable bus mastering to prevent DMA activity - Save PCI configuration space - Disable the PCI device - Disable wake from D3 (S0i3 does not require Wake-on-LAN) - Set the device to D3hot Resume path: - Restore the PCI power state to D0 - Restore PCI configuration space - Enable the PCI device - Re-enable bus mastering - Re-enable device interrupts - Clear the PHY low-power mode - Power up the network interface This allows systems using amd-xgbe to reach the deepest suspend state when entering modern standby (S0i3). Signed-off-by: Raju Rangoju --- Changes since v1: - Added a new helper function xgbe_pci_synchronize_irqs() that synchronizes all registered IRQs to ensure that no pending IRQs are left when the device is suspended. drivers/net/ethernet/amd/xgbe/xgbe-pci.c | 67 ++++++++++++++++++++++++ 1 file changed, 67 insertions(+) diff --git a/drivers/net/ethernet/amd/xgbe/xgbe-pci.c b/drivers/net/etherne= t/amd/xgbe/xgbe-pci.c index 9e09b269cb1d..dbdd791380a4 100644 --- a/drivers/net/ethernet/amd/xgbe/xgbe-pci.c +++ b/drivers/net/ethernet/amd/xgbe/xgbe-pci.c @@ -360,19 +360,67 @@ static void xgbe_pci_remove(struct pci_dev *pdev) xgbe_free_pdata(pdata); } =20 +static void xgbe_pci_synchronize_irqs(struct xgbe_prv_data *pdata) +{ + unsigned int i; + + /* Synchronize main device interrupt */ + synchronize_irq(pdata->dev_irq); + + /* Synchronize ECC interrupt if separate from main device interrupt */ + if (pdata->vdata->ecc_support && pdata->dev_irq !=3D pdata->ecc_irq) + synchronize_irq(pdata->ecc_irq); + + /* Synchronize I2C interrupt if separate from main device interrupt */ + if (pdata->vdata->i2c_support && pdata->dev_irq !=3D pdata->i2c_irq) + synchronize_irq(pdata->i2c_irq); + + /* Synchronize AN interrupt if separate from main device interrupt */ + if (pdata->dev_irq !=3D pdata->an_irq) + synchronize_irq(pdata->an_irq); + + /* Synchronize per-channel DMA interrupts */ + if (pdata->per_channel_irq) { + for (i =3D 0; i < pdata->channel_count; i++) + synchronize_irq(pdata->channel[i]->dma_irq); + } +} + static int xgbe_pci_suspend(struct device *dev) { struct xgbe_prv_data *pdata =3D dev_get_drvdata(dev); struct net_device *netdev =3D pdata->netdev; + struct pci_dev *pdev =3D to_pci_dev(dev); int ret =3D 0; =20 if (netif_running(netdev)) ret =3D xgbe_powerdown(netdev); =20 + /* Disable all device interrupts to prevent spurious wakeups */ + XP_IOWRITE(pdata, XP_INT_EN, 0x0); + + /* Ensure no IRQ handlers are still executing before powering down. + * This prevents race conditions where an IRQ handler could access + * invalid register state after the device is disabled. + */ + xgbe_pci_synchronize_irqs(pdata); + + /* Set PHY to low-power mode */ pdata->lpm_ctrl =3D XMDIO_READ(pdata, MDIO_MMD_PCS, MDIO_CTRL1); pdata->lpm_ctrl |=3D MDIO_CTRL1_LPOWER; XMDIO_WRITE(pdata, MDIO_MMD_PCS, MDIO_CTRL1, pdata->lpm_ctrl); =20 + /* Disable bus mastering to prevent DMA activity */ + pci_clear_master(pdev); + + /* Save PCI configuration state and disable device */ + pci_save_state(pdev); + pci_disable_device(pdev); + + /* Disable wake from D3 - required for S0i3 deep sleep */ + pci_wake_from_d3(pdev, false); + pci_set_power_state(pdev, PCI_D3hot); + return ret; } =20 @@ -380,10 +428,29 @@ static int xgbe_pci_resume(struct device *dev) { struct xgbe_prv_data *pdata =3D dev_get_drvdata(dev); struct net_device *netdev =3D pdata->netdev; + struct pci_dev *pdev =3D to_pci_dev(dev); int ret =3D 0; =20 + /* Restore PCI power state */ + pci_set_power_state(pdev, PCI_D0); + + /* Restore PCI configuration state */ + pci_restore_state(pdev); + + /* Enable PCI device */ + ret =3D pci_enable_device(pdev); + if (ret) { + dev_err(dev, "pci_enable_device failed: %d\n", ret); + return ret; + } + + /* Re-enable bus mastering */ + pci_set_master(pdev); + + /* Re-enable all device interrupts */ XP_IOWRITE(pdata, XP_INT_EN, 0x1fffff); =20 + /* Clear PHY low-power mode */ pdata->lpm_ctrl &=3D ~MDIO_CTRL1_LPOWER; XMDIO_WRITE(pdata, MDIO_MMD_PCS, MDIO_CTRL1, pdata->lpm_ctrl); =20 --=20 2.34.1