Hi Geert,

On Monday 24 Oct 2016 11:14:11 Geert Uytterhoeven wrote:

On Mon, Oct 24, 2016 at 11:03 AM, Laurent Pinchart wrote:

quoted

--- /dev/null
+++ b/Documentation/devicetree/bindings/media/renesas,fdp1.txt

@@ -0,0 +1,33 @@
+Renesas R-Car Fine Display Processor (FDP1)
+-------------------------------------------
+
+The FDP1 is a de-interlacing module which converts interlaced video to
+progressive video. It is capable of performing pixel format conversion

between +YCbCr/YUV formats and RGB formats. Only YCbCr/YUV formats are
supported as +an input to the module.
+
+ - compatible: Must be the following
+
+   - "renesas,fdp1" for generic compatible
+
+ - reg: the register base and size for the device registers
+ - interrupts : interrupt specifier for the FDP1 instance
+ - clocks: reference to the functional clock
+ - renesas,fcp: reference to the FCPF connected to the FDP1
+
+Optional properties:
+ - power-domains : power-domain property defined with a power domain
specifier

                      "power domain"?

quoted

+                            to respective power domain.

Still, too many power domains in one sentence?

How about

 - power-domains : reference to the power domain that the FDP1 belongs to, if
   any.

-- 
Regards,

Laurent Pinchart

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html