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>
Wed, 21 Feb 2018 00:49:33 +0000 (00:49 +0000)
commita948b2a9b25105b916fcfad78cbaa7e55cbfd960
tree118c9b62372d4de6a242120830f89a6c676a29d6
parente118b67fe39901b45d63316f11760de38fd87b19
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