From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751762AbbCTVFx (ORCPT ); Fri, 20 Mar 2015 17:05:53 -0400 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:36385 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751380AbbCTVFw (ORCPT ); Fri, 20 Mar 2015 17:05:52 -0400 Date: Fri, 20 Mar 2015 22:05:49 +0100 From: Pavel Machek To: Bryan Wu Cc: =?iso-8859-1?Q?Rapha=EBl?= Teysseyre , Joe Xue , "rpurdie@rpsys.net" , Linux LED Subsystem , lkml Subject: Re: [PATCH v2] leds: Add arbitrary pattern trigger Message-ID: <20150320210549.GA10475@amd> References: <1426862522.48232.3.camel@localhost> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi! > > Add a new led trigger supporting arbitrary patterns. > > This is useful for embedded systems with no screen or network access. > > > > Export two sysfs attributes: "pattern", and "repeat". > > > > When "repeat"=-1, repeat the pattern indefinitely. Otherwise, > > repeat it the specified number of times. "pattern" specifies > > the pattern as comma separated couples of "brightness duration" > > values. See detailled documentation in patch. > > > > Thanks for pushing this. But as we discussed in previous patches > provided by Joe Xue, we won't merge this kind of complex trigger patch > into kernel. I think we need to think about a new LED API to user > space and make it easier to handle. So, how do you propose to control intelligent LED controllers, such as the one in N900? This pattern engine quite nicely matches the stuff hardware can do. And for a cell phone, it is quite important to be able to use the hardware LED pattern engine, to save power. (And so that the LED keeps working while machine is suspended -- important for Android phones). Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html