From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V2 1/2] dt/bindings: serial: bcm2835: add binding documentation for bcm2835-aux-uart
Date: Mon, 11 Jan 2016 13:21:17 +0100 [thread overview]
Message-ID: <4288762.bCLxQBDvy4@wuerfel> (raw)
In-Reply-To: <175E89CE-581C-48AB-B7FF-6E9A41B033D5@martin.sperl.org>
On Sunday 10 January 2016 11:49:19 Martin Sperl wrote:
> The existing dts arch/arm/boot/dts/bcm2835.dtsi already contains:
> uart0: uart at 7e201000 {
> compatible = "brcm,bcm2835-pl011", "arm,pl011", "arm,primecell";
> reg = <0x7e201000 0x1000>;
> interrupts = <2 25>;
> clocks = <&clocks BCM2835_CLOCK_UART>,
> <&clocks BCM2835_CLOCK_VPU>;
> clock-names = "uartclk", "apb_pclk";
> arm,primecell-periphid = <0x00241011>;
> };
> for the primary uart.
>
> So I was just following that example and would hesitate to
> use a different nomenclature in the same file (even though I
> saw serial examples elsewhere).
>
> I guess this is a more general thing that needs to get resolved.
>
> If you tell me to re-spin the patch because of this then I can do that,
> but that would not solve the issue with uart0.
Please just add another patch to rename the existing uarts appropriately
first, then add this one on top with the correct name.
Arnd
next prev parent reply other threads:[~2016-01-11 12:21 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-09 13:07 [PATCH V2 0/2] serial: bcm2835: add bcm2835 auxiliar uart driver kernel at martin.sperl.org
2016-01-09 13:07 ` [PATCH V2 1/2] dt/bindings: serial: bcm2835: add binding documentation for bcm2835-aux-uart kernel at martin.sperl.org
2016-01-09 20:53 ` Arnd Bergmann
2016-01-10 10:49 ` Martin Sperl
2016-01-11 12:21 ` Arnd Bergmann [this message]
2016-01-11 13:57 ` Martin Sperl
2016-01-11 14:40 ` Arnd Bergmann
2016-01-28 6:39 ` Eric Anholt
2016-01-09 13:07 ` [PATCH V2 2/2] serial: bcm2835: add driver " kernel at martin.sperl.org
2016-01-11 9:52 ` Stefan Wahren
2016-01-11 15:25 ` Peter Korsgaard
2016-01-10 15:05 ` [PATCH V2 0/2] serial: bcm2835: add bcm2835 auxiliar uart driver Stefan Wahren
2016-01-10 15:45 ` Martin Sperl
2016-01-10 21:17 ` Stefan Wahren
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4288762.bCLxQBDvy4@wuerfel \
--to=arnd@arndb.de \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox