Message ID | 20181210090640.4495-1-mjourdan@baylibre.com (mailing list archive) |
---|---|
State | Changes Requested |
Headers | show |
Series | [1/2] dt-bindings: meson: add specific simplefb bindings | expand |
Hi Maxime, On 10/12/2018 10:06, 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@baylibre.com> > --- > .../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" > +- amlogic,pipeline, one of: > + "vpu-cvbs" > + "vpu-hdmi" > + > +Example: > + > +chosen { > + #address-cells = <2>; > + #size-cells = <2>; > + ranges; > + > + framebuffer@0 { > + 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>; > + }; > +}; > You should add a cover-letter for this patchset ! Neil
Hi Neil, On Wed, Dec 12, 2018 at 10:54 AM Neil Armstrong <narmstrong@baylibre.com> wrote: > > Hi Maxime, > > On 10/12/2018 10:06, 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@baylibre.com> > > --- > > .../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" > > +- amlogic,pipeline, one of: > > + "vpu-cvbs" > > + "vpu-hdmi" > > + > > +Example: > > + > > +chosen { > > + #address-cells = <2>; > > + #size-cells = <2>; > > + ranges; > > + > > + framebuffer@0 { > > + 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>; > > + }; > > +}; > > > > You should add a cover-letter for this patchset ! > > Neil Sorry about that, I completely forgot. Thanks for the reminder, will repost. Maxime
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@baylibre.com> > --- > .../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 >
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" +- amlogic,pipeline, one of: + "vpu-cvbs" + "vpu-hdmi" + +Example: + +chosen { + #address-cells = <2>; + #size-cells = <2>; + ranges; + + framebuffer@0 { + 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>; + }; +};
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@baylibre.com> --- .../display/simple-framebuffer-meson.txt | 33 +++++++++++++++++++ 1 file changed, 33 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/simple-framebuffer-meson.txt