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

bootdelay

Use the default bootdelay from distro_bootcmd.

Gbp-Pq: Topic odroid-xu3
Gbp-Pq: Name bootdelay

Convert odroid to use distro_bootcmd.

Gbp-Pq: Topic odroid
Gbp-Pq: Name 0001-Convert-odroid-to-use-distro_bootcmd.patch

omap5_distro_bootcmd

===================================================================

Gbp-Pq: Topic am57xx
Gbp-Pq: Name omap5_distro_bootcmd

i4x4-ram

Gbp-Pq: Topic mx6cuboxi
Gbp-Pq: Name i4x4-ram.patch

ensure-config-sandbox-for-make-env

Ensure that CONFIG_SANDBOX is set when running "make env", avoiding a
failure to build caused by config_distro_bootcmd.h following the wrong
codepath...

Gbp-Pq: Name ensure-config-sandbox-for-make-env.patch

sh4-fix-linker-name-prefix

Gbp-Pq: Name sh4-fix-linker-name-prefix.patch

board-spl-rule

Gbp-Pq: Topic arndale
Gbp-Pq: Name board-spl-rule.diff

n900-bootz-raw-initrd

Enable booting of zImage/vmlinuz and initrd without requiring the use of
mkimage to create uImage/uInitrd.

Gbp-Pq: Name n900-bootz-raw-initrd.diff

mx53loco

Enables support for ext4, the "load" command, and using bootz with raw initrds.

Gbp-Pq: Name mx53loco

udoo-standardized-variables

Adds standardized variables, fdtfile, fdt_addr_r, ramdisk_addr_r and
kernel_addr_r.

Gbp-Pq: Topic udoo
Gbp-Pq: Name udoo-standardized-variables

udoo_quad-support

Gbp-Pq: Topic udoo
Gbp-Pq: Name udoo_quad-support.patch

hurd

===================================================================

Gbp-Pq: Name hurd.diff

Makefile-add-kwb-target-to-all

Add u-boot.kwb to "make all" target on Kirkwood.

Gbp-Pq: Name Makefile-add-kwb-target-to-all.patch

tools-generic-builds

Gbp-Pq: Name tools-generic-builds.patch

no-force-CROSS_COMPILE-powerpc

===================================================================

Gbp-Pq: Name no-force-CROSS_COMPILE-powerpc.diff

mipsel-native-endianness

[MIPS] Fix little-endian build with non-ELDK toolchains

We've been in trouble for a long time when cross compiling with non-ELDK
toolchains.  This is caused by -EB passed to CPPFLAGS incorrectly, by the
lack of an endian specifier to LDFLAGS, and by wrong OUTPUT_FORMATs.

We're going to implement two workarounds.  One is the endianness specifier
bugfix not to pass -EB / -EL to CPPFLAGS unless ELDK toolchain is used.
Note that ELDK and non-ELDK toolchains know their default endianness, so
the endianness specifier may not be necessary in principle.

The other is removal of OUTPUT_FORMAT in *.lds files.  If we have this,
it doesn't work unless an endianness specifier is added to LDFLAGS.  As
we haven't added that to LDFLAGS so far, it must have not worked properly,
except ELDK; I don't know why and how ELDK works, though.

With these two changes, all objects will be generated and linked in the
toolchain's default endianness.  Then MAKEALL mips_el will work even with
non-ELDK toolchain.

Note that Linux/MIPS kernel has CONFIG_CPU_BIG_ENDIAN and
CONFIG_CPU_LITTLE_ENDIAN alternatives to allow users to compile kernels
with a toolchain for the other endianness. But U-Boot does not have such
feature for now, and it's another story.

Signed-off-by: Shinya Kuribayashi <skuribay@ruby.dti.ne.jp>
Gbp-Pq: Name mipsel-native-endianness.diff

add-debian-revision-to-u-boot-version

Add the debian revision to the U-boot version, which is displayed at
boot and can be helpful to determine which specific version is used.

Gbp-Pq: Name add-debian-revision-to-u-boot-version

u-boot (2017.11+dfsg1-2) unstable; urgency=medium

  [ Vagrant Cascadian ]
  * u-boot-tools: Fix broken FIT image generation by building tools-only
    target with an empty defconfig.
  * Run basic tests for mkimage/dumpimage.

  [ Marek Vasut ]
  * Backport DH iMX6 DDR configuration fix (Closes: #882123).

  [ Vagrant Cascadian ]
  * debian/control: Bump Standards-Version 4.1.2, no changes.
  * debian/rules: Use dpkg/architecture.mk instead of manually calling
    dpkg-architecture.

[dgit import unpatched u-boot 2017.11+dfsg1-2]

Import u-boot_2017.11+dfsg1-2.debian.tar.xz

[dgit import tarball u-boot 2017.11+dfsg1-2 u-boot_2017.11+dfsg1-2.debian.tar.xz]

Import u-boot_2017.11+dfsg1.orig.tar.xz

[dgit import orig u-boot_2017.11+dfsg1.orig.tar.xz]