Received-SPF: pass (google.com: domain of linux-kernel+bounces-171998-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>
Subject: [PATCH v4 00/12] PCI: imx6: Fix\rename\clean up and add lut
 information for imx95
Date: Tue, 07 May 2024 14:45:38 -0400
Message-Id: <20240507-pci2_upstream-v4-0-e8c80d874057@nxp.com>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
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?VWxBWGJGcDIyUFR5MG42dXpMcmFZSlJyZW9DQ24wR1h0aXFtdmNpWFR1eUll?=
 =?utf-8?B?ajBmdDZMaWlCQldpSmRtQXc5L2lOMlZmN2EwNXNaNTNBRC95ZTJDRFdlV3Jv?=
 =?utf-8?B?YU9RYm1QaVpZUGp1SWhlVVl5ZjZYOTJkNXhIUGQ2eExMVFB4QnhpSDEvc3E1?=
 =?utf-8?B?eVlEaUJRN3M4ejRlY1BMNWNNTkkzWXh2L3FuMk5VY1ZvaW9ZTlVKcnQwbGtn?=
 =?utf-8?B?Qnh5THJWQW95VCt0cDRZTXhyOStFREMweE5PWm9NVDJjcEsrKy9hN2g0ZFR2?=
 =?utf-8?B?NksrQU5mSW11bjE1cThLYXM1WWZHV3l4WUFLL1c4OFlIMnJRVzdnc2ROQ0R2?=
 =?utf-8?B?cXBDRHJKS0gyYkp4YmYxTWJVV3BDSHBqR2JQQXI5cGhZOGViU2VCc3ZGWGQw?=
 =?utf-8?B?K3ZoTkp2cGxTTUZzT1phcnpvUzcwYkY3YWZOQkk3VU5tWS80bVpnTjh4Qk0y?=
 =?utf-8?B?MVNCUnovMXJSREwxQ29pSzQrdng1UllSblpJNnpiRW9DMVVwaE9VdytRK28x?=
 =?utf-8?B?L0JoQmM0N3VabmhCZDMzeFNTbzR5dWw0TVhBYlhyaDJydnZpUUI0QUZuT2RB?=
 =?utf-8?B?S2lrVHhOVFpZOFNqcE54aUhURmtpanpweTFKeG9SenpxRkZlOFI5Um8rcVJz?=
 =?utf-8?B?Qk5OdWhoUE5Pbm9kdDBRT3ljVU9KSlh5YWh2ZGFwY1BncGIzSmtHUEU0Y2dN?=
 =?utf-8?B?b2lmWlA3L1kyTlg0Q1NJOUd3RTZSaEFuS2FGMitER0JFRHNMTytxRVA4Zy9s?=
 =?utf-8?B?cW9heG5oMDJ1b0RRb0JPbG8rMmxOWFFONGdjU0hnMk9WK21Na2Q1NDFtM21H?=
 =?utf-8?B?ejVQa3NOTzYwaW52UUFzZHNZNzYvVmRSRklpcDI1UnhnNmdHb0c0cmxqbzBT?=
 =?utf-8?B?VUtsRmhkNUFZWlF2NVdVM2RlUDdrc0R6RVZpbEpER0Q3SEdveHRpaFNva0RK?=
 =?utf-8?B?N1I4WTJDamxJQ1N5RlBib0hGemQ0Ulg2R0ZRc3YxNUxHSzVRcXVUQlpqMG0v?=
 =?utf-8?B?RS9COEkwZ1d1c1dGS29TV1gzMlF0SmFrazBnSC9RaDZsN3VWMG5sVG1ETEFK?=
 =?utf-8?B?K2FKemFrdlRzRzhhV01QMDAzUndKeUJrdFNkVSs2NGtTNUR3d3UvZk5zUTBz?=
 =?utf-8?B?Um1aWUNRQjdLa241UFZ6MGZ3QzdhZVh6MnBxeTh3Wmw0Zko3QlUwNEprL0tv?=
 =?utf-8?B?c3ZqVEtXTTZXRTFrSmxxWjJoa1RqdUFaWTcyMWxBUkNZdTExemM0NUFJbXls?=
 =?utf-8?B?ZmNsb29kamEvdXZHUUdPS3l3dm1iZStaa25WR1EwaTd1VCtueHJaT29TZkw5?=
 =?utf-8?B?cU80Z0s2bzVlNHBoZjdZZ3lsS1F4TWpFMklzUHd4a1BJTG9IYWlWNFRHZ0tP?=
 =?utf-8?B?VldtYVl2WXFVWXdNZ2pDczUzRjJVMTU5eUQ4R1pubCsrcFJjREpzdWtIM211?=
 =?utf-8?B?Umx3cTRvdk5pYkZ5WDFqajZzZUZ4NU9CZ0g5aDYzcVZjTmQ0TE90bFZEZSsw?=
 =?utf-8?B?bkw0M3dlcy8vc3FkcHQ3TmUzQXhRc2h0S2xleXA3Ukx0MzlxbjhRUkxGV3Vx?=
 =?utf-8?B?b1RhdWhmT3dYUCtKT1BjMjBvcTEvU2NuSzZUVnZXRVJiUzhTZ29JRWIrMUFw?=
 =?utf-8?B?RnlucE0xT01hOURjaDdnaXh2eUhkaUlIak1vaVd5c2FBYUM5b2xvOFRSK1ov?=
 =?utf-8?B?ZThKZk1iSXNyYWVycmE2OUtjMkJmVHpCRmMycjdsTy96NXpvbk1GMEN6Q1VL?=
 =?utf-8?B?UlFPVUcvN3lNOXhJUjFBWjVUMktoYlFQZTNWY3ljajQzcFBLY0NiZXhzajYv?=
 =?utf-8?B?QnNiSHZDQ1Rvbmp4WEYyWFFkUDFlVDJydTUwVnZUeTI4dHhGS3piUzBQMnhG?=
 =?utf-8?B?STlGcGM2NUlrdjB3eVg2dGk3NXRSV1JwQjNQSDV0S0FCc2RCZFlBeExkbExo?=
 =?utf-8?B?QnVrUlpNcVpQTlpPQm9BQzJUWkszTURYbjVPdEFKTFJRQXk0azFNaHphZ2Vr?=
 =?utf-8?B?LzI2TisxcStRL1FndXI5eXJST3I4UHZDVmZtNTVUcUdPMTI3QTdoTjBHMGxw?=
 =?utf-8?B?VTlCZUZtTVNJTldRNjc3cXIyMEgwSGNwM01qa28vOFJpOHVKRWdUZ3Z2SFZq?=
 =?utf-8?Q?gehipZUDrEcDBH8L+//cbG5ze?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ae20b9ad-e65b-4f56-9a7f-08dc6ec5fc68
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 May 2024 18:46:20.3344
 (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: Gc12jDa0RzCNXDx3L1pCLXKovSdEriEhiM9fOOl0yYFtkyFxW37XAsDfP+Fl+FRlYVO7zYgmpvo1OXI186WKBg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB8204

Fixed 8mp EP mode problem.

imx6 actaully for all imx chips (imx6*, imx7*, imx8*, imx9*). To avoid     
confuse, rename all imx6_* to imx_*, IMX6_* to IMX_*. pci-imx6.c to        
pci-imx.c to avoid confuse.                                                

Using callback to reduce switch case for core reset and refclk.            

Add imx95 iommux and its stream id information.                            

Base on linux-pci/controller/imx

To: Richard Zhu <hongxing.zhu@nxp.com>
To: Lucas Stach <l.stach@pengutronix.de>
To: Lorenzo Pieralisi <lpieralisi@kernel.org>
To: Krzysztof Wilczyński <kw@linux.com>
To: Rob Herring <robh@kernel.org>
To: Bjorn Helgaas <bhelgaas@google.com>
To: Shawn Guo <shawnguo@kernel.org>
To: Sascha Hauer <s.hauer@pengutronix.de>
To: Pengutronix Kernel Team <kernel@pengutronix.de>
To: Fabio Estevam <festevam@gmail.com>
To: NXP Linux Team <linux-imx@nxp.com>
To: Philipp Zabel <p.zabel@pengutronix.de>
To: Liam Girdwood <lgirdwood@gmail.com>
To: Mark Brown <broonie@kernel.org>
To: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
To: Conor Dooley <conor+dt@kernel.org>
Cc: linux-pci@vger.kernel.org
Cc: imx@lists.linux.dev
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
Cc: bpf@vger.kernel.org
Cc: devicetree@vger.kernel.org
Signed-off-by: Frank Li <Frank.Li@nxp.com>

Changes in v4:                                                             
- Improve comment message for patch 1 and 2.
- Rework commit message for patch 3 and add mani's review tag
- Remove file rename patch and update maintainer patch
- [PATCH v3 06/11] PCI: imx: Simplify switch-case logic by involve set_ref_clk callback
	remove extra space.
	keep original comments format (wrap at 80 column width)
	update error message "'Failed to enable PCIe REFCLK'"
- PATCH v3 07/11] PCI: imx: Simplify switch-case logic by involve core_reset callback
	keep exact the logic as original code
- Add patch to update comment about workaround ERR010728
- Add patch about help function imx_pcie_match_device()
- Using bus device notify to update LUT information for imx95 to avoid
parse iommu-map and msi-map in driver code.  Bus notify will better and
only update lut when device added.
- split patch call PHY interface function.
- Improve commit message for imx8q. remove local-address dts proptery. and
use standard "range" to convert cpu address to bus address.             
- Check entry in cpu_fix function is too late. Check it at probe
- Link to v3: https://lore.kernel.org/r/20240402-pci2_upstream-v3-0-803414bdb430@nxp.com

Changes in v3:
- Add an EP fixed patch
  PCI: imx6: Fix PCIe link down when i.MX8MM and i.MX8MP PCIe is EP mode
  PCI: imx6: Fix i.MX8MP PCIe EP can not trigger MSI
- Add 8qxp rc support
dt-bing yaml pass binding check
make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- -j8  dt_binding_check DT_SCHEMA_FILES=fsl,imx6q-pcie.yaml
  LINT    Documentation/devicetree/bindings
  DTEX    Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.example.dts
  CHKDT   Documentation/devicetree/bindings/processed-schema.json
  SCHEMA  Documentation/devicetree/bindings/processed-schema.json
  DTC_CHK Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.example.dtb

- Link to v2: https://lore.kernel.org/r/20240304-pci2_upstream-v2-0-ad07c5eb6d67@nxp.com

Changes in v2:
- remove file to 'pcie-imx.c'
- keep CONFIG unchange.
- Link to v1: https://lore.kernel.org/r/20240227-pci2_upstream-v1-0-b952f8333606@nxp.com

---
Frank Li (8):
      PCI: imx6: Rename imx6_* with imx_*
      PCI: imx6: Introduce SoC specific callbacks for controlling REFCLK
      PCI: imx6: Simplify switch-case logic by involve core_reset callback
      PCI: imx6: Improve comment for workaround ERR010728
      PCI: imx6: Add help function imx_pcie_match_device()
      PCI: imx6: Config look up table(LUT) to support MSI ITS and IOMMU for i.MX95
      PCI: imx6: Consolidate redundant if-checks
      PCI: imx6: Call: Common PHY API to set mode, speed, and submode

Richard Zhu (4):
      PCI: imx6: Fix establish link failure in EP mode for iMX8MM and iMX8MP
      PCI: imx6: Fix i.MX8MP PCIe EP's occasional failure to trigger MSI
      dt-bindings: imx6q-pcie: Add i.MX8Q pcie compatible string
      PCI: imx6: Add i.MX8Q PCIe root complex (RC) support

 .../devicetree/bindings/pci/fsl,imx6q-pcie.yaml    |   16 +
 drivers/pci/controller/dwc/pci-imx6.c              | 1193 ++++++++++++--------
 2 files changed, 736 insertions(+), 473 deletions(-)
---
base-commit: 9d8b196fd12e52820a40c21297a97ea6186aa87e
change-id: 20240227-pci2_upstream-0cdd19a15163

Best regards,
---
Frank Li <Frank.Li@nxp.com>