From: Jani Nikula <jani.nikula@linux.intel.com>
To: "Daniel Thompson" <daniel.thompson@linaro.org>,
"Stéphane Marchesin" <stephane.marchesin@gmail.com>
Cc: "Hans de Goede" <hdegoede@redhat.com>,
"Martin Gräßlin" <mgraesslin@kde.org>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>
Subject: Re: KMS backlight ABI proposition
Date: Fri, 24 Feb 2017 13:41:53 +0200 [thread overview]
Message-ID: <878tovx1by.fsf@intel.com> (raw)
In-Reply-To: <08f3ec93-fda7-ee53-d290-12ecd42dcaac@linaro.org>
On Fri, 24 Feb 2017, Daniel Thompson <daniel.thompson@linaro.org> wrote:
> On 24/02/17 08:43, Jani Nikula wrote:
> >> - "Some PWM based backlight allow adjusting the PWM modulation
> >> frequency." you don't need a motivation for *why* I would want to
> >> change the mod freq on the fly, actually in my experience you
> >> shouldn't since this can lead to flickery backlights.
> >
> > The modulation frequency is usually an OEM design choice. The higher
> > the
> > frequency, less flicker, but also fewer user distinguishable levels of
> > backlight (signal rise/fall times come into play). Occasionally there
> > have been requests to be able to adjust the frequency. We can of course
> > decide that's an implementation detail and not let userspace change it.
>
> In principle the backlight max level could also change during monitor
> hotplug; recent versions DDC/CI include a backlight properties. We have
> no driver for this right now but conversation is starting (and proper
> backlight support in DRM connectors will probably help it happen faster).
>
> I'm afraid I have no idea if this is "real" or now. Right now I have no
> clue how many monitors implement it. For regular desktop monitors which
> mirror old school brightness/contrast in their UI then 0 wouldn't
> surprise me much ;-)
Thanks for the reminder. I think so far this as worked by using the DDC
I2C device nodes directly. And it really works already to some extent,
on some displays. It would be nice to have this controlled the same way
as embedded diplays.
Another interesting aspect with DDC/CI brightness control, one that it
shares with eDP DPCD and DSI DCS brightness controls, is that it could
all happen within DRM, mostly in a generic, driver independent
manner. There would not be a need for a backlight class device or sysfs
API at all, except for legacy needs.
BR,
Jani.
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2017-02-24 11:41 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-17 12:58 KMS backlight ABI proposition Martin Peres
2017-02-20 12:46 ` Martin Peres
2017-02-20 14:11 ` Daniel Thompson
2017-02-22 15:05 ` Jani Nikula
2017-02-22 15:18 ` Martin Peres
2017-02-22 16:20 ` Hans de Goede
2017-02-23 8:55 ` Jani Nikula
2017-02-23 13:44 ` Hans de Goede
2017-02-20 16:25 ` Thierry Reding
2017-02-22 15:38 ` Jani Nikula
2017-02-20 19:27 ` Dave Airlie
2017-02-20 19:57 ` Hans de Goede
2017-02-22 15:08 ` Martin Peres
2017-02-20 22:40 ` Alex Deucher
2017-02-20 23:01 ` Jasper St. Pierre
2017-02-22 14:00 ` Jani Nikula
2017-02-22 16:35 ` Jasper St. Pierre
2017-02-22 15:48 ` Jani Nikula
2017-02-20 20:09 ` Hans de Goede
2017-02-22 14:14 ` Jani Nikula
2017-02-22 19:07 ` Stéphane Marchesin
2017-02-23 8:40 ` Jani Nikula
2017-02-23 13:38 ` Hans de Goede
2017-02-23 17:31 ` Stéphane Marchesin
2017-02-24 8:43 ` Jani Nikula
2017-02-24 8:55 ` Hans de Goede
2017-02-24 9:34 ` Jani Nikula
2017-02-24 9:46 ` Hans de Goede
2017-02-24 9:48 ` Hans de Goede
2017-02-24 9:59 ` Hans de Goede
2017-02-24 10:23 ` Martin Peres
2017-02-24 10:44 ` Hans de Goede
2017-02-24 12:56 ` Martin Peres
2017-02-24 11:16 ` Daniel Thompson
2017-02-24 11:41 ` Jani Nikula [this message]
2017-02-23 17:41 ` Jasper St. Pierre
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=878tovx1by.fsf@intel.com \
--to=jani.nikula@linux.intel.com \
--cc=daniel.thompson@linaro.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=hdegoede@redhat.com \
--cc=mgraesslin@kde.org \
--cc=stephane.marchesin@gmail.com \
/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.