On 11/05/2023 02:23, Yoshihiro Shimoda wrote: > Hi Krzysztof, > >> From: Krzysztof Kozlowski, Sent: Wednesday, May 10, 2023 7:03 PM >> >> On Wed, 10 May 2023 15:22:30 +0900, Yoshihiro Shimoda wrote: >>> Document bindings for Renesas R-Car Gen4 and R-Car S4-8 (R8A779F0) >>> PCIe endpoint module. >>> >>> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> >>> Reviewed-by: Rob Herring <robh@xxxxxxxxxx> >>> Reviewed-by: Serge Semin <fancer.lancer@xxxxxxxxx> >>> Acked-by: Manivannan Sadhasivam <mani@xxxxxxxxxx> >>> --- >>> .../bindings/pci/rcar-gen4-pci-ep.yaml | 98 +++++++++++++++++++ >>> 1 file changed, 98 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/pci/rcar-gen4-pci-ep.yaml >>> >> >> BTW, you keep not-ccing me. Since long time. I don't understand why. I >> don't have the emails in inbox, so I won't be responding to your >> patchset. > > I'm sorry. This is my bad. My using script for sending PCI patches didn't describe > your email address and I didn't realize that until now. Today I added your email > address and Conor's email address into my script. > >> >> My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' >> on your patch (DT_CHECKER_FLAGS is new in v5.13): >> >> yamllint warnings/errors: >> >> dtschema/dtc warnings/errors: >> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/pci/rcar-gen4-pci-ep.example.dtb: >> pcie-ep@e65d0000: reg: [[0, 3864854528, 0, 8192], [0, 3864864768, 0, 2048], [0, 3864866816, 0, 8192], [0, 3864875008, >> 0, 4608], [0, 3864879616, 0, 3584], [0, 4261412864, 0, 4194304]] is too long >> From schema: >> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/pci/rcar-gen4-pci-ep.yaml >> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/pci/rcar-gen4-pci-ep.example.dtb: >> pcie-ep@e65d0000: reg-names: ['dbi', 'dbi2', 'atu', 'dma', 'app', 'addr_space'] is too long >> From schema: >> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/pci/rcar-gen4-pci-ep.yaml > > I believe these errors disappear if we apply the patch [16/22] of this patch series. > So, we can ignore these errors. Yes, seems so. Best regards, Krzysztof
