linux-kernel - Re: [PATCH 1/1] dt-bindings: mfd: convert mxs-lradc.txt to yaml format

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <174803207693.2936926.13970067014189882058.robh@kernel.org>
Date: Fri, 23 May 2025 15:27:56 -0500
From: "Rob Herring (Arm)" <robh@...nel.org>
To: Frank Li <Frank.Li@....com>
Cc: Fabio Estevam <festevam@...il.com>, 
 Krzysztof Kozlowski <krzk+dt@...nel.org>, 
 Sascha Hauer <s.hauer@...gutronix.de>, Conor Dooley <conor+dt@...nel.org>, 
 Lee Jones <lee@...nel.org>, devicetree@...r.kernel.org, imx@...ts.linux.dev, 
 Pengutronix Kernel Team <kernel@...gutronix.de>, 
 Shawn Guo <shawnguo@...nel.org>, linux-arm-kernel@...ts.infradead.org, 
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/1] dt-bindings: mfd: convert mxs-lradc.txt to yaml
 format


On Fri, 23 May 2025 15:18:43 -0400, Frank Li wrote:
> Convert mxs-lradc.txt to yaml format.
> 
> Additional changes:
> - Only keep one example.
> - Change node name to adc.
> - Add clocks and #io-channel-cells.
> 
> Signed-off-by: Frank Li <Frank.Li@....com>
> ---
>  .../bindings/mfd/fsl,imx28-lradc.yaml         | 85 +++++++++++++++++++
>  .../devicetree/bindings/mfd/mxs-lradc.txt     | 45 ----------
>  2 files changed, 85 insertions(+), 45 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/mfd/fsl,imx28-lradc.yaml
>  delete mode 100644 Documentation/devicetree/bindings/mfd/mxs-lradc.txt
> 

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

yamllint warnings/errors:

dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/mfd/fsl,imx28-lradc.example.dtb: fake-interrupt-controller: #interrupt-cells: 9 is greater than the maximum of 8
	from schema $id: http://devicetree.org/schemas/dt-core.yaml#

doc reference errors (make refcheckdocs):

See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20250523191844.563177-1-Frank.Li@nxp.com

The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.

If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:

pip3 install dtschema --upgrade

Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.