From owner-freebsd-acpi@FreeBSD.ORG Fri Feb 12 19:48:27 2010 Return-Path: Delivered-To: freebsd-acpi@FreeBSD.org Received: from [127.0.0.1] (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by hub.freebsd.org (Postfix) with ESMTP id 0D815106568F; Fri, 12 Feb 2010 19:48:27 +0000 (UTC) (envelope-from jkim@FreeBSD.org) From: Jung-uk Kim To: freebsd-acpi@FreeBSD.org Date: Fri, 12 Feb 2010 14:48:17 -0500 User-Agent: KMail/1.6.2 References: <201002121409.49779.jkim@FreeBSD.org> In-Reply-To: MIME-Version: 1.0 Content-Disposition: inline Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201002121448.20071.jkim@FreeBSD.org> Cc: Daniel Walter Subject: Re: Call for testers acpi_dell 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: Fri, 12 Feb 2010 19:48:27 -0000 On Friday 12 February 2010 02:37 pm, Daniel Walter wrote: > On Fri, 12 Feb 2010, Jung-uk Kim wrote: > > On Friday 12 February 2010 01:33 pm, Daniel Walter wrote: > >> On Fri, 12 Feb 2010, Jung-uk Kim wrote: > >>> On Friday 12 February 2010 12:48 pm, Daniel Walter wrote: > >>>> Hi all, > >>>> > >>>> I've written a small kernel driver to support Dell Mini 1011 > >>>> (aka Dell Mini 10v) Brightness Controll Buttons. Since I do > >>>> not have any other Dell Notebooks, I'd like you to test this > >>>> module on other Dell Notebooks as well. > >>>> > >>>> This driver is in an early stage, but future releases will > >>>> also support the rfkill switch and CRT switch. > >>>> > >>>> Currently following features are implemented: > >>>> * Increase / Decrease Brightness > >>>> * Change brightness if AC is connected / disconnected > >>>> > >>>> TODO: > >>>> * Add CRT Support > >>>> * Add Sysctl to define brightness levels for eco and ac > >>>> powerprofile * Add Support for RF-Kill Switch > >>>> > >>>> Any suggestions are welcome > >>> > >>> I just quickly glanced at it but I failed see anything quite > >>> specific to Dell laptops except for "TODO: Add support for > >>> RF-Kill Switch". acpi_video(4) should be able to do what you > >>> want. Am I missing something here? > >>> > >>> Jung-uk Kim > >> > >> hi, > >> > >> Yes it is possible to control the brightness using acpi_video(4) > >> via sysctl but there is no support for the FN-Buttons. This > >> module adds the support for those buttons. > > > > I see. However, my point is the probe function does not seem > > specific to Dell laptops. In fact, you are just checking _BCL > > transition, i.e., effectively it just makes a generic acpi_video > > notify handler for brightness change. > > > > Jung-uk Kim > > Oh, I've completly overlooked this one. Sorry for that, I should > replace the probing with a Dell specific probing instead. Do you > have any suggestions to this one ? It totally depends on DSDT of your laptop model and you will have to maintain the list of these ACPI handles, unfortunately. Please see acpi_asus.c for example. If you are interested in generic acpi_video(4) enhancement, you just need to extend acpi_video.c to get these notifications, i.e., 0x86 and 0x87. FYI, acpi_video can only handle 0x80 and 0x81 events ATM. Please read ACPI specification Appendix B. Video Extensions, especially B.6 and B.7 are what you are looking for. Jung-uk Kim