From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751631AbbALIcK (ORCPT ); Mon, 12 Jan 2015 03:32:10 -0500 Received: from mailout3.w1.samsung.com ([210.118.77.13]:24273 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750991AbbALIcG (ORCPT ); Mon, 12 Jan 2015 03:32:06 -0500 X-AuditID: cbfec7f4-b7f126d000001e9a-a9-54b386840377 Message-id: <54B38682.5080605@samsung.com> Date: Mon, 12 Jan 2015 09:32:02 +0100 From: Jacek Anaszewski User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130804 Thunderbird/17.0.8 MIME-version: 1.0 To: Rob Herring Cc: linux-leds@vger.kernel.org, "linux-media@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , Kyungmin Park , Bartlomiej Zolnierkiewicz , Pavel Machek , Bryan Wu , Richard Purdie , sakari.ailus@iki.fi, Sylwester Nawrocki , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala Subject: Re: [PATCH/RFC v10 03/19] DT: leds: Add led-sources property References: <1420816989-1808-1-git-send-email-j.anaszewski@samsung.com> <1420816989-1808-4-git-send-email-j.anaszewski@samsung.com> In-reply-to: Content-type: text/plain; charset=UTF-8; format=flowed Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFupmkeLIzCtJLcpLzFFi42I5/e/4Vd2Wts0hBmtmsVhsnLGe1eLozolM FvOPnGO16H+zkNXi3KuVjBZnm96wW1zeNYfNYuubdYwWPRu2slosvX6RyeLuqaNsFhOmr2Wx aN17hN3i+7dvbBa7dz1ltTj8pp3V4sz+lWwOgh5r5q1h9Ljc18vksXPWXXaPlcu/sHkc/rqQ xWPTqk42jz3zf7B69G1ZxeixYvV3do/Pm+QCuKK4bFJSczLLUov07RK4MloOvWQu6BetWLm8 g62B8ZpAFyMnh4SAicT7V/cYIWwxiQv31rN1MXJxCAksZZS4/u8oI4TzkVGicUobK0gVr4CW xOpHs9hAbBYBVYmlvR/B4mwChhI/X7xmArFFBSIk/pzeB1UvKPFj8j0WEFtEQF3i2/LjYBuY BZaySpx/PYMdJCEs4Coxu30xM8S2M4wS73d3g03iFAiWWL7wNJjNLGAm8ahlHTOELS+xec1b 5gmMArOQLJmFpGwWkrIFjMyrGEVTS5MLipPScw31ihNzi0vz0vWS83M3MUIi8MsOxsXHrA4x CnAwKvHwTpDZHCLEmlhWXJl7iFGCg1lJhNe1DCjEm5JYWZValB9fVJqTWnyIkYmDU6qBUfbc 47Dw931i1astbt4R7eKMCmRYrDHla7LFrvzZjConp2XufTV/fcU+z18v/1+4LHDN9A6vSof1 3uhzxeVeKy6fqJrbrLNZ+LnL49vhlnnraubXSv+Y5hj39TPPUQ7DF9WcyluutTl/m3qo7VPr Oa6b2Vd69+aI/7Z0vSl1jPm67flVdqkbfimxFGckGmoxFxUnAgAVCc7LngIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/09/2015 07:33 PM, Rob Herring wrote: > On Fri, Jan 9, 2015 at 9:22 AM, Jacek Anaszewski > wrote: >> Add a property for defining the device outputs the LED >> represented by the DT child node is connected to. >> >> Signed-off-by: Jacek Anaszewski >> Acked-by: Kyungmin Park >> Cc: Bryan Wu >> Cc: Richard Purdie >> Cc: Rob Herring >> Cc: Pawel Moll >> Cc: Mark Rutland >> Cc: Ian Campbell >> Cc: Kumar Gala >> --- >> Documentation/devicetree/bindings/leds/common.txt | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/leds/common.txt b/Documentation/devicetree/bindings/leds/common.txt >> index a2c3f7a..29295bf 100644 >> --- a/Documentation/devicetree/bindings/leds/common.txt >> +++ b/Documentation/devicetree/bindings/leds/common.txt >> @@ -1,6 +1,10 @@ >> Common leds properties. >> >> Optional properties for child nodes: >> +- led-sources : Array of bits signifying the LED current regulator outputs the >> + LED represented by the child node is connected to (1 - the LED >> + is connected to the output, 0 - the LED isn't connected to the >> + output). > > Sorry, I just don't understand this. In some Flash LED devices one LED can be connected to one or more electric current outputs, which allows for multiplying the maximum current allowed for the LED. Each sub-LED is represented by a child node in the DT binding of the Flash LED device and it needs to declare which outputs it is connected to. In the example below the led-sources property is a two element array, which means that the flash LED device has two current outputs, and the bits signify if the LED is connected to the output. Do your doubts stem from the ambiguity of the word "current" or the form of the description itself is unclear? Probably there should be explicit explanation added that the size of the array depends on the number of current outputs of the flash LED device. >> - label : The label for this LED. If omitted, the label is >> taken from the node name (excluding the unit address). >> >> @@ -33,6 +37,7 @@ system-status { >> >> camera-flash { >> label = "Flash"; >> + led-sources = <1 0>; >> max-microamp = <50000>; >> flash-max-microamp = <320000>; >> flash-timeout-us = <500000>; >> -- >> 1.7.9.5 >> > -- Best Regards, Jacek Anaszewski