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

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 laurent.pinchart+renesas@ideasonboard.com Reviewed-by: Geert Uytterhoeven geert+renesas@glider.be

Reviewed-by: Niklas Söderlund niklas.soderlund+renesas@ragnatech.se

---

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