From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753118AbcGVHMK (ORCPT ); Fri, 22 Jul 2016 03:12:10 -0400 Received: from mailout4.w1.samsung.com ([210.118.77.14]:14334 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751726AbcGVHMI (ORCPT ); Fri, 22 Jul 2016 03:12:08 -0400 X-AuditID: cbfec7f4-f796c6d000001486-e5-5791c74301c7 Message-id: <5791C742.2070308@samsung.com> Date: Fri, 22 Jul 2016 09:12:02 +0200 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: "H. Nikolaus Schaller" Cc: Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Richard Purdie , drivshin.allworx@gmail.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-leds@vger.kernel.org, kernel@pyra-handheld.com, marek@goldelico.com, letux-kernel@openphoenux.org, Andrey Utkin Subject: Re: [PATCH v6 2/2] Bindings documentation for ISSI is31fl319x driver References: <403335d22f6a8e131bc44077f97e499970e61245.1468928850.git.hns@goldelico.com> <20160720012457.GA10971@rob-hp-laptop> <5791BEF3.5060404@samsung.com> <09C8D3C3-0CA9-48EB-B7B1-FCADAB2BA23A@goldelico.com> In-reply-to: <09C8D3C3-0CA9-48EB-B7B1-FCADAB2BA23A@goldelico.com> Content-type: text/plain; charset=ISO-8859-1; format=flowed Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrEIsWRmVeSWpSXmKPExsVy+t/xy7rOxyeGG/x+pWbx/9s1Fov5R86x WiyePIvdov/NQlaLH9u+Mlmce7WS0eLS1xqLrX8usVlc3jWHzWLrm3WMFv+WbmGzWHr9IpPF hOlrWSz+79nBbrF711NWB36PNfPWMHpc7utl8lj0cj67x85Zd9k91rw/xeyxcvkXNo9NqzrZ PFom7WL3+NLSzOyxZ/4PVo/Pm+QCuKO4bFJSczLLUov07RK4MpZ03WQuOMFbcbzjMmMD4wTu LkZODgkBE4lz3VPZIGwxiQv31gPZXBxCAksZJS48OcIC4TxjlJi3bxUrSBWvgJZE48HtYB0s AqoScw+cZASx2QQMJX6+eM0EYosKREj8Ob0Pql5Q4sfkeywgtoiAnkTn9z9gQ5kFLjNL3J60 FaxIWMBXYlfrV1aIbY1MEmdebweaysHBKeAoce2EIkgNs4C1xMpJ2xghbHmJzWveMk9gFJiF ZMcsJGWzkJQtYGRexSiaWppcUJyUnmuoV5yYW1yal66XnJ+7iRESc192MC4+ZnWIUYCDUYmH 98TTCeFCrIllxZW5hxglOJiVRHhZjk4MF+JNSaysSi3Kjy8qzUktPsQozcGiJM47d9f7ECGB 9MSS1OzU1ILUIpgsEwenVAOj9jyNzhQDCXU2vzUlz60L/ASaNDu4i7Y+lmwsqE12j6yXKGQ3 M7TQYFhtI706RPDjDOdF2gldcqZL9zqJJa1IqfBcaNXIXOxjwfHywuLnCUEzNl26KOzT/f3g VckZ5yI/N/843PDc4ZvO87mT73/21DncebUyTW+f55eMr0sdbh9/V8/9foMSS3FGoqEWc1Fx IgAj1rUVtQIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 07/22/2016 08:42 AM, H. Nikolaus Schaller wrote: > Hi, > >> Am 22.07.2016 um 08:36 schrieb Jacek Anaszewski : >> >> On 07/20/2016 03:24 AM, Rob Herring wrote: >>> On Tue, Jul 19, 2016 at 01:47:31PM +0200, H. Nikolaus Schaller wrote: >>>> Signed-off-by: H. Nikolaus Schaller >>>> --- >>>> .../devicetree/bindings/leds/is31fl319x.txt | 59 ++++++++++++++++++++++ >>>> 1 file changed, 59 insertions(+) >>>> create mode 100644 Documentation/devicetree/bindings/leds/is31fl319x.txt >>>> >>>> diff --git a/Documentation/devicetree/bindings/leds/is31fl319x.txt b/Documentation/devicetree/bindings/leds/is31fl319x.txt >>>> new file mode 100644 >>>> index 0000000..03287c0 >>>> --- /dev/null >>>> +++ b/Documentation/devicetree/bindings/leds/is31fl319x.txt >>>> @@ -0,0 +1,59 @@ >>>> +LEDs connected to is31fl319x LED controller chip >>>> + >>>> +Required properties: >>>> +- compatible : Should be any of >>>> + "issi,is31fl3190" >>>> + "issi,is31fl3191" >>>> + "issi,is31fl3193" >>>> + "issi,is31fl3196" >>>> + "issi,is31fl3199" >>>> + "si-en,sn3199". >>>> +- #address-cells: Must be 1. >>>> +- #size-cells: Must be 0. >>>> +- reg: 0x64, 0x65, 0x66, 0x67. >>> >>> This is an OR or AND? OR it seems. With that, >> >> Nikolaus, could you please make it explicit? > > Yes, Rob's intuition is right. An AND does > obviously not make sense for a reg property. > > It is an OR. > > So if you perfer, please substitute > > - reg: 0x64, 0x65, 0x66, or 0x67. Updated the for-4.9 branch accordingly. I also added "led-" prefix to the file name to match it with the vast majority of the remaining ones. -- Best regards, Jacek Anaszewski