From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759096AbcDAOHi (ORCPT ); Fri, 1 Apr 2016 10:07:38 -0400 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:37573 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751227AbcDAOHe (ORCPT ); Fri, 1 Apr 2016 10:07:34 -0400 Date: Fri, 1 Apr 2016 16:07:31 +0200 From: Pavel Machek To: Jacek Anaszewski Cc: Heiner Kallweit , pali.rohar@gmail.com, sre@kernel.org, khilman@kernel.org, aaro.koskinen@iki.fi, ivo.g.dimitrov.75@gmail.com, Patrik Bachan , serge@hallyn.com, Greg KH , linux-leds@vger.kernel.org, Benjamin Tissoires , Linux Kernel Mailing List , Linux USB Mailing List Subject: Re: [PATCH v5 1/4] leds: core: add generic support for RGB Color LED's Message-ID: <20160401140731.GE11860@amd> References: <56D608ED.2090406@gmail.com> <20160329100258.GA24964@amd> <56FAE7A8.2070302@gmail.com> <20160329214323.GA10455@amd> <56FB893C.60203@samsung.com> <20160330130319.GB19994@amd> <20160401125540.GC11860@amd> <56FE779B.9020602@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <56FE779B.9020602@samsung.com> 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! > >>>pavel@duo:~$ ls -1 /sys/class/leds/ > >>>tpacpi:green:batt > >>>tpacpi:orange:batt > >>> > >>>This is physically 2 leds but hidden under one indicator, so you got > >>>"off", "green", "orange" and "green+orange". > >> > >>That's a good example. As long as you can recognize green+orange as > >>separate lights/colors > >>(w/o magnifying glass) I wouldn't call it "a LED with multiple colors" > >>but "multiple > >>LED devices". > > > >Well, that's how it is currently handled. But for the user, it looks > >as a LED with multiple colors. > > > >>In my use case we talk about RGB LEDs like the commonly used 5050 SMD RGB LEDs. > >>And it's not only about using a handful of discrete colors but about > >>displaying any arbitrary > >>color. > >>So far the kernel exposes the physical RGB LEDs as separate LEDs only > >>and I can't use > >>a trigger to e.g. set "magenta with 50% brightness". > > > >Why not? > > > >What do you do if you want to display magenta on your LCD? > > > >You compute RGB values, then display them. > > The main drawback is that you can't set the colour at one go, > but have to set brightness of each LED class device (R,G,B) > separately. It incurs delays between setting each colour component. Yeah. Well, on some hardware, that's just the way it is. If the leds are separate devices on i2c, you can't really set them in one go. But some hardware has hardware pattern controls, and it can set them atomically. > It is also impossible to set arbitrary colour from a trigger. > Similarly blinking with arbitrarily chosen colour from RGB palette > is impossible if separate colour components are treated as > separate LEDs. Yes, see the proposal in the other mail. I believe we should have separate R, G, B LED devices, and separate pattern controller. Best regards, Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html