Re: [PATCH v8 1/2] dt-bindings: edac: al-mc-edac: Amazon's Annapurna Labs Memory Controller EDAC

[Shenhar, Talel]

On 7/27/2020 9:14 PM, Rob Herring wrote:
> On Sun, 26 Jul 2020 22:15:54 +0300, Talel Shenhar wrote:
> > Document Amazon's Annapurna Labs Memory Controller EDAC SoC binding.
> >
> > Signed-off-by: Talel Shenhar <talel@xxxxxxxxxx>
> > Reviewed-by: Rob Herring <robh@xxxxxxxxxx>
> > ---
> >   .../bindings/edac/amazon,al-mc-edac.yaml      | 52 +++++++++++++++++++
> >   1 file changed, 52 insertions(+)
> >   create mode 100644 Documentation/devicetree/bindings/edac/amazon,al-mc-edac.yaml
>
> My bot found errors running 'make dt_binding_check' on your patch:
>
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/edac/amazon,al-mc-edac.example.dt.yaml: example-0: edac@f0080000:reg:0: [0, 4027056128, 0, 65536] is too long
>
>
> See https://patchwork.ozlabs.org/patch/1336532
>
> If you already ran 'make dt_binding_check' and didn't see the above
> error(s), then make sure dt-schema is up to date:
>
> pip3 install git+https://github.com/devicetree-org/dt-schema.git@master --upgrade
>
> Please check and re-submit.

Thanks Rob,

Indeed I am not able to see this error.

Even tried updating dt-schema.

Even targeted check didn't give anything:

linux-ras$ make dt_binding_check 
DT_SCHEMA_FILES=Documentation/devicetree/bindings/edac/amazon,al-mc-edac.yaml
 CHKDT Documentation/devicetree/bindings/edac/amazon,al-mc-edac.yaml
 DTC 
Documentation/devicetree/bindings/edac/amazon,al-mc-edac.example.dt.yaml
 CHECK 
Documentation/devicetree/bindings/edac/amazon,al-mc-edac.example.dt.yaml

ill look into it and publish v9.

>