public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Dmitry Torokhov <dtor@insightbb.com>
To: Richard Purdie <rpurdie@rpsys.net>
Cc: Jan Kratochvil <honza@jikos.cz>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Support for controlling leds on xbox 360 pad.
Date: Tue, 29 May 2007 23:17:11 -0400	[thread overview]
Message-ID: <200705292317.14023.dtor@insightbb.com> (raw)
In-Reply-To: <1180474885.5822.94.camel@localhost.localdomain>

On Tuesday 29 May 2007 17:41, Richard Purdie wrote:
> 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.
> 

I will also venture to say that you only want to grant access to
the user currently logged on console, not any random user logged
in over the network so you really want to dynamically manage
premissions. The kernel just provides sensible defaults.

-- 
Dmitry

  reply	other threads:[~2007-05-30  3:50 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-29 21:11 [PATCH] Support for controlling leds on xbox 360 pad Jan Kratochvil
2007-05-29 21:29 ` Jan Kratochvil
2007-05-29 21:41   ` Richard Purdie
2007-05-30  3:17     ` Dmitry Torokhov [this message]
2007-05-30 14:56     ` Dmitry Torokhov
2007-05-30 16:32       ` Richard Purdie
2007-05-30 21:37         ` Jan Kratochvil
2007-06-03 18:02           ` Jan Kratochvil
2007-06-03 18:07             ` Jan Kratochvil
2007-06-15  4:15             ` Dmitry Torokhov
2007-06-15  5:06               ` Dmitry Torokhov
2007-05-31 14:07 ` Pavel Machek
2007-05-31 15:53   ` Jan Kratochvil
2007-05-31 16:09     ` Richard Purdie

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200705292317.14023.dtor@insightbb.com \
    --to=dtor@insightbb.com \
    --cc=honza@jikos.cz \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rpurdie@rpsys.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox