On Wed, Mar 18, 2015 at 1:53 PM, Lee Jones <lee@xxxxxxxxxx> 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 <lkundrak@xxxxx> >> >> >> >>Signed-off-by: Lubomir Rintel <lkundrak@xxxxx> >> >>Signed-off-by: Craig McGeachie <slapdau@xxxxxxxxxxxx> >> >>Signed-off-by: Eric Anholt <eric@xxxxxxxxxx> >> >>--- >> >> >> >> >> >>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 <lee.jones@xxxxxxxxxx> >> > >> >>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" -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
