Re: [PATCHv2 2/3] dt-bindings: documentation: add clock bindings information for Agilex

From: Rob Herring
Date: Tue Mar 10 2020 - 14:36:21 EST


On Mon, 9 Mar 2020 12:16:52 -0500, Dinh Nguyen wrote:
> Document the Agilex clock bindings, and add the clock header file. The
> clock header is an enumeration of all the different clocks on the Agilex
> platform.
>
> Signed-off-by: Dinh Nguyen <dinguyen@xxxxxxxxxx>
> ---
> v2: convert original document to YAML
> ---
> .../bindings/clock/intc,agilex.yaml | 79 +++++++++++++++++++
> include/dt-bindings/clock/agilex-clock.h | 70 ++++++++++++++++
> 2 files changed, 149 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/clock/intc,agilex.yaml
> create mode 100644 include/dt-bindings/clock/agilex-clock.h
>

My bot found errors running 'make dt_binding_check' on your patch:

Documentation/devicetree/bindings/clock/intc,agilex.yaml: while scanning a block scalar
in "<unicode string>", line 36, column 5
found a tab character where an indentation space is expected
in "<unicode string>", line 37, column 1
Documentation/devicetree/bindings/Makefile:12: recipe for target 'Documentation/devicetree/bindings/clock/intc,agilex.example.dts' failed
make[1]: *** [Documentation/devicetree/bindings/clock/intc,agilex.example.dts] Error 1
make[1]: *** Waiting for unfinished jobs....
warning: no schema found in file: Documentation/devicetree/bindings/clock/intc,agilex.yaml
/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/clock/intc,agilex.yaml: ignoring, error parsing file
Makefile:1262: recipe for target 'dt_binding_check' failed
make: *** [dt_binding_check] Error 2

See https://patchwork.ozlabs.org/patch/1251669
Please check and re-submit.