Bug#877074: u-boot-exynos: Default environment for Odroid XU3 has wrong console var
authorPeter Lebbing <peter@digitalbrains.com>
Thu, 28 Sep 2017 12:40:24 +0000 (14:40 +0200)
committerVagrant Cascadian <vagrant@debian.org>
Tue, 5 Dec 2017 23:43:23 +0000 (23:43 +0000)
commit2da7ad62ca514b570d26ab6345e466504c7c6a72
tree03c59a8e1eb52c3246d8dd27557054c8c44be4a9
parent3a1c099092e78cc73a121b6ec7df403ce7e2fdcb
Bug#877074: u-boot-exynos: Default environment for Odroid XU3 has wrong console var

Package: u-boot-exynos
Version: 2017.09+dfsg1-1
Severity: normal
Tags: patch

The odroid-xu3 platform passes a wrong "console" argument on the kernel commandline:

console=console=ttySAC2,115200n8

>From the source, it's clear where this comes from:
include/configs/odroid_xu3.h:
#define CONFIG_DEFAULT_CONSOLE         "console=ttySAC2,115200n8\0"
#define CONFIG_EXTRA_ENV_SETTINGS \
[...]
        "console=" CONFIG_DEFAULT_CONSOLE \

/etc/flash-kernel/bootscript/bootscr.uboot-generic:
  setenv bootargs "${bootargs} console=${console}"

The patch below makes the situation equal to what Debian patches in odroid.h: drop it from CONFIG_DEFAULT_CONSOLE. Judging from a grep in include/configs, I think the same situation exists for s5pc210_universal.h, s5p_goni.h, trats2.h and trats.h, but I don't have these devices. There are many more header files that have a "console=" in their CONFIG_DEFAULT_CONSOLE, but I don't see that definition being subsequently used somewhere, so I don't know whether it's being used wrongly or not.

Thanks for your efforts,

Peter.

*** src/debian/u-boot-2017.09+dfsg1/debian/patches/odroid-xu3/double-console-string

Gbp-Pq: Topic odroid-xu3
Gbp-Pq: Name 877074-u-boot-exynos-Default-environment-for-Odr.patch
include/configs/odroid_xu3.h