Re: [PATCH 0/3] Generic sysfs support for ACPI ALS and other ALS devices

[Pavel Machek <pavel@ucw.cz>]

Hi!

> > > Could you please look at the documentation about ALS sysfs class in the
> > > patch I sent out just now, and comment on that one please? thanks!
> > 
> > I did. The interface is too ugly to live.
> 
> Hi, Pavel,
> 
> I tried to convert the ALS sysfs I/F to two attributes only, i.e.
> illuminance and adjustment.
> But I found several potential problems.
> 1. the illuminance to display adjustment mappings can not be convert to
> a brightness level smoothly.
> for example,
> 	illuminance	adjustment
> 1	600		70
> 2	900		100
> 3	1500		120
> when the current illuminance is not one of the values listed in the
> mappings, e.g. 750, the ALS driver don't have enough knowledge to get
> the proper display adjustment, especially that a proper display
> adjustment would be easy to select a proper brightness level.
> We'd better leave this to user space, which is more flexible.

Well, what interpolation does ACPI specs suggest to do? Maybe it is
easier to have linear interpolation in the kernel than to have ugly
20-file interface?

> 2. I don't know if there will be laptops exporting buggy mappings that
> needs to be overridden some day, but the current sysfs I/F is easy for
> expanding.
> what do you think?

If you have buggy laptop, fix the laptop. (You can still export the
raw values... and btw if you have buggy laptop and want to work around
it, maybe the workaround is easier/better done in kernel?)

								Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html