On 17:43-20200914, Kishon Vijay Abraham I wrote: > Hi Nishanth, > > On 14/09/20 5:22 pm, Nishanth Menon wrote: > > On 16:53-20200914, Kishon Vijay Abraham I wrote: > >> Hi Rob, > >> > >> On 02/09/20 1:07 pm, Kishon Vijay Abraham I wrote: > >>> Hi Rob, > >>> > >>> On 02/09/20 10:24 am, Kishon Vijay Abraham I wrote: > >>>> Hi Nishanth, > >>>> > >>>> On 01/09/20 8:22 pm, Nishanth Menon wrote: > >>>>> On 19:36-20200901, Kishon Vijay Abraham I wrote: > >>>>>> Add PCIe device tree node (both RC and EP) for the four > >>>>>> PCIe instances here. > >>>>>> > >>>>>> Signed-off-by: Kishon Vijay Abraham I <kishon@xxxxxx> > >>>>>> --- > >>>>>> arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 218 > >>>>>> ++++++++++++++++++++++ > >>>>>> arch/arm64/boot/dts/ti/k3-j721e.dtsi | 5 +- > >>>>>> 2 files changed, 222 insertions(+), 1 deletion(-) > >>>>> > >>>>> > >>>>> Did you look at the diff of the dtbs_check before and after this > >>>>> series? I see: https://pastebin.ubuntu.com/p/9fyfrTjx9M/ > >>>> > >>>> I didn't see any errors when I checked for individual bindings > >>>> a0393678@a0393678-ssd:~/repos/linux$ mkconfig64 dtbs_check > >>>> DT_SCHEMA_FILES="Documentation/devicetree/bindings/pci/ti,j721e-pci-ep.yaml" > >>>> > >>>> SCHEMA Documentation/devicetree/bindings/processed-schema.yaml > >>>> DTC arch/arm64/boot/dts/ti/k3-am654-base-board.dt.yaml > >>>> DTC arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dt.yaml > >>>> CHECK arch/arm64/boot/dts/ti/k3-am654-base-board.dt.yaml > >>>> CHECK arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dt.yaml > >>>> a0393678@a0393678-ssd:~/repos/linux$ mkconfig64 dtbs_check > >>>> DT_SCHEMA_FILES="Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml" > >>>> > >>>> SCHEMA Documentation/devicetree/bindings/processed-schema.yamlsimple-bus > >>>> DTC arch/arm64/boot/dts/ti/k3-am654-base-board.dt.yaml > >>>> DTC arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dt.yaml > >>>> CHECK arch/arm64/boot/dts/ti/k3-am654-base-board.dt.yaml > >>>> CHECK arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dt.yaml > >>> > >>> Can you give hint on why I get ranges is too long error > >>> https://pastebin.ubuntu.com/p/cPm2tg3dcV/ which I give mkconfig64 > >>> dtbs_check but don't see an error when I include "DT_SCHEMA_FILES"? > >> > >> I debugged this and looks like PCIe "ranges" property conflicts with > >> simple-bus. > >> > >> A diff like below helps to solve the issue > >> diff --git a/schemas/simple-bus.yaml b/schemas/simple-bus.yaml > >> index 248ac9d..ed818ef 100644 > >> --- a/schemas/simple-bus.yaml > >> +++ b/schemas/simple-bus.yaml > >> @@ -39,7 +39,7 @@ patternProperties: > >> oneOf: > >> - items: > >> minItems: 3 > >> - maxItems: 6 > >> + maxItems: 7 > >> minItems: 0 > >> maxItems: 1024 > >> - $ref: "types.yaml#/definitions/flag" > >> > >> If this looks right to you, I can post a patch for it. Without fixing > >> this we would see false errors for PCIe DT nodes. > > > > https://github.com/devicetree-org/dt-schema/commit/3baf308b01786788e3ccb9824fce6d7136b21214 > > > > Part of v2020.08.1? Am i missing something? > > I think I was using a different repo > https://github.com/robherring/dt-schema.git. > > Do you see the following issues you reported earlier with v2020.08.1? > > +/home/nmenon/Src/opensource/linux/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dt.yaml: > bus@100000: pcie@2900000:ranges: [[16777216, 0, 268439552, 0, 268439552, > 0, 65536], [33554432, 0, 268505088, 0, 268505088, 0, 134148096]] is not > valid under any of the given schemas (Possible causes of the failure): > + > /home/nmenon/Src/opensource/linux/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dt.yaml: > bus@100000: pcie@2900000:ranges: True was expected > + > /home/nmenon/Src/opensource/linux/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dt.yaml: > bus@100000: pcie@2900000:ranges:0: [16777216, 0, 268439552, 0, > 268439552, 0, 65536] is too long > + > /home/nmenon/Src/opensource/linux/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dt.yaml: > bus@100000: pcie@2900000:ranges:1: [33554432, 0, 268505088, 0, > 268505088, 0, 134148096] is too long Arrgh.. you are right.. Looks like one of my PCs still had 2020.8 instead of 2020.8.1. We just need to figure out what is going on with syscon.yaml as against ti,j721e-system-controller.yaml.. -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
