linux-kernel - Re: [PATCH] dt-bindings: firmware: xilinx: Fix versal-fpga node name

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: <4a5c810a-ad59-44b2-8926-4d34204649d3@amd.com>
Date: Mon, 22 Jan 2024 14:01:56 +0100
From: Michal Simek <michal.simek@....com>
To: linux-kernel@...r.kernel.org, monstr@...str.eu, michal.simek@...inx.com,
 git@...inx.com
Cc: Conor Dooley <conor+dt@...nel.org>, Jassi Brar
 <jassisinghbrar@...il.com>,
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
 Mark Brown <broonie@...nel.org>, Moritz Fischer <mdf@...nel.org>,
 Naman Trivedi Manojbhai <naman.trivedimanojbhai@....com>,
 Rob Herring <robh+dt@...nel.org>,
 Shubhrajyoti Datta <shubhrajyoti.datta@....com>, Tom Rix <trix@...hat.com>,
 Wu Hao <hao.wu@...el.com>, Xu Yilun <yilun.xu@...el.com>,
 devicetree@...r.kernel.org, kishore Manne <nava.kishore.manne@....com>,
 linux-arm-kernel@...ts.infradead.org, linux-fpga@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: firmware: xilinx: Fix versal-fpga node name



On 12/19/23 15:31, Michal Simek wrote:
> Based on commit 83a368a3fc8a ("docs: dt-bindings: add DTS Coding Style
> document") using underscore ('_') in node name is not recommended that's
> why switch to dash ('-').
> 
> Signed-off-by: Michal Simek <michal.simek@....com>
> ---
> 
>   .../bindings/firmware/xilinx/xlnx,zynqmp-firmware.yaml        | 4 ++--
>   Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml  | 2 +-
>   2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/firmware/xilinx/xlnx,zynqmp-firmware.yaml b/Documentation/devicetree/bindings/firmware/xilinx/xlnx,zynqmp-firmware.yaml
> index 8e584857ddd4..3d578f98ae2c 100644
> --- a/Documentation/devicetree/bindings/firmware/xilinx/xlnx,zynqmp-firmware.yaml
> +++ b/Documentation/devicetree/bindings/firmware/xilinx/xlnx,zynqmp-firmware.yaml
> @@ -41,7 +41,7 @@ properties:
>     "#power-domain-cells":
>       const: 1
>   
> -  versal_fpga:
> +  versal-fpga:
>       $ref: /schemas/fpga/xlnx,versal-fpga.yaml#
>       description: Compatible of the FPGA device.
>       type: object
> @@ -84,7 +84,7 @@ examples:
>         compatible = "xlnx,versal-firmware";
>         method = "smc";
>   
> -      versal_fpga: versal_fpga {
> +      versal_fpga: versal-fpga {
>           compatible = "xlnx,versal-fpga";
>         };
>   
> diff --git a/Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml b/Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml
> index 26f18834caa3..80833462f620 100644
> --- a/Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml
> +++ b/Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml
> @@ -26,7 +26,7 @@ additionalProperties: false
>   
>   examples:
>     - |
> -    versal_fpga: versal_fpga {
> +    versal_fpga: versal-fpga {
>            compatible = "xlnx,versal-fpga";
>       };
>   

Applied.
M