On Mon, Dec 10, 2018 at 10:06:39AM +0100, Maxime Jourdan wrote: > Similar to simple-framebuffer-sunxi, we support different display pipelines > that the firmware is free to choose from. > > This documents the "amlogic,simple-framebuffer" compatible and the > "amlogic,pipeline" extension. > > Signed-off-by: Maxime Jourdan <mjourdan@xxxxxxxxxxxx> > --- > .../display/simple-framebuffer-meson.txt | 33 +++++++++++++++++++ > 1 file changed, 33 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/simple-framebuffer-meson.txt > > diff --git a/Documentation/devicetree/bindings/display/simple-framebuffer-meson.txt b/Documentation/devicetree/bindings/display/simple-framebuffer-meson.txt > new file mode 100644 > index 000000000000..122a5c005cd9 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/simple-framebuffer-meson.txt > @@ -0,0 +1,33 @@ > +Meson specific Simple Framebuffer bindings > + > +This binding documents meson specific extensions to the simple-framebuffer > +bindings. The meson simplefb u-boot code relies on the devicetree containing > +pre-populated simplefb nodes. > + > +These extensions are intended so that u-boot can select the right node based > +on which pipeline is being used. As such they are solely intended for > +firmware / bootloader use, and the OS should ignore them. > + > +Required properties: > +- compatible: "amlogic,simple-framebuffer" And a fallback? > +- amlogic,pipeline, one of: > + "vpu-cvbs" > + "vpu-hdmi" > + > +Example: > + > +chosen { > + #address-cells = <2>; > + #size-cells = <2>; > + ranges; > + > + framebuffer@0 { Without 'reg', you shouldn't have a unit-address. > + compatible = "amlogic,simple-framebuffer", > + "simple-framebuffer"; > + amlogic,pipeline = "vpu-hdmi"; > + clocks = <&clkc CLKID_HDMI_PCLK>, > + <&clkc CLKID_CLK81>, > + <&clkc CLKID_GCLK_VENCI_INT0>; > + power-domains = <&pwrc_vpu>; > + }; > +}; > -- > 2.19.2 >