From: Hans de Goede <hdegoede@redhat.com>
To: "Rafael J. Wysocki" <rjw@rjwysocki.net>, Aaron Lu <aaron.lu@intel.com>
Cc: Julian Wollrath <jwollrath@web.de>, linux-acpi@vger.kernel.org
Subject: Re: [Regression]: Changing brightness does not work with v3.16-rc4
Date: Wed, 16 Jul 2014 08:29:41 +0200 [thread overview]
Message-ID: <53C61BD5.8080501@redhat.com> (raw)
In-Reply-To: <2231725.PEI454q0UP@vostro.rjw.lan>
Hi,
On 07/16/2014 02:43 AM, Rafael J. Wysocki wrote:
> On Tuesday, July 15, 2014 10:45:29 PM Aaron Lu wrote:
>> On 07/15/2014 10:05 PM, Hans de Goede wrote:
>
> [cut]
>
>>> On 07/15/2014 04:00 PM, Aaron Lu wrote:
>>>
>>> > If we have to consider the following config:
>>> > 1 ACPI backlight interface works(or can work with some cmdline option);
>>>
>>> And the BIOS advertises native win8 support.
>>
>> Right.
>> So this condition should be:
>> 1 ACPI backlight interface works in Win8 mode;
>>
>> This means, for Julian's system, the commit 751109aad583 doesn't need to
>> be reverted as the ACPI backlight interface on his system only works
>> when the acpi_osi="!Windows 2012" is added. In that case, the
>> video.use_native_backlight=X doesn't make any difference anymore.
>
> But Julian reports that he needs to revert that commit to make things work
> as before for him. What gives?
Likely what is happening is that this commit is still affecting Julian because
we check if the bios is advertising win8 support, not if we are advertising win8
support ourselves.
Julian, I've jumped into this thread halfway, so I'm not 100% clear on your
original problem. From what I've read sofar you need acpi_osi="!Windows 2012"
for backlight control to work. That in itself is a sign that things are not
as they should be, and this is actually what the new default of
video.use_native_backlight=1 is trying to fix.
If you boot the new kernel without acpi_osi="!Windows 2012" and without
video.use_native_backlight=X (so using the new default of
video.use_native_backlight=1), what does ls /sys/class/backlight say then ?
If there is a directory in there can you then cd in to that directory,
and do: "cat max_brightness" and then "echo value > brightness" (the latter
as root) with value being a number, once half of max_brightness, once
full brightness and see if this controls your backlight brightness.
Assuming this test succeeds, is your problem resolved then? And if not
what is the problem?
Regards,
Hans
next prev parent reply other threads:[~2014-07-16 6:34 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-12 16:06 [Regression]: Changing brightness does not work with v3.16-rc4 Julian Wollrath
2014-07-12 16:24 ` Julian Wollrath
2014-07-14 18:21 ` Rafael J. Wysocki
2014-07-14 18:34 ` Julian Wollrath
2014-07-14 19:02 ` Rafael J. Wysocki
2014-07-14 19:10 ` Rafael J. Wysocki
2014-07-14 18:56 ` Julian Wollrath
2014-07-14 19:17 ` Rafael J. Wysocki
2014-07-15 6:06 ` Aaron Lu
2014-07-15 12:27 ` Rafael J. Wysocki
2014-07-15 14:00 ` Aaron Lu
2014-07-15 14:05 ` Hans de Goede
2014-07-15 14:45 ` Aaron Lu
2014-07-16 0:43 ` Rafael J. Wysocki
2014-07-16 6:29 ` Hans de Goede [this message]
2014-07-16 7:19 ` Aaron Lu
2014-07-16 8:13 ` Hans de Goede
2014-07-16 8:30 ` Aaron Lu
2014-07-16 8:35 ` Hans de Goede
2014-07-16 11:50 ` Julian Wollrath
2014-07-16 11:52 ` Hans de Goede
2014-07-15 21:38 ` Julian Wollrath
2014-07-16 2:36 ` Aaron Lu
2014-07-16 11:36 ` Julian Wollrath
2014-07-14 2:06 ` Aaron Lu
2014-07-14 12:42 ` Julian Wollrath
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=53C61BD5.8080501@redhat.com \
--to=hdegoede@redhat.com \
--cc=aaron.lu@intel.com \
--cc=jwollrath@web.de \
--cc=linux-acpi@vger.kernel.org \
--cc=rjw@rjwysocki.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox