diff mbox series

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

Message ID 20250217095226.12606-1-shubhrajyoti.datta@amd.com (mailing list archive)
State Handled Elsewhere, archived
Headers show
Series dt-bindings: power: reset: xilinx: Make "interrupts" property optional | expand

Commit Message

Shubhrajyoti Datta Feb. 17, 2025, 9:52 a.m. UTC
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(-)

Comments

Michal Simek Feb. 17, 2025, 11:09 a.m. UTC | #1
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
Rob Herring (Arm) Feb. 19, 2025, 11:15 p.m. UTC | #2
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>
diff mbox series

Patch

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