From mboxrd@z Thu Jan 1 00:00:00 1970 From: Josh Cartwright Subject: Re: [PATCH 6/6] documentation: bindings: document PMIC8921/8058 RTC Date: Mon, 10 Mar 2014 12:05:26 -0500 Message-ID: <20140310170526.GK18529@joshc.qualcomm.com> References: <1394047776-13827-1-git-send-email-joshc@codeaurora.org> <1394047776-13827-7-git-send-email-joshc@codeaurora.org> <5317900F.5030803@codeaurora.org> <20140306000058.GE18529@joshc.qualcomm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-arm-msm-owner@vger.kernel.org To: Rob Herring Cc: Stephen Boyd , Alessandro Zummo , "linux-kernel@vger.kernel.org" , linux-arm-msm , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Rob Landley , "devicetree@vger.kernel.org" , "linux-doc@vger.kernel.org" List-Id: devicetree@vger.kernel.org Hey Rob- Thanks for the reply. On Mon, Mar 10, 2014 at 10:35:25AM -0500, Rob Herring wrote: > On Wed, Mar 5, 2014 at 6:00 PM, Josh Cartwright wrote: > > On Wed, Mar 05, 2014 at 12:58:55PM -0800, Stephen Boyd wrote: > >> On 03/05/14 11:29, Josh Cartwright wrote: > >> > +- interrupts: interrupt list for the RTC, must contain a single interrupt > >> > + specifier for the alarm interrupt > >> > +- interrupt-names: corresponding interrupt names for the interrupts listed in > >> > + the 'interrupts' property, must contain: > >> > + "alarm" - summary interrupt for PMIC peripherals > >> > >> optional interrupt-names? > > > > It isn't clear to me why these should be made optional, I hope Rob > > provides some clarification in the sdhci-msm thread. > > Because reg and interrupt names are relatively new and reluctantly > added by DT maintainers. Personally, I think it was a mistake and it > is simply Linux specific information leaking into the DT, but it did > make transition to DT easier. I don't necessarily buy the Linux-specific argument in general. If a devices' datasheet clearly gives names to register regions and interrupts, what about reflecting these names in the bindings is Linux-specific? Now, there are probably abuses of this, where the reg-names and interrupt-names are abused to ensure driver compatibility with devices described in board files, and only in that case will I agree is Linux-specific and should be strongly discouraged. > The requirement is still the ordering of reg and interrupts fields > must be defined and you cannot rely on the names to define the order. Should this requirement also exist for other -names properties? > It is quite pointless here since you only have 1 field. Indeed in the interrupt case it is worthless, as there is only one alarm interrupt. However for registers I do plan to extend this binding in the future to document a newer RTC which does split registers across multiple named address regions. Thanks again, Josh -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation