| Message ID | 1723534943-28499-2-git-send-email-hongxing.zhu@nxp.com |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | Add dbi2 and atu for i.MX8M PCIe EP | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | warning | total: 0 errors, 1 warnings, 29 lines checked |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On 13/08/2024 09:42, Richard Zhu wrote: > Add reg-name: "dbi2", "atu" for i.MX8M PCIe Endpoint. > > For i.MX8M PCIe EP, the dbi2 and atu addresses are pre-defined in the > driver. This method is not good. > > In commit b7d67c6130ee ("PCI: imx6: Add iMX95 Endpoint (EP) support"), > Frank suggests to fetch the dbi2 and atu from DT directly. This commit is > preparation to do that for i.MX8M PCIe EP. > > These changes wouldn't break driver function. When "dbi2" and "atu" > properties are present, i.MX PCIe driver would fetch the according base > addresses from DT directly. If only two reg properties are provided, i.MX > PCIe driver would fall back to the old method. > > Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com> > Reviewed-by: Frank Li <Frank.Li@nxp.com> This is some random tagging! You add tags, then drop, then add other people. No, it does not work like this. Where was this tag given? Where are other tags? Why did you drop them? Why this is not explained? <form letter> This is a friendly reminder during the review process. It looks like you received a tag and forgot to add it. If you do not know the process, here is a short explanation: Please add Acked-by/Reviewed-by/Tested-by tags when posting new versions, under or above your Signed-off-by tag. Tag is "received", when provided in a message replied to you on the mailing list. Tools like b4 can help here. However, there's no need to repost patches *only* to add the tags. The upstream maintainer will do that for tags received on the version they apply. https://elixir.bootlin.com/linux/v6.5-rc3/source/Documentation/process/submitting-patches.rst#L577 If a tag was not added on purpose, please state why and what changed. </form letter> Best regards, Krzysztof
On Tue, 13 Aug 2024 15:42:20 +0800, Richard Zhu wrote: > Add reg-name: "dbi2", "atu" for i.MX8M PCIe Endpoint. > > For i.MX8M PCIe EP, the dbi2 and atu addresses are pre-defined in the > driver. This method is not good. > > In commit b7d67c6130ee ("PCI: imx6: Add iMX95 Endpoint (EP) support"), > Frank suggests to fetch the dbi2 and atu from DT directly. This commit is > preparation to do that for i.MX8M PCIe EP. > > These changes wouldn't break driver function. When "dbi2" and "atu" > properties are present, i.MX PCIe driver would fetch the according base > addresses from DT directly. If only two reg properties are provided, i.MX > PCIe driver would fall back to the old method. > > Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com> > Reviewed-by: Frank Li <Frank.Li@nxp.com> > --- > .../devicetree/bindings/pci/fsl,imx6q-pcie-ep.yaml | 13 +++++++++---- > 1 file changed, 9 insertions(+), 4 deletions(-) > Please add Acked-by/Reviewed-by tags when posting new versions. However, there's no need to repost patches *only* to add the tags. The upstream maintainer will do that for acks received on the version they apply. If a tag was not added on purpose, please state why and what changed. Missing tags: Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
> -----Original Message----- > From: Rob Herring (Arm) <robh@kernel.org> > Sent: 2024年8月14日 0:33 > To: Hongxing Zhu <hongxing.zhu@nxp.com> > Cc: shawnguo@kernel.org; linux-pci@vger.kernel.org; > devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; > conor+dt@kernel.org; imx@lists.linux.dev; linux-kernel@vger.kernel.org; > kernel@pengutronix.de; l.stach@pengutronix.de; krzk+dt@kernel.org > Subject: Re: [PATCH v5 1/4] dt-bindings: imx6q-pcie: Add reg-name "dbi2" and > "atu" for i.MX8M PCIe Endpoint > > > On Tue, 13 Aug 2024 15:42:20 +0800, Richard Zhu wrote: > > Add reg-name: "dbi2", "atu" for i.MX8M PCIe Endpoint. > > > > For i.MX8M PCIe EP, the dbi2 and atu addresses are pre-defined in the > > driver. This method is not good. > > > > In commit b7d67c6130ee ("PCI: imx6: Add iMX95 Endpoint (EP) support"), > > Frank suggests to fetch the dbi2 and atu from DT directly. This commit > > is preparation to do that for i.MX8M PCIe EP. > > > > These changes wouldn't break driver function. When "dbi2" and "atu" > > properties are present, i.MX PCIe driver would fetch the according > > base addresses from DT directly. If only two reg properties are > > provided, i.MX PCIe driver would fall back to the old method. > > > > Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com> > > Reviewed-by: Frank Li <Frank.Li@nxp.com> > > --- > > .../devicetree/bindings/pci/fsl,imx6q-pcie-ep.yaml | 13 > > +++++++++---- > > 1 file changed, 9 insertions(+), 4 deletions(-) > > > > > Please add Acked-by/Reviewed-by tags when posting new versions. However, > there's no need to repost patches *only* to add the tags. The upstream > maintainer will do that for acks received on the version they apply. > > If a tag was not added on purpose, please state why and what changed. > > Missing tags: > > Reviewed-by: Rob Herring (Arm) <robh@kernel.org> Hi Rob: Oops, I'm really sorry about that I missing this reviewed-by tag in v5 by my mistake. Thank you very much for your notice and kindly help. Hi Shawn: Can you help to pick-up Rob's reviewed-by tag? Thanks in advanced. Best Regards Richard Zhu > >
On Wed, Aug 14, 2024 at 01:49:30AM +0000, Hongxing Zhu wrote: > > Please add Acked-by/Reviewed-by tags when posting new versions. However, > > there's no need to repost patches *only* to add the tags. The upstream > > maintainer will do that for acks received on the version they apply. > > > > If a tag was not added on purpose, please state why and what changed. > > > > Missing tags: > > > > Reviewed-by: Rob Herring (Arm) <robh@kernel.org> > Hi Rob: > Oops, I'm really sorry about that I missing this reviewed-by tag in v5 by > my mistake. > Thank you very much for your notice and kindly help. > > Hi Shawn: > Can you help to pick-up Rob's reviewed-by tag? > Thanks in advanced. Hmm, this one should go via PCI tree. Shawn
Hi Shawn: Thanks for your help. Hi Krzysztof: Can you help to pick up the #1 patch? Thanks in advanced. Best Regards Richard Zhu > -----Original Message----- > From: Shawn Guo <shawnguo2@yeah.net> > Sent: 2024年8月31日 21:00 > To: Hongxing Zhu <hongxing.zhu@nxp.com> > Cc: Rob Herring (Arm) <robh@kernel.org>; shawnguo@kernel.org; > linux-pci@vger.kernel.org; devicetree@vger.kernel.org; > linux-arm-kernel@lists.infradead.org; conor+dt@kernel.org; > imx@lists.linux.dev; linux-kernel@vger.kernel.org; kernel@pengutronix.de; > l.stach@pengutronix.de; krzk+dt@kernel.org > Subject: Re: [PATCH v5 1/4] dt-bindings: imx6q-pcie: Add reg-name "dbi2" and > "atu" for i.MX8M PCIe Endpoint > > On Wed, Aug 14, 2024 at 01:49:30AM +0000, Hongxing Zhu wrote: > > > Please add Acked-by/Reviewed-by tags when posting new versions. > > > However, there's no need to repost patches *only* to add the tags. > > > The upstream maintainer will do that for acks received on the version they > apply. > > > > > > If a tag was not added on purpose, please state why and what changed. > > > > > > Missing tags: > > > > > > Reviewed-by: Rob Herring (Arm) <robh@kernel.org> > > Hi Rob: > > Oops, I'm really sorry about that I missing this reviewed-by tag in v5 > > by my mistake. > > Thank you very much for your notice and kindly help. > > > > Hi Shawn: > > Can you help to pick-up Rob's reviewed-by tag? > > Thanks in advanced. > > Hmm, this one should go via PCI tree. > > Shawn
On Mon, Sep 02, 2024 at 02:08:05AM +0000, Hongxing Zhu wrote: > Hi Shawn: > Thanks for your help. > > Hi Krzysztof: > Can you help to pick up the #1 patch? > Thanks in advanced. Add Bjorn Helgaas, I am not sure why origial To/Cc list have not bjorn. Bjorn: Could you please take care this patch, which already acked by Rob? We don't want to missed this merge windows for this trivial binding change. Frank > > Best Regards > Richard Zhu > > > -----Original Message----- > > From: Shawn Guo <shawnguo2@yeah.net> > > Sent: 2024年8月31日 21:00 > > To: Hongxing Zhu <hongxing.zhu@nxp.com> > > Cc: Rob Herring (Arm) <robh@kernel.org>; shawnguo@kernel.org; > > linux-pci@vger.kernel.org; devicetree@vger.kernel.org; > > linux-arm-kernel@lists.infradead.org; conor+dt@kernel.org; > > imx@lists.linux.dev; linux-kernel@vger.kernel.org; kernel@pengutronix.de; > > l.stach@pengutronix.de; krzk+dt@kernel.org > > Subject: Re: [PATCH v5 1/4] dt-bindings: imx6q-pcie: Add reg-name "dbi2" and > > "atu" for i.MX8M PCIe Endpoint > > > > On Wed, Aug 14, 2024 at 01:49:30AM +0000, Hongxing Zhu wrote: > > > > Please add Acked-by/Reviewed-by tags when posting new versions. > > > > However, there's no need to repost patches *only* to add the tags. > > > > The upstream maintainer will do that for acks received on the version they > > apply. > > > > > > > > If a tag was not added on purpose, please state why and what changed. > > > > > > > > Missing tags: > > > > > > > > Reviewed-by: Rob Herring (Arm) <robh@kernel.org> > > > Hi Rob: > > > Oops, I'm really sorry about that I missing this reviewed-by tag in v5 > > > by my mistake. > > > Thank you very much for your notice and kindly help. > > > > > > Hi Shawn: > > > Can you help to pick-up Rob's reviewed-by tag? > > > Thanks in advanced. > > > > Hmm, this one should go via PCI tree. > > > > Shawn >
Hello, > Add reg-name: "dbi2", "atu" for i.MX8M PCIe Endpoint. > > For i.MX8M PCIe EP, the dbi2 and atu addresses are pre-defined in the > driver. This method is not good. > > In commit b7d67c6130ee ("PCI: imx6: Add iMX95 Endpoint (EP) support"), > Frank suggests to fetch the dbi2 and atu from DT directly. This commit is > preparation to do that for i.MX8M PCIe EP. > > These changes wouldn't break driver function. When "dbi2" and "atu" > properties are present, i.MX PCIe driver would fetch the according base > addresses from DT directly. If only two reg properties are provided, i.MX > PCIe driver would fall back to the old method. Applied to dt-bindings, thank you! [1/1] dt-bindings: PCI: imx6q-pcie: Add reg-name "dbi2" and "atu" for i.MX8M PCIe Endpoint https://git.kernel.org/pci/pci/c/2f309c988b7c Krzysztof
diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie-ep.yaml b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie-ep.yaml index a06f75df8458..84ca12e8b25b 100644 --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie-ep.yaml +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie-ep.yaml @@ -65,12 +65,14 @@ allOf: then: properties: reg: - minItems: 2 - maxItems: 2 + minItems: 4 + maxItems: 4 reg-names: items: - const: dbi - const: addr_space + - const: dbi2 + - const: atu - if: properties: @@ -129,8 +131,11 @@ examples: pcie_ep: pcie-ep@33800000 { compatible = "fsl,imx8mp-pcie-ep"; - reg = <0x33800000 0x000400000>, <0x18000000 0x08000000>; - reg-names = "dbi", "addr_space"; + reg = <0x33800000 0x100000>, + <0x18000000 0x8000000>, + <0x33900000 0x100000>, + <0x33b00000 0x100000>; + reg-names = "dbi", "addr_space", "dbi2", "atu"; clocks = <&clk IMX8MP_CLK_HSIO_ROOT>, <&clk IMX8MP_CLK_HSIO_AXI>, <&clk IMX8MP_CLK_PCIE_ROOT>;
Add reg-name: "dbi2", "atu" for i.MX8M PCIe Endpoint. For i.MX8M PCIe EP, the dbi2 and atu addresses are pre-defined in the driver. This method is not good. In commit b7d67c6130ee ("PCI: imx6: Add iMX95 Endpoint (EP) support"), Frank suggests to fetch the dbi2 and atu from DT directly. This commit is preparation to do that for i.MX8M PCIe EP. These changes wouldn't break driver function. When "dbi2" and "atu" properties are present, i.MX PCIe driver would fetch the according base addresses from DT directly. If only two reg properties are provided, i.MX PCIe driver would fall back to the old method. Signed-off-by: Richard Zhu <hongxing.zhu@nxp.com> Reviewed-by: Frank Li <Frank.Li@nxp.com> --- .../devicetree/bindings/pci/fsl,imx6q-pcie-ep.yaml | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-)