From: Pavel Machek <pavel@ucw.cz>
To: Ondrej Jirman <megous@megous.com>
Cc: linux-sunxi@googlegroups.com,
Thierry Reding <thierry.reding@gmail.com>,
Sam Ravnborg <sam@ravnborg.org>, David Airlie <airlied@linux.ie>,
Daniel Vetter <daniel@ffwll.ch>, Rob Herring <robh+dt@kernel.org>,
Maxime Ripard <mripard@kernel.org>, Chen-Yu Tsai <wens@csie.org>,
Linus Walleij <linus.walleij@linaro.org>,
Icenowy Zheng <icenowy@aosc.io>,
dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
Samuel Holland <samuel@sholland.org>,
Martijn Braam <martijn@brixit.nl>, Luca Weiss <luca@z3ntu.xyz>,
Bhushan Shah <bshah@kde.org>
Subject: Re: [PATCH v3 4/5] arm64: dts: sun50i-a64-pinephone: Enable LCD support on PinePhone
Date: Fri, 29 May 2020 18:27:28 +0200 [thread overview]
Message-ID: <20200529162728.GB3709@amd> (raw)
In-Reply-To: <20200513212451.1919013-5-megous@megous.com>
[-- Attachment #1: Type: text/plain, Size: 1265 bytes --]
Hi!
> PinePhone uses PWM backlight and a XBD599 LCD panel over DSI for
> display.
>
> Backlight levels curve was optimized by Martijn Braam using a
> lux meter.
If it was possible to preserve lux values for individual settings in
the comment somewhere... that would be nice :-).
One day, it would be nice to have brightness settings in lux, so I
could easily set matching levels on multiple devices, for example...
Best regards,
Pavel
> +
> +&backlight {
> + power-supply = <®_ldo_io0>;
> + /*
> + * PWM backlight circuit on this PinePhone revision was changed since
> + * 1.0, and the lowest PWM duty cycle that doesn't lead to backlight
> + * being off is around 20%. Duty cycle for the lowest brightness level
> + * also varries quite a bit between individual boards, so the lowest
> + * value here was chosen as a safe default.
> + */
> + brightness-levels = <
> + 774 793 814 842
> + 882 935 1003 1088
> + 1192 1316 1462 1633
> + 1830 2054 2309 2596
> + 2916 3271 3664 4096>;
> + num-interpolated-steps = <50>;
> + default-brightness-level = <400>;
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Pavel Machek <pavel@ucw.cz>
To: Ondrej Jirman <megous@megous.com>
Cc: devicetree@vger.kernel.org, Samuel Holland <samuel@sholland.org>,
David Airlie <airlied@linux.ie>,
Linus Walleij <linus.walleij@linaro.org>,
Bhushan Shah <bshah@kde.org>, Chen-Yu Tsai <wens@csie.org>,
Maxime Ripard <mripard@kernel.org>,
linux-kernel@vger.kernel.org, Martijn Braam <martijn@brixit.nl>,
linux-sunxi@googlegroups.com, Rob Herring <robh+dt@kernel.org>,
Thierry Reding <thierry.reding@gmail.com>,
dri-devel@lists.freedesktop.org, Daniel Vetter <daniel@ffwll.ch>,
Sam Ravnborg <sam@ravnborg.org>, Luca Weiss <luca@z3ntu.xyz>,
linux-arm-kernel@lists.infradead.org,
Icenowy Zheng <icenowy@aosc.io>
Subject: Re: [PATCH v3 4/5] arm64: dts: sun50i-a64-pinephone: Enable LCD support on PinePhone
Date: Fri, 29 May 2020 18:27:28 +0200 [thread overview]
Message-ID: <20200529162728.GB3709@amd> (raw)
In-Reply-To: <20200513212451.1919013-5-megous@megous.com>
[-- Attachment #1.1: Type: text/plain, Size: 1265 bytes --]
Hi!
> PinePhone uses PWM backlight and a XBD599 LCD panel over DSI for
> display.
>
> Backlight levels curve was optimized by Martijn Braam using a
> lux meter.
If it was possible to preserve lux values for individual settings in
the comment somewhere... that would be nice :-).
One day, it would be nice to have brightness settings in lux, so I
could easily set matching levels on multiple devices, for example...
Best regards,
Pavel
> +
> +&backlight {
> + power-supply = <®_ldo_io0>;
> + /*
> + * PWM backlight circuit on this PinePhone revision was changed since
> + * 1.0, and the lowest PWM duty cycle that doesn't lead to backlight
> + * being off is around 20%. Duty cycle for the lowest brightness level
> + * also varries quite a bit between individual boards, so the lowest
> + * value here was chosen as a safe default.
> + */
> + brightness-levels = <
> + 774 793 814 842
> + 882 935 1003 1088
> + 1192 1316 1462 1633
> + 1830 2054 2309 2596
> + 2916 3271 3664 4096>;
> + num-interpolated-steps = <50>;
> + default-brightness-level = <400>;
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]
[-- Attachment #2: Type: text/plain, Size: 176 bytes --]
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: Pavel Machek <pavel@ucw.cz>
To: Ondrej Jirman <megous@megous.com>
Cc: devicetree@vger.kernel.org, Samuel Holland <samuel@sholland.org>,
David Airlie <airlied@linux.ie>, Bhushan Shah <bshah@kde.org>,
Chen-Yu Tsai <wens@csie.org>,
linux-kernel@vger.kernel.org, Martijn Braam <martijn@brixit.nl>,
linux-sunxi@googlegroups.com, Rob Herring <robh+dt@kernel.org>,
Thierry Reding <thierry.reding@gmail.com>,
dri-devel@lists.freedesktop.org, Sam Ravnborg <sam@ravnborg.org>,
Luca Weiss <luca@z3ntu.xyz>,
linux-arm-kernel@lists.infradead.org,
Icenowy Zheng <icenowy@aosc.io>
Subject: Re: [PATCH v3 4/5] arm64: dts: sun50i-a64-pinephone: Enable LCD support on PinePhone
Date: Fri, 29 May 2020 18:27:28 +0200 [thread overview]
Message-ID: <20200529162728.GB3709@amd> (raw)
In-Reply-To: <20200513212451.1919013-5-megous@megous.com>
[-- Attachment #1.1: Type: text/plain, Size: 1265 bytes --]
Hi!
> PinePhone uses PWM backlight and a XBD599 LCD panel over DSI for
> display.
>
> Backlight levels curve was optimized by Martijn Braam using a
> lux meter.
If it was possible to preserve lux values for individual settings in
the comment somewhere... that would be nice :-).
One day, it would be nice to have brightness settings in lux, so I
could easily set matching levels on multiple devices, for example...
Best regards,
Pavel
> +
> +&backlight {
> + power-supply = <®_ldo_io0>;
> + /*
> + * PWM backlight circuit on this PinePhone revision was changed since
> + * 1.0, and the lowest PWM duty cycle that doesn't lead to backlight
> + * being off is around 20%. Duty cycle for the lowest brightness level
> + * also varries quite a bit between individual boards, so the lowest
> + * value here was chosen as a safe default.
> + */
> + brightness-levels = <
> + 774 793 814 842
> + 882 935 1003 1088
> + 1192 1316 1462 1633
> + 1830 2054 2309 2596
> + 2916 3271 3664 4096>;
> + num-interpolated-steps = <50>;
> + default-brightness-level = <400>;
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2020-05-29 16:27 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-13 21:24 [PATCH v3 0/5] Add support for PinePhone LCD panel Ondrej Jirman
2020-05-13 21:24 ` Ondrej Jirman
2020-05-13 21:24 ` Ondrej Jirman
2020-05-13 21:24 ` [PATCH v3 1/5] dt-bindings: vendor-prefixes: Add Xingbangda Ondrej Jirman
2020-05-13 21:24 ` Ondrej Jirman
2020-05-13 21:24 ` Ondrej Jirman
2020-05-28 2:40 ` Rob Herring
2020-05-28 2:40 ` Rob Herring
2020-05-28 2:40 ` Rob Herring
2020-05-13 21:24 ` [PATCH v3 2/5] dt-bindings: panel: Add binding for Xingbangda XBD599 panel Ondrej Jirman
2020-05-13 21:24 ` Ondrej Jirman
2020-05-13 21:24 ` Ondrej Jirman
2020-05-26 11:22 ` Linus Walleij
2020-05-26 11:22 ` Linus Walleij
2020-05-26 11:22 ` Linus Walleij
2020-05-28 2:41 ` Rob Herring
2020-05-28 2:41 ` Rob Herring
2020-05-28 2:41 ` Rob Herring
2020-05-13 21:24 ` [PATCH v3 3/5] drm: panel: Add Xingbangda XBD599 panel (ST7703 controller) Ondrej Jirman
2020-05-13 21:24 ` Ondrej Jirman
2020-05-13 21:24 ` Ondrej Jirman
2020-05-26 11:32 ` Linus Walleij
2020-05-26 11:32 ` Linus Walleij
2020-05-26 11:32 ` Linus Walleij
2020-06-17 2:00 ` [linux-sunxi] " Ondřej Jirman
2020-06-17 2:00 ` Ondřej Jirman
2020-06-17 2:00 ` Ondřej Jirman
2020-05-28 3:31 ` Samuel Holland
2020-05-28 3:31 ` Samuel Holland
2020-05-28 3:31 ` Samuel Holland
2020-05-13 21:24 ` [PATCH v3 4/5] arm64: dts: sun50i-a64-pinephone: Enable LCD support on PinePhone Ondrej Jirman
2020-05-13 21:24 ` Ondrej Jirman
2020-05-13 21:24 ` Ondrej Jirman
2020-05-29 16:27 ` Pavel Machek [this message]
2020-05-29 16:27 ` Pavel Machek
2020-05-29 16:27 ` Pavel Machek
2020-05-13 21:24 ` [PATCH v3 5/5] arm64: dts: sun50i-a64-pinephone: Add touchscreen support Ondrej Jirman
2020-05-13 21:24 ` Ondrej Jirman
2020-05-13 21:24 ` Ondrej Jirman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200529162728.GB3709@amd \
--to=pavel@ucw.cz \
--cc=airlied@linux.ie \
--cc=bshah@kde.org \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=icenowy@aosc.io \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sunxi@googlegroups.com \
--cc=luca@z3ntu.xyz \
--cc=martijn@brixit.nl \
--cc=megous@megous.com \
--cc=mripard@kernel.org \
--cc=robh+dt@kernel.org \
--cc=sam@ravnborg.org \
--cc=samuel@sholland.org \
--cc=thierry.reding@gmail.com \
--cc=wens@csie.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.