Messages in this thread |  | | | Date | Sat, 26 Jan 2008 22:00:45 -0800 | | From | Andrew Morton <> | | Subject | Re: [PATCH] Rationalise ACPI backlight implementation | |
> On Thu, 24 Jan 2008 16:44:48 -0500 Len Brown <lenb@kernel.org> wrote:
> On Tuesday 25 December 2007 21:03, Matthew Garrett wrote:
> > The sysfs backlight class provides no mechanism for querying the
> > acceptable brightness for a backlight. The ACPI spec states that values
> > are only valid if they are reported as available by the firmware. Since
> > we can't provide that information to userspace, instead collapse the
> > range to the number of actual values that can be set.
> >
> > Signed-off-by: Matthew Garrett <mjg59@srcf.ucam.org>
>
> I wish we did this in the first place.
> But doing it now is an API change -- since
> with the old way 100 always meant 100% brightness, yes?
>
> so my concern is that if we change what "10" means, somebody like akpm
> with an existing script gets grumpy.
It takes more than that to make me grumpy. I've been very grumpy lately.
- Create a new /sys node with a new name which has the new semantics.
- Deprecate the old /sys entry by emitting an angry printk when someone
uses it.
- Wait 12 months
- Kill the old one.
|  |