Re: LED devices & poll() for brightness attribute

From	Pavel Machek <pavel@ucw.cz>
Newsgroups	linux.kernel
Subject	Re: LED devices & poll() for brightness attribute
Date	2017-02-23 22:20 +0100
Message-ID	<te8XT-6NT-5@gated-at.bofh.it> (permalink)
References	<tdB5T-7N5-3@gated-at.bofh.it> <te2Su-2AJ-3@gated-at.bofh.it> <te8bw-6hU-7@gated-at.bofh.it> <te8uS-6oc-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Hi!

> > > > 2) there may be more then one
> > > 
> > > Yes and script can be adjusted to use specific one (config option
> > > for script).
> > 
> > No, you should autoconfigure it,
> 
> Why? Because you prefer autoconfiguration and all other people must use 
> it only? Does not seems a good argument.
> 
> For machine which does not change, static policy file is enough too. And 
> for this simple case it will work correctly.
> 
> Why everybody must be forced to use some autoconfiguration and does not 
> allow user to use his own manual configuration?

Why everybody must be forced to poll() on brightness file?

> > actually. See N900, there are 6 leds controlling one backlight.
> 
> I'm not saying that one script must be universal for everything. My 
> script is enough for machine with single keyboard led. And for other 
> people who have only one backlight keyboard it will work fine too.

> For other cases something other could be written...

For other cases, you need to communicate with the desktop.

> > And I don't want led/brightness file to be used for interprocess
> > communication.
> 
> You are not forced to use something. I'm proposing extending current ABI 
> which can be used for processes who want those notification.

Your proposal does not help you with your problem, unless you get
desktop people to poll() on the brightness files, and do something
inteligent with the results.

Hmm. And doing something inteligent will be quite hard given that we
have 6 of those files on N900.

So, if you seriously want to do this, can you

1) write specification explaining how desktops should poll() the
brightness files

2) get someone to volunteer to implement that specification for at
least GNOME and KDE?

									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Thread

LED devices & poll() for brightness attribute Pali Rohár <pali.rohar@gmail.com> - 2017-02-22 10:10 +0100
  Re: LED devices & poll() for brightness attribute Pavel Machek <pavel@ucw.cz> - 2017-02-22 13:30 +0100
    Re: LED devices & poll() for brightness attribute Pali Rohár <pali.rohar@gmail.com> - 2017-02-22 13:50 +0100
      Re: LED devices & poll() for brightness attribute Pavel Machek <pavel@ucw.cz> - 2017-02-22 22:00 +0100
        Re: LED devices & poll() for brightness attribute Pali Rohár <pali.rohar@gmail.com> - 2017-02-22 22:20 +0100
          Re: LED devices & poll() for brightness attribute Pavel Machek <pavel@ucw.cz> - 2017-02-23 15:40 +0100
            Re: LED devices & poll() for brightness attribute Pali Rohár <pali.rohar@gmail.com> - 2017-02-23 15:50 +0100
              Re: LED devices & poll() for brightness attribute Pavel Machek <pavel@ucw.cz> - 2017-02-23 21:30 +0100
                Re: LED devices & poll() for brightness attribute Pali Rohár <pali.rohar@gmail.com> - 2017-02-23 21:50 +0100
                Re: LED devices & poll() for brightness attribute Pavel Machek <pavel@ucw.cz> - 2017-02-23 22:20 +0100
                Re: LED devices & poll() for brightness attribute Pali Rohár <pali.rohar@gmail.com> - 2017-02-24 10:00 +0100
                Re: LED devices & poll() for brightness attribute Pali Rohár <pali.rohar@gmail.com> - 2017-02-24 10:00 +0100
                Re: LED devices & poll() for brightness attribute Hans de Goede <hdegoede@redhat.com> - 2017-02-24 10:00 +0100

csiph-web