Re: [PATCH v9 01/20] dt-bindings: PCI: Add PLDA XpressRICH PCIe host common properties

Conor Dooley
2023-10-25T10:28:32Z

[-- Attachment #1.1: Type: text/plain, Size: 1547 bytes --]

Hi John,

On Mon, Oct 23, 2023 at 11:09:50PM -0400, John Clark wrote:
> > On 2023/10/20 19:04, Krzysztof Kozlowski wrote:
> > > On 20/10/2023 12:43, Minda Chen wrote:
> > > > Add PLDA XpressRICH PCIe host common properties dt-binding doc.
> > > > Microchip PolarFire PCIe host using PLDA IP.
> > > > Move common properties from Microchip PolarFire PCIe host
> > > > to PLDA files.
> > > >
> > > > Signed-off-by: Minda Chen <minda.chen@starfivetech.com>
> > > > Reviewed-by: Hal Feng <hal.feng@starfivetech.com>
> > > > Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
> > > > Reviewed-by: Rob Herring <robh@kernel.org>
> > > > ---
> > > > .../bindings/pci/microchip,pcie-host.yaml | 55 +-------------
> > > > .../pci/plda,xpressrich3-axi-common.yaml | 75 +++++++++++++++++++
> > > Where was this patch reviewed?
> > >
> > > Best regards,
> > > Krzysztof
> > >
> > This Conor's review tag. v2 : https://patchwork.kernel.org/project/linux-pci/patch/20230727103949.26149-2-minda.chen@starfivetech.com/
> > This is Rob's review tag v3: https://patchwork.kernel.org/project/linux-pci/patch/20230814082016.104181-2-minda.chen@starfivetech.com/
>
> Tested-by: John Clark <inindev@gmail.com>:
> https://github.com/inindev/visionfive2/tree/main/kernel/patches

I suspect you don't mean that you tested this individual dt-binding, but
rather that you tested the whole series. If so, you should probably
provide this tested-by against the cover-letter instead of this bindings
patch.

Cheers,
Conor.

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 161 bytes --]