On 2016/5/14 0:49, Jon Mason wrote: > > > On Thu, May 12, 2016 at 10:09 PM, Kefeng Wang <wangkefeng.wang@xxxxxxxxxx <mailto:wangkefeng.wang@xxxxxxxxxx>> wrote: > > > > On 2016/5/12 22:46, Jon Mason wrote: > > > > > > On Thu, May 12, 2016 at 2:16 AM, Kefeng Wang <wangkefeng.wang@xxxxxxxxxx <mailto:wangkefeng.wang@xxxxxxxxxx> <mailto:wangkefeng.wang@xxxxxxxxxx <mailto:wangkefeng.wang@xxxxxxxxxx>>> wrote: > > > > > > > > On 2016/5/12 6:56, Jon Mason wrote: > > > Add all of the UARTs present on NS2 and enable them in the SVK device > > > tree file. Also, do some magic to make sure that uart3 is discovered as > > > ttyS0 (as that is the console UART). > > > > > > Signed-off-by: Jon Mason <jonmason@xxxxxxxxxxxx <mailto:jonmason@xxxxxxxxxxxx> <mailto:jonmason@xxxxxxxxxxxx <mailto:jonmason@xxxxxxxxxxxx>>> > > > --- > > > arch/arm64/boot/dts/broadcom/ns2-svk.dts | 16 ++++++++++++++++ > > > arch/arm64/boot/dts/broadcom/ns2.dtsi | 30 ++++++++++++++++++++++++++++++ > > > 2 files changed, 46 insertions(+) > > > > > > diff --git a/arch/arm64/boot/dts/broadcom/ns2-svk.dts b/arch/arm64/boot/dts/broadcom/ns2-svk.dts > > > index 7cd3640..b062a44 100644 > > > --- a/arch/arm64/boot/dts/broadcom/ns2-svk.dts > > > +++ b/arch/arm64/boot/dts/broadcom/ns2-svk.dts > > > @@ -40,10 +40,14 @@ > > > > > > aliases { > > > serial0 = &uart3; > > > + serial1 = &uart0; > > > + serial2 = &uart1; > > > + serial3 = &uart2; > > > }; > > > > > > chosen { > > > stdout-path = "serial0:115200n8"; > > > + bootargs = "earlycon=uart8250,mmio32,0x66130000"; > > > > Hi Jon, > > > > I submit a patch[1], with it, we can use earlycon without option to enable early console > > for "snps,dw-apb-uart", could you help me to test it, thanks. > > > > > > Adding your change and removing the line above does not cause earlycon to work for me. Is there any additional changes necessary for this to work? > With my patch, the bootargs still need contain earlycon, but no need uart8250,mmio32,0x66130000; > > chosen { > stdout-path = "serial0:115200n8"; > bootargs = "earlycon"; > } > > Hope it would work. > > > I can confirm that with your change and the change to the bootargs you describe above, it works as desired. Was your change already accepted? > Great, thanks a lot. it is still being reviewing for now and waiting for response for now. BRs, Kefeng > Thanks, > Jon > > > > BRs, > Kefeng > > > > > Thanks, > > Jon > > > > > > > > BRs, > > Kefeng > > > > > > [1] https://lkml.org/lkml/2016/5/11/34 > > > > > }; > > > > > > memory { > > > @@ -68,6 +72,18 @@ > > > status = "ok"; > > > }; > > > > > > +&uart0 { > > > + status = "ok"; > > > +}; > > > + > > > +&uart1 { > > > + status = "ok"; > > > +}; > > > + > > > +&uart2 { > > > + status = "ok"; > > > +}; > > > + > > > &uart3 { > > > status = "ok"; > > > }; > > > diff --git a/arch/arm64/boot/dts/broadcom/ns2.dtsi b/arch/arm64/boot/dts/broadcom/ns2.dtsi > > > index 788ed8f..c77a9e8 100644 > > > --- a/arch/arm64/boot/dts/broadcom/ns2.dtsi > > > +++ b/arch/arm64/boot/dts/broadcom/ns2.dtsi > > > @@ -357,6 +357,36 @@ > > > status = "disabled"; > > > }; > > > > > > + uart0: serial@66100000 { > > > + compatible = "snps,dw-apb-uart"; > > > + reg = <0x66100000 0x100>; > > > + interrupts = <GIC_SPI 390 IRQ_TYPE_LEVEL_HIGH>; > > > + clocks = <&iprocslow>; > > > + reg-shift = <2>; > > > + reg-io-width = <4>; > > > + status = "disabled"; > > > + }; > > > + > > > + uart1: serial@66110000 { > > > + compatible = "snps,dw-apb-uart"; > > > + reg = <0x66110000 0x100>; > > > + interrupts = <GIC_SPI 391 IRQ_TYPE_LEVEL_HIGH>; > > > + clocks = <&iprocslow>; > > > + reg-shift = <2>; > > > + reg-io-width = <4>; > > > + status = "disabled"; > > > + }; > > > + > > > + uart2: serial@66120000 { > > > + compatible = "snps,dw-apb-uart"; > > > + reg = <0x66120000 0x100>; > > > + interrupts = <GIC_SPI 392 IRQ_TYPE_LEVEL_HIGH>; > > > + clocks = <&iprocslow>; > > > + reg-shift = <2>; > > > + reg-io-width = <4>; > > > + status = "disabled"; > > > + }; > > > + > > > uart3: serial@66130000 { > > > compatible = "snps,dw-apb-uart"; > > > reg = <0x66130000 0x100>; > > > > > > > > > -- 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