On Mon, 28 Jan 2008 01:25:50 +0000 Matthew Garrett <mjg59@xxxxxxxxxxxxx> wrote: > On Sat, Jan 26, 2008 at 10:00:45PM -0800, Andrew Morton wrote: > > - Create a new /sys node with a new name which has the new semantics. > > The semantics are the same as they always have been - values between 0 > and max_brightness are valid values. If you've made assumptions about > what max_brightness is, then that's a bug in the userspace application > rather than a change in the semantics of the interface. > WTH? My (utterly comedic chase-crap-around-the-tree) brightness script was: ( 0 sh -c "echo $1 > /proc/acpi/sony/brightness" 0 sh -c "echo $1 > /proc/acpi/sony/brightness_default" 0 sh -c "echo $1 > /sys/class/backlight/sony/brightness" 0 sh -c "echo $1 > /sys/class/backlight/thinkpad_screen/brightness" ) 2>/dev/null And yes, I had an rc.local command which assumed that 7 (later 8) is max brightness. You cannot seriously tell me that if we are to change this range from 0-8 up to 0-100 then this is not a backwards-incompatible change in semantics. My /sys/class/backlight/ directory is presently empty. Rather than trying to find out why, I think I'll just collapse in laughter. Guys, sort it out, please. - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
