From: Pavel Machek <pavel@ucw.cz>
To: Jacek Anaszewski <jacek.anaszewski@gmail.com>
Cc: Tony Lindgren <tony@atomide.com>,
Jacek Anaszewski <j.anaszewski@samsung.com>,
Hans de Goede <hdegoede@redhat.com>,
linux-leds@vger.kernel.org, linux-omap@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, Darren Hart <dvhart@infradead.org>
Subject: Re: PM regression with LED changes in next-20161109
Date: Fri, 11 Nov 2016 23:06:32 +0100 [thread overview]
Message-ID: <20161111220631.GA10983@amd> (raw)
In-Reply-To: <f7c9b2be-0003-f342-a4c7-802532faa424@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 723 bytes --]
Hi!
> >Hmm. So userland can read the LED state, and it can get _some_ value
> >back, but it can not know if it is current state or not.
> >
> >I don't think that's a good interface. I see it is from 2008... is
> >someone using it? Maybe it is too late for revert.
>
> I can imagine it being used in flash LED use case. E.g. one
> could use oneshot trigger to trigger flash strobe, and then
> he could periodically read brightness file to check, for whatever
> reason, whether the flash is strobing.
I'm pretty sure nobody does that for flah strobe.
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: pavel@ucw.cz (Pavel Machek)
To: linux-arm-kernel@lists.infradead.org
Subject: PM regression with LED changes in next-20161109
Date: Fri, 11 Nov 2016 23:06:32 +0100 [thread overview]
Message-ID: <20161111220631.GA10983@amd> (raw)
In-Reply-To: <f7c9b2be-0003-f342-a4c7-802532faa424@gmail.com>
Hi!
> >Hmm. So userland can read the LED state, and it can get _some_ value
> >back, but it can not know if it is current state or not.
> >
> >I don't think that's a good interface. I see it is from 2008... is
> >someone using it? Maybe it is too late for revert.
>
> I can imagine it being used in flash LED use case. E.g. one
> could use oneshot trigger to trigger flash strobe, and then
> he could periodically read brightness file to check, for whatever
> reason, whether the flash is strobing.
I'm pretty sure nobody does that for flah strobe.
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20161111/e6116c19/attachment.sig>
next prev parent reply other threads:[~2016-11-11 22:06 UTC|newest]
Thread overview: 102+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-09 19:23 PM regression with LED changes in next-20161109 Tony Lindgren
2016-11-09 19:23 ` Tony Lindgren
2016-11-09 20:45 ` Jacek Anaszewski
2016-11-09 20:45 ` Jacek Anaszewski
2016-11-10 8:49 ` Hans de Goede
2016-11-10 8:49 ` Hans de Goede
2016-11-10 12:56 ` Jacek Anaszewski
2016-11-10 12:56 ` Jacek Anaszewski
2016-11-10 13:04 ` Hans de Goede
2016-11-10 13:04 ` Hans de Goede
2016-11-10 13:55 ` Jacek Anaszewski
2016-11-10 13:55 ` Jacek Anaszewski
2016-11-10 16:36 ` Pavel Machek
2016-11-10 16:36 ` Pavel Machek
2016-11-10 16:29 ` Pavel Machek
2016-11-10 16:29 ` Pavel Machek
2016-11-10 16:44 ` Hans de Goede
2016-11-10 16:44 ` Hans de Goede
2016-11-10 20:48 ` Pavel Machek
2016-11-10 20:48 ` Pavel Machek
2016-11-11 8:25 ` Hans de Goede
2016-11-11 8:25 ` Hans de Goede
2016-11-10 17:55 ` Tony Lindgren
2016-11-10 17:55 ` Tony Lindgren
2016-11-10 20:29 ` Pavel Machek
2016-11-10 20:29 ` Pavel Machek
2016-11-10 21:34 ` Jacek Anaszewski
2016-11-10 21:34 ` Jacek Anaszewski
2016-11-11 12:01 ` Pavel Machek
2016-11-11 12:01 ` Pavel Machek
2016-11-11 17:03 ` Jacek Anaszewski
2016-11-11 17:03 ` Jacek Anaszewski
2016-11-11 19:28 ` Hans de Goede
2016-11-11 19:28 ` Hans de Goede
2016-11-11 22:12 ` Pavel Machek
2016-11-11 22:12 ` Pavel Machek
2016-11-12 8:03 ` Hans de Goede
2016-11-12 8:03 ` Hans de Goede
2016-11-13 9:10 ` Three different LED brightnesses (was Re: PM regression with LED changes in next-20161109) Pavel Machek
2016-11-13 9:10 ` Pavel Machek
2016-11-13 9:44 ` Hans de Goede
2016-11-13 9:44 ` Hans de Goede
2016-11-13 20:45 ` Pavel Machek
2016-11-13 20:45 ` Pavel Machek
2016-11-12 10:24 ` PM regression with LED changes in next-20161109 Jacek Anaszewski
2016-11-12 10:24 ` Jacek Anaszewski
2016-11-12 10:33 ` Hans de Goede
2016-11-12 10:33 ` Hans de Goede
2016-11-12 10:33 ` Hans de Goede
2016-11-12 19:14 ` Jacek Anaszewski
2016-11-12 19:14 ` Jacek Anaszewski
2016-11-12 21:14 ` Hans de Goede
2016-11-12 21:14 ` Hans de Goede
2016-11-13 11:44 ` Jacek Anaszewski
2016-11-13 11:44 ` Jacek Anaszewski
2016-11-13 13:52 ` Hans de Goede
2016-11-13 13:52 ` Hans de Goede
2016-11-14 9:12 ` Jacek Anaszewski
2016-11-14 9:12 ` Jacek Anaszewski
2016-11-14 9:12 ` Jacek Anaszewski
2016-11-14 12:51 ` Hans de Goede
2016-11-14 12:51 ` Hans de Goede
2016-11-15 10:01 ` Jacek Anaszewski
2016-11-15 10:01 ` Jacek Anaszewski
2016-11-15 10:09 ` Hans de Goede
2016-11-15 10:09 ` Hans de Goede
2016-11-15 10:31 ` LEDs that change brightness "itself" -- that's a trigger. " Pavel Machek
2016-11-15 10:31 ` Pavel Machek
2016-11-15 10:58 ` Jacek Anaszewski
2016-11-15 10:58 ` Jacek Anaszewski
2016-11-15 11:11 ` Pavel Machek
2016-11-15 11:11 ` Pavel Machek
2016-11-15 11:21 ` Hans de Goede
2016-11-15 11:21 ` Hans de Goede
2016-11-15 11:48 ` Pavel Machek
2016-11-15 11:48 ` Pavel Machek
2016-11-15 12:06 ` Hans de Goede
2016-11-15 12:06 ` Hans de Goede
2016-11-15 12:11 ` Pavel Machek
2016-11-15 12:11 ` Pavel Machek
2016-11-15 13:28 ` Jacek Anaszewski
2016-11-15 13:28 ` Jacek Anaszewski
2016-11-15 13:48 ` Hans de Goede
2016-11-15 13:48 ` Hans de Goede
2016-11-15 14:04 ` Jacek Anaszewski
2016-11-15 14:04 ` Jacek Anaszewski
2016-11-15 14:30 ` Hans de Goede
2016-11-15 14:30 ` Hans de Goede
2016-11-15 14:41 ` Jacek Anaszewski
2016-11-15 14:41 ` Jacek Anaszewski
2016-11-17 22:12 ` Hans de Goede
2016-11-17 22:12 ` Hans de Goede
2016-11-15 11:17 ` Hans de Goede
2016-11-15 11:17 ` Hans de Goede
2016-11-14 8:31 ` Pavel Machek
2016-11-14 8:31 ` Pavel Machek
2016-11-11 22:06 ` Pavel Machek [this message]
2016-11-11 22:06 ` Pavel Machek
2016-11-10 8:34 ` Hans de Goede
2016-11-10 8:34 ` Hans de Goede
2016-11-10 15:11 ` Tony Lindgren
2016-11-10 15:11 ` Tony Lindgren
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=20161111220631.GA10983@amd \
--to=pavel@ucw.cz \
--cc=dvhart@infradead.org \
--cc=hdegoede@redhat.com \
--cc=j.anaszewski@samsung.com \
--cc=jacek.anaszewski@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-leds@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=tony@atomide.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.