From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH 2/4 v3] dt/bindings: Add binding for BCM2835 mailbox driver Date: Wed, 18 Mar 2015 09:15:40 +0000 Message-ID: <20150318091540.GK3318@x1> References: <1426213936-4139-1-git-send-email-eric@anholt.net> <1426213936-4139-2-git-send-email-eric@anholt.net> <20150317172758.GC3318@x1> <5508A75A.4070203@broadcom.com> <20150318082321.GH3318@x1> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jassi Brar Cc: Scott Branden , Eric Anholt , Devicetree List , linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-arm-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On Wed, 18 Mar 2015, Jassi Brar wrote: > On Wed, Mar 18, 2015 at 1:53 PM, Lee Jones wrote: > > On Tue, 17 Mar 2015, Scott Branden wrote: > >> On 15-03-17 10:27 AM, Lee Jones wrote: > >> >On Thu, 12 Mar 2015, Eric Anholt wrote: > >> > > >> >>From: Lubomir Rintel > >> >> > >> >>Signed-off-by: Lubomir Rintel > >> >>Signed-off-by: Craig McGeachie > >> >>Signed-off-by: Eric Anholt > >> >>--- > >> >> > >> >> > >> >>v2: Split into a separate patch for submitting to the devicetree list. > >> >> Consistently start node docs with a capital letter. device's > >> >> address in the example shouldn't have "0x". Drop machine-specific > >> >> interrupt numbers from the docs. (changes by anholt). > >> >> > >> >>v3: Move the file to just bcm2835-mbox.txt, clean up formatting > >> >> (changes by anholt, from review by Lee Jones). > >> > > >> >Thanks for fixing up. > >> > > >> >>.../devicetree/bindings/mailbox/bcm2835-mbox.txt | 19 +++++++++++++++++++ > >> >> 1 file changed, 19 insertions(+) > >> >> create mode 100644 Documentation/devicetree/bindings/mailbox/bcm2835-mbox.txt > >> > > >> >I don't see any unruliness or causes of controversy. > >> > > >> >Acked-by: Lee Jones > >> > > >> >>diff --git a/Documentation/devicetree/bindings/mailbox/bcm2835-mbox.txt b/Documentation/devicetree/bindings/mailbox/bcm2835-mbox.txt > >> >>new file mode 100644 > >> >>index 0000000..0bb2b9d > >> >>--- /dev/null > >> >>+++ b/Documentation/devicetree/bindings/mailbox/bcm2835-mbox.txt > >> I am trying to standarize all broadcom device tree bindings in the > >> format "brcm,binding.txt". > >> > >> Could you please rename this file to brcm,bcm2835-mbox.txt for > >> consistency in bindings? > > > > The file name is governed by the maintainer of the subsystem you're > > applying documentation for. If this were an MFD submission, I would > > not accept a the format matching the compatible string for instance. > > I like consistency and commas in file names creeps me out. > > > > This however, is Jassi's call. > > > I don't really have a strong opinion here. I tend to respect Broadcom > platforms' convention of using "brcm,binding.txt" I'm fine with this Eric. If you change it back my Ack can reside. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html