linux-kernel - Re: [PATCH] Support for controlling leds on xbox 360 pad.

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <1180474885.5822.94.camel@localhost.localdomain>
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/