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>
Mon, 2 Apr 2018 01:20:06 +0000 (02:20 +0100)
commit9546b7edc013903b5a77034aadc1e1a1dceedaeb
tree3642ab5fb21d0c5b33c03c2f96a5b3ab5601dce1
parent0f97fe36afb440064351d9d457e4a73736869712
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.

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