linux-kernel - Re: [PATCH V3] dt-bindings: mailbox: ti, secure-proxy: Convert to json schema

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: <20210416191254.GA3779553@robh.at.kernel.org>
Date:   Fri, 16 Apr 2021 14:12:54 -0500
From:   Rob Herring <robh@...nel.org>
To:     Nishanth Menon <nm@...com>
Cc:     devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        s-anna@...com, linux-kernel@...r.kernel.org,
        Jassi Brar <jassisinghbrar@...il.com>
Subject: Re: [PATCH V3] dt-bindings: mailbox: ti, secure-proxy: Convert to
 json schema

On Thu, 15 Apr 2021 19:59:53 -0500, Nishanth Menon wrote:
> Convert the ti,secure-proxy to yaml for better checks and documentation.
> Differences being mostly in the examples:
> - Dropped the example usage of mailbox client, it is better done in
>   tisci node definition.
> - Switched reg usage for address-cells and size-cells 1 - aligned with
>   schema checks as well.
> - included header in example for a buildable example.
> 
> While at this, lets make sure to support upto 100 rx threads even though
> typically upto 1 threads is practically in use.
> 
> NOTE: The following checkpatch warning is generated since we do include the header
> in the example, but this is a false positive warning.
>   WARNING: DT binding docs and includes should be a separate patch. See:
>   Documentation/devicetree/bindings/submitting-patches.rst
> 
> Signed-off-by: Nishanth Menon <nm@...com>
> ---
> 
> Changes since v2:
>  - Subject line rewording to indicate json schema rather than yaml
>  - Review comment: Dropped "-" in interrupt-names pattern so that it will match all
>    entries
>  - Minor commit message formatting changes
> 
> V2: https://lore.kernel.org/linux-arm-kernel/20210413224535.30910-1-nm@ti.com/
> V1: https://lore.kernel.org/linux-arm-kernel/20210413171230.5872-1-nm@ti.com/
> 
>  .../bindings/mailbox/ti,secure-proxy.txt      | 50 ------------
>  .../bindings/mailbox/ti,secure-proxy.yaml     | 79 +++++++++++++++++++
>  2 files changed, 79 insertions(+), 50 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/mailbox/ti,secure-proxy.txt
>  create mode 100644 Documentation/devicetree/bindings/mailbox/ti,secure-proxy.yaml
> 

Reviewed-by: Rob Herring <robh@...nel.org>