Re: [PATCH 2.6.18-mm2] acpi: add backlight support to the sony_acpi driver

Richard Hughes
2006-10-10T14:55:25Z

On Tue, 2006-10-10 at 22:32 +0800, Yu Luming wrote:
> >From my understanding, a cute userspace App shouldn't have this kind
> of logic:
> if (is DELL )
> invoke libsmbios
> if (is foo)
> invoke libfoo,
> if (is bar)
> invoke libbar,
> ....
> else
> operate on /sys/class/backlight/ ,.,..

This is what HAL has at the moment[1]. And it's hell to maintain, but
works for a lot of users.

> It should be:
> just write/read file in /sys/class/backlight ,....

That would make things much easier IMO.

Richard.

[1]
http://gitweb.freedesktop.org/?p=hal.git;a=blob;h=3ff9284be440a7197b0de9b5f0234761c3397cb1;hb=dbffafacbf7b9143d82547b9eabe61d1a5b8fffc;f=tools/linux/hal-system-lcd-get-brightness-linux