Received-SPF: pass (google.com: domain of linux-kernel+bounces-73340-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: Frank Li <Frank.Li@nxp.com>
To: lpieralisi@kernel.org
Cc: Frank.li@nxp.com,
	bhelgaas@google.com,
	cassel@kernel.org,
	conor+dt@kernel.org,
	devicetree@vger.kernel.org,
	festevam@gmail.com,
	helgaas@kernel.org,
	hongxing.zhu@nxp.com,
	imx@lists.linux.dev,
	kernel@pengutronix.de,
	krzysztof.kozlowski+dt@linaro.org,
	krzysztof.kozlowski@linaro.org,
	kw@linux.com,
	l.stach@pengutronix.de,
	linux-arm-kernel@lists.infradead.org,
	linux-imx@nxp.com,
	linux-kernel@vger.kernel.org,
	linux-pci@vger.kernel.org,
	manivannan.sadhasivam@linaro.org,
	robh@kernel.org,
	s.hauer@pengutronix.de,
	shawnguo@kernel.org
Subject: [PATCH v11 12/14] PCI: imx6: Add epc_features in imx6_pcie_drvdata
Date: Tue, 20 Feb 2024 11:19:22 -0500
Message-Id: <20240220161924.3871774-13-Frank.Li@nxp.com>
In-Reply-To: <20240220161924.3871774-1-Frank.Li@nxp.com>
References: <20240220161924.3871774-1-Frank.Li@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?yBp75FEf6vkHZCOkjsI0LWziylMygFjz+fSnjumYIFzdSz0MKlCmGrl79VTq?=
 =?us-ascii?Q?/23fxkzOvhAkTx9rA4ms3BJlNuPNLg4D+ib0c9E1y49fEJYzfEE/LzPY3OI6?=
 =?us-ascii?Q?nXynLuyWzdqSWa+6UXl9LhzExgqNn6HWc4wbaen6UnPMyjGPkhmu1l3egcDu?=
 =?us-ascii?Q?26fAnTU+QRC6oQHEeUlAvTfr3KDrGZtJByQRQYdtGUEI9g2wg2xltSr/F9Lt?=
 =?us-ascii?Q?IViQylGi1mDWywIjb+w9dvb+YwMIYRbGLN/YAw/hq7X75RUeyIrd6HJD6CTy?=
 =?us-ascii?Q?w2HbNY5jhfAIMYlDVkj0ejxcxLhOpM1W1nCzGTAHXu97SsBK+IhTrZF0cAbC?=
 =?us-ascii?Q?wrZqm245BpfTdKMKOW95djEV+w0rRRQ53QoQkm0m+3b2Vi23YghdMMZX1Yy+?=
 =?us-ascii?Q?IvThrKRcNtjsZXqydsmlE3kt25Ro+OFdpvoowDZQKKHFOfnesEBHjKKE2FOB?=
 =?us-ascii?Q?LsvNcH/T4sdo8h7gWuAdvOTzc2eFa9VOo4ERGCHcSlWVsRfnpz7eMmT1D+ZR?=
 =?us-ascii?Q?enAa5uE1YmTXpgbtql56JxWkaifaeO/keEXJBRluef8eyXW5IlG3X6cvm3/2?=
 =?us-ascii?Q?bxMcBm2m2fkSmhCVNyX/LuEDYrlcA5Z4TOh0LYFCL4bKDF1j3gp6e0SChLez?=
 =?us-ascii?Q?Rh1tKWRTROGgJmm8EBnHCa6OIdvtx9WSYgH7ec1XalA20cgNtCVFZDlH2TsU?=
 =?us-ascii?Q?dqHVMruPBB69EbFRS05INLj0DIGZIcn2ELHxQUCnyh5TW01jNe5a6NuhrJ8L?=
 =?us-ascii?Q?IW39rPVnxPwch1oKbdMtdsS1slqH/afNYkT9cTexMk+odpLNlVj9qhP5jM1X?=
 =?us-ascii?Q?vuCceO+dmmyJejzFl6MHWsIFO1tYMTYulkKPkoDUKzONH/q6R93MoE/F1hEk?=
 =?us-ascii?Q?JbrsNf2wBdhtALEhQc88+p4qfG4dFTDT4nFD8MGC2xc81o4QP/wLI3PZ/tqv?=
 =?us-ascii?Q?I7sfe2Y67AUT01GgEwqf/YaZGBNEHUhOPnodODfe8ZA6Ror7PKlc0LThuAyj?=
 =?us-ascii?Q?drTP8Aw6cw39sqiJsgG7FyXu+MmvUMbGlwaVZljMXuju0P+e9RMpOkVzOuho?=
 =?us-ascii?Q?LDhM9AlRuGpEyjXtY06PVZxNuTjd9TFDnOBgLXh6VxOGPKq9odbcbHWNw3qS?=
 =?us-ascii?Q?QIncPEAgMoirLiMgsSBRkAt+BgIOKkK9HChjK68/qX0LpA+DN0ax4YYr3NgN?=
 =?us-ascii?Q?tg/g+Do0B6k/TJWr07Vkwh1eOAeKIIAGRFkkyezwBIuqoEwvbuprjNzZydX4?=
 =?us-ascii?Q?FY2vKhYK8FOLtNlCcY0hNoZihqP17xutaqtnGcENQd8ybCBibsqM5JIH71Ns?=
 =?us-ascii?Q?YPiS8FRofJbXSsLfWlFZFR/MtHdAXSPe4CnneK6+yXQWyxfor2JKJCZR4kpN?=
 =?us-ascii?Q?l81oU6MleW9kqr9fL0k/1olsPWqQSMGaW0ZdI7FL9DDwDH6Ea3iFnC/x8T1e?=
 =?us-ascii?Q?rRCaZwH4VdrIT3W0hRydzp4CQAgrWbKwxQwHpgEzCcfYQ/2ijVZn7/ON0J8a?=
 =?us-ascii?Q?I/FA+/8Zy7KEqFsPX9guz7JWx5DjxPQXtpr5/yJI9Y7wHtS6s5y31eXxc5BL?=
 =?us-ascii?Q?fnv/yzOlD5Pv1rkEp8FARPuk5DPqpdzcD7WM+02e?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: dbdff3f7-e48a-4a07-7525-08dc322fe510
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Feb 2024 16:20:46.9999
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 2bXoHXpub57pCgSKzPX+XvI+hQqsP8NTGsWJDU9xtoizPFFp8+uC8U87ELwr9OrPx3ZuLzJKnd7HezHyYx+OEA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VE1PR04MB7295

The i.MX EP exhibits variations in epc_features among different EP
configurations. This introduces the addition of epc_features in
imx6_pcie_drvdata to accommodate these differences. It's important to note
that there are no functional changes in this commit; instead, it lays the
groundwork for supporting i.MX95 EP functions.

Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---

Notes:
    Change from v7 to v8
    - Added  Manivannan Sadhasivam's review tag
    Change from v6 to v7
    - none
    Change from v5 to v6
    - add missed maxitems.
    - add comments about reuse linux,pci-domain as controller id.
    linux,pci-domain have not defined at PCI endpoint side.
    
    Change from v1 to v3
    - new patch at v3

 drivers/pci/controller/dwc/pci-imx6.c | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/drivers/pci/controller/dwc/pci-imx6.c b/drivers/pci/controller/dwc/pci-imx6.c
index 3009c3ffa1415..b714edac6d566 100644
--- a/drivers/pci/controller/dwc/pci-imx6.c
+++ b/drivers/pci/controller/dwc/pci-imx6.c
@@ -105,6 +105,7 @@ struct imx6_pcie_drvdata {
 	const u32 ltssm_mask;
 	const u32 mode_off[IMX6_PCIE_MAX_INSTANCES];
 	const u32 mode_mask[IMX6_PCIE_MAX_INSTANCES];
+	const struct pci_epc_features *epc_features;
 	int (*init_phy)(struct imx6_pcie *pcie);
 };
 
@@ -1053,7 +1054,10 @@ static const struct pci_epc_features imx8m_pcie_epc_features = {
 static const struct pci_epc_features*
 imx6_pcie_ep_get_features(struct dw_pcie_ep *ep)
 {
-	return &imx8m_pcie_epc_features;
+	struct dw_pcie *pci = to_dw_pcie_from_ep(ep);
+	struct imx6_pcie *imx6_pcie = to_imx6_pcie(pci);
+
+	return imx6_pcie->drvdata->epc_features;
 }
 
 static const struct dw_pcie_ep_ops pcie_ep_ops = {
@@ -1533,6 +1537,7 @@ static const struct imx6_pcie_drvdata drvdata[] = {
 		.mode_mask[0] = IMX6Q_GPR12_DEVICE_TYPE,
 		.mode_off[1] = IOMUXC_GPR12,
 		.mode_mask[1] = IMX8MQ_GPR12_PCIE2_CTRL_DEVICE_TYPE,
+		.epc_features = &imx8m_pcie_epc_features,
 		.init_phy = imx8mq_pcie_init_phy,
 	},
 	[IMX8MM_EP] = {
@@ -1544,6 +1549,7 @@ static const struct imx6_pcie_drvdata drvdata[] = {
 		.clks_cnt = ARRAY_SIZE(imx8mm_clks),
 		.mode_off[0] = IOMUXC_GPR12,
 		.mode_mask[0] = IMX6Q_GPR12_DEVICE_TYPE,
+		.epc_features = &imx8m_pcie_epc_features,
 	},
 	[IMX8MP_EP] = {
 		.variant = IMX8MP_EP,
@@ -1554,6 +1560,7 @@ static const struct imx6_pcie_drvdata drvdata[] = {
 		.clks_cnt = ARRAY_SIZE(imx8mm_clks),
 		.mode_off[0] = IOMUXC_GPR12,
 		.mode_mask[0] = IMX6Q_GPR12_DEVICE_TYPE,
+		.epc_features = &imx8m_pcie_epc_features,
 	},
 };
 
-- 
2.34.1