From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756981AbbGQG7x (ORCPT ); Fri, 17 Jul 2015 02:59:53 -0400 Received: from lists.s-osg.org ([54.187.51.154]:53350 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756866AbbGQG7v (ORCPT ); Fri, 17 Jul 2015 02:59:51 -0400 Subject: Re: [PATCH v2 4/4] mfd: max77686: Split out regulator part from the DT binding To: Krzysztof Kozlowski References: <1437114567-17629-1-git-send-email-javier@osg.samsung.com> <1437114567-17629-5-git-send-email-javier@osg.samsung.com> <55A8A4BE.1030904@osg.samsung.com> Cc: devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org, Sergei Shtylyov , linux-kernel@vger.kernel.org, Mark Brown , Lee Jones , linux-arm-kernel@lists.infradead.org From: Javier Martinez Canillas Message-ID: <55A8A7E2.7020201@osg.samsung.com> Date: Fri, 17 Jul 2015 08:59:46 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.0.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Krzysztof, On 07/17/2015 08:49 AM, Krzysztof Kozlowski wrote: > 2015-07-17 15:46 GMT+09:00 Javier Martinez Canillas : >> Hello Krzysztof, >> >> On 07/17/2015 08:42 AM, Krzysztof Kozlowski wrote: >>> 2015-07-17 15:29 GMT+09:00 Javier Martinez Canillas : >>>> The Maxim MAX77686 PMIC is a multi-function device with regulators, >>>> clocks and a RTC. The DT bindings for the clocks are in a separate >>>> file but the bindings for the regulators are inside the mfd part. >>>> >>>> To make it consistent with the clocks portion of the binding and >>>> because is more natural to look for regulator bindings under the >>>> bindings/regulator sub-directory, split the regulator portion of >>>> the DT binding and add it as a separate file. >>>> >>>> Signed-off-by: Javier Martinez Canillas >>>> >>>> --- >>>> >>>> Changes in v2: >>>> - Use a generic name for the max77686 node in the regulator example. >>>> >>>> Documentation/devicetree/bindings/mfd/max77686.txt | 58 +----------------- >>>> .../devicetree/bindings/regulator/max77686.txt | 71 ++++++++++++++++++++++ >>>> 2 files changed, 74 insertions(+), 55 deletions(-) >>>> create mode 100644 Documentation/devicetree/bindings/regulator/max77686.txt >>> >>> Putting regulator bindings under regulator directory seems logical so >>> I am fine with it. >>> >>> Reviewed-by: Krzysztof Kozlowski >>> >> >> Thanks a lot for your review. For completeness, I talked with Lee Jones >> over IRC yesterday asking him his opinion on whether MFD DT binding doc >> should contain all the info or if it should be split across different >> subsystems and he prefers the later. That's why I re-spun this series. > > Okay, this sounds good. > > One more thing - can you update the maintainers file? The bindings > documentation is mentioned there for max77686. > Sure, I see that also the max77802 files are missing in that section. I'll included that as well. But I'll wait to see if Lee pick this series before posting the patch for MAINTAINERS. > Best regards, > Krzysztof > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America