Call for testers acpi_dell

[Daniel Walter]

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 ?

Daniel Walter