Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755992AbaLJKvA (ORCPT ); Wed, 10 Dec 2014 05:51:00 -0500 Received: from nblzone-211-213.nblnetworks.fi ([83.145.211.213]:36375 "EHLO hillosipuli.retiisi.org.uk" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1755542AbaLJKu6 (ORCPT ); Wed, 10 Dec 2014 05:50:58 -0500 Date: Wed, 10 Dec 2014 12:50:51 +0200 From: Sakari Ailus To: Jacek Anaszewski Cc: linux-leds@vger.kernel.org, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, kyungmin.park@samsung.com, b.zolnierkie@samsung.com, pavel@ucw.cz, cooloney@gmail.com, rpurdie@rpsys.net, s.nawrocki@samsung.com, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, Andrzej Hajda , Lee Jones , Chanwoo Choi , devicetree@vger.kernel.org Subject: Re: [PATCH/RFC v9 06/19] DT: Add documentation for the mfd Maxim max77693 Message-ID: <20141210105051.GN15559@valkosipuli.retiisi.org.uk> References: <1417622814-10845-1-git-send-email-j.anaszewski@samsung.com> <1417622814-10845-7-git-send-email-j.anaszewski@samsung.com> <20141204100706.GP14746@valkosipuli.retiisi.org.uk> <54804840.4030202@samsung.com> <54881A1F.2080607@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <54881A1F.2080607@samsung.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Jacek, On Wed, Dec 10, 2014 at 11:02:07AM +0100, Jacek Anaszewski wrote: > Hi Sakari, > > On 12/04/2014 12:40 PM, Jacek Anaszewski wrote: > > >On 12/04/2014 11:07 AM, Sakari Ailus wrote: > >>Hi Jacek, > >> > >>On Wed, Dec 03, 2014 at 05:06:41PM +0100, Jacek Anaszewski wrote: > >>>This patch adds device tree binding documentation for > >>>the flash cell of the Maxim max77693 multifunctional device. > >>> > >>>Signed-off-by: Jacek Anaszewski > >>>Signed-off-by: Andrzej Hajda > >>>Acked-by: Kyungmin Park > >>>Cc: Lee Jones > >>>Cc: Chanwoo Choi > >>>Cc: Bryan Wu > >>>Cc: Richard Purdie > >>>Cc: Rob Herring > >>>Cc: Pawel Moll > >>>Cc: Mark Rutland > >>>Cc: Ian Campbell > >>>Cc: Kumar Gala > >>>Cc: > >>>--- > >>> Documentation/devicetree/bindings/mfd/max77693.txt | 89 > >>>++++++++++++++++++++ > >>> 1 file changed, 89 insertions(+) > >>> > >>>diff --git a/Documentation/devicetree/bindings/mfd/max77693.txt > >>>b/Documentation/devicetree/bindings/mfd/max77693.txt > >>>index 01e9f30..25a6e78 100644 > >>>--- a/Documentation/devicetree/bindings/mfd/max77693.txt > >>>+++ b/Documentation/devicetree/bindings/mfd/max77693.txt > >>>@@ -41,7 +41,66 @@ Optional properties: > >>> To get more informations, please refer to documentaion. > >>> [*] refer Documentation/devicetree/bindings/pwm/pwm.txt > >>> > >>>+- led : the LED submodule device node > >>>+ > >>>+There are two led outputs available - fled1 and fled2. Each of them can > >>>+control a separate led or they can be connected together to double > >>>+the maximum current for a single connected led. One led is represented > >>>+by one child node. > >>>+ > >>>+Required properties: > >>>+- compatible : Must be "maxim,max77693-led". > >>>+ > >>>+Optional properties: > >>>+- maxim,fleds : Array of current outputs in order: fled1, fled2. > >>>+ Note: both current outputs can be connected to a single led > >>>+ Possible values: > >>>+ MAX77693_LED_FLED_UNUSED - the output is left disconnected, > >>>+ MAX77693_LED_FLED_USED - a diode is connected to the output. > >> > >>As you have a LED sub-nodes for each LED already, isn't this redundant? > > > >Well, it seems so :) > > I agreed here recklessly. This property allows to describe the If this is reckless then we're doing very, very well. :-D > situation when one LED is connected to both outputs. Single sub-node > can describe two type of designs: one LED connected to a single > output or one LED connected to both outputs. Therefore additional > property is needed to assess what is the actual case. Which output do you say such LED is connected then? I wonder if the reg property could be made an array, so you could say the LED is connected to this and that output. The advantage would be that this still works even if you have three outputs. -- Regards, Sakari Ailus e-mail: sakari.ailus@iki.fi XMPP: sailus@retiisi.org.uk -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/