| Message ID | 1477299818-31935-3-git-send-email-laurent.pinchart+renesas@ideasonboard.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Mon, Oct 24, 2016 at 11:03 AM, Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> wrote: > --- /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"? > + to respective power domain. Still, too many power domains in one sentence? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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: > > --- /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"? > > > + 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.
Hi Laurent, On Mon, Oct 24, 2016 at 11:46 AM, Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote: > On Monday 24 Oct 2016 11:14:11 Geert Uytterhoeven wrote: >> On Mon, Oct 24, 2016 at 11:03 AM, Laurent Pinchart wrote: >> > --- /dev/null >> > +++ b/Documentation/devicetree/bindings/media/renesas,fdp1.txt >> > +Optional properties: >> > + - power-domains : power-domain property defined with a power domain >> > specifier >> "power domain"? >> >> > + 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. Much better. Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/media/renesas,fdp1.txt b/Documentation/devicetree/bindings/media/renesas,fdp1.txt new file mode 100644 index 000000000000..e6abd2a17e66 --- /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 + to respective power domain. + + +Device node example +------------------- + + fdp1@fe940000 { + compatible = "renesas,fdp1"; + reg = <0 0xfe940000 0 0x2400>; + interrupts = <GIC_SPI 262 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&cpg CPG_MOD 119>; + power-domains = <&sysc R8A7795_PD_A3VP>; + renesas,fcp = <&fcpf0>; + };