From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Renninger Subject: Re: [PATCH, RFC] [1/3] Generic in-kernel AC status Date: Tue, 14 Feb 2006 18:44:30 +0100 Message-ID: <43F216FE.7050101@suse.de> References: <20060208125753.GA25562@srcf.ucam.org> <20060210080643.GA14763@suse.de> <20060210121913.GA4974@elf.ucw.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mx1.suse.de ([195.135.220.2]:14041 "EHLO mx1.suse.de") by vger.kernel.org with ESMTP id S1422702AbWBNRoc (ORCPT ); Tue, 14 Feb 2006 12:44:32 -0500 In-Reply-To: <20060210121913.GA4974@elf.ucw.cz> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Pavel Machek Cc: Stefan Seyfried , Matthew Garrett , linux-pm@lists.osdl.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org Pavel Machek wrote: > On P=E1 10-02-06 09:06:43, Stefan Seyfried wrote: >> On Wed, Feb 08, 2006 at 12:57:53PM +0000, Matthew Garrett wrote: >>> The included patch adds support for power management methods to reg= ister=20 >>> callbacks in order to allow drivers to check if the system is on AC= or=20 >>> not. Following patches add support to ACPI and APM. Feedback welcom= e. >> Ok. Maybe i am not seeing the point. But why do we need this in the = kernel? >> Can't we handles this easily in userspace? >=20 > Some kernel parts need to now: for example powernow-k8: some > frequencies are not allowed when you are running off battery. [Just > now it is solved by not allowing those frequencies at all unless ACPI > is available; quite an ugly solution.] >=20 Allowed CPUfreqs are exported via _PPC. This is why a lot hardware sends an ac_adapter and a processor event when (un)plugging ac adapter. Limiting cpufreq already works nice that way. AMD64 laptops are booting with lower freqs per default until they are pushed up, so there shouldn't be anything critical? =46or the brightness part, I don't see any "laptop is going to explode" issue. I always hated the brightness going down when I unplugged ac on M$ and had to push ten times the brightness "up button" before I could go on working... Shouldn't it be: =20 ac Event ---> userspace <--- user config | |=20 brightness <-------| Whether the ac brightness will be set when going to ac, or whatever brightness, should be configurable in userspace IMO. This is a one liner in the acpid config? However, I also like the general /sys/../brightness file very much! Thomas - To unsubscribe from this list: send the line "unsubscribe linux-acpi" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html