From owner-freebsd-acpi@FreeBSD.ORG Sat Feb 13 02:26:03 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 937471065672; Sat, 13 Feb 2010 02:26:03 +0000 (UTC) (envelope-from jkim@FreeBSD.org) From: Jung-uk Kim To: freebsd-acpi@FreeBSD.org Date: Fri, 12 Feb 2010 21:25:54 -0500 User-Agent: KMail/1.6.2 References: <201002121541.29429.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: <201002122125.56882.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: Sat, 13 Feb 2010 02:26:03 -0000 On Friday 12 February 2010 04:38 pm, Daniel Walter wrote: > On Fri, 12 Feb 2010, Jung-uk Kim wrote: > > On Friday 12 February 2010 03:12 pm, Daniel Walter wrote: > >> On Fri, 12 Feb 2010, Jung-uk Kim wrote: > >>> 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 > >> > >> Hi I'll take a look at my DSDT again ... > >> > >> I've also will take a look at Appendix B of the current ACPI > >> spec to see if 0x86 and 0x87 are OEM or part of the standard. If > >> they are not completly out of spec I'll patch acpi_video(4) and > >> send it again to the list. > > > > 0x86/0x87 is part of the spec. and it should be handled by > > acpi_video IMHO. > > > > Jung-uk Kim > > hi, > > as you suggested I've written a patch for acpi_video(4) to support > the 0x86 and 0x87 notifications. the support for 0x85, 0x88, 0x89 > (also specified in ACPI Standard Appendix B7) is not yet > implemented. > > Could anyone with a ACPI sending 0x86 and 0x87 test the patch ? > > thanks for the help and pointing my in the right direction. I just went ahead and committed a rewritten version. :-) Thanks! Jung-uk Kim