[PATCH 4/5] arm: mvebu: Espressobin: Define console env

[Pali Rohár]

On Wednesday 26 August 2020 15:22:26 Andre Heider wrote:
> Hi,
> 
> On 24/08/2020 16:25, Pali Rohár wrote:
> > Some distribution bootscripts append console env into boot cmdline.
> > So define console env correctly to have early console access.
> > 
> > Signed-off-by: Pali Rohár <pali at kernel.org>
> > ---
> >   include/configs/mvebu_armada-37xx.h | 2 ++
> >   1 file changed, 2 insertions(+)
> > 
> > diff --git a/include/configs/mvebu_armada-37xx.h b/include/configs/mvebu_armada-37xx.h
> > index 6feceda757..b473072976 100644
> > --- a/include/configs/mvebu_armada-37xx.h
> > +++ b/include/configs/mvebu_armada-37xx.h
> > @@ -17,6 +17,7 @@
> >   #define CONFIG_SYS_BAUDRATE_TABLE	{ 9600, 19200, 38400, 57600, \
> >   					  115200, 230400, 460800, 921600 }
> > +#define CONFIG_DEFAULT_CONSOLE "console=ttyMV0,115200 earlycon=ar3700_uart,0xd0012000"
> >   /*
> >    * For booting Linux, the board info and command line data
> > @@ -99,6 +100,7 @@
> >   	"kernel_addr=0x7000000\0"	\
> >   	"kernel_addr_r=0x7000000\0"	\
> >   	"ramdisk_addr_r=0x8000000\0"	\
> > +	"console=" CONFIG_DEFAULT_CONSOLE "\0" \
> >   	BOOTENV
> >   #endif /* _CONFIG_MVEBU_ARMADA_37XX_H */
> > 
> 
> this breaks booting debian with systemd for me:
> [    4.632197] systemd-udevd[90]: Starting version 241
> [    4.639324] systemd-udevd[91]: Failed to create udev control event
> source: Operation not permitted
> 
> "env delete console" and it works again.

This is really suspicious. Why systemd has problems with control socket
when console= parameter is specified on boot command line? And if
console= parameter is problematic on command line, why console output is
working fine? For me it looks like this variable controls something
different, not (only) console output. Maybe some bootscript sets
something more when it see u-boot env ${console} non-empty?

> The device tree already has 'stdout-path = "serial0:115200n8";' btw.

I see. But it does not initialize earlycon.