All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pekka Paalanen <ppaalanen@gmail.com>
To: Hans de Goede <hdegoede@redhat.com>
Cc: Sebastian Wick <sebastian.wick@redhat.com>,
	Christoph Grenz <christophg+lkml@grenz-bonn.de>,
	Martin Roukala <martin.roukala@mupuf.org>,
	"dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>,
	wayland <wayland-devel@lists.freedesktop.org>,
	Yusuf Khan <yusisamerican@gmail.com>
Subject: Re: [RFC v2] drm/kms: control display brightness through drm_connector properties
Date: Mon, 3 Oct 2022 13:32:22 +0300	[thread overview]
Message-ID: <20221003133222.7f34862f@eldfell> (raw)
In-Reply-To: <44df4468-955b-0226-67e7-89467291d38f@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 1563 bytes --]

On Mon, 3 Oct 2022 11:44:56 +0200
Hans de Goede <hdegoede@redhat.com> wrote:

> One example mentioned here is that laptops with Intel integrated
> graphics may have some "perceived brightness" mapping table
> in their Video BIOS Tables (VBT) it would be interesting to use
> this and to export the curve coming from that to userspace as
> extra information (including allow userspace to write it).
> Since in this case (unlike many other cases) at least this
> curve is done in the kernel I guess we can then also add a boolean
> property to disable the curve (making it linear) in case that
> is helpful for HDR scenarios.

Hi Hans,

what if you would export that curve to userspace and not apply it in
the kernel, aside from the min-luminance=0 offset?

I'm not sure if that was your intention, I didn't see it clearly said.
Of course this can be only about curves that are supposed to be applied
by the OS and not applied in firmware or hardware. Call it "software
curve"?

Let userspace apply that curve if it happens to exist. Then, if we get
some other definition replacing that curve on some hardware, the kernel
could just expose the other thing as a new thing, and the old curve API
would not be interfering. Userspace that does not understand the new
thing (and the old curve property is not populated) would still be able
to control the brightness, just not in as pleasant way.

It would also make using a custom curve a completely userspace thing with
no need for the kernel to support overwriting it.


Thanks,
pq

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2022-10-03 10:32 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-09 10:12 [RFC v2] drm/kms: control display brightness through drm_connector properties Hans de Goede
2022-09-09 13:39 ` Simon Ser
2022-09-09 13:53   ` Pekka Paalanen
2022-09-09 14:01     ` Simon Ser
2022-09-09 14:39   ` Hans de Goede
2022-09-28 10:04 ` Jani Nikula
2022-09-28 10:57   ` Ville Syrjälä
2022-09-28 11:14     ` Ville Syrjälä
2022-09-29 18:06       ` Sebastian Wick
2022-09-30  7:39         ` Pekka Paalanen
2022-09-30 14:20           ` Sebastian Wick
2022-09-30 14:30             ` Jani Nikula
2022-09-30 14:44             ` Ville Syrjälä
2022-09-30 14:49               ` Simon Ser
2022-09-30 15:26               ` Pekka Paalanen
2022-09-30 16:17                 ` Sebastian Wick
2022-10-03  8:37                   ` Pekka Paalanen
2022-10-03  9:29                     ` Ville Syrjälä
2022-10-03 10:16                       ` Pekka Paalanen
2022-10-03  9:44                   ` Hans de Goede
2022-10-03 10:32                     ` Pekka Paalanen [this message]
2022-10-03 11:02                       ` Hans de Goede
2022-10-03  9:02     ` Hans de Goede
2022-10-03  8:53   ` Hans de Goede

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=20221003133222.7f34862f@eldfell \
    --to=ppaalanen@gmail.com \
    --cc=christophg+lkml@grenz-bonn.de \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=hdegoede@redhat.com \
    --cc=martin.roukala@mupuf.org \
    --cc=sebastian.wick@redhat.com \
    --cc=wayland-devel@lists.freedesktop.org \
    --cc=yusisamerican@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.