Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Alexander Stein <alexander.stein@ew.tq-group.com>
To:     Hongxing Zhu <hongxing.zhu@nxp.com>
Cc:     "p.zabel@pengutronix.de" <p.zabel@pengutronix.de>,
        "l.stach@pengutronix.de" <l.stach@pengutronix.de>,
        "bhelgaas@google.com" <bhelgaas@google.com>,
        "lorenzo.pieralisi@arm.com" <lorenzo.pieralisi@arm.com>,
        "robh@kernel.org" <robh@kernel.org>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "kernel@pengutronix.de" <kernel@pengutronix.de>,
        dl-linux-imx <linux-imx@nxp.com>
Subject: Re: (EXT) RE: (EXT) [PATCH v1 4/7] dt-bindings: imx6q-pcie: Add iMX8MP PCIe compatible string
Date:   Mon, 07 Mar 2022 09:42:02 +0100
Message-ID: <4386341.LvFx2qVVIh@steina-w>
Organization: TQ-Systems GmbH
In-Reply-To: <AS8PR04MB8676905699E85011E625BFE68C089@AS8PR04MB8676.eurprd04.prod.outlook.com>
References: <1646634556-23779-1-git-send-email-hongxing.zhu@nxp.com> <11939148.O9o76ZdvQC@steina-w> <AS8PR04MB8676905699E85011E625BFE68C089@AS8PR04MB8676.eurprd04.prod.outlook.com>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

Hi Richard,

Am Montag, 7. M=C3=A4rz 2022, 09:19:52 CET schrieb Hongxing Zhu:
> > -----Original Message-----
> > From: Alexander Stein <alexander.stein@ew.tq-group.com>
> > Sent: 2022=E5=B9=B43=E6=9C=887=E6=97=A5 16:03
> > To: Hongxing Zhu <hongxing.zhu@nxp.com>
> > Cc: p.zabel@pengutronix.de; l.stach@pengutronix.de; bhelgaas@google.com;
> > lorenzo.pieralisi@arm.com; robh@kernel.org; shawnguo@kernel.org;
> > linux-arm-kernel@lists.infradead.org; devicetree@vger.kernel.org;
> > linux-pci@vger.kernel.org; linux-arm-kernel@lists.infradead.org;
> > linux-kernel@vger.kernel.org; kernel@pengutronix.de; dl-linux-imx
> > <linux-imx@nxp.com>; Hongxing Zhu <hongxing.zhu@nxp.com>
> > Subject: Re: (EXT) [PATCH v1 4/7] dt-bindings: imx6q-pcie: Add iMX8MP
> > PCIe
> > compatible string
> >=20
> > Hi Richard,
> >=20
> > thanks for providing a patch supporting PCie on iMX8MP.
> >=20
> > Am Montag, 7. M=C3=A4rz 2022, 07:29:13 CET schrieb Richard Zhu:
> >=20
> > > Add i.MX8MP PCIe compatible string.
> > >
> > >
> > >
> > > Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com>
> > > ---
> > >=20
> > >  Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > >
> > >
> > > diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml
> > > b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml index
> > > 36c8a06d17a0..252e5b72aee0 100644
> > > --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml
> > > +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml
> > > @@ -26,6 +26,7 @@ properties:
> > >=20
> > >        - fsl,imx7d-pcie
> > >        - fsl,imx8mq-pcie
> > >        - fsl,imx8mm-pcie
> > >=20
> > > +      - fsl,imx8mp-pcie
> > >
> > >
> > >
> > >    reg:
> > >   =20
> > >      items:
> >=20
> >=20
> > Which branch is this based on? I don't have 'fsl,imx8mm-pcie' entry in =
my
> > tree.
 Another patch 7 also doesn't apply cleanly.
> >=20
>=20
> Hi Alexander:
> Thanks for your quick reply.

Thanks for your fast response. I just wanted to give it a try on our own=20
custom iMX8MP board. I did see the PCIe RC, but not EP (yet), no link up fo=
r=20
now.

> In the past days, I had summit some patches, and wait for ack or reviewed
> tags.
 But they might have some conflictions when do the codes merge.
> I'm waiting for the ack, and will re-base them in a proper sequence later.
>=20
> About the #4 patch apply, it's better to pick the following commit, since=
 it
> had
 Lucas' Reviewed-by tag, and wait for merge.
> https://patchwork.ozlabs.org/project/linux-pci/patch/1646293805-18248-1-g=
it-> send-email-hongxing.zhu@nxp.com/
=20
> About the #7 patch, I do the changes based on the following patch-set.
> https://patchwork.ozlabs.org/project/linux-pci/cover/1645760667-10510-1-g=
it-> send-email-hongxing.zhu@nxp.com/
=20
> I would add the dependency patch-set later in the cover-letter later.
> Sorry to bring in-conveniency to you.

Thanks I'll retry with the dependencies updated, so I hopefully won't miss=
=20
something.

> BTW, If you want to tests PCIe on i.MX8MM EVK board too, the following dts
> changes should be cherry-picked from Shawn's git.
> b4d36c10bf17 arm64: dts: imx8mm-evk: Add the pcie support on imx8mm evk
> board
 aaeba6a8e226 arm64: dts: imx8mm: Add the pcie support
> cfc5078432ca arm64: dts: imx8mm: Add the pcie phy support

Thanks I'll keep the imx8mm support in mind.

Best regards
Alexander