> On 09.01.2016, at 21:53, Arnd Bergmann <arnd@xxxxxxxx> wrote:
>
> On Saturday 09 January 2016 13:07:32 kernel@xxxxxxxxxxxxxxxx wrote:
>> +
>> + uart1: uart@7e215040 {
>> + compatible = "brcm,bcm2835-aux-uart";
>> + reg = <0x7e215040 0x40>;
>> + interrupts = <1 29>;
>> + clocks = <&aux BCM2835_AUX_CLOCK_UART>;
>> + };
>>
>
> The node name should always be "serial@...", not "uart@..."
>
> Arnd
The existing dts arch/arm/boot/dts/bcm2835.dtsi already contains:
uart0: uart@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.
Thanks,
Martin
--
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
