linux-kernel - Re: [RESEND] [PATCH v2 2/4] dt-bindings: Add jdi lt070me05000 panel bindings

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20160824114534.GA3169@ulmo.ba.sec>
Date:   Wed, 24 Aug 2016 13:45:34 +0200
From:   Thierry Reding <thierry.reding@...il.com>
To:     Vinay Simha BN <simhavcs@...il.com>
Cc:     Archit Taneja <archit.taneja@...il.com>,
        John Stultz <john.stultz@...aro.org>,
        Sumit Semwal <sumit.semwal@...aro.org>,
        David Airlie <airlied@...ux.ie>,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        "open list:DRM PANEL DRIVERS" <dri-devel@...ts.freedesktop.org>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@...r.kernel.org>,
        open list <linux-kernel@...r.kernel.org>
Subject: Re: [RESEND] [PATCH v2 2/4] dt-bindings: Add jdi lt070me05000 panel
 bindings

On Sun, Jul 31, 2016 at 08:45:21PM +0530, Vinay Simha BN wrote:
> Add documentation for lt070me05000 panel
> 
> Cc: Archit Taneja <archit.taneja@...il.com>
> Cc: John Stultz <john.stultz@...aro.org>
> Cc: Thierry Reding <thierry.reding@...il.com>
> Cc: Sumit Semwal <sumit.semwal@...aro.org>
> Signed-off-by: Vinay Simha BN <simhavcs@...il.com>
> Acked-by: Rob Herring <robh@...nel.org>
> 
> ---
> v2:
>  * incorporated rob herring and thierry reviews
>    gpio to gpios, gpio to regulator using fixed regulators
>    and pwm backlight is removed, since it is controlled by
>    dcs commands
> ---
>  .../bindings/display/panel/jdi,lt070me05000.txt    | 57 ++++++++++++++++++++++
>  1 file changed, 57 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/display/panel/jdi,lt070me05000.txt
> 
> diff --git a/Documentation/devicetree/bindings/display/panel/jdi,lt070me05000.txt b/Documentation/devicetree/bindings/display/panel/jdi,lt070me05000.txt
> new file mode 100644
> index 0000000..613b76f
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/display/panel/jdi,lt070me05000.txt
> @@ -0,0 +1,57 @@
> +JDI model LT070ME05000 1200x1920 7" DSI Panel
> +
> +Required properties:
> +- compatible: should be "jdi,lt070me05000"
> +- vddp-supply: phandle of the regulator that provides the supply voltage
> +  Power IC supply (3-5V)
> +- dcdc_en-supply: phandle of the regulator that provides the supply voltage
> +  Power IC supply enable, High active
> +- vcc-supply: phandle of the regulator that provides the supply voltage
> +  IOVCC , power supply for LCM (1.8V)

I was just going to apply this and wanted to add some information about
how many lanes the panel uses, so I looked at the datasheet linked to in
patch 4/4:

	http://panelone.net/en/7-0-inch/JDI_LT070ME05000_7.0_inch-datasheet

and noticed that the power supplies in the binding here don't match the
ones listed in the datasheet. Granted, the datasheet is somewhat meager,
but it lists the following three input voltages:

	- IOVCC at 1.8 V
	- VDD at 3.0 V
	- VDDP at 3.0 V

This matches in part what the above binding describes, but it's not
quite right, so I'm wondering if the name for dcdc_en-supply came from
the board schematics and reflects the name of the rail on the board
design that provides this rather than the input voltage on the panel.

I think we should follow the names in the panel datasheet so that people
can more easily reference the correct regulators when using this panel
on a different board. I'd suggest:

	- iovcc-supply: phandle of the regulator that supplies power to
	  the IOVCC voltage input (1.8 V)
	- vdd-supply: phandle of the regulator that supplies power to
	  the VDD voltage input (3.0 V)
	- vddp-supply: phandle of the regulator that supplies power to
	  the VDDP voltage input (3.0 V)

Any objections?

Thierry

Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)