From owner-freebsd-acpi@FreeBSD.ORG Tue Mar 30 08:47:17 2010 Return-Path: Delivered-To: freebsd-acpi@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 71B98106566B for ; Tue, 30 Mar 2010 08:47:17 +0000 (UTC) (envelope-from rpaulo@gmail.com) Received: from mail-bw0-f216.google.com (mail-bw0-f216.google.com [209.85.218.216]) by mx1.freebsd.org (Postfix) with ESMTP id EE0B18FC12 for ; Tue, 30 Mar 2010 08:47:16 +0000 (UTC) Received: by bwz8 with SMTP id 8so5418269bwz.3 for ; Tue, 30 Mar 2010 01:47:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:sender:subject:mime-version :content-type:from:in-reply-to:date:cc:content-transfer-encoding :message-id:references:to:x-mailer; bh=xvzm/rVnRJJanocNDz16YluT2UElIp2fIGC1cNuhdGI=; b=LpDChi2H/lI8PE+/qviiS5NcVHf5ytxI6ieJFh2GpISwzLU1GhB+Bd6YDTO/nBPLav Anin+C57T066upyseR4ELwHIkWGWoqpcrPQ4Y/jkTHmkt2vLuozv/onrSLUf8d5ufxhV gKePbwu83151Egx+DovFlsIz5YvY8NICgUyWQ= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=sender:subject:mime-version:content-type:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to:x-mailer; b=Cn3vOtL//18Edu13TQLpNBxygjyajwV2sq1WhqhD36CZtGFT5KRL2wpVv1eRAbgFT9 iN0z9IVJCezjdxHLXOSwshxhypJpzrL9fRZ/YyZP9ckXSV5Wj+2KB9KJzOMoQ6/sQ0k6 fczWMxCSUWnES1pmdYGnMUWkV3qHlaeKsGJJs= Received: by 10.204.174.194 with SMTP id u2mr6498936bkz.40.1269938835708; Tue, 30 Mar 2010 01:47:15 -0700 (PDT) Received: from [10.0.10.2] (54.81.54.77.rev.vodafone.pt [77.54.81.54]) by mx.google.com with ESMTPS id s17sm46065845bkd.4.2010.03.30.01.47.14 (version=TLSv1/SSLv3 cipher=RC4-MD5); Tue, 30 Mar 2010 01:47:14 -0700 (PDT) Sender: Rui Paulo Mime-Version: 1.0 (Apple Message framework v1078) Content-Type: text/plain; charset=us-ascii From: Rui Paulo In-Reply-To: <4BB170E7.3030407@obluda.cz> Date: Tue, 30 Mar 2010 09:47:12 +0100 Content-Transfer-Encoding: quoted-printable Message-Id: <8D820BAA-C1B9-408C-AAF3-EA69612D2C46@freebsd.org> References: <4BB170E7.3030407@obluda.cz> To: Dan Lukes X-Mailer: Apple Mail (2.1078) Cc: freebsd-acpi@freebsd.org Subject: Re: Brightness change on notebook that follow ACPI specification (par. B.7) X-BeenThere: freebsd-acpi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: ACPI and power management development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 30 Mar 2010 08:47:17 -0000 On 30 Mar 2010, at 04:32, Dan Lukes wrote: >=20 > Most notebooks have special keys (mostly Fn+something) to change = brightness of LCD display. >=20 > Some of them (my notebok, for example) follows the ACPI specification = (paragraph B.7) how to announce the user request for brightness change = to OS. >=20 > I implemented such handling as part of acpi-video module. >=20 > It's verified to work on my HP Mini 5101 & FreeBSD 8.0. >=20 > No test on other notebook done as I have no other notebook. >=20 > No test on other OS version, but verified that patched module can be = compiled on 6.4-R, 7.2-R, 8.0-R. I assume (not verified) it's compilable = on all 6.x-8.x. I assume (not verified) it should work on all of these = versions. >=20 > If someone want to test it then the patch is here: > http://www.freebsd.cz/~dan/patch-acpi-video >=20 > installation: > cd /usr/src ; patch < patch-acpi-video > then compile the module: > http://www.freebsd.org/doc/handbook/acpi-debug.html#ACPI-DEBUGOUTPUT > then kldload acpi_video > then try keys for brightness change. >=20 > I'm interested to know on which notebook and FreeBSD version is = verified to be (not)working. >=20 > If it doesn't work it may be either: > 1. bug in my patch > 2. your notebook doesn't follow ACPI specification (par. B.7) >=20 > If you are skilled user and you want to know more (and you have = compiled ACPI modules with ACPI_DEBUG) you can set > debug.acpi.level=3D"ACPI_LV_ALL_EXCEPTIONS" > then look on console messages what happens when you press the special = keys. >=20 > Dan I see nothing wrong with this patch and I think it can be committed, but = I would like others to take a look. -- Rui Paulo