On Tue, 2007-05-29 at 23:29 +0200, Jan Kratochvil wrote: > I have question, probably for Richard. Why is > /sys/class/leds/whatsoever/brightness mode set to 0644? Is it really > necessary? > I feel like I'll be happy to allow anybody to change the state of this led. > (Ok > this maybe doesn't apply to other leds)
Permissions management of the LEDs is outside the scope of kernel. If you need users to have access to them, change the permissions in userspace to grant access. > > So this patch uses led subystem in such way that it makes led in > > /sys/class/leds/xpad:[0-9][0-9]/ for each attached xbox 360 pad. > > I have the impression that the led subsystem doesn't actually count with > possibility to have more then one device of one type, right? In my case there > can by lots of pads, so I made it on my own, using counter. Is it ok? It leaves it up to the driver concerned. In your case it makes sense so counting is fine. Don't put the colon in though, just have it as xpad0 etc. as the name. Anything after the first colon is defined as an optional colour (see Documentation/leds-class.txt). Richard - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
