All of lore.kernel.org
 help / color / mirror / Atom feed
From: Niel Lambrechts <niel.lambrechts@gmail.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: "linux.kernel" <linux-kernel@vger.kernel.org>,
	Henrique de Moraes Holschuh <hmh@hmh.eng.br>,
	linux-acpi@vger.kernel.org
Subject: Re: 2.6.30-rc1 (latest git): thinkpad-acpi: cannot control brightness with hotkeys
Date: Sun, 12 Apr 2009 00:55:12 +0200	[thread overview]
Message-ID: <49E11FD0.9040403@gmail.com> (raw)
In-Reply-To: <20090411150555.8b13ed18.akpm@linux-foundation.org>

On 04/12/2009 12:05 AM, Andrew Morton wrote:
> (cc's added)
>
> On Sat, 11 Apr 2009 23:07:16 +0200 Niel Lambrechts <niel.lambrechts@gmail.com> wrote:
>
>   
>> Hi there,
>>
>> With latest git kernel, I'm no longer able to control my Thinkpad (W500)
>> brightness, although I can still control the backlight, bluetooth etc.
>>     
>
> So 2.6.29 was OK?
>
>   
>> I originally got some errors (below), but I was able to get around them
>> by changing /etc/modprobe.d/thinkpad_acpi
>> from the (default OPENSUSE)
>> options thinkpad_acpi experimental=1 hotkey=enable,0xffffff
>> to
>> options thinkpad_acpi experimental=1
>>
>> Any ideas on how I can regain control over the brightness gas-pedal?
>>
>> Niel
>>
>> dmesg:
>> thinkpad_acpi: ThinkPad ACPI Extras v0.22
>> thinkpad_acpi: http://ibm-acpi.sf.net/
>> thinkpad_acpi: ThinkPad BIOS 6FET50WW (1.20 ), EC 7VHT12WW-1.01
>> thinkpad_acpi: Lenovo ThinkPad W500, model 40622XG
>> thinkpad_acpi: radio switch found; radios are enabled
>> thinkpad_acpi: This ThinkPad has standard ACPI backlight brightness
>> control, supported by the ACPI video driver
>> thinkpad_acpi: Disabling thinkpad-acpi brightness events by default...
>> thinkpad_acpi: procfs hotkey enable/disable: access by process with PID 890
>> WARNING: at drivers/platform/x86/thinkpad_acpi.c:2954
>> hotkey_enabledisable_warn+0x3e/0x43 [thinkpad_acpi]()
>> thinkpad_acpi: hotkey enable/disable functionality has been removed from
>> the driver. Hotkeys are always enabled.
>> Modules linked in: thinkpad_acpi(+) snd_hda_intel(+) rfkill
>> snd_hda_codec rtc_cmos snd_hwdep mac80211 video i2c_i801 rtc_core
>> snd_pcm ohci1394 led_class battery ac output snd_timer snd soundcore
>> snd_page_alloc e1000e intel_agp button joydev wmi i2c_core sr_mod
>> rtc_lib cfg80211 sg ieee1394 iTCO_wdt agpgart cdrom nvram
>> iTCO_vendor_support sd_mod ehci_hcd uhci_hcd usbcore ext4 jbd2 crc16 edd
>> ext3 mbcache jbd fan ahci libata scsi_mod thermal processor
>>  [<f81a3f7c>] hotkey_enabledisable_warn+0x3e/0x43 [thinkpad_acpi]
>>  [<f81a5234>] hotkey_write+0x68/0x162 [thinkpad_acpi]
>>  [<f7c63f62>] thinkpad_acpi_module_init+0x742/0x839 [thinkpad_acpi]
>>  [<f7c63820>] ? thinkpad_acpi_module_init+0x0/0x839 [thinkpad_acpi]
>> Registered led device: tpacpi::thinklight
>> thinkpad_acpi: Standard ACPI backlight interface available, not loading
>> native one.
>>
>>     
>
>
>   
Could it be related to this commit?

commit 0e501834f8c2ba7de2a56e332d346dcf4ac0b593
Author: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Date:   Sat Apr 4 04:25:53 2009 +0000

    thinkpad-acpi: rework brightness support

    Refactor and redesign the brightness control backend...

Regards,
Niel

  parent reply	other threads:[~2009-04-11 22:54 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-11 21:07 2.6.30-rc1 (latest git): thinkpad-acpi: cannot control brightness with hotkeys Niel Lambrechts
2009-04-11 22:05 ` Andrew Morton
2009-04-11 22:44   ` Niel Lambrechts
2009-04-11 22:55   ` Niel Lambrechts [this message]
2009-04-11 23:10     ` Niel Lambrechts
2009-04-12  5:04       ` Henrique de Moraes Holschuh
2009-04-12  4:58   ` Henrique de Moraes Holschuh
2009-04-12 16:14   ` Henrique de Moraes Holschuh
2009-04-12 18:37     ` Niel Lambrechts
2009-04-13  1:21       ` yakui_zhao
2009-04-13 15:16         ` Niel Lambrechts
2009-04-13 20:34           ` Henrique de Moraes Holschuh
2009-04-14 23:59             ` 2.6.30-rc1 (latest git): thinkpad-acpi: cannot control brightness with hotkeys (BISECTED) Niel Lambrechts
2009-04-15  0:41               ` Maxim Levitsky
2009-04-15  5:51                 ` Niel Lambrechts
2009-04-15  1:50               ` Zhang Rui
2009-04-15  1:51                 ` Zhang Rui
2009-04-15  6:53                 ` Niel Lambrechts
2009-04-15 12:35                   ` Peter Zijlstra
2009-04-15 16:49               ` Matthew Garrett
2009-04-15 19:55                 ` Niel Lambrechts
2009-04-15 20:01                   ` Matthew Garrett
2009-04-15 20:57                     ` Niel Lambrechts
2009-04-15 21:00                       ` Matthew Garrett
2009-04-16  1:02                 ` yakui_zhao
2009-04-13  1:53       ` 2.6.30-rc1 (latest git): thinkpad-acpi: cannot control brightness with hotkeys Zhang Rui
2009-04-13 15:18         ` Niel Lambrechts
2009-04-13 18:20           ` Niel Lambrechts
2009-04-14  1:02             ` Zhang Rui
2009-04-14  3:03               ` Zhang Rui
2009-04-11 22:17 ` Arkadiusz Miskiewicz
2009-04-11 22:39   ` Arkadiusz Miskiewicz
2009-04-12 22:38 ` 2.6.30-rc1 (latest git): ACPI video backlight support broken. (WAS:thinkpad-acpi: cannot control brightness with hotkeys ) Maxim Levitsky

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=49E11FD0.9040403@gmail.com \
    --to=niel.lambrechts@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=hmh@hmh.eng.br \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@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.