Received-SPF: pass (google.com: domain of linux-kernel+bounces-12200-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
From: Frank Li <Frank.Li@nxp.com>
To: krzysztof.kozlowski@linaro.org
Cc: Frank.Li@nxp.com,
	bhelgaas@google.com,
	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,
	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,
	lpieralisi@kernel.org,
	manivannan.sadhasivam@linaro.org,
	robh@kernel.org,
	s.hauer@pengutronix.de,
	shawnguo@kernel.org
Subject: [PATCH v7 04/16] dt-bindings: imx6q-pcie: Add linux,pci-domain as required for iMX8MQ
Date: Wed, 27 Dec 2023 13:27:15 -0500
Message-Id: <20231227182727.1747435-5-Frank.Li@nxp.com>
In-Reply-To: <20231227182727.1747435-1-Frank.Li@nxp.com>
References: <20231227182727.1747435-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?xD0tJjjLNketT5puMGBZrvxrpeeOOYJybZzEewBeFnUZdNyRsSe5BKP0F6cx?=
 =?us-ascii?Q?CvQrANtyudOoOQJzsUptoEeCl6uMto9vXC3YFfYRkVbrjG/0UnRKioJXYXZV?=
 =?us-ascii?Q?ClHJYWKSujtnZlhXLpC+S6UVaQ9RHnWROVX4R4Bev7j1HA6k3r+kABwYenV5?=
 =?us-ascii?Q?SQPoSMrWVtEjIq1+s2KbdgKa55xUNl3PGlM7zFWum48nnxMRYtB2aXOPvog6?=
 =?us-ascii?Q?vla11qMaCnIqFyM4OPP25OtoTKq1kVmocaYHttukUj/GNyypKC74yztWEB4Z?=
 =?us-ascii?Q?+1/B8XX+sw1To5HBw7L+iYGLvvlwSzx/ccBj7x5RJ/7v1dAEkjCO8vEHrbFu?=
 =?us-ascii?Q?rAKQVwHq4tHxFLRnXlkjjW9HT+H4QeD4uj2loYGAXBPtWmOCXL9jIeJBYItj?=
 =?us-ascii?Q?iXZKEraQiHr9AeH/y33j2LaWcOFnA3BswBGzYW2JEEvzKj8Ti+NBbkhKQvx0?=
 =?us-ascii?Q?KlTCNME6eaoI4L6lFtEypJls1zIxv7ZH7cZM8iyNHLNzbWz8Q0lvsLh1sWda?=
 =?us-ascii?Q?hWg2lkmyC/maydGedP8uAY+UnpncAU8tRlRU+6+MlULtaKSbPA7uRUhzU1z9?=
 =?us-ascii?Q?EyFhMtRvZ+dQK0BCZMwoo9PmKYKI37cy7aJtFqiYAA+zoWtod1ymM2t5xqDt?=
 =?us-ascii?Q?nqXnuLRIUr1PQp2hjy8SXZlBQ4VsNSNs+ZvCixKodi+zuc1eJ10JGsJw0J5r?=
 =?us-ascii?Q?ELhqghdLsE3IroagPwnat8EGmu69M/EfEtrMsyAOJG7r4mAph10mms9nNIrM?=
 =?us-ascii?Q?jJAr/EvYdG79HrnguEvGr3DeB/4oIGgIZ/h8JRvlXSO63P7PHujKvQxITLsh?=
 =?us-ascii?Q?h4Ne/RGG8WSuxAic4YmqfkQrVz9u3jYc2FXqlT3p6QyitGErlogGZvLFLSl6?=
 =?us-ascii?Q?W9qCuvio7ja8wmWLsAMKiwjSVe8FIgC11GqNCj9dUucWUdqgimUudMrND9TN?=
 =?us-ascii?Q?81c6kDPb/E1KcPxI8OzzsYPnZUvFI0zyDnhgwziEeW8w9iwE0YvLUw6K4VP5?=
 =?us-ascii?Q?7W7m3B4d7h2PcJ7EUPP+pCBtqV1JW3DmwTViy/W7QmwSgGiSIgQIju/+AL1N?=
 =?us-ascii?Q?vLzh64G9XvDHWD65tamx5GCapr8y2RZ8dVgdr4conZE+Q1kXKeSU9whNJQqz?=
 =?us-ascii?Q?rLYFTgEh6X6S4iJl4Hg0P7Cb/Tr6piyhlLQAC/u0Xlu7xWnl+Obx5hf2YdZ+?=
 =?us-ascii?Q?zqpgsqOl3ONHe6Q8OLgg3s5pQox6vVXHhn9yJOVohznVzozLIY6IJwBp+7GW?=
 =?us-ascii?Q?k/iDAT7i22O+QYANuXtHN1zLIzj/IsHmwb6ChQ7VH2BCC+XUq0OD5+HW1qBQ?=
 =?us-ascii?Q?2kPX9+YTQNvCN7njII6izQ+EKqsiejiFpuPWhRsbXRaGsmaJtGqp6Sec+68b?=
 =?us-ascii?Q?2DhsFx+DiSJUEnijQFn33M7YaMdgT+UBHmBWkzwZ7hHXzAD3ryRKUOwKu7x6?=
 =?us-ascii?Q?uEBRAUVrx5G1LJ3veeICTSnoB06JWR9WrqXGZMJI//+3Rq7foGjKo5lrl4Ss?=
 =?us-ascii?Q?4z80jJMM5U3lbkBW5A7uOyXsb6bFBvdHqewGAAueDZc5iqIakAaP37cwjmj0?=
 =?us-ascii?Q?AD5vKS/sKGOdIBA909Y=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 897c047d-ba04-4f56-d7e4-08dc07099477
X-MS-Exchange-CrossTenant-AuthSource: AM6PR04MB4838.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Dec 2023 18:28:10.8447
 (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: MnFey9Gyjnb1sF6iCLzGWx1wrP7W8Qs8Vqi+lRzDUdct5Ag9uXSaLief/mwoZqn7u+vhZ8wYYDe5T6s3aClGJA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU2PR04MB9050

iMX8MQ have two pci controllers. Adds "linux,pci-domain" as required
proptery for iMX8MQ to indicate pci controller index.

This adjustment paves the way for eliminating the hardcoded check on the
base register for acquiring the controller_id.

	...
	if (dbi_base->start == IMX8MQ_PCIE2_BASE_ADDR)
		imx6_pcie->controller_id = 1;
	...

The controller_id is crucial and utilized for certain register bit
positions. It must align precisely with the controller index in the SoC.
An auto-incremented ID don't fit this case. The DTS or fuse configurations
may deactivate specific PCI controllers.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---

Notes:
    Change from v5 to v6
    - rework commit message to explain why need required and why auto increase
    id not work
    
    Change from v4 to v5
    - new patch at v5

 .../bindings/pci/fsl,imx6q-pcie-common.yaml           | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie-common.yaml b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie-common.yaml
index d91b639ae7ae7..8f39b4e6e8491 100644
--- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie-common.yaml
+++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie-common.yaml
@@ -265,6 +265,17 @@ allOf:
             - const: apps
             - const: turnoff
 
+  - if:
+      properties:
+        compatible:
+          contains:
+            enum:
+              - fsl,imx8mq-pcie
+              - fsl,imx8mq-pcie-ep
+    then:
+      required:
+        - linux,pci-domain
+
 additionalProperties: true
 
 ...
-- 
2.34.1