From nobody Sat Feb 7 09:30:19 2026 Received: from PH7PR06CU001.outbound.protection.outlook.com (mail-westus3azon11010035.outbound.protection.outlook.com [52.101.201.35]) (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 DE55536E479; Fri, 30 Jan 2026 11:55:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.201.35 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769774135; cv=fail; b=Q3Gfxw91Mw85t/c+WwFTV1a0/sMMpnJ8jqYuKcHPg+qlw+vDwh1zbJxGd4FV6FK4c6ASKqyXCvlhTlDlcTYW+jDEsidfmiROnat6UHW7JmdkeSRcDVAYkA4kYtwnDvDQGjbdSFuiysmgTtBho7I8bh0AcQBVMx9HNjVTO29S63o= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769774135; c=relaxed/simple; bh=kewQh/AP5AA2ULKiUq9LNH59/IlFyPWuuw+zF2PLap0=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=U1om0bRafb314X5LVfssfntk6ExkDYeqC16tHC919Z19YBz4rcjcoHcl14akq7UvJMtou7t2Jr9kmZ/ZrX+FTqFVjTSaptVCotoRAju0ToRSTtC+7vgjHAuKiAj1tnjg/c6l6IOz3yiULQ4L5BKd08DT0CjWNlPnfW9kDVQfGd4= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ti.com; spf=pass smtp.mailfrom=ti.com; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b=WnDBvEag; arc=fail smtp.client-ip=52.101.201.35 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ti.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ti.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b="WnDBvEag" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=V7rK8G5moddWhgr3WtZVrJOQmxYj5laXXJZxFiCrXBgcoIGC96PdBpJkBpWcYuw5H07W3pYCJjzaw4Bkx28VFc5yyfAnf3UMjxFZinsYSoEta52Faz7rxvP+Ugb2o5XQhAjF3wSfoTTA6sIcKiVe2CN338qBtxQJP59sFIQ0JwUUtsCYC6tyr4CIbs87YzmmgV1RIcBdf6tXfCCejfVMSZO1zZnRux/Fj0c7CR2ab1VsOaGKD9z0relsxV7o3rqSH3vsWkdlWcm7dQCjPAkmEKo0EtnyiiDHua+YXLaowrJjKjvqRGUcV86vYxuhOESpb4dzjbm3fLnWew32e2CYJg== 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=ynrmTnhHEvOsBv7JJA1c8TJTgsCQOqlOi9WaBkHSXvI=; b=oWFUvMxtGSd2bJF7px0hlygKFIh2OFFZH+ocfxoMgrc1uDFP/P1qkCgTBJGHoqSMatGgo3jxcpt6Tz/uAuBDe+fPeEOxc4D0K1bKPidzrHwJnL47I4FpxAAKqErPFcY+jsUMEn7EvkDEgTDFc3iZiRltowKS+OmgPgoFappsdacW9T8KtyS5NcLgOO4izyQNqt9xhCPe+DSrRYYrS+iLVw8WLmrCR/eRo4DGb3yqgEPIByN+JWPsVsAr41zWstVMETOnKJwJk0fHqoZqE8/43xaMOlqWzQC3sMOUR34e+sZrMM7A9eL6bxiBiry3pGQ69Jxo88mokgZVpZFrhY97xA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 198.47.21.195) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=ti.com; dmarc=pass (p=quarantine sp=none pct=100) action=none header.from=ti.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ynrmTnhHEvOsBv7JJA1c8TJTgsCQOqlOi9WaBkHSXvI=; b=WnDBvEag4xbR+kkWIOs8/hZW2V6TKdjpi06fEh63codtDB65RehetUSgbeM2W6qkrgvGj0frHH/CBJQ2K5pwBwXNvYeHrOwnXmVLskgZB1dRCT7G4lKLyqOEBAKiaytdURpf/KD3x7CSJa8bYeFZymI8r7CKNPMaPSK6otHWvxI= Received: from BYAPR08CA0045.namprd08.prod.outlook.com (2603:10b6:a03:117::22) by DS4PPFAE15A2261.namprd10.prod.outlook.com (2603:10b6:f:fc00::d3f) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9564.9; Fri, 30 Jan 2026 11:55:32 +0000 Received: from MWH0EPF000971E8.namprd02.prod.outlook.com (2603:10b6:a03:117:cafe::8d) by BYAPR08CA0045.outlook.office365.com (2603:10b6:a03:117::22) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9564.11 via Frontend Transport; Fri, 30 Jan 2026 11:55:30 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 198.47.21.195) smtp.mailfrom=ti.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=ti.com; Received-SPF: Pass (protection.outlook.com: domain of ti.com designates 198.47.21.195 as permitted sender) receiver=protection.outlook.com; client-ip=198.47.21.195; helo=flwvzet201.ext.ti.com; pr=C Received: from flwvzet201.ext.ti.com (198.47.21.195) by MWH0EPF000971E8.mail.protection.outlook.com (10.167.243.68) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9564.3 via Frontend Transport; Fri, 30 Jan 2026 11:55:29 +0000 Received: from DFLE212.ent.ti.com (10.64.6.70) by flwvzet201.ext.ti.com (10.248.192.32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.20; Fri, 30 Jan 2026 05:55:27 -0600 Received: from DFLE202.ent.ti.com (10.64.6.60) by DFLE212.ent.ti.com (10.64.6.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.20; Fri, 30 Jan 2026 05:55:27 -0600 Received: from lelvem-mr05.itg.ti.com (10.180.75.9) by DFLE202.ent.ti.com (10.64.6.60) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.20 via Frontend Transport; Fri, 30 Jan 2026 05:55:27 -0600 Received: from a0507033-hp.dhcp.ti.com (a0507033-hp.dhcp.ti.com [172.24.231.225]) by lelvem-mr05.itg.ti.com (8.18.1/8.18.1) with ESMTP id 60UBtIu11665999; Fri, 30 Jan 2026 05:55:23 -0600 From: Aksh Garg To: , , , , , , , , , , CC: , , , Aksh Garg Subject: [PATCH v5 1/3] PCI: dwc: ep: Fix resizable BAR support for multi-PF configurations Date: Fri, 30 Jan 2026 17:25:14 +0530 Message-ID: <20260130115516.515082-2-a-garg7@ti.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20260130115516.515082-1-a-garg7@ti.com> References: <20260130115516.515082-1-a-garg7@ti.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-C2ProcessedOrg: 333ef613-75bf-4e12-a4b1-8e3623f5dcea X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: MWH0EPF000971E8:EE_|DS4PPFAE15A2261:EE_ X-MS-Office365-Filtering-Correlation-Id: 1b33c3f8-a669-4a8b-e91e-08de5ff676f8 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|1800799024|82310400026|36860700013|921020; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?6tuLG/v5Sx8EqfNDYiVBU/X7VEqzYVsmU86cgAs0IEb/NT7h6DgP2GheSbLN?= =?us-ascii?Q?qRSkdpQJAqeMUhKYzXif9xXIn8rmBxfRWqhJ+p4hSrhIb089272/RRUDZ8rh?= =?us-ascii?Q?JCCclTIaKVsy+SWBAsyo9e5PADAWjWei7GnDuEZp/kM8+gokUH1O8qG+g844?= =?us-ascii?Q?30MLoB1M4jQASQlzIQiImgu6Y62XWNQJ+4B6fDtB4cC5VcPxQXr7oeZHazbU?= =?us-ascii?Q?6K2T0rx6B13gfdfZIJP0McrcaLHKN9APPYnWrhorTrnDOyCnDm1yueAvCkQ4?= =?us-ascii?Q?Sxiq/P3lA+IofljJlTP7qogmSYXnR4z/oQSAAtFpD2/wEtUhMFoMybBLGEYJ?= =?us-ascii?Q?k2ESo6IxzekbyaM9sjISOJWvhsaekW2XdicLJikxiyWMVN46PeGuyFNYx1sW?= =?us-ascii?Q?Nha4pek3j7/MpItriHyP3oRP5HjqDibkMTEpV+HWCclNpey/UYL+eJwZ+tB7?= =?us-ascii?Q?unihEvfv33DA2vyuNToODbA7NFXWTjzzO5qhdDjcshE0bcntVbayTle+04r1?= =?us-ascii?Q?9pKKda7QWgGjanMvmlj3mw0nfakEHqLsg8+J/laNUh7gW8QPe1HAXrSliCxI?= =?us-ascii?Q?bFuv3+sEqQLP1g9hAcS27bgh118pWB/722mpo4uisG/p7o7m8jxDM9/5s9DP?= =?us-ascii?Q?X2JWn7GN7sSeHSS3jm7/sKT1fvklp4p3WG6tuRtCHqynutwD1047vPjTx6pT?= =?us-ascii?Q?e9yOfx09sqAY1HW5plBrkKO7kfZQz02M6AlmS5rr5/r4w/mP9l8uqdBNm/+b?= =?us-ascii?Q?cvwLoLAObAXycjkoMHvrOz+Rdumt8EiS1y/tCVxW/82ud2T7PDos4JBdA31M?= =?us-ascii?Q?Pg3WYlk51Bhb3KDXG7+z8swOOH5F10y4LDRwT0urZ1CUNAx4yo5rqT9iAv5A?= =?us-ascii?Q?g+Ey0NGxp3FWJ7yNTk6PJrOJnd4zNf1UK9vZ8QC47CEa7MMYGpbhxzo0v7i4?= =?us-ascii?Q?yUTCMbggzpbH0pudlfhHvPVHKFm18exjphxXbOtYTl+nwevWv7nidgPl+W+t?= =?us-ascii?Q?1ynCs4BZr0y24KTib4YfCUHpCeu7HCNfN9fjVPxQGpIbp5RhbDFTKkqRcI/0?= =?us-ascii?Q?AGzwF3rHmUB9DBoZkp8Mg1a6/Mo2cEsBBm2O+UUkNdPy++h+2eqj6xLgVptV?= =?us-ascii?Q?aJ0VoJXwzqlYzqNlAnbVLYhxUDEMkugdOgrl+eO55n4wvQoD8MkgcgqKB0M4?= =?us-ascii?Q?FNRxf4MkpIZJWuUPIsivB0SsLvKfKg18oZFGRC9dOBTxAbSquVONTbd9JHVY?= =?us-ascii?Q?Bt1hEg5zej3eD4gnq8N83sCLGryJw6q6oXZ4UlsxuU/LRSuXErFZeefMFbAh?= =?us-ascii?Q?uePHeia8h97H4q0uzA3Eq63DWTmqbCHve+t7gG7xViXKgOnCTOr9AfEvE6Ra?= =?us-ascii?Q?mWZU8ww+1RyytQaOcl3MtHQmbUBFDuJqtjGppH8IY9gaUZk3UvlAPWYXeJ2h?= =?us-ascii?Q?ey/S5LsJl/TsLf/BVutgaleTSa3cz/iDUiIRCGmoS3+RrC7ebHmRjUMOPbk5?= =?us-ascii?Q?9H2K4Fu9yPDyCvV6vGuHLvTcf6g/66kE5Ilx2Y42mkrN7O4SqicHZKqgNT7A?= =?us-ascii?Q?Gc6iupt8FdimyWyTsjCqsegkI/G83yRmN1esy6MCzq2NuXisk+UZNQ6vYrIz?= =?us-ascii?Q?sT9HhUlHutuJqAQ4Dnw/f5DhN2DIv21WKzpgG7mXoMUzo2Ej1b4QNKhaMKMn?= =?us-ascii?Q?DwLd5a2Oteq6g+gH4lkdYKxclUM=3D?= X-Forefront-Antispam-Report: CIP:198.47.21.195;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:flwvzet201.ext.ti.com;PTR:ErrorRetry;CAT:NONE;SFS:(13230040)(376014)(7416014)(1800799024)(82310400026)(36860700013)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: hTBJu+1/v4qHZsm/TULWXX2JdOB0X7biI3pIUsQ1P3Qey3soTGmYaOIYMPxzn40HeLkOwf04+GuMeLN6ZyTiPpbxfudfMfjXpQJxbzJ93yX+qOaIKKunXPV/eMMQvRGM6b3clPLUJ1RYN3W7mDdvykYhnOZ+6f1rwCCJ8adTM/PRKa+hI15Czjh57+clu5iv8QAFGyjR/+y+KejlqLK4D8+KDpJs6zq5m5kEyyt4X2wxaXhhxeWmhLmykW6UVrruNfb9O0XLQ6dpbpD2HDoUkH+NHbWkW8HSIX64krnrIKdgfMApQUb7PsZT5IxuVzva1z7B5XVrTSmZYLXVNdzrly2SSy6pt7tdLRpzrTw9+YR0aBNxtOynrMSagcrqV4ZPLghWoj5v5UVlojDE9NeAoyukyuTqfT759FqwFVfESeYC2rpHo1ctTJrFVhdIXSqb X-OriginatorOrg: ti.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Jan 2026 11:55:29.4437 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 1b33c3f8-a669-4a8b-e91e-08de5ff676f8 X-MS-Exchange-CrossTenant-Id: e5b49634-450b-4709-8abb-1e2b19b982b7 X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=e5b49634-450b-4709-8abb-1e2b19b982b7;Ip=[198.47.21.195];Helo=[flwvzet201.ext.ti.com] X-MS-Exchange-CrossTenant-AuthSource: MWH0EPF000971E8.namprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS4PPFAE15A2261 Content-Type: text/plain; charset="utf-8" The resizable BAR support added by the commit 3a3d4cabe681 ("PCI: dwc: ep: Allow EPF drivers to configure the size of Resizable BARs") incorrectly configures the resizable BARs only for the first Physical Function (PF0) in EP mode. The resizable BAR configuration functions use generic dw_pcie_*_dbi operations instead of physical function specific dw_pcie_ep_*_dbi operations. This causes resizable BAR configuration to always target PF0 regardless of the requested function number. Additionally, dw_pcie_ep_init_non_sticky_registers() only initializes resizable BAR registers for PF0, leaving other PFs unconfigured during the execution of this function. Fix this by using physical function specific configuration space access operations throughout the resizable BAR code path and initializing registers for all the physical functions that support resizable BARs. Fixes: 3a3d4cabe681 ("PCI: dwc: ep: Allow EPF drivers to configure the size= of Resizable BARs") Signed-off-by: Aksh Garg Reviewed-by: Niklas Cassel Reviewed-by: Frank Li --- Changes from v4 to v5: - None Changes from v3 to v4: - Rebased the patch, no functional change Changes from v2 to v3: - Changed function name from __dw_pcie_ep_init_non_sticky_registers() to dw_pcie_ep_init_rebar_registers() Changes from v1 to v2: - Fixed the suggested nit v4: https://lore.kernel.org/all/20260129091753.490167-2-a-garg7@ti.com/ v3: https://lore.kernel.org/all/20260127085010.446116-2-a-garg7@ti.com/ v2: https://lore.kernel.org/all/20260122082538.309122-2-a-garg7@ti.com/ v1: https://lore.kernel.org/all/20260121054214.274429-2-a-garg7@ti.com/ .../pci/controller/dwc/pcie-designware-ep.c | 48 ++++++++++++------- 1 file changed, 32 insertions(+), 16 deletions(-) diff --git a/drivers/pci/controller/dwc/pcie-designware-ep.c b/drivers/pci/= controller/dwc/pcie-designware-ep.c index 855b2e58c338..1cc2985bab03 100644 --- a/drivers/pci/controller/dwc/pcie-designware-ep.c +++ b/drivers/pci/controller/dwc/pcie-designware-ep.c @@ -75,6 +75,13 @@ static u8 dw_pcie_ep_find_capability(struct dw_pcie_ep *= ep, u8 func_no, u8 cap) cap, NULL, ep, func_no); } =20 +static u16 dw_pcie_ep_find_ext_capability(struct dw_pcie_ep *ep, + u8 func_no, u8 cap) +{ + return PCI_FIND_NEXT_EXT_CAP(dw_pcie_ep_read_cfg, 0, + cap, NULL, ep, func_no); +} + static int dw_pcie_ep_write_header(struct pci_epc *epc, u8 func_no, u8 vfu= nc_no, struct pci_epf_header *hdr) { @@ -350,22 +357,22 @@ static void dw_pcie_ep_clear_bar(struct pci_epc *epc,= u8 func_no, u8 vfunc_no, ep->epf_bar[bar] =3D NULL; } =20 -static unsigned int dw_pcie_ep_get_rebar_offset(struct dw_pcie *pci, +static unsigned int dw_pcie_ep_get_rebar_offset(struct dw_pcie_ep *ep, u8 = func_no, enum pci_barno bar) { u32 reg, bar_index; unsigned int offset, nbars; int i; =20 - offset =3D dw_pcie_find_ext_capability(pci, PCI_EXT_CAP_ID_REBAR); + offset =3D dw_pcie_ep_find_ext_capability(ep, func_no, PCI_EXT_CAP_ID_REB= AR); if (!offset) return offset; =20 - reg =3D dw_pcie_readl_dbi(pci, offset + PCI_REBAR_CTRL); + reg =3D dw_pcie_ep_readl_dbi(ep, func_no, offset + PCI_REBAR_CTRL); nbars =3D FIELD_GET(PCI_REBAR_CTRL_NBAR_MASK, reg); =20 for (i =3D 0; i < nbars; i++, offset +=3D PCI_REBAR_CTRL) { - reg =3D dw_pcie_readl_dbi(pci, offset + PCI_REBAR_CTRL); + reg =3D dw_pcie_ep_readl_dbi(ep, func_no, offset + PCI_REBAR_CTRL); bar_index =3D FIELD_GET(PCI_REBAR_CTRL_BAR_IDX, reg); if (bar_index =3D=3D bar) return offset; @@ -386,7 +393,7 @@ static int dw_pcie_ep_set_bar_resizable(struct dw_pcie_= ep *ep, u8 func_no, u32 rebar_cap, rebar_ctrl; int ret; =20 - rebar_offset =3D dw_pcie_ep_get_rebar_offset(pci, bar); + rebar_offset =3D dw_pcie_ep_get_rebar_offset(ep, func_no, bar); if (!rebar_offset) return -EINVAL; =20 @@ -416,16 +423,16 @@ static int dw_pcie_ep_set_bar_resizable(struct dw_pci= e_ep *ep, u8 func_no, * 1 MB to 128 TB. Bits 31:16 in PCI_REBAR_CTRL define "supported sizes" * bits for sizes 256 TB to 8 EB. Disallow sizes 256 TB to 8 EB. */ - rebar_ctrl =3D dw_pcie_readl_dbi(pci, rebar_offset + PCI_REBAR_CTRL); + rebar_ctrl =3D dw_pcie_ep_readl_dbi(ep, func_no, rebar_offset + PCI_REBAR= _CTRL); rebar_ctrl &=3D ~GENMASK(31, 16); - dw_pcie_writel_dbi(pci, rebar_offset + PCI_REBAR_CTRL, rebar_ctrl); + dw_pcie_ep_writel_dbi(ep, func_no, rebar_offset + PCI_REBAR_CTRL, rebar_c= trl); =20 /* * The "selected size" (bits 13:8) in PCI_REBAR_CTRL are automatically * updated when writing PCI_REBAR_CAP, see "Figure 3-26 Resizable BAR * Example for 32-bit Memory BAR0" in DWC EP databook 5.96a. */ - dw_pcie_writel_dbi(pci, rebar_offset + PCI_REBAR_CAP, rebar_cap); + dw_pcie_ep_writel_dbi(ep, func_no, rebar_offset + PCI_REBAR_CAP, rebar_ca= p); =20 dw_pcie_dbi_ro_wr_dis(pci); =20 @@ -1023,20 +1030,17 @@ void dw_pcie_ep_deinit(struct dw_pcie_ep *ep) } EXPORT_SYMBOL_GPL(dw_pcie_ep_deinit); =20 -static void dw_pcie_ep_init_non_sticky_registers(struct dw_pcie *pci) +static void dw_pcie_ep_init_rebar_registers(struct dw_pcie_ep *ep, u8 func= _no) { - struct dw_pcie_ep *ep =3D &pci->ep; unsigned int offset; unsigned int nbars; enum pci_barno bar; u32 reg, i, val; =20 - offset =3D dw_pcie_find_ext_capability(pci, PCI_EXT_CAP_ID_REBAR); - - dw_pcie_dbi_ro_wr_en(pci); + offset =3D dw_pcie_ep_find_ext_capability(ep, func_no, PCI_EXT_CAP_ID_REB= AR); =20 if (offset) { - reg =3D dw_pcie_readl_dbi(pci, offset + PCI_REBAR_CTRL); + reg =3D dw_pcie_ep_readl_dbi(ep, func_no, offset + PCI_REBAR_CTRL); nbars =3D FIELD_GET(PCI_REBAR_CTRL_NBAR_MASK, reg); =20 /* @@ -1057,16 +1061,28 @@ static void dw_pcie_ep_init_non_sticky_registers(st= ruct dw_pcie *pci) * the controller when RESBAR_CAP_REG is written, which * is why RESBAR_CAP_REG is written here. */ - val =3D dw_pcie_readl_dbi(pci, offset + PCI_REBAR_CTRL); + val =3D dw_pcie_ep_readl_dbi(ep, func_no, offset + PCI_REBAR_CTRL); bar =3D FIELD_GET(PCI_REBAR_CTRL_BAR_IDX, val); if (ep->epf_bar[bar]) pci_epc_bar_size_to_rebar_cap(ep->epf_bar[bar]->size, &val); else val =3D BIT(4); =20 - dw_pcie_writel_dbi(pci, offset + PCI_REBAR_CAP, val); + dw_pcie_ep_writel_dbi(ep, func_no, offset + PCI_REBAR_CAP, val); } } +} + +static void dw_pcie_ep_init_non_sticky_registers(struct dw_pcie *pci) +{ + struct dw_pcie_ep *ep =3D &pci->ep; + u8 funcs =3D ep->epc->max_functions; + u8 func_no; + + dw_pcie_dbi_ro_wr_en(pci); + + for (func_no =3D 0; func_no < funcs; func_no++) + dw_pcie_ep_init_rebar_registers(ep, func_no); =20 dw_pcie_setup(pci); dw_pcie_dbi_ro_wr_dis(pci); --=20 2.34.1 From nobody Sat Feb 7 09:30:19 2026 Received: from BL0PR03CU003.outbound.protection.outlook.com (mail-eastusazon11012039.outbound.protection.outlook.com [52.101.53.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 935B736F40C; Fri, 30 Jan 2026 11:55:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.53.39 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769774136; cv=fail; b=B9+hM5VKwyjnxF+xSnNDt/g8jjBVlbgIDmhgWu7Hsr6e7G9O81PgKXaCemUB8fBvHQnrpa5Yb6mgO0YATg2WRRNhFePC96Ok6Zs9XttXu/2o6M6QwN6eiyw71Gr5CPc3aeQobXZRc1BnCqnifl3xlV5mnWTR2gTy9GiKqhWYTbc= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769774136; c=relaxed/simple; bh=dnt3Ywyo3kRrN1O9kcVc+pHLI6DZ+oRycSjBrSKmnY0=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=N/eikJImToy9NtSWTzEjTOhFQzu1D6v4Tx0WwmOk5yBRxJ3tjhXRRYXC08/GkXSYCKue+lYdH0kXXxEStDo5/W+k76zOk1PkMUhO6bp+wkEquk3e7HKOjl/7Stxkwp9hXxaPdswYEA/wiIbfWTqMXMd5cxlaIy90PfcbRB8mXew= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ti.com; spf=pass smtp.mailfrom=ti.com; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b=gtV5l4r5; arc=fail smtp.client-ip=52.101.53.39 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ti.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ti.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b="gtV5l4r5" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=A3UlGQbUJsxvDv+SLXFxqAs6d4RzEB3HeVC30DQUOt3LkDlzUOLTccvbFQhIsyg309NeaegYGLJe/MocmZOzDsLSpOqVGXms3s+VCzFsWMrikQ0qexSh8isncci+jbPQ+HPIiP+4UXODSAtscpCZbxnSqZ8AVuEJDcqEFVwyjCTrl0BlyzmbuuCEM4AAzHT3/0apoYpHLn/QLvdyh5qBEcR1Ohd5arUD/8oBtFggdgF9+hrLpnK4Sx9vohzMZ1jvvS2ITOvxEv7U3TMnrkq2HThNZ/a85cOfjtwHmb9muDJguIHvdUtR1CIKT1rr7Mlp6HmNbfR+LIdojt2ljKc0RA== 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=odZKYb45py2tu0D13lN9N/LELubn2OG8kN46xOHwpjw=; b=nxkFz/iUu+VUJleJMlQ1csnlcFFrfyWZVZMKDHe2IRzMV/n/9119GN1LQrXAdHtFBu4RHTWWiTXAqdf75JMMGAOhpmgxdHAG9lnYfe7vwfzFMkmMLrQ104GlYffcLemddWofFruhyoX7h2tDzrDOq8yj/Cws1HbTJQVn3W1xiakQ33t+pCGKtpgcwRPQ03OfbJyUHn+k1PzsttC3rF29ArhBK5RYPVlNQt2JfVMUhRzhXQM1m3Ks8r55RzncUacyAKyIsrwsveuRS3pnYkIiXvhs9nY2llaCQd/VD/6NNnAKbkvl8vNE7xdWhsz4liB+Ca0THIF+rR/TNAcuyUYPPQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 198.47.23.194) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=ti.com; dmarc=pass (p=quarantine sp=none pct=100) action=none header.from=ti.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=odZKYb45py2tu0D13lN9N/LELubn2OG8kN46xOHwpjw=; b=gtV5l4r5KyuI0/qH6JV4nGbRtdZUIbAcjN3C2IouvViGrLybN1jH1Et3jCNNn8Qnp9EvYrY1PKNns+BMIJmpspHfANOg/An3el8qIIQYDDXZY3QhPUImQT+E9X3kEHxuRU+aOZTONvK4HuEsp5i7VYSTUgiGh0SAO1L0VX/eWf8= Received: from BY5PR13CA0027.namprd13.prod.outlook.com (2603:10b6:a03:180::40) by SA3PR10MB6998.namprd10.prod.outlook.com (2603:10b6:806:31c::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9564.8; Fri, 30 Jan 2026 11:55:32 +0000 Received: from SJ5PEPF000001D6.namprd05.prod.outlook.com (2603:10b6:a03:180:cafe::41) by BY5PR13CA0027.outlook.office365.com (2603:10b6:a03:180::40) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9587.5 via Frontend Transport; Fri, 30 Jan 2026 11:55:33 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 198.47.23.194) smtp.mailfrom=ti.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=ti.com; Received-SPF: Pass (protection.outlook.com: domain of ti.com designates 198.47.23.194 as permitted sender) receiver=protection.outlook.com; client-ip=198.47.23.194; helo=lewvzet200.ext.ti.com; pr=C Received: from lewvzet200.ext.ti.com (198.47.23.194) by SJ5PEPF000001D6.mail.protection.outlook.com (10.167.242.58) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9564.3 via Frontend Transport; Fri, 30 Jan 2026 11:55:32 +0000 Received: from DLEE207.ent.ti.com (157.170.170.95) by lewvzet200.ext.ti.com (10.4.14.103) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.20; Fri, 30 Jan 2026 05:55:31 -0600 Received: from DLEE204.ent.ti.com (157.170.170.84) by DLEE207.ent.ti.com (157.170.170.95) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.20; Fri, 30 Jan 2026 05:55:31 -0600 Received: from lelvem-mr05.itg.ti.com (10.180.75.9) by DLEE204.ent.ti.com (157.170.170.84) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.20 via Frontend Transport; Fri, 30 Jan 2026 05:55:31 -0600 Received: from a0507033-hp.dhcp.ti.com (a0507033-hp.dhcp.ti.com [172.24.231.225]) by lelvem-mr05.itg.ti.com (8.18.1/8.18.1) with ESMTP id 60UBtIu21665999; Fri, 30 Jan 2026 05:55:27 -0600 From: Aksh Garg To: , , , , , , , , , , CC: , , , Aksh Garg Subject: [PATCH v5 2/3] PCI: dwc: ep: Add per-PF BAR and inbound ATU mapping support Date: Fri, 30 Jan 2026 17:25:15 +0530 Message-ID: <20260130115516.515082-3-a-garg7@ti.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20260130115516.515082-1-a-garg7@ti.com> References: <20260130115516.515082-1-a-garg7@ti.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-C2ProcessedOrg: 333ef613-75bf-4e12-a4b1-8e3623f5dcea X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ5PEPF000001D6:EE_|SA3PR10MB6998:EE_ X-MS-Office365-Filtering-Correlation-Id: d38ad26d-2855-427c-62de-08de5ff67898 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|36860700013|1800799024|7416014|376014|82310400026|921020; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?5KSrDIFlDOCbgDwXoS0+IkQRMl5cYBAnyppvZ0X7XTMVEsy46Gzs0kCSsXqr?= =?us-ascii?Q?ZamItENWRNpsR9pETWGv3jeZU/H9BGt5p/TIfxUcMWyvisiQvaZehGQQWBW1?= =?us-ascii?Q?dfZZbuG5Pm1bev3Xp5OGfUjInEPVjxY1rFkLHNkVOjw5U2g0/HqW4B907I5x?= =?us-ascii?Q?zxglvg3rLb9EYVLltltXS48Wag1rDsPQkGV+Igg2nlsjglb+6uWEfHMS8w13?= =?us-ascii?Q?g/QTeRYmfysTeWORlpN/fd3ahmh4kRHOcTY/WzDLZQOQx62kyPNOqHv7N5lc?= =?us-ascii?Q?CeLKH31cQbdJpRJM9UbapmWN2nmKoxl7AG2nzH0CcqqadmvIl9ZABKcAWEZ0?= =?us-ascii?Q?luXrcQKcLYbu6RDBQ30tREx0652uHozYevAjab90QcpuopKg+/lyxsKSj2+F?= =?us-ascii?Q?ftehsjYFsDmDgyu6dQ8JB2tpT57MktHS/5jGlnpbWYj3rTwNYMw5mCUZVQzB?= =?us-ascii?Q?50aZHMCb4WEsmpsG02bbhSxVtX8ki+mnSsidSvan0lFMend7kkP9gUu66yeE?= =?us-ascii?Q?oAIotvK5DU1G1zC31IeJoOV2LdapWNejn8PyjXBottFyh91ki7x4blddYv+e?= =?us-ascii?Q?zHLAEvxhSx2HJAaJiUKICXJEZdoAMBPWWusgjbAyJdPlHLz4iPcVc3VCT/Ff?= =?us-ascii?Q?mpfYtCFSTcXAZfHIBbZcUN67qpTdZ6BoOA2sOR/Xllq82Tskc//6r9dfZTsJ?= =?us-ascii?Q?MnOSemDZ968zeAPkeTMgh5oJHJPpG0wL1+ut/STKM7hdHmnUYtVLNJW87QDV?= =?us-ascii?Q?PwMWxcyofF0Ps/M3xbkRZUzPJxgJd4+aOC0mYniKLl2F/mSlbftQ1yzTWeoB?= =?us-ascii?Q?vTspI7vA9tu/40JqQIS/wIDArpFqxnt7zBfi/4KOZg3FpK0MKfCLI9SnoNFw?= =?us-ascii?Q?pozTknzfjG/9E5QZIF+xmthwmkwMLYaT2KVIazKZNgkV/7HvocHH8THDz+5k?= =?us-ascii?Q?4Z5LhChaMuWWeVo0MQOTN0BN2CtJlLD7Pr9H8TMIqOqKozv8ZvuFzH1yIITl?= =?us-ascii?Q?MHFeORPGo9d+iIsiiGcIWt8CSiIdaYBXrSh8MAvkJbA1zvnuGnT4RO/JJFf9?= =?us-ascii?Q?mWGP4fEOXsjadGB01o/dNSCuMJI1Bv9+xLo/uqFvazg03im3k6k42Bn1HmiL?= =?us-ascii?Q?RApG5MjA6/GrhsRFXuba/jZ/FK8Ts+ypm1jYdtSLkT4xApeFyGqzjO8COQQW?= =?us-ascii?Q?dlyGwNF9RYjqYMD84ogMGVIDNJ64MkrxPwHem4JSaV24DXebJxDEOi0uovX6?= =?us-ascii?Q?7FXicDeYFry+zd4o9SEHlpUuoRRGk60ZgDfX8F1CxbvAm1V8bXzK7vFBFx5d?= =?us-ascii?Q?V9xata3oh4XFq+RNuSqfHnvS1HKPULVeQuPbRguEJsXb2H2wQhPnns9zbyAY?= =?us-ascii?Q?7OGW2GQYHYFEBFXUMT6JvpSmpFybXzajTiujZQjFEUnD9z4tm4MWXyUNzEJR?= =?us-ascii?Q?U2JxlZEZRckVTOsump6gK/Z2F52udaEXf+9waQuMDYVtzbLW8Hj9AhLLGrOj?= =?us-ascii?Q?S+Rjne+7EOu+jwt+1an5/lgvFUaBO9tSLrcgVZD+0g2f5R9kshSSNlZ2AXv6?= =?us-ascii?Q?OXL98vejRQhbhPIPICR0Bxa8yKkOjaZkStP2f5CZU7zd9o0M9qQI/5DIHZzS?= =?us-ascii?Q?QugQ/E1NxoRNl1ghwDEC+Py0TJzFqtOSEPXM8swB9G6YbG5d12TIXclJgxi8?= =?us-ascii?Q?U4b4v1w62ERrV7zhJroI5n87QzM=3D?= X-Forefront-Antispam-Report: CIP:198.47.23.194;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:lewvzet200.ext.ti.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(36860700013)(1800799024)(7416014)(376014)(82310400026)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: +T/YDGmuVjhDBMbIpV+z+O98aPvRvm2zG4xipz4LoHpZkeRz95EdUTSTRZnOFE2Rg/xc/6g7DnP2NpCP9vOijjt8Nu2rUMhZ6tt4JhayIzY5G5ZSE6gxkfElt2J8fsAeoLsWGLlwX0/4PL/l43KdE2RSblA0W5shhhoq7n7p5Ub7E0FBLbu+ex6BY+LPxNQJSWv959tvgd+cfvzzZNtiQ5j6nPeQ1wjIpqvmgiQgI3RdkeLjHs7+pbhv28VVUR54qQpmzv4A4F4EBvGmDh5wR/NjYESiYZNXwTcXJPNLJCo4krdE07kZ5zRhX+TmBbovsN6RtwmO2NUymsFrs9Cm5W1ZSAAtKXA+ZPeVOzWoXyr2361QGkuuRmyTDPadRIt5oFeBoFmoAq2GojpXdfuJz4HoIt4EapJfh/i0hGdPwSiz1u2X2HAmW+wibnAQM3bd X-OriginatorOrg: ti.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Jan 2026 11:55:32.1700 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: d38ad26d-2855-427c-62de-08de5ff67898 X-MS-Exchange-CrossTenant-Id: e5b49634-450b-4709-8abb-1e2b19b982b7 X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=e5b49634-450b-4709-8abb-1e2b19b982b7;Ip=[198.47.23.194];Helo=[lewvzet200.ext.ti.com] X-MS-Exchange-CrossTenant-AuthSource: SJ5PEPF000001D6.namprd05.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA3PR10MB6998 Content-Type: text/plain; charset="utf-8" The commit 24ede430fa49 ("PCI: designware-ep: Add multiple PFs support for DWC") added support for multiple PFs in the DWC driver, but the implementation was incomplete. It did not properly support MSI/MSI-X, as well as BAR and inbound ATU mapping for multiple PFs. The MSI/MSI-X issue was later fixed by commit 47a062609a30 ("PCI: designware-ep: Modify MSI and MSIX CAP way of finding") by introducing a per-PF struct dw_pcie_ep_func. However, even with both commits, the multiple PF support in the driver remains broken because BAR configuration and ATU mappings are managed globally in struct dw_pcie_ep, meaning all PFs share the same BAR-to-ATU mapping table. This causes one PF's EPF to overwrite the address translation of another PF's EPF in the internal ATU region, creating conflicts when multiple physical functions attempt to configure their BARs independently. The commit cfbc98dbf44d ("PCI: dwc: ep: Support BAR subrange inbound mapping via Address Match Mode iATU") later introduced Address Match Mode support, which suffers from the same multi-PF conflict issue. Fix this by moving the required members from struct dw_pcie_ep to struct dw_pcie_ep_func, similar to what commit 47a062609a30 ("PCI: designware-ep: Modify MSI and MSIX CAP way of finding") did for MSI/MSI-X capability support, to allow proper multi-function endpoint operation, where each PF can configure its BARs and corresponding internal ATU region without interfering with other PFs. Fixes: 24ede430fa49 ("PCI: designware-ep: Add multiple PFs support for DWC") Fixes: cfbc98dbf44d ("PCI: dwc: ep: Support BAR subrange inbound mapping vi= a Address Match Mode iATU") Signed-off-by: Aksh Garg Reviewed-by: Niklas Cassel Reviewed-by: Frank Li --- Changes from v4 to v5: - Added NULL checker for ep_func in dw_pcie_ep_clear_ib_maps() Changes from v3 to v4: - Fix the similar conflict created by the commit cfbc98dbf44d Changes from v2 to v3: - None Changes from v1 to v2: - Fixed the suggested nits - Rephrased the commit message with a proper Fixes tag v4: https://lore.kernel.org/all/20260129091753.490167-3-a-garg7@ti.com/ v3: https://lore.kernel.org/all/20260127085010.446116-3-a-garg7@ti.com/ v2: https://lore.kernel.org/all/20260122082538.309122-3-a-garg7@ti.com/ v1: https://lore.kernel.org/all/20260121054214.274429-3-a-garg7@ti.com/ .../pci/controller/dwc/pcie-designware-ep.c | 79 +++++++++++-------- drivers/pci/controller/dwc/pcie-designware.h | 12 +-- 2 files changed, 54 insertions(+), 37 deletions(-) diff --git a/drivers/pci/controller/dwc/pcie-designware-ep.c b/drivers/pci/= controller/dwc/pcie-designware-ep.c index 1cc2985bab03..6d3c35dd280f 100644 --- a/drivers/pci/controller/dwc/pcie-designware-ep.c +++ b/drivers/pci/controller/dwc/pcie-designware-ep.c @@ -115,11 +115,15 @@ static int dw_pcie_ep_ib_atu_bar(struct dw_pcie_ep *e= p, u8 func_no, int type, int ret; u32 free_win; struct dw_pcie *pci =3D to_dw_pcie_from_ep(ep); + struct dw_pcie_ep_func *ep_func =3D dw_pcie_ep_get_func_from_ep(ep, func_= no); =20 - if (!ep->bar_to_atu[bar]) + if (!ep_func) + return -EINVAL; + + if (!ep_func->bar_to_atu[bar]) free_win =3D find_first_zero_bit(ep->ib_window_map, pci->num_ib_windows); else - free_win =3D ep->bar_to_atu[bar] - 1; + free_win =3D ep_func->bar_to_atu[bar] - 1; =20 if (free_win >=3D pci->num_ib_windows) { dev_err(pci->dev, "No free inbound window\n"); @@ -137,33 +141,37 @@ static int dw_pcie_ep_ib_atu_bar(struct dw_pcie_ep *e= p, u8 func_no, int type, * Always increment free_win before assignment, since value 0 is used to = identify * unallocated mapping. */ - ep->bar_to_atu[bar] =3D free_win + 1; + ep_func->bar_to_atu[bar] =3D free_win + 1; set_bit(free_win, ep->ib_window_map); =20 return 0; } =20 -static void dw_pcie_ep_clear_ib_maps(struct dw_pcie_ep *ep, enum pci_barno= bar) +static void dw_pcie_ep_clear_ib_maps(struct dw_pcie_ep *ep, u8 func_no, en= um pci_barno bar) { + struct dw_pcie_ep_func *ep_func =3D dw_pcie_ep_get_func_from_ep(ep, func_= no); struct dw_pcie *pci =3D to_dw_pcie_from_ep(ep); struct device *dev =3D pci->dev; unsigned int i, num; u32 atu_index; u32 *indexes; =20 + if (!ep_func) + return; + /* Tear down the BAR Match Mode mapping, if any. */ - if (ep->bar_to_atu[bar]) { - atu_index =3D ep->bar_to_atu[bar] - 1; + if (ep_func->bar_to_atu[bar]) { + atu_index =3D ep_func->bar_to_atu[bar] - 1; dw_pcie_disable_atu(pci, PCIE_ATU_REGION_DIR_IB, atu_index); clear_bit(atu_index, ep->ib_window_map); - ep->bar_to_atu[bar] =3D 0; + ep_func->bar_to_atu[bar] =3D 0; } =20 /* Tear down all Address Match Mode mappings, if any. */ - indexes =3D ep->ib_atu_indexes[bar]; - num =3D ep->num_ib_atu_indexes[bar]; - ep->ib_atu_indexes[bar] =3D NULL; - ep->num_ib_atu_indexes[bar] =3D 0; + indexes =3D ep_func->ib_atu_indexes[bar]; + num =3D ep_func->num_ib_atu_indexes[bar]; + ep_func->ib_atu_indexes[bar] =3D NULL; + ep_func->num_ib_atu_indexes[bar] =3D 0; if (!indexes) return; for (i =3D 0; i < num; i++) { @@ -248,6 +256,7 @@ static int dw_pcie_ep_validate_submap(struct dw_pcie_ep= *ep, static int dw_pcie_ep_ib_atu_addr(struct dw_pcie_ep *ep, u8 func_no, int t= ype, const struct pci_epf_bar *epf_bar) { + struct dw_pcie_ep_func *ep_func =3D dw_pcie_ep_get_func_from_ep(ep, func_= no); const struct pci_epf_bar_submap *submap =3D epf_bar->submap; struct dw_pcie *pci =3D to_dw_pcie_from_ep(ep); enum pci_barno bar =3D epf_bar->barno; @@ -258,7 +267,7 @@ static int dw_pcie_ep_ib_atu_addr(struct dw_pcie_ep *ep= , u8 func_no, int type, unsigned int i; u32 *indexes; =20 - if (!epf_bar->num_submap || !submap || !epf_bar->size) + if (!ep_func || !epf_bar->num_submap || !submap || !epf_bar->size) return -EINVAL; =20 ret =3D dw_pcie_ep_validate_submap(ep, submap, epf_bar->num_submap, @@ -279,8 +288,8 @@ static int dw_pcie_ep_ib_atu_addr(struct dw_pcie_ep *ep= , u8 func_no, int type, if (!indexes) return -ENOMEM; =20 - ep->ib_atu_indexes[bar] =3D indexes; - ep->num_ib_atu_indexes[bar] =3D 0; + ep_func->ib_atu_indexes[bar] =3D indexes; + ep_func->num_ib_atu_indexes[bar] =3D 0; =20 for (i =3D 0; i < epf_bar->num_submap; i++) { size =3D submap[i].size; @@ -308,11 +317,11 @@ static int dw_pcie_ep_ib_atu_addr(struct dw_pcie_ep *= ep, u8 func_no, int type, =20 set_bit(free_win, ep->ib_window_map); indexes[i] =3D free_win; - ep->num_ib_atu_indexes[bar] =3D i + 1; + ep_func->num_ib_atu_indexes[bar] =3D i + 1; } return 0; err: - dw_pcie_ep_clear_ib_maps(ep, bar); + dw_pcie_ep_clear_ib_maps(ep, func_no, bar); return ret; } =20 @@ -346,15 +355,16 @@ static void dw_pcie_ep_clear_bar(struct pci_epc *epc,= u8 func_no, u8 vfunc_no, struct dw_pcie_ep *ep =3D epc_get_drvdata(epc); struct dw_pcie *pci =3D to_dw_pcie_from_ep(ep); enum pci_barno bar =3D epf_bar->barno; + struct dw_pcie_ep_func *ep_func =3D dw_pcie_ep_get_func_from_ep(ep, func_= no); =20 - if (!ep->epf_bar[bar]) + if (!ep_func || !ep_func->epf_bar[bar]) return; =20 __dw_pcie_ep_reset_bar(pci, func_no, bar, epf_bar->flags); =20 - dw_pcie_ep_clear_ib_maps(ep, bar); + dw_pcie_ep_clear_ib_maps(ep, func_no, bar); =20 - ep->epf_bar[bar] =3D NULL; + ep_func->epf_bar[bar] =3D NULL; } =20 static unsigned int dw_pcie_ep_get_rebar_offset(struct dw_pcie_ep *ep, u8 = func_no, @@ -481,12 +491,16 @@ static int dw_pcie_ep_set_bar(struct pci_epc *epc, u8= func_no, u8 vfunc_no, { struct dw_pcie_ep *ep =3D epc_get_drvdata(epc); struct dw_pcie *pci =3D to_dw_pcie_from_ep(ep); + struct dw_pcie_ep_func *ep_func =3D dw_pcie_ep_get_func_from_ep(ep, func_= no); enum pci_barno bar =3D epf_bar->barno; size_t size =3D epf_bar->size; enum pci_epc_bar_type bar_type; int flags =3D epf_bar->flags; int ret, type; =20 + if (!ep_func) + return -EINVAL; + /* * DWC does not allow BAR pairs to overlap, e.g. you cannot combine BARs * 1 and 2 to form a 64-bit BAR. @@ -500,22 +514,22 @@ static int dw_pcie_ep_set_bar(struct pci_epc *epc, u8= func_no, u8 vfunc_no, * calling clear_bar() would clear the BAR's PCI address assigned by the * host). */ - if (ep->epf_bar[bar]) { + if (ep_func->epf_bar[bar]) { /* * We can only dynamically change a BAR if the new BAR size and * BAR flags do not differ from the existing configuration. */ - if (ep->epf_bar[bar]->barno !=3D bar || - ep->epf_bar[bar]->size !=3D size || - ep->epf_bar[bar]->flags !=3D flags) + if (ep_func->epf_bar[bar]->barno !=3D bar || + ep_func->epf_bar[bar]->size !=3D size || + ep_func->epf_bar[bar]->flags !=3D flags) return -EINVAL; =20 /* * When dynamically changing a BAR, tear down any existing * mappings before re-programming. */ - if (ep->epf_bar[bar]->num_submap || epf_bar->num_submap) - dw_pcie_ep_clear_ib_maps(ep, bar); + if (ep_func->epf_bar[bar]->num_submap || epf_bar->num_submap) + dw_pcie_ep_clear_ib_maps(ep, func_no, bar); =20 /* * When dynamically changing a BAR, skip writing the BAR reg, as @@ -574,7 +588,7 @@ static int dw_pcie_ep_set_bar(struct pci_epc *epc, u8 f= unc_no, u8 vfunc_no, if (ret) return ret; =20 - ep->epf_bar[bar] =3D epf_bar; + ep_func->epf_bar[bar] =3D epf_bar; =20 return 0; } @@ -969,7 +983,7 @@ int dw_pcie_ep_raise_msix_irq(struct dw_pcie_ep *ep, u8= func_no, bir =3D FIELD_GET(PCI_MSIX_TABLE_BIR, tbl_offset); tbl_offset &=3D PCI_MSIX_TABLE_OFFSET; =20 - msix_tbl =3D ep->epf_bar[bir]->addr + tbl_offset; + msix_tbl =3D ep_func->epf_bar[bir]->addr + tbl_offset; msg_addr =3D msix_tbl[(interrupt_num - 1)].msg_addr; msg_data =3D msix_tbl[(interrupt_num - 1)].msg_data; vec_ctrl =3D msix_tbl[(interrupt_num - 1)].vector_ctrl; @@ -1032,11 +1046,14 @@ EXPORT_SYMBOL_GPL(dw_pcie_ep_deinit); =20 static void dw_pcie_ep_init_rebar_registers(struct dw_pcie_ep *ep, u8 func= _no) { - unsigned int offset; - unsigned int nbars; + struct dw_pcie_ep_func *ep_func =3D dw_pcie_ep_get_func_from_ep(ep, func_= no); + unsigned int offset, nbars; enum pci_barno bar; u32 reg, i, val; =20 + if (!ep_func) + return; + offset =3D dw_pcie_ep_find_ext_capability(ep, func_no, PCI_EXT_CAP_ID_REB= AR); =20 if (offset) { @@ -1063,8 +1080,8 @@ static void dw_pcie_ep_init_rebar_registers(struct dw= _pcie_ep *ep, u8 func_no) */ val =3D dw_pcie_ep_readl_dbi(ep, func_no, offset + PCI_REBAR_CTRL); bar =3D FIELD_GET(PCI_REBAR_CTRL_BAR_IDX, val); - if (ep->epf_bar[bar]) - pci_epc_bar_size_to_rebar_cap(ep->epf_bar[bar]->size, &val); + if (ep_func->epf_bar[bar]) + pci_epc_bar_size_to_rebar_cap(ep_func->epf_bar[bar]->size, &val); else val =3D BIT(4); =20 diff --git a/drivers/pci/controller/dwc/pcie-designware.h b/drivers/pci/con= troller/dwc/pcie-designware.h index 8f170122ad78..43d7606bc987 100644 --- a/drivers/pci/controller/dwc/pcie-designware.h +++ b/drivers/pci/controller/dwc/pcie-designware.h @@ -471,6 +471,12 @@ struct dw_pcie_ep_func { u8 func_no; u8 msi_cap; /* MSI capability offset */ u8 msix_cap; /* MSI-X capability offset */ + u8 bar_to_atu[PCI_STD_NUM_BARS]; + struct pci_epf_bar *epf_bar[PCI_STD_NUM_BARS]; + + /* Only for Address Match Mode inbound iATU */ + u32 *ib_atu_indexes[PCI_STD_NUM_BARS]; + unsigned int num_ib_atu_indexes[PCI_STD_NUM_BARS]; }; =20 struct dw_pcie_ep { @@ -480,17 +486,11 @@ struct dw_pcie_ep { phys_addr_t phys_base; size_t addr_size; size_t page_size; - u8 bar_to_atu[PCI_STD_NUM_BARS]; phys_addr_t *outbound_addr; unsigned long *ib_window_map; unsigned long *ob_window_map; void __iomem *msi_mem; phys_addr_t msi_mem_phys; - struct pci_epf_bar *epf_bar[PCI_STD_NUM_BARS]; - - /* Only for Address Match Mode inbound iATU */ - u32 *ib_atu_indexes[PCI_STD_NUM_BARS]; - unsigned int num_ib_atu_indexes[PCI_STD_NUM_BARS]; =20 /* MSI outbound iATU state */ bool msi_iatu_mapped; --=20 2.34.1 From nobody Sat Feb 7 09:30:19 2026 Received: from DM5PR21CU001.outbound.protection.outlook.com (mail-centralusazon11011062.outbound.protection.outlook.com [52.101.62.62]) (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 4EC981E4AF; Fri, 30 Jan 2026 11:55:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.62.62 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769774141; cv=fail; b=DkSXu6DQt2HWLi1yQ5JpdPk66DL8lkVa5yUxchWTe0fmcdpMGMHDwyAjk6+k7DWTeiulaHgLy9PAZF+eMhGhNUgynQaMbQj7ZyyT9Kp8rKBqTQ8nuA4j9u+LEGkwWwYOCEB06Smy3LkLB5cSVAiQA+yd6MHroVu2fwKPcHsJQA4= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769774141; c=relaxed/simple; bh=bsYwCZ9BuKmoQyXibzW95YrBYz5+Xz2NFFT4nffoqHE=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=VuboIwxRUIRBEEVt3t4kHomPdT0DpDuwvKPUvD578D94HmOqncJx9yfMcFlKiOFqW8nLYLvZJQ1a78rb2e6p6E/0CxNYjwmK8ZIwbPHdtLMHFwaAM2B8912hd91b1ooGsgABepkf5310F/OR07bNWYeHsg0nIuU+aGkCNyNNHjc= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ti.com; spf=pass smtp.mailfrom=ti.com; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b=GtJYm79G; arc=fail smtp.client-ip=52.101.62.62 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ti.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ti.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b="GtJYm79G" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=Ig3t1HweU7cl239CRAstORK2DBxKSU+wb4IJvG5taJ7z6CP5Ey6uya9RDnmDh2vigrvUCLCsFfCSmlFkVKr+QeCXXfZ4GIY+xJRerTT7dYehFZ5jigu0uJPVFzTQ9GKgY4dLsG0v/ITdPCvCP810Fgph15LTguhZoZXzIjDMPKUT7b50V5RgKzXyhmLkOZoIda0CT5hNMCxJ5PM57hAv/rirdBybPmG6h5MoC728gXPTbRu6Cm5Ryt69Hj+VAJgLc640KL78r3uto9uq8TOcxm3iITD6xZmwThmC3wz/sskeU1HsuD6sjypkSCFD2pWWhqmR2EcfKNgO7odl474NYg== 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=UVKnmmVqfFnVvMxr7iaP7N1+OH7UwiI7sT/BN0qqEQU=; b=eJZLILQpl9/qzrAsJVZcxQh+w2Ywnuq8JtdE+QVyuJB8WJHTaGsTF/HhwVnyDM0S1nmIS2qIxXCLxyCCNeSdBlPB/yvOn19M2vNg8rq+5lk2PwShRhi8QwZAoCKLjfZKRWzXXbEeTUXAjBx+xO4weev2uxMsPD4TkzpMx7EbeH5vSBFIlEIYeUffCz5jErBRgBfygB3V2MyI+zGLot5UEFmPYIciqKip8m9ONYfqWuVQGw3rSqjK76/qE4+P2OB8Buhc+oZQf8kWLK8q268da7fBRahHk3qSFq/OZhg0d+c8GTusPjTYAx4a3DS1U5QAPlHEqCbLuPFXowju7/P+Kg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 198.47.23.194) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=ti.com; dmarc=pass (p=quarantine sp=none pct=100) action=none header.from=ti.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=UVKnmmVqfFnVvMxr7iaP7N1+OH7UwiI7sT/BN0qqEQU=; b=GtJYm79G5hPZmTxakgu4d+nqkfsz90POTRNxVhxLNZIh+ncruN1S8Rn0Xc37wcY3+gzJwW+GpEZRBXVTE/U9DSTuXC9UFzutXgEiRRwqwW67y+HHcyX7LqSZK2WwISVUv5RfS3abzoCqhAnt4pCK4f5CZTVtzdfueKmHdoEwc2s= Received: from BY5PR17CA0006.namprd17.prod.outlook.com (2603:10b6:a03:1b8::19) by CH3PR10MB6740.namprd10.prod.outlook.com (2603:10b6:610:144::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9564.7; Fri, 30 Jan 2026 11:55:36 +0000 Received: from SJ5PEPF000001D0.namprd05.prod.outlook.com (2603:10b6:a03:1b8:cafe::99) by BY5PR17CA0006.outlook.office365.com (2603:10b6:a03:1b8::19) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9564.11 via Frontend Transport; Fri, 30 Jan 2026 11:55:38 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 198.47.23.194) smtp.mailfrom=ti.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=ti.com; Received-SPF: Pass (protection.outlook.com: domain of ti.com designates 198.47.23.194 as permitted sender) receiver=protection.outlook.com; client-ip=198.47.23.194; helo=lewvzet200.ext.ti.com; pr=C Received: from lewvzet200.ext.ti.com (198.47.23.194) by SJ5PEPF000001D0.mail.protection.outlook.com (10.167.242.52) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9564.3 via Frontend Transport; Fri, 30 Jan 2026 11:55:36 +0000 Received: from DLEE205.ent.ti.com (157.170.170.85) by lewvzet200.ext.ti.com (10.4.14.103) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.20; Fri, 30 Jan 2026 05:55:35 -0600 Received: from DLEE201.ent.ti.com (157.170.170.76) by DLEE205.ent.ti.com (157.170.170.85) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.20; Fri, 30 Jan 2026 05:55:35 -0600 Received: from lelvem-mr05.itg.ti.com (10.180.75.9) by DLEE201.ent.ti.com (157.170.170.76) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.20 via Frontend Transport; Fri, 30 Jan 2026 05:55:35 -0600 Received: from a0507033-hp.dhcp.ti.com (a0507033-hp.dhcp.ti.com [172.24.231.225]) by lelvem-mr05.itg.ti.com (8.18.1/8.18.1) with ESMTP id 60UBtIu31665999; Fri, 30 Jan 2026 05:55:31 -0600 From: Aksh Garg To: , , , , , , , , , , CC: , , , Aksh Garg Subject: [PATCH v5 3/3] PCI: dwc: ep: Add comment explaining controller-level PTM access Date: Fri, 30 Jan 2026 17:25:16 +0530 Message-ID: <20260130115516.515082-4-a-garg7@ti.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20260130115516.515082-1-a-garg7@ti.com> References: <20260130115516.515082-1-a-garg7@ti.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-C2ProcessedOrg: 333ef613-75bf-4e12-a4b1-8e3623f5dcea X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ5PEPF000001D0:EE_|CH3PR10MB6740:EE_ X-MS-Office365-Filtering-Correlation-Id: 9cd4d47c-00ec-4aca-bd34-08de5ff67b3f X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|7416014|1800799024|376014|36860700013|82310400026|921020; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?iS/cPyqh4CDBimlUW8tFbNlVnU44CUaDw7LDsPQprDOR3mPLQAkrB0pjdqFz?= =?us-ascii?Q?JrrqaCPBz7SwsQ/4192x5cgGbNDkKUL2KHN7aPnvH6i+UZ72KQWiCUgFhWyH?= =?us-ascii?Q?AuLTU0jklzWzjGEYw6kEQ3iXuIkmE59Z8cgFJJgsOElfi3gm1/WDoQGWb4dX?= =?us-ascii?Q?/oIqDMoGw5oIvIWxM9DPl8M4P68ElP8D1j+g3xym2dTxxmIYMG8TjPVB6yAW?= =?us-ascii?Q?k+Ad3+sKzUsB0OH6PopazZDrCOXFoR2Y8PDkRrvqt9/YTiry9Ran6A+3e0Hk?= =?us-ascii?Q?E9zl5ziJkPGW/qgAMnggjeMeyjGuF6WW0WLjVIwLKn6YM3zP4EpbuI/bhd1n?= =?us-ascii?Q?7lIiQ+gJjW5D+hbhISD/1FoETkv2qyuXGnECFIFArC24cSWMsHH4AZLumVNw?= =?us-ascii?Q?TlqA0M2/+qEX9ONb/6aNg1HEG3i/o2/h5DHjQOUg2Xp2wNVB2sumZtfrWh+B?= =?us-ascii?Q?kMdDrMhI4o0NHG/qnOSAarjKNJZIBXFshHWHtO8UJxfRlFEnNubKO+4eRreP?= =?us-ascii?Q?QC8HmgcYSrfQTHukhL4FocUQdZpvmAmR8W2vkcJmG8IdFh/jjZWaXZOck7aN?= =?us-ascii?Q?2e9NQYoaQhMH66p4DrdxZN0FDVJ/hnM2hBuh38u3t9Ey93pHsuaLFlf+c84N?= =?us-ascii?Q?BuxwSki3mgjzZ9hQqE9ze1pL5ufcEvS3aqNzlr2B4x7isOlUMgTeKjIERfGV?= =?us-ascii?Q?JDoXeTGeMIQ102vzOE6zwXVghD77Znpz22w4UY0uECkXJMPno4Lb+9trPPuG?= =?us-ascii?Q?hBstiKzo0jPk9JrknZ4iMo7dauZPjLFboJ3sH1Qt8OG+VMYMjAnrpVlmNejX?= =?us-ascii?Q?OJPhvW2T9CTdXd1zWmHQoySPGI2Odf9UbM9voP3BbTH61rPvAf+hIHuTpNuS?= =?us-ascii?Q?1bjqhlg1is+QykTBl5W7XIy62M7d30/CE9nMCb6RxGrV0kFemkx7Jm8hwfTW?= =?us-ascii?Q?kMHSz49lIscSr8wu9sTGyzU3A8FEYbndEpi/rhekjXpBFnBQRsEy4FGt62Po?= =?us-ascii?Q?1W8L4wF9ySNYrFWI1w99emS0lxqqP+JdwcYy00CrAB/MuGX2jvlbaiQgTihO?= =?us-ascii?Q?HvFT+2OcUIu+8IN/xuYp1QfcxlozPDkynoW01PsN6alDnryyGzgNr3blcvpX?= =?us-ascii?Q?jz1v+ojsFxJvzJRPYT/v645ahOnfufGS5OjCRm5in3LkiM1aMCqAVyKUtOr4?= =?us-ascii?Q?h6G6uMYYfuL02m+ljhwCNxiXFpjheZ25+YpD+Hl1KpaLQRJ3aCG4zDbFR4o/?= =?us-ascii?Q?ZAiU19fMLe6HSfh+klH58ryYSnCj5vucLCHdO7+a3ByyJCEimbPZrbQDlN3d?= =?us-ascii?Q?qlq8eJ1LMW/ukeWQg/+y3YHN0DJ17yEJoh5BPJdYbl1B9hGS6pYQMy1/sxkT?= =?us-ascii?Q?SMQrwDHDKLjj9we5e/najjNkSdkpC+3vQ1XYH3BxYxQ5k84Xdj1Gfb9E0PWa?= =?us-ascii?Q?eTeC3Ll2JXAbJt/q/HLhv+MZudfWFUI1OIWuDiXMsUBkeYjhzRtU4x1cTlqJ?= =?us-ascii?Q?bOAXGnXUwg1zECFaq6MF8+uvoywtkkwT9Ov/KE1/kyhus1/B3Ksj7VZOJMGj?= =?us-ascii?Q?GhyfVp/AKTjNBiNCGNp8Zxxwxm04L/093SPyTRhUx4rRPhT9cVfNUp6XonG/?= =?us-ascii?Q?QV3u1Jmy278WfLhlPm7ZKATF1xWFZytZ/hjPipNL4VeoFP3C4sQ8iij6AOjD?= =?us-ascii?Q?ptfZYbTTpICL6KOmeGaEBu0RWzU=3D?= X-Forefront-Antispam-Report: CIP:198.47.23.194;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:lewvzet200.ext.ti.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(7416014)(1800799024)(376014)(36860700013)(82310400026)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: 1kR+a0ZhAKckMtZ+erjgXnqhpsY8RckJvorZd+ng5+YYfVNJSRTrShe9uZJp8fMrRbnPw9Y8eK+uVXqEIhzUhkW8QxK1PtID9/ZgxTfFfTKuMyMy6fqiw81/AyLQ+MtjzMLXNT2fKfrG0MB9FBwmy+CQOZ8Nl9WMQNla4SPkF+3Bhx8aPRxzTAUYQ2gRSvseVEZIYkQTJvgNnAYtfPhqtaEnwhIaL/NbOUgkMUORVF3fiFd2qZ06fgY6yy8dsw0EQM/7ud8agWTAM0pIhTwBubZvEtoUCm85B1QrxsZsMRPeCWIgCAJ7Yt70K8qIh5GuiLvet7ZBTf/jlSAK77Pf+CEcKHU6w82NbZvG9XbrIHRFQCaOdRcMKNHgwk20W68pXIXDFYgTK1ZrYFtlOgIavMKatdPg6Y3IDquTbl1a2Q1ZZ5QRcdLvsC2bi3kZY89P X-OriginatorOrg: ti.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Jan 2026 11:55:36.6256 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 9cd4d47c-00ec-4aca-bd34-08de5ff67b3f X-MS-Exchange-CrossTenant-Id: e5b49634-450b-4709-8abb-1e2b19b982b7 X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=e5b49634-450b-4709-8abb-1e2b19b982b7;Ip=[198.47.23.194];Helo=[lewvzet200.ext.ti.com] X-MS-Exchange-CrossTenant-AuthSource: SJ5PEPF000001D0.namprd05.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR10MB6740 Content-Type: text/plain; charset="utf-8" PCIe r6.0, section 7.9.15 requires PTM capability in exactly one function to control all PTM-capable functions. This makes PTM registers controller-level rather than per-function. Add a comment explaining why PTM capability registers are accessed using the standard DBI accessors instead of func_no indexed per-function accessors. Suggested-by: Niklas Cassel Signed-off-by: Aksh Garg Reviewed-by: Niklas Cassel Reviewed-by: Frank Li --- Changes from v4 to v5: - None Changes from v3 to v4: - Rebase Changes from v2 to v3: - Fixed the suggested nits v4: https://lore.kernel.org/all/20260129091753.490167-4-a-garg7@ti.com/ v3: https://lore.kernel.org/all/20260127085010.446116-4-a-garg7@ti.com/ v2: https://lore.kernel.org/all/20260122082538.309122-4-a-garg7@ti.com/ drivers/pci/controller/dwc/pcie-designware-ep.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/drivers/pci/controller/dwc/pcie-designware-ep.c b/drivers/pci/= controller/dwc/pcie-designware-ep.c index 6d3c35dd280f..dd48e60b513c 100644 --- a/drivers/pci/controller/dwc/pcie-designware-ep.c +++ b/drivers/pci/controller/dwc/pcie-designware-ep.c @@ -1187,6 +1187,16 @@ int dw_pcie_ep_init_registers(struct dw_pcie_ep *ep) if (ep->ops->init) ep->ops->init(ep); =20 + /* + * PCIe r6.0, section 7.9.15 states that for endpoints that support PTM, + * this capability structure is required in exactly one function, which + * controls the PTM behavior of all PTM capable functions. This indicates + * the PTM capability structure represents controller-level registers + * rather than per-function registers. + * + * Therefore, PTM capability registers are configured using the standard = DBI + * accessors, instead of func_no indexed per-function accessors. + */ ptm_cap_base =3D dw_pcie_find_ext_capability(pci, PCI_EXT_CAP_ID_PTM); =20 /* --=20 2.34.1