| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Tue, 29 May 2007 22:41:25 +0100 From: Richard Purdie <rpurdie@...ys.net> To: Jan Kratochvil <honza@...os.cz> Cc: dtor@...l.ru, linux-kernel@...r.kernel.org Subject: Re: [PATCH] Support for controlling leds on xbox 360 pad. 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 majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists