From: Petter Reinholdtsen <pere@hungry.com>
To: dri-devel@lists.freedesktop.org
Subject: Re: Packard Bell EasyNote LV need i915.invert_brightness=1
Date: Wed, 26 Jun 2013 13:30:01 +0200 [thread overview]
Message-ID: <2flr4fpyurq.fsf@diskless.uio.no> (raw)
In-Reply-To: <20130625125810.GT5823@phenom.ffwll.local>
[Daniel Vetter]
> Nah, silence just means that your patch fell through the crack while I've
> travelled around a bit. Thanks for poking, it's merged now for 3.11 (with
> cc: stable) to my drm-intel-next-queued branch. I've slightly bikeshedded
> the patch though for formatting.
Right. Thank you.
Btw, a related question. On the laptop in question, even when I use
i915.invert_brightness=1, the kernel still seem to be confused about the
brightness values. This causes the KDE system powerdevil to turn off
the screen when I log into KDE.
<URL: https://bugs.kde.org/show_bug.cgi?id=321580 > got some details
about the problem, which can be seen from this shell run:
% for interface in /sys/class/backlight/*; do echo $interface; cat $interface/max_brightness; cat $interface/actual_brightness; done
/sys/class/backlight/acpi_video0
100
0
%
The max_brightness value is 100, and the current brightness is 0, which
actually is full brightness (and not black screen). This fools
powerdevil into turning off the screen while believing it turn it on.
Should not the ACPI backlight system also know about the inverted
brightness setting, and adjust the /sys/ interface to get these values
to behave consistently across all hardware models?
Is there some other quirk table for acpi that also need to be updated?
--
Happy hacking
Petter Reinholdtsen
next prev parent reply other threads:[~2013-06-26 11:30 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20130603085755.GC11674@ulrik.uio.no>
2013-06-11 8:28 ` Packard Bell EasyNote LV need i915.invert_brightness=1 Petter Reinholdtsen
2013-06-11 14:08 ` Daniel Vetter
2013-06-11 15:13 ` Petter Reinholdtsen
2013-06-24 21:57 ` Petter Reinholdtsen
2013-06-25 12:58 ` Daniel Vetter
2013-06-26 11:30 ` Petter Reinholdtsen [this message]
2013-06-26 17:08 ` Daniel Vetter
2013-06-26 17:35 ` Petter Reinholdtsen
2013-06-27 12:49 ` Daniel Vetter
2013-06-27 13:03 ` Petter Reinholdtsen
2013-06-27 13:51 ` Daniel Vetter
2013-06-27 15:05 ` Petter Reinholdtsen
2013-06-28 12:15 ` Daniel Vetter
2013-06-28 12:16 ` Daniel Vetter
2013-06-29 10:48 ` Petter Reinholdtsen
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=2flr4fpyurq.fsf@diskless.uio.no \
--to=pere@hungry.com \
--cc=dri-devel@lists.freedesktop.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.