Re: [PATCH v2 3/8] dt-bindings: media: renesas, fcp: Add resets and iommus properties

2020-07-13 Thread Rob Herring
On Sun, 21 Jun 2020 03:47:29 +0300, Laurent Pinchart wrote:
> The resets and iommus properties are used in DT sources in the kernel
> tree. Document them, and make resets mandatory. The iommus property is
> optional as not all platforms wire the FCP to a functional IOMMU.
> 
> Signed-off-by: Laurent Pinchart 
> Reviewed-by: Geert Uytterhoeven 
> ---
>  Documentation/devicetree/bindings/media/renesas,fcp.yaml | 9 +
>  1 file changed, 9 insertions(+)
> 

Acked-by: Rob Herring 
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH v2 3/8] dt-bindings: media: renesas,fcp: Add resets and iommus properties

2020-07-01 Thread Niklas Söderlund
Hi Laurent,

Thanks for your patch.

On 2020-06-21 03:47:29 +0300, Laurent Pinchart wrote:
> The resets and iommus properties are used in DT sources in the kernel
> tree. Document them, and make resets mandatory. The iommus property is
> optional as not all platforms wire the FCP to a functional IOMMU.
> 
> Signed-off-by: Laurent Pinchart 
> Reviewed-by: Geert Uytterhoeven 

Reviewed-by: Niklas Söderlund 

> ---
>  Documentation/devicetree/bindings/media/renesas,fcp.yaml | 9 +
>  1 file changed, 9 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/media/renesas,fcp.yaml 
> b/Documentation/devicetree/bindings/media/renesas,fcp.yaml
> index eeca7e255420..b6cf2958e6c9 100644
> --- a/Documentation/devicetree/bindings/media/renesas,fcp.yaml
> +++ b/Documentation/devicetree/bindings/media/renesas,fcp.yaml
> @@ -31,14 +31,21 @@ properties:
>clocks:
>  maxItems: 1
>  
> +  iommus:
> +maxItems: 1
> +
>power-domains:
>  maxItems: 1
>  
> +  resets:
> +maxItems: 1
> +
>  required:
>- compatible
>- reg
>- clocks
>- power-domains
> +  - resets
>  
>  additionalProperties: false
>  
> @@ -53,5 +60,7 @@ examples:
>  reg = <0xfea2f000 0x200>;
>  clocks = <&cpg CPG_MOD 602>;
>  power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> +resets = <&cpg 602>;
> +iommus = <&ipmmu_vi0 9>;
>  };
>  ...
> -- 
> Regards,
> 
> Laurent Pinchart
> 

-- 
Regards,
Niklas Söderlund
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[PATCH v2 3/8] dt-bindings: media: renesas, fcp: Add resets and iommus properties

2020-06-20 Thread Laurent Pinchart
The resets and iommus properties are used in DT sources in the kernel
tree. Document them, and make resets mandatory. The iommus property is
optional as not all platforms wire the FCP to a functional IOMMU.

Signed-off-by: Laurent Pinchart 
Reviewed-by: Geert Uytterhoeven 
---
 Documentation/devicetree/bindings/media/renesas,fcp.yaml | 9 +
 1 file changed, 9 insertions(+)

diff --git a/Documentation/devicetree/bindings/media/renesas,fcp.yaml 
b/Documentation/devicetree/bindings/media/renesas,fcp.yaml
index eeca7e255420..b6cf2958e6c9 100644
--- a/Documentation/devicetree/bindings/media/renesas,fcp.yaml
+++ b/Documentation/devicetree/bindings/media/renesas,fcp.yaml
@@ -31,14 +31,21 @@ properties:
   clocks:
 maxItems: 1
 
+  iommus:
+maxItems: 1
+
   power-domains:
 maxItems: 1
 
+  resets:
+maxItems: 1
+
 required:
   - compatible
   - reg
   - clocks
   - power-domains
+  - resets
 
 additionalProperties: false
 
@@ -53,5 +60,7 @@ examples:
 reg = <0xfea2f000 0x200>;
 clocks = <&cpg CPG_MOD 602>;
 power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
+resets = <&cpg 602>;
+iommus = <&ipmmu_vi0 9>;
 };
 ...
-- 
Regards,

Laurent Pinchart

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel