[v6,1/2] dt-bindings: optee driver interrupt can be a per-cpu interrupt

Message ID 20230322132212.3646972-1-etienne.carriere@linaro.org
State New
Headers
Series [v6,1/2] dt-bindings: optee driver interrupt can be a per-cpu interrupt |

Commit Message

Etienne Carriere March 22, 2023, 1:22 p.m. UTC
  Explicit in optee firmware device tree bindings that the interrupt
used by optee driver for async notification can be a peripheral
interrupt or a per-cpu interrupt.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
---
No change since v5

No change since v4

Changes since v3:
- Patch added in this v4 to address review comments.
---
 .../devicetree/bindings/arm/firmware/linaro,optee-tz.yaml      | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
  

Comments

Sumit Garg March 22, 2023, 2:56 p.m. UTC | #1
On Wed, 22 Mar 2023 at 18:52, Etienne Carriere
<etienne.carriere@linaro.org> wrote:
>
> Explicit in optee firmware device tree bindings that the interrupt
> used by optee driver for async notification can be a peripheral
> interrupt or a per-cpu interrupt.
>
> Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
> ---
> No change since v5
>

Reviewed-by: Sumit Garg <sumit.garg@linaro.org>

-Sumit

> No change since v4
>
> Changes since v3:
> - Patch added in this v4 to address review comments.
> ---
>  .../devicetree/bindings/arm/firmware/linaro,optee-tz.yaml      | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/firmware/linaro,optee-tz.yaml b/Documentation/devicetree/bindings/arm/firmware/linaro,optee-tz.yaml
> index d4dc0749f9fd..5d033570b57b 100644
> --- a/Documentation/devicetree/bindings/arm/firmware/linaro,optee-tz.yaml
> +++ b/Documentation/devicetree/bindings/arm/firmware/linaro,optee-tz.yaml
> @@ -28,7 +28,8 @@ properties:
>      maxItems: 1
>      description: |
>        This interrupt which is used to signal an event by the secure world
> -      software is expected to be edge-triggered.
> +      software is expected to be either a per-cpu interrupt or an
> +      edge-triggered peripheral interrupt.
>
>    method:
>      enum: [smc, hvc]
> --
> 2.25.1
>
  
Jens Wiklander March 30, 2023, 9:23 a.m. UTC | #2
On Wed, Mar 22, 2023 at 08:26:22PM +0530, Sumit Garg wrote:
> On Wed, 22 Mar 2023 at 18:52, Etienne Carriere
> <etienne.carriere@linaro.org> wrote:
> >
> > Explicit in optee firmware device tree bindings that the interrupt
> > used by optee driver for async notification can be a peripheral
> > interrupt or a per-cpu interrupt.
> >
> > Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
> > ---
> > No change since v5
> >
> 
> Reviewed-by: Sumit Garg <sumit.garg@linaro.org>

I'm picking up this.

Thanks,
Jens

> 
> -Sumit
> 
> > No change since v4
> >
> > Changes since v3:
> > - Patch added in this v4 to address review comments.
> > ---
> >  .../devicetree/bindings/arm/firmware/linaro,optee-tz.yaml      | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/firmware/linaro,optee-tz.yaml b/Documentation/devicetree/bindings/arm/firmware/linaro,optee-tz.yaml
> > index d4dc0749f9fd..5d033570b57b 100644
> > --- a/Documentation/devicetree/bindings/arm/firmware/linaro,optee-tz.yaml
> > +++ b/Documentation/devicetree/bindings/arm/firmware/linaro,optee-tz.yaml
> > @@ -28,7 +28,8 @@ properties:
> >      maxItems: 1
> >      description: |
> >        This interrupt which is used to signal an event by the secure world
> > -      software is expected to be edge-triggered.
> > +      software is expected to be either a per-cpu interrupt or an
> > +      edge-triggered peripheral interrupt.
> >
> >    method:
> >      enum: [smc, hvc]
> > --
> > 2.25.1
> >
  

Patch

diff --git a/Documentation/devicetree/bindings/arm/firmware/linaro,optee-tz.yaml b/Documentation/devicetree/bindings/arm/firmware/linaro,optee-tz.yaml
index d4dc0749f9fd..5d033570b57b 100644
--- a/Documentation/devicetree/bindings/arm/firmware/linaro,optee-tz.yaml
+++ b/Documentation/devicetree/bindings/arm/firmware/linaro,optee-tz.yaml
@@ -28,7 +28,8 @@  properties:
     maxItems: 1
     description: |
       This interrupt which is used to signal an event by the secure world
-      software is expected to be edge-triggered.
+      software is expected to be either a per-cpu interrupt or an
+      edge-triggered peripheral interrupt.
 
   method:
     enum: [smc, hvc]