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

[Michal Simek <michal.simek@xxxxxxx>]

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@xxxxxxx>
> ---
>
>   .../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