Received-SPF: pass (google.com: domain of linux-kernel+bounces-128145-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
From: Frank Li <Frank.Li@nxp.com>
Date: Tue, 02 Apr 2024 10:33:38 -0400
Subject: [PATCH v3 02/11] PCI: imx6: Fix i.MX8MP PCIe EP can not trigger
 MSI
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Message-Id: <20240402-pci2_upstream-v3-2-803414bdb430@nxp.com>
References: <20240402-pci2_upstream-v3-0-803414bdb430@nxp.com>
In-Reply-To: <20240402-pci2_upstream-v3-0-803414bdb430@nxp.com>
To: Richard Zhu <hongxing.zhu@nxp.com>, 
 Lucas Stach <l.stach@pengutronix.de>, 
 Lorenzo Pieralisi <lpieralisi@kernel.org>, 
 =?utf-8?q?Krzysztof_Wilczy=C5=84ski?= <kw@linux.com>, 
 Rob Herring <robh@kernel.org>, Bjorn Helgaas <bhelgaas@google.com>, 
 Shawn Guo <shawnguo@kernel.org>, Sascha Hauer <s.hauer@pengutronix.de>, 
 Pengutronix Kernel Team <kernel@pengutronix.de>, 
 Fabio Estevam <festevam@gmail.com>, NXP Linux Team <linux-imx@nxp.com>, 
 Philipp Zabel <p.zabel@pengutronix.de>, Liam Girdwood <lgirdwood@gmail.com>, 
 Mark Brown <broonie@kernel.org>, 
 Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>, 
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, 
 Conor Dooley <conor+dt@kernel.org>
Cc: linux-pci@vger.kernel.org, imx@lists.linux.dev, 
 linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, 
 bpf@vger.kernel.org, devicetree@vger.kernel.org, 
 Frank Li <Frank.Li@nxp.com>, Jason Liu <jason.hui.liu@nxp.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?aFFmdWZMUmdzMnI4SU9xZUtCejQ2cVdUNTNCOFJTYzNyVXQ4c3hhdjVkZlQ5?=
 =?utf-8?B?YlpicHkvd24rWHZsMVdTVTJhM3Y5ckU5QUVNSEJDdXZQcFl0QjVLK29acnNS?=
 =?utf-8?B?SXJVVzQzelhmUDRBRDdrSUlud0JUVnQ5WHpaS04rbjMzNlM0OVFGZERyQWF2?=
 =?utf-8?B?WjR2c0lZT3VvYmMrVlc5elJCTDlMN2xHSTdPL1lXMGRCek1lVmdlMmcxb0p3?=
 =?utf-8?B?NFVhV203MmZxZ0c4ZkwzeEdqVWhDTXg4WlZMSTNQSk9RVnF0YnpOK0M4R1BP?=
 =?utf-8?B?U3M2THpUN0pUaUhuay9WSTRoeG0rYjNpNlR1aDBvdGlrV1NSbjZuZ1FlZStE?=
 =?utf-8?B?T3JhT01wUFZveGdHVEhmNEo1UVNzUTdnV0FPTkU3SldFMURoSFYxNFVaWGNy?=
 =?utf-8?B?Y2dhRTF5bytLR1I0V2ZXYzlRQ0o0UnViU1NQNGUxcDJIUzFYekgvbjcrZTBN?=
 =?utf-8?B?UEptaTVnVmZTazF4QkJNK1FjTEJmRk1zZWxpSzhBNlhleVpYUjBuSVIrbGhj?=
 =?utf-8?B?MlpzTlAxNXlOTzZYUDMwci85dFB6L1VYZmxoSTM4b3RPMFpXZTNZcFE4Q2V1?=
 =?utf-8?B?dlJxOHIxbnlLZ09EYS93N2xwczF1VEJCTGdYb1NFMDRvc21HaHB4K255QzZq?=
 =?utf-8?B?ZEg2VGUzRFhYRXdnWFJmRWZYWTBEVnZwbi85aXd5TlV5YmIyOUsrS2VSWjl1?=
 =?utf-8?B?STZDMUh2MWx5MHVLRWM3LzIrQWNOSGFhQmlqWHQ5Y0ttaTJWZFk5MVFMOVZm?=
 =?utf-8?B?dUwyQjFYNkdsYkM0bEFXN1luVy9Td201dm5vYm54d0I5bUtkMnJsbmpNNDhx?=
 =?utf-8?B?ZmxQZDk2Z2doQnJOOXo3Q1h0b3FQWS84RXp1QzdLR2RoM1B2RHZES0xLMGl6?=
 =?utf-8?B?M01lakV0VzdCSDJnZ1QwQ2pUTDBqWlBtdUNyaGVuSWp5dGhPRmpld1BHK3Q2?=
 =?utf-8?B?WnpJd28vWGhzb3dzaVdzcGhmTXdlYjExNkp6VnJIb3IxeFRIc1ZhYXdyY1ZP?=
 =?utf-8?B?T2t4Y3RsNmhRcW9YWitSZExiWHI3VFREakVxRERCUDZQK1NSOHNNTXpOUll0?=
 =?utf-8?B?b2x6Ri9wMHRUWGpFM0NBTTlNeE1XSHk4VGVvWnFUaUFrcW84bDRBbFpZNVNS?=
 =?utf-8?B?UXFwY1Y2Qk56b0c5V0g5SGhMT2dDbS9TNFFHdi8yNEliYW9pRUp2SzlzSTFY?=
 =?utf-8?B?Mzh0aHNpRlhrRlF1ekhoOHlXWlc5d3ZXTHE4SEpGRUczbTNIQlBKMG1Ib2V4?=
 =?utf-8?B?cWNwL3JsenRDajRORTlaZVlMQzVEbWJCNEZUQy9KYWY4c251TEhSMHRBUXlY?=
 =?utf-8?B?NE9QM0UydE9sNlFhcjA4dGJPTE8ySkhaYmJqYURCYmJWTWJWOENxZWpRaEhN?=
 =?utf-8?B?RjZJclM0d0ZFTjRLNXBIdmJvQVM4eHZSWU5vNnVkcU9EM1EvVUJEaXpVRjNC?=
 =?utf-8?B?LzB6UTNRR2xpSDlaY2MzVFlqUU1kV2QybmlWQ09ualMyVWx1ck44UnVQS1Q5?=
 =?utf-8?B?YWY0TUpPbzJKMjRuWW8zanhjWVNab2poWVZKaDdLekpYUzFzKzlLMFhhTWND?=
 =?utf-8?B?eFMvenAzQ3laVWJESy9uU3JOOFdaL1RIVHNobEtXcWFucTJaWm1nVVJ4U0Y2?=
 =?utf-8?B?QnNrbjR4NjZmZDRhU1ZDYXA1NHd2ZGN5OVFLWWdEOS9oNXdWcFNoTGFtdlFa?=
 =?utf-8?B?Szc0OFdEby8vUnRZT0FvY3JzdmdQMnhEQ1lFTzZZWXRyME1PY1VLbEdrMkJY?=
 =?utf-8?B?amJxd0VtY3Y0RTFuTnpaYS9hdmVZSkRZS3dSSnZvNlh2ZVNkaVY0bWtBdzNB?=
 =?utf-8?B?ajQ5VmltVHBoTUJoSWhjRUtkcS9TVkhDbzl2MVJCYXVjMVZqbSsvZmw0T3JD?=
 =?utf-8?B?NEJETTZoZ1RJSGN1VFAvQU1qeW4yM3Q0TDJFQTg0ZThBUWRvaytHWnlzaFds?=
 =?utf-8?B?dTdLaHJRSmp5Y0JtWnMrS3k3cWdoQ0JpdGZlL2tnRkl3aFMxb0JLQ0t2cmpJ?=
 =?utf-8?B?c1h4L2lrV3lzZ1RnMDZOV3JRUXNLUXRBZWsrc3I0VDN6bjF5bzB5eUd0U3RZ?=
 =?utf-8?B?bEFZbkZta1IvRkJLRVJTT1QwdytKbElzWEllZ2xkUVR3ako1UE9xMHViM01J?=
 =?utf-8?Q?lgCRW6N9POAQuo6FMUeroF5mc?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b2e5d60f-08d6-4790-6341-08dc5321f3fc
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Apr 2024 14:34:07.3435
 (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: VGC2SLRAZ4fES+DOF6BtFygjzVBjlptlAs2HYNK8+TAzO6+qcNnWYQUt9PWWTmsmYRyxRSVKqQu0lInv77WwvQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB10036

From: Richard Zhu <hongxing.zhu@nxp.com>

Fix i.MX8MP PCIe EP can't trigger MSI issue.
There is one 64Kbytes minimal requirement on i.MX8M PCIe outbound
region configuration.

EP uses Bar0 to set the outboud region to configure the MSI setting.
Set the page_size to "epc_features->align" to meet the requirement,
let the MSI can be triggered successfully.

Fixes: 1bd0d43dcf3b ("PCI: imx6: Clean up addr_space retrieval code")
Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com>
Acked-by: Jason Liu <jason.hui.liu@nxp.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 drivers/pci/controller/dwc/pci-imx6.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/pci/controller/dwc/pci-imx6.c b/drivers/pci/controller/dwc/pci-imx6.c
index e43eda6b33ca7..6c4d25b92225e 100644
--- a/drivers/pci/controller/dwc/pci-imx6.c
+++ b/drivers/pci/controller/dwc/pci-imx6.c
@@ -1118,6 +1118,8 @@ static int imx6_add_pcie_ep(struct imx6_pcie *imx6_pcie,
 	if (imx6_check_flag(imx6_pcie, IMX6_PCIE_FLAG_SUPPORT_64BIT))
 		dma_set_mask_and_coherent(dev, DMA_BIT_MASK(64));
 
+	ep->page_size = imx6_pcie->drvdata->epc_features->align;
+
 	ret = dw_pcie_ep_init(ep);
 	if (ret) {
 		dev_err(dev, "failed to initialize endpoint\n");

-- 
2.34.1