Linux-Devicetree Archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: vf-colibri: define stdout-path property
@ 2015-07-03  8:06 Stefan Agner
  2015-07-13 15:10 ` Shawn Guo
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Agner @ 2015-07-03  8:06 UTC (permalink / raw
  To: shawn.guo, kernel
  Cc: devicetree, linux-arm-kernel, linux-kernel, Stefan Agner

Define Vybrid's UART0, connected to the Colibri pinout UART_A, as
standard output.

Signed-off-by: Stefan Agner <stefan@agner.ch>
---
 arch/arm/boot/dts/vf-colibri-eval-v3.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi b/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
index 2cbe663..cb199ae 100644
--- a/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
+++ b/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
@@ -10,6 +10,7 @@
 / {
 	chosen {
 		bootargs = "console=ttyLP0,115200";
+		stdout-path = "serial0:115200n8";
 	};
 
 	clk16m: clk16m {
-- 
2.4.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] ARM: dts: vf-colibri: define stdout-path property
  2015-07-03  8:06 [PATCH] ARM: dts: vf-colibri: define stdout-path property Stefan Agner
@ 2015-07-13 15:10 ` Shawn Guo
  2015-07-15 14:05   ` Stefan Agner
  0 siblings, 1 reply; 4+ messages in thread
From: Shawn Guo @ 2015-07-13 15:10 UTC (permalink / raw
  To: Stefan Agner
  Cc: shawn.guo, kernel, devicetree, linux-arm-kernel, linux-kernel

On Fri, Jul 03, 2015 at 10:06:39AM +0200, Stefan Agner wrote:
> Define Vybrid's UART0, connected to the Colibri pinout UART_A, as
> standard output.
> 
> Signed-off-by: Stefan Agner <stefan@agner.ch>
> ---
>  arch/arm/boot/dts/vf-colibri-eval-v3.dtsi | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi b/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
> index 2cbe663..cb199ae 100644
> --- a/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
> +++ b/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
> @@ -10,6 +10,7 @@
>  / {
>  	chosen {
>  		bootargs = "console=ttyLP0,115200";
> +		stdout-path = "serial0:115200n8";

With this change, can bootargs just be dropped?

Shawn

>  	};
>  
>  	clk16m: clk16m {
> -- 
> 2.4.4
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ARM: dts: vf-colibri: define stdout-path property
  2015-07-13 15:10 ` Shawn Guo
@ 2015-07-15 14:05   ` Stefan Agner
  2015-07-17  2:55     ` Shawn Guo
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Agner @ 2015-07-15 14:05 UTC (permalink / raw
  To: Shawn Guo
  Cc: shawn.guo-QSEj5FYQhm4dnm+yROfE0A, kernel-bIcnvbaLZ9MEGnE8C9+IrQ,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA

On 2015-07-13 17:10, Shawn Guo wrote:
> On Fri, Jul 03, 2015 at 10:06:39AM +0200, Stefan Agner wrote:
>> Define Vybrid's UART0, connected to the Colibri pinout UART_A, as
>> standard output.
>>
>> Signed-off-by: Stefan Agner <stefan-XLVq0VzYD2Y@public.gmane.org>
>> ---
>>  arch/arm/boot/dts/vf-colibri-eval-v3.dtsi | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi b/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
>> index 2cbe663..cb199ae 100644
>> --- a/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
>> +++ b/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
>> @@ -10,6 +10,7 @@
>>  / {
>>  	chosen {
>>  		bootargs = "console=ttyLP0,115200";
>> +		stdout-path = "serial0:115200n8";
> 
> With this change, can bootargs just be dropped?

I guess it would break a (fairly old) kernel which does not support
stdout-path yet?

However, the newer Vybrid DT's anyway do not work on old kernels due to
the interrupt hierarchy changes...

Will send a v2 without default bootargs.

--
Stefan
--
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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ARM: dts: vf-colibri: define stdout-path property
  2015-07-15 14:05   ` Stefan Agner
@ 2015-07-17  2:55     ` Shawn Guo
  0 siblings, 0 replies; 4+ messages in thread
From: Shawn Guo @ 2015-07-17  2:55 UTC (permalink / raw
  To: Stefan Agner
  Cc: devicetree, shawn.guo, linux-arm-kernel, kernel, linux-kernel

On Wed, Jul 15, 2015 at 04:05:50PM +0200, Stefan Agner wrote:
> On 2015-07-13 17:10, Shawn Guo wrote:
> > On Fri, Jul 03, 2015 at 10:06:39AM +0200, Stefan Agner wrote:
> >> Define Vybrid's UART0, connected to the Colibri pinout UART_A, as
> >> standard output.
> >>
> >> Signed-off-by: Stefan Agner <stefan@agner.ch>
> >> ---
> >>  arch/arm/boot/dts/vf-colibri-eval-v3.dtsi | 1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >> diff --git a/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi b/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
> >> index 2cbe663..cb199ae 100644
> >> --- a/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
> >> +++ b/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
> >> @@ -10,6 +10,7 @@
> >>  / {
> >>  	chosen {
> >>  		bootargs = "console=ttyLP0,115200";
> >> +		stdout-path = "serial0:115200n8";
> > 
> > With this change, can bootargs just be dropped?
> 
> I guess it would break a (fairly old) kernel which does not support
> stdout-path yet?

The DT ABI requires that the new kernel doesn't break anything
with old DTB, but doesn't require the compatibility between new DTB
and old kernel.

Shawn

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-07-17  2:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-03  8:06 [PATCH] ARM: dts: vf-colibri: define stdout-path property Stefan Agner
2015-07-13 15:10 ` Shawn Guo
2015-07-15 14:05   ` Stefan Agner
2015-07-17  2:55     ` Shawn Guo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).