linux-kernel - Re: [PATCH v2] dt-bindings: watchdog: brcm,kona-wdt: convert txt file to yaml

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: <168044604098.2412538.8062482147729168054.robh@kernel.org>
Date:   Sun, 02 Apr 2023 09:43:30 -0500
From:   Rob Herring <robh@...nel.org>
To:     Nikita Bondarenko <n2h9z4@...il.com>
Cc:     krzysztof.kozlowski+dt@...aro.org, rjui@...adcom.com,
        linux@...ck-us.net, f.fainelli@...il.com, robh+dt@...nel.org,
        bcm-kernel-feedback-list@...adcom.com,
        linux-watchdog@...r.kernel.org, sbranden@...adcom.com,
        wim@...ux-watchdog.org, linux-kernel@...r.kernel.org,
        devicetree@...r.kernel.org
Subject: Re: [PATCH v2] dt-bindings: watchdog: brcm,kona-wdt: convert txt
 file to yaml


On Sun, 02 Apr 2023 15:55:55 +0200, Nikita Bondarenko wrote:
> Remove device tree binding in txt
> Add device tree binding in json-schema

Write complete sentences. The commit msg purpose is not to explain the 
diff, we can read that. The purpose is to explain why though we don't 
need much reasoning why for conversions. What is useful is any fixes or 
changes you made to the binding (only because the existing binding 
didn't match reality). For example, there's a mismatch in the 
compatible. If you aren't adjusting the schema, then it means you think 
the .dts file is wrong and it needs to be fixed.

> 
> 

Single blank line here.

> Signed-off-by: Nikita Bondarenko <n2h9z4@...il.com>
> ---
> 
> Changes in v2, according to review comments:
> - use subject prefixes matching the subsystem in patch subject
> - add commit message
> - update the list of maintainers: add here all maintainers returned by scripts/get_maintainer.p who are marked as "maintainer:BROADCOM BCM281XX/BCM11XXX/BCM216XX"
> - remove quotes in - $ref: watchdog.yaml#
> - remove description for reg
> - leave one example
> - add all SoCs mentioned in txt file description to compatible enum values
> 
> Both checks (dtbs_check, dt_binding_check) return no error:
> 
> make  DT_SCHEMA_FILES=Documentation/devicetree/bindings/watchdog/brcm,kona-wdt.yaml ARCH=arm dtbs_check
>   LINT    Documentation/devicetree/bindings
>   CHKDT   Documentation/devicetree/bindings/processed-schema.json
>   SCHEMA  Documentation/devicetree/bindings/processed-schema.json

Doesn't match what I get below...

> 
> make  DT_SCHEMA_FILES=Documentation/devicetree/bindings/watchdog/brcm,kona-wdt.yaml ARCH=arm dt_binding_check
>   LINT    Documentation/devicetree/bindings
>   CHKDT   Documentation/devicetree/bindings/processed-schema.json
>   SCHEMA  Documentation/devicetree/bindings/processed-schema.json
>   DTEX    Documentation/devicetree/bindings/watchdog/brcm,kona-wdt.example.dts
>   DTC_CHK Documentation/devicetree/bindings/watchdog/brcm,kona-wdt.example.dtb
> 
>  .../bindings/watchdog/brcm,kona-wdt.txt       | 15 -------
>  .../bindings/watchdog/brcm,kona-wdt.yaml      | 45 +++++++++++++++++++
>  2 files changed, 45 insertions(+), 15 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/watchdog/brcm,kona-wdt.txt
>  create mode 100644 Documentation/devicetree/bindings/watchdog/brcm,kona-wdt.yaml
> 

Running 'make dtbs_check' with the schema in this patch gives the
following warnings. Consider if they are expected or the schema is
incorrect. These may not be new warnings.

Note that it is not yet a requirement to have 0 warnings for dtbs_check.
This will change in the future.

Full log is available here: https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20230402135555.62507-1-n2h9z4@gmail.com


watchdog@...02f40: compatible: ['brcm,bcm11351-wdt', 'brcm,kona-wdt'] is too long
	arch/arm/boot/dts/bcm28155-ap.dtb

watchdog@...02f40: Unevaluated properties are not allowed ('compatible' was unexpected)
	arch/arm/boot/dts/bcm28155-ap.dtb