linux-fpga.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: firmware: xilinx: Fix versal-fpga node name
@ 2023-12-19 14:31 Michal Simek
  2023-12-20  7:54 ` Krzysztof Kozlowski
  2024-01-22 13:01 ` Michal Simek
  0 siblings, 2 replies; 4+ messages in thread
From: Michal Simek @ 2023-12-19 14:31 UTC (permalink / raw)
  To: linux-kernel, monstr, michal.simek, git
  Cc: Conor Dooley, Jassi Brar, Krzysztof Kozlowski, Mark Brown,
	Moritz Fischer, Naman Trivedi Manojbhai, Rob Herring,
	Shubhrajyoti Datta, Tom Rix, Wu Hao, Xu Yilun, devicetree,
	kishore Manne, linux-arm-kernel, linux-fpga

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@amd.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";
     };
 
-- 
2.36.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: firmware: xilinx: Fix versal-fpga node name
  2023-12-19 14:31 [PATCH] dt-bindings: firmware: xilinx: Fix versal-fpga node name Michal Simek
@ 2023-12-20  7:54 ` Krzysztof Kozlowski
  2023-12-21 13:47   ` Xu Yilun
  2024-01-22 13:01 ` Michal Simek
  1 sibling, 1 reply; 4+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-20  7:54 UTC (permalink / raw)
  To: Michal Simek, linux-kernel, monstr, michal.simek, git
  Cc: Conor Dooley, Jassi Brar, Krzysztof Kozlowski, Mark Brown,
	Moritz Fischer, Naman Trivedi Manojbhai, Rob Herring,
	Shubhrajyoti Datta, Tom Rix, Wu Hao, Xu Yilun, devicetree,
	kishore Manne, linux-arm-kernel, linux-fpga

On 19/12/2023 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@amd.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
>  

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: firmware: xilinx: Fix versal-fpga node name
  2023-12-20  7:54 ` Krzysztof Kozlowski
@ 2023-12-21 13:47   ` Xu Yilun
  0 siblings, 0 replies; 4+ messages in thread
From: Xu Yilun @ 2023-12-21 13:47 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Michal Simek, linux-kernel, monstr, michal.simek, git,
	Conor Dooley, Jassi Brar, Krzysztof Kozlowski, Mark Brown,
	Moritz Fischer, Naman Trivedi Manojbhai, Rob Herring,
	Shubhrajyoti Datta, Tom Rix, Wu Hao, Xu Yilun, devicetree,
	kishore Manne, linux-arm-kernel, linux-fpga

On Wed, Dec 20, 2023 at 08:54:33AM +0100, Krzysztof Kozlowski wrote:
> On 19/12/2023 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@amd.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
> >  
> 
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Acked-by: Xu Yilun <yilun.xu@intel.com>

> 
> Best regards,
> Krzysztof
> 
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: firmware: xilinx: Fix versal-fpga node name
  2023-12-19 14:31 [PATCH] dt-bindings: firmware: xilinx: Fix versal-fpga node name Michal Simek
  2023-12-20  7:54 ` Krzysztof Kozlowski
@ 2024-01-22 13:01 ` Michal Simek
  1 sibling, 0 replies; 4+ messages in thread
From: Michal Simek @ 2024-01-22 13:01 UTC (permalink / raw)
  To: linux-kernel, monstr, michal.simek, git
  Cc: Conor Dooley, Jassi Brar, Krzysztof Kozlowski, Mark Brown,
	Moritz Fischer, Naman Trivedi Manojbhai, Rob Herring,
	Shubhrajyoti Datta, Tom Rix, Wu Hao, Xu Yilun, devicetree,
	kishore Manne, linux-arm-kernel, linux-fpga



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@amd.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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2024-01-22 13:02 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-12-19 14:31 [PATCH] dt-bindings: firmware: xilinx: Fix versal-fpga node name Michal Simek
2023-12-20  7:54 ` Krzysztof Kozlowski
2023-12-21 13:47   ` Xu Yilun
2024-01-22 13:01 ` Michal Simek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).