Re: [PATCHv2 05/15] ARM: mvebu: add stdout-path to all armada-*.dts

2015-03-03 Thread Thomas Petazzoni
Dear Rob Herring, On Fri, 27 Feb 2015 11:42:43 -0600, Rob Herring wrote: It is worth mentioning that this also happens with stdout-path = uart0:115200n8; which was explicitly suggested by Mark in his e-mail. uart0 is an alias to the proper DT node, since stdout-path = uart0;

Re: [PATCHv2 05/15] ARM: mvebu: add stdout-path to all armada-*.dts

2015-02-28 Thread Thomas Petazzoni
Dear Rob Herring, On Fri, 27 Feb 2015 11:42:43 -0600, Rob Herring wrote: Unfortunately, this doesn't seem to work at all: This must have all been tested on QEMU (or Fast Models?) which ignores baud rate. Ok. So it gets the right port name, the right idx, just the options are

Re: [PATCHv2 05/15] ARM: mvebu: add stdout-path to all armada-*.dts

2015-02-27 Thread Thomas Petazzoni
Rob, Mark, On Mon, 23 Feb 2015 10:50:26 -0600, Rob Herring wrote: On Mon, Feb 23, 2015 at 10:00 AM, Gregory CLEMENT Not exactly, stdout-path allows for removing console from the command line. earlyprintk is a debug/developer option, so it should not be in a default command line IMO. So

Re: [PATCHv2 05/15] ARM: mvebu: add stdout-path to all armada-*.dts

2015-02-27 Thread Rob Herring
On Fri, Feb 27, 2015 at 10:48 AM, Thomas Petazzoni thomas.petazz...@free-electrons.com wrote: Rob, Mark, On Mon, 23 Feb 2015 10:50:26 -0600, Rob Herring wrote: On Mon, Feb 23, 2015 at 10:00 AM, Gregory CLEMENT Not exactly, stdout-path allows for removing console from the command line.

Re: [PATCHv2 05/15] ARM: mvebu: add stdout-path to all armada-*.dts

2015-02-23 Thread Mark Rutland
On Fri, Feb 20, 2015 at 05:04:24PM +, Thomas Petazzoni wrote: This commit adds the stdout-path property in /chosen for all Armada boards that were not yet carrying this property. Signed-off-by: Thomas Petazzoni thomas.petazz...@free-electrons.com --- arch/arm/boot/dts/armada-370-db.dts

Re: [PATCHv2 05/15] ARM: mvebu: add stdout-path to all armada-*.dts

2015-02-23 Thread Gregory CLEMENT
Hi Thomas, On 20/02/2015 18:04, Thomas Petazzoni wrote: This commit adds the stdout-path property in /chosen for all Armada boards that were not yet carrying this property. I though the main motivation for using the stdout-path property was for removing the earlyprintk in the command line.

Re: [PATCHv2 05/15] ARM: mvebu: add stdout-path to all armada-*.dts

2015-02-23 Thread Thomas Petazzoni
Dear Rob Herring, On Mon, 23 Feb 2015 10:50:26 -0600, Rob Herring wrote: Not exactly, stdout-path allows for removing console from the command line. earlyprintk is a debug/developer option, so it should not be in a default command line IMO. So bootargs should be removed entirely. Ok, will

Re: [PATCHv2 05/15] ARM: mvebu: add stdout-path to all armada-*.dts

2015-02-23 Thread Rob Herring
On Mon, Feb 23, 2015 at 10:00 AM, Gregory CLEMENT gregory.clem...@free-electrons.com wrote: Hi Thomas, On 20/02/2015 18:04, Thomas Petazzoni wrote: This commit adds the stdout-path property in /chosen for all Armada boards that were not yet carrying this property. I though the main

[PATCHv2 05/15] ARM: mvebu: add stdout-path to all armada-*.dts

2015-02-20 Thread Thomas Petazzoni
This commit adds the stdout-path property in /chosen for all Armada boards that were not yet carrying this property. Signed-off-by: Thomas Petazzoni thomas.petazz...@free-electrons.com --- arch/arm/boot/dts/armada-370-db.dts | 1 + arch/arm/boot/dts/armada-370-mirabox.dts |