linux-kernel - RE: [PATCH v7 0/7] Add the iMX8MP PCIe support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <AS8PR04MB8676F2D976E3BF6A17D567A68C7A9@AS8PR04MB8676.eurprd04.prod.outlook.com>
Date:   Fri, 2 Sep 2022 09:16:49 +0000
From:   Hongxing Zhu <hongxing.zhu@....com>
To:     Hongxing Zhu <hongxing.zhu@....com>,
        "p.zabel@...gutronix.de" <p.zabel@...gutronix.de>,
        "l.stach@...gutronix.de" <l.stach@...gutronix.de>,
        "bhelgaas@...gle.com" <bhelgaas@...gle.com>,
        "lorenzo.pieralisi@....com" <lorenzo.pieralisi@....com>,
        "robh@...nel.org" <robh@...nel.org>,
        "shawnguo@...nel.org" <shawnguo@...nel.org>,
        "vkoul@...nel.org" <vkoul@...nel.org>,
        "alexander.stein@...tq-group.com" <alexander.stein@...tq-group.com>,
        "marex@...x.de" <marex@...x.de>,
        "richard.leitner@...ux.dev" <richard.leitner@...ux.dev>
CC:     "linux-phy@...ts.infradead.org" <linux-phy@...ts.infradead.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "kernel@...gutronix.de" <kernel@...gutronix.de>,
        dl-linux-imx <linux-imx@....com>
Subject: RE: [PATCH v7 0/7] Add the iMX8MP PCIe support

Sorry, get the wrong v6 patches sending out.
The correct one is re-issued.
Please ignore this one.

Best Regards
Richard Zhu

> -----Original Message-----
> From: Richard Zhu <hongxing.zhu@....com>
> Sent: 2022年9月2日 16:56
> To: p.zabel@...gutronix.de; l.stach@...gutronix.de; bhelgaas@...gle.com;
> lorenzo.pieralisi@....com; robh@...nel.org; shawnguo@...nel.org;
> vkoul@...nel.org; alexander.stein@...tq-group.com; marex@...x.de;
> richard.leitner@...ux.dev
> Cc: linux-phy@...ts.infradead.org; devicetree@...r.kernel.org;
> linux-pci@...r.kernel.org; linux-arm-kernel@...ts.infradead.org;
> linux-kernel@...r.kernel.org; kernel@...gutronix.de; dl-linux-imx
> <linux-imx@....com>
> Subject: [PATCH v7 0/7] Add the iMX8MP PCIe support
> 
> Based on the 6.0-rc1 of the pci/next branch.
> This series adds the i.MX8MP PCIe support and tested on i.MX8MP EVK board
> when one PCIe NVME device is used.
> 
> - i.MX8MP PCIe has reversed initial PERST bit value refer to
> i.MX8MQ/i.MX8MM.
>   Add the PHY PERST explicitly for i.MX8MP PCIe PHY.
> - Add the i.MX8MP PCIe PHY support in the i.MX8M PCIe PHY driver.
>   And share as much as possible codes with i.MX8MM PCIe PHY.
> - Add the i.MX8MP PCIe support in binding document, DTS files, and PCIe
>   driver.
> 
> Main changes v6-->v7:
> - Add "Reviewed-by: Lucas Stach <l.stach@...gutronix.de>" into first three
>   patches.
> - Use "const *char" to replace the static allocation.
> 
> Main changes v5-->v6:
> - To avoid code duplication when find the gpr syscon regmap, add the
>   gpr compatible into the drvdata.
> - Add one missing space before one curly brace in 3/7 of v5 series.
> - 4/7 of v5 had been applied by Phillipp, thanks. For ease of tests, still
>   keep it in v6.
> 
> Main changes v4-->v5:
> - Use Lucas' approach, let blk-ctrl driver do the hsio-mix resets.
> - Fetch the iomuxc-gpr regmap by the different phandles.
> 
> Main changes v3-->v4:
> - Regarding Phillipp's suggestions, add fix tag into the first commit.
> - Add Reviewed and Tested tags.
> 
> Main changes v2-->v3:
> - Fix the schema checking error in the PHY dt-binding patch.
> - Inspired by Lucas, the PLL configurations might not required when
>   external OSC is used as PCIe referrence clock. It's true. Remove all
>   the HSIO PLL bit manipulations, and PCIe works fine on i.MX8MP EVK board
>   with one NVME device is used.
> - Drop the #4 patch of v2, since it had been applied by Rob.
> 
> Main changes v1-->v2:
> - It's my fault forget including Vinod, re-send v2 after include Vinod
>   and linux-phy@...ts.infradead.org.
> - List the basements of this patch-set. The branch, codes changes and so on.
> - Clean up some useless register and bit definitions in #3 patch.
> 
> Documentation/devicetree/bindings/phy/fsl,imx8-pcie-phy.yaml |  16
> ++++++++--
> arch/arm64/boot/dts/freescale/imx8mp-evk.dts                 |  53
> ++++++++++++++++++++++++++++++
> arch/arm64/boot/dts/freescale/imx8mp.dtsi                    |  43
> +++++++++++++++++++++++++
> drivers/pci/controller/dwc/pci-imx6.c                        |  27
> ++++++++++++++--
> drivers/phy/freescale/phy-fsl-imx8m-pcie.c                   | 144
> +++++++++++++++++++++++++++++++++++++++++++++++++++++++-----------
> ----------------
> drivers/reset/reset-imx7.c                                   |   1 +
> drivers/soc/imx/imx8mp-blk-ctrl.c                            |  10
> ++++++
> 7 files changed, 242 insertions(+), 52 deletions(-)
> 
> [PATCH v6 1/7] dt-binding: phy: Add iMX8MP PCIe PHY binding [PATCH v6 2/7]
> arm64: dts: imx8mp: Add iMX8MP PCIe support [PATCH v6 3/7] arm64: dts:
> imx8mp-evk: Add PCIe support [PATCH v6 4/7] reset: imx7: Fix the iMX8MP
> PCIe PHY PERST support [PATCH v6 5/7] soc: imx: imx8mp-blk-ctrl: handle PCIe
> PHY resets [PATCH v6 6/7] phy: freescale: imx8m-pcie: Add i.MX8MP PCIe PHY
> [PATCH v6 7/7] PCI: imx6: Add i.MX8MP PCIe support