All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mattia Dongili <malattia@linux.it>
To: Karol Babioch <karol@babioch.de>
Cc: platform-driver-x86@vger.kernel.org,
	ACPI Devel Mailing List <linux-acpi@vger.kernel.org>
Subject: Re: no brightness with acpi video since 3.3 [was Re: Issues with the sony-platform module]
Date: Sat, 16 Jun 2012 11:52:24 +0900	[thread overview]
Message-ID: <20120616025223.GA5200@kamineko.org> (raw)
In-Reply-To: <4FDBC183.7050203@babioch.de>

On Sat, Jun 16, 2012 at 01:13:07AM +0200, Karol Babioch wrote:
> Hi,
> 
> Am 16.06.2012 00:37, schrieb Mattia Dongili:
> > ok, let's forget about gnome popups for a moment, can you try using the
> > sysfs files directly?
> I suppose you expect me to use the "acpi_backlight=vendor" parameter?

yes correct.

> Because when leaving it out, I don't have any sony subdirectory within
> /sys/class/backlight. The only directory is
> /sys/class/backlight/acpi_video0, but changing the brightness directly
> there doesn't change anything.

basically that vendor parameter forces acpi to give up backlight control
to the platform driver.

> With the "acpi_backlight=vendor" boot paramater I got a sony sub
> directory (as well as a "nv" subdirectory), but see yourself:
> 
> [johnpatcher@vpcs ~]$ ls -l /sys/class/backlight/
> insgesamt 0
> lrwxrwxrwx 1 root root 0 16. Jun 2012  nv_backlight ->
> ../../devices/pci0000:00/0000:00:01.0/0000:01:00.0/drm/card0/card0-LVDS-1/nv_backlight
> lrwxrwxrwx 1 root root 0 16. Jun 01:00 sony ->
> ../../devices/virtual/backlight/sony
> 
> I tried to change the brightness via sysfs with the following command:
> 
> [root@vpcs sony]# cat actual_brightness brightness max_brightness
> -13
> -13
> 242

This is a known issue (at least to me) but it shouldn't really be a
problem as it existed for quite some time. Reading the brightness level
at boot returns an uninitialized value so the driver displays that
negative number. After echoing below you should see the actual value you
set.

> [root@vpcs sony]# echo 150 > brightness
> 
> But nothing changed. dmesg reports the following:
...
> [  461.256523] sony_laptop: found handle 0x0137 (offset: 0x0b)
> [  461.386744] sony_laptop: __call_snc_method: [SN07:0x0000000000a3010b]
> [  461.386746] sony_laptop: called SN07 with 0xa3010b (result: 0x0000)

this looks ok. I'm wondering if the nouveau driver may be getting in the
way somehow or if I broke backlight support in sony-laptop.

Can you try applying the sony-laptop patches on 3.2, boot with
acpi_backlight=vendor and see what is the state there?

> Interestingly enough I can change the brightness via
> /sys/class/backlight/nv_backlight:
...
> To be honest, I don't understand what is going on here. Why are there

there are various (potentially conflicting) ways to control the
brightness levels, acpi, platform and raw (via the video driver).

-- 
mattia
:wq!

  reply	other threads:[~2012-06-16  2:52 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-14 17:33 Issues with the sony-platform module Karol Babioch
2012-06-14 20:39 ` no brightness with acpi video since 3.3 [was Re: Issues with the sony-platform module] Mattia Dongili
2012-06-15  7:45   ` Karol Babioch
2012-06-15 11:36     ` Mattia Dongili
2012-06-15 12:08       ` Karol Babioch
2012-06-15 15:17       ` Karol Babioch
2012-06-15 22:37         ` Mattia Dongili
2012-06-15 23:13           ` Karol Babioch
2012-06-16  2:52             ` Mattia Dongili [this message]
2012-06-16 11:24               ` Karol Babioch
2012-06-16 13:07                 ` Mattia Dongili
2012-06-16 18:07                   ` Karol Babioch
2012-06-16 23:16                     ` Mattia Dongili
2012-06-17 10:50                       ` Karol Babioch
2012-06-18 22:19                         ` Mattia Dongili
2012-06-19 10:48                           ` Karol Babioch
2012-06-19  0:59   ` Zhang Rui
2012-06-19 13:57     ` Karol Babioch

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=20120616025223.GA5200@kamineko.org \
    --to=malattia@linux.it \
    --cc=karol@babioch.de \
    --cc=linux-acpi@vger.kernel.org \
    --cc=platform-driver-x86@vger.kernel.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.