On Tue, Feb 28, 2017 at 4:07 AM, Yannick FERTRE <yannick.fertre@xxxxxx> wrote: > On 02/24/2017 03:06 PM, Rob Herring wrote: >> On Fri, Feb 24, 2017 at 2:18 AM, Yannick FERTRE <yannick.fertre@xxxxxx> wrote: >>> On 02/21/2017 03:07 PM, Rob Herring wrote: >>>> On Mon, Feb 20, 2017 at 5:01 AM, Yannick FERTRE <yannick.fertre@xxxxxx> wrote: >>>>> On 02/16/2017 03:34 AM, Rob Herring wrote: >>>>>> On Fri, Feb 10, 2017 at 03:54:44PM +0100, Yannick Fertre wrote: >>>>>>> Signed-off-by: Yannick Fertre <yannick.fertre@xxxxxx> >>>>>>> --- >>>>>>> .../bindings/display/panel/ampire,am-480272h3tmqw-t01h.txt | 7 +++++++ >>>>>>> 1 file changed, 7 insertions(+) >>>>>>> create mode 100644 Documentation/devicetree/bindings/display/panel/ampire,am-480272h3tmqw-t01h.txt [...] >>>>> This binding is similar to all panel bindings which are link to panel >>>>> simple. Details on gpio & power supply are describe with simple-panel >>>>> binding. >>>> >>>> simple-panel.txt provides what possible standard properties there are, >>>> not whether a panel uses them. The power supply needs to be explicit >>>> because I don't know if you forgot and there's really 2 supplies or >>>> you are using "power-supply" for a single supply. >>>> >>>> Rob >>>> > > > Hi Rob, > ok, I'll push a V4 version with all details on properties. > I write bellow a draft of binding: > > Ampire AM-480272H3TMQW-T01H 4.3" WQVGA TFT LCD panel > > This binding is compatible with the simple-panel binding, which is specified > in simple-panel.txt in this directory. > > Required properties: > - compatible: should be "ampire,am-480272h3tmqw-t01h" > > Optional properties: > - power-supply: regulator to provide the supply voltage > - ddc-i2c-bus: phandle of an I2C controller used for DDC EDID probing Doesn't look like there's any DDC connection to this panel. > - enable-gpios: GPIO pin to enable or disable the panel Need to state this is active high. > - backlight: phandle of the backlight device attached to the panel > > Optional nodes: > - Video port for RGB input. > > Example: > panel_rgb: panel-rgb { > compatible = "ampire,am-480272h3tmqw-t01h"; > enable-gpios = <&gpioa 8 1>; > port { > panel_in_rgb: endpoint { > remote-endpoint = <&controler_out_rgb>; > }; > }; > }; > > Best regards _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel