[PATCH] dt-bindings: power: reset: xilinx: Make "interrupts" property optional

Shubhrajyoti Datta posted 1 patch 10 months ago
.../devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml       | 1 -
1 file changed, 1 deletion(-)
[PATCH] dt-bindings: power: reset: xilinx: Make "interrupts" property optional
Posted by Shubhrajyoti Datta 10 months ago
The "interrupts" property in the ZynqMP power/reset binding was previously
marked as required. However, there are multiple mechanisms for
handling power/reset events, including:
-Event management registration,
-Mailbox (mboxes),
-Interrupts (interrupts).

When event management support is available (default on Versal SoC), the
"interrupts" property is not used hence not required.

Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>
---

 .../devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml       | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml b/Documentation/devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml
index 799831636194..079ad977b907 100644
--- a/Documentation/devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml
+++ b/Documentation/devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml
@@ -46,7 +46,6 @@ properties:
 
 required:
   - compatible
-  - interrupts
 
 additionalProperties: false
 
-- 
2.17.1
Re: [PATCH] dt-bindings: power: reset: xilinx: Make "interrupts" property optional
Posted by Michal Simek 10 months ago

On 2/17/25 10:52, Shubhrajyoti Datta wrote:
> The "interrupts" property in the ZynqMP power/reset binding was previously
> marked as required. However, there are multiple mechanisms for
> handling power/reset events, including:
> -Event management registration,
> -Mailbox (mboxes),
> -Interrupts (interrupts).
> 
> When event management support is available (default on Versal SoC), the
> "interrupts" property is not used hence not required.
> 
> Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>
> ---
> 
>   .../devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml       | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml b/Documentation/devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml
> index 799831636194..079ad977b907 100644
> --- a/Documentation/devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml
> +++ b/Documentation/devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml
> @@ -46,7 +46,6 @@ properties:
>   
>   required:
>     - compatible
> -  - interrupts
>   
>   additionalProperties: false
>   

Applied.
M
Re: [PATCH] dt-bindings: power: reset: xilinx: Make "interrupts" property optional
Posted by Rob Herring (Arm) 10 months ago
On Mon, 17 Feb 2025 15:22:26 +0530, Shubhrajyoti Datta wrote:
> The "interrupts" property in the ZynqMP power/reset binding was previously
> marked as required. However, there are multiple mechanisms for
> handling power/reset events, including:
> -Event management registration,
> -Mailbox (mboxes),
> -Interrupts (interrupts).
> 
> When event management support is available (default on Versal SoC), the
> "interrupts" property is not used hence not required.
> 
> Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>
> ---
> 
>  .../devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml       | 1 -
>  1 file changed, 1 deletion(-)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>
Re: [PATCH] dt-bindings: power: reset: xilinx: Make "interrupts" property optional
Posted by Michal Simek 10 months ago

On 2/17/25 10:52, Shubhrajyoti Datta wrote:
> The "interrupts" property in the ZynqMP power/reset binding was previously
> marked as required. However, there are multiple mechanisms for
> handling power/reset events, including:
> -Event management registration,
> -Mailbox (mboxes),
> -Interrupts (interrupts).
> 
> When event management support is available (default on Versal SoC), the
> "interrupts" property is not used hence not required.
> 
> Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>
> ---
> 
>   .../devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml       | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml b/Documentation/devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml
> index 799831636194..079ad977b907 100644
> --- a/Documentation/devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml
> +++ b/Documentation/devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml
> @@ -46,7 +46,6 @@ properties:
>   
>   required:
>     - compatible
> -  - interrupts
>   
>   additionalProperties: false
>   

Acked-by: Michal Simek <michal.simek@amd.com>

Thanks,
Michal