[U-Boot] [PATCH] Revert "arm: config: fix default console only to specify the device"

Anand Moon linux.amoon at gmail.com
Mon Mar 25 06:45:23 UTC 2019


Hi Seung-Woo,

On Tue, 20 Nov 2018 at 11:25, Seung-Woo Kim <sw0312.kim at samsung.com> wrote:
>
> This reverts commit 232ed3ca534708527a9515c7c41bc3542949525c.
>
> In exynos boards, ${console} is used to set bootargs but it sets
> without "console=", so CONFIG_DEFAULT_CONSOLE for these boards is
> designated with "console=" but it is removed. So revert the commit.
>
> References for using ${console} in
>    board/samsung/common/bootscripts/autoboot.cmd
>    board/samsung/common/bootscripts/bootzimg.cmd
>
> Signed-off-by: Seung-Woo Kim <sw0312.kim at samsung.com>
> ---
>  include/configs/odroid.h            | 4 ++--
>  include/configs/odroid_xu3.h        | 4 ++--
>  include/configs/s5p_goni.h          | 4 ++--
>  include/configs/s5pc210_universal.h | 4 ++--
>  include/configs/trats.h             | 4 ++--
>  include/configs/trats2.h            | 4 ++--
>  6 files changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/include/configs/odroid.h b/include/configs/odroid.h
> index ad77242e38..c3520bb15f 100644
> --- a/include/configs/odroid.h
> +++ b/include/configs/odroid.h
> @@ -40,7 +40,7 @@
>  /* Console configuration */
>
>  #define CONFIG_BOOTCOMMAND             "run distro_bootcmd ; run autoboot"
> -#define CONFIG_DEFAULT_CONSOLE         "ttySAC1,115200n8"
> +#define CONFIG_DEFAULT_CONSOLE         "console=ttySAC1,115200n8\0"
>
>  #define CONFIG_SYS_INIT_SP_ADDR        (CONFIG_SYS_LOAD_ADDR \
>                                         - GENERATED_GBL_DATA_SIZE)
> @@ -157,7 +157,7 @@
>                 "elif test -e mmc 0 uImage; then; " \
>                         "run boot_uimg;" \
>                 "fi;\0" \
> -       "console=" CONFIG_DEFAULT_CONSOLE "\0" \
> +       "console=" CONFIG_DEFAULT_CONSOLE \
>         "mmcbootdev=0\0" \
>         "mmcbootpart=1\0" \
>         "mmcrootdev=0\0" \
> diff --git a/include/configs/odroid_xu3.h b/include/configs/odroid_xu3.h
> index f683ee46e3..3b9a945e7c 100644
> --- a/include/configs/odroid_xu3.h
> +++ b/include/configs/odroid_xu3.h
> @@ -30,7 +30,7 @@
>
>  #define CONFIG_SYS_INIT_SP_ADDR        (CONFIG_SYS_LOAD_ADDR - 0x1000000)
>
> -#define CONFIG_DEFAULT_CONSOLE         "ttySAC2,115200n8"
> +#define CONFIG_DEFAULT_CONSOLE         "console=ttySAC2,115200n8\0"
>
>  /* USB */
>  #define CONFIG_USB_EHCI_EXYNOS
> @@ -99,7 +99,7 @@
>         MEM_LAYOUT_ENV_SETTINGS \
>         BOOTENV \
>         "rootfstype=ext4\0" \
> -       "console=" CONFIG_DEFAULT_CONSOLE "\0"\
> +       "console=" CONFIG_DEFAULT_CONSOLE \
>         "fdtfile=exynos5422-odroidxu3.dtb\0" \
>         "boardname=odroidxu3\0" \
>         "mmcbootdev=0\0" \
> diff --git a/include/configs/s5p_goni.h b/include/configs/s5p_goni.h
> index 87ddc20a52..ff634d91dd 100644
> --- a/include/configs/s5p_goni.h
> +++ b/include/configs/s5p_goni.h
> @@ -87,7 +87,7 @@
>
>  #define CONFIG_BOOTCOMMAND     "run mmcboot"
>
> -#define CONFIG_DEFAULT_CONSOLE "ttySAC2,115200n8"
> +#define CONFIG_DEFAULT_CONSOLE "console=ttySAC2,115200n8\0"
>
>  #define CONFIG_RAMDISK_BOOT    "root=/dev/ram0 rw rootfstype=ext4" \
>                 " ${console} ${meminfo}"
> @@ -134,7 +134,7 @@
>         "bootchart=set opts init=/sbin/bootchartd; run bootcmd\0" \
>         "verify=n\0" \
>         "rootfstype=ext4\0" \
> -       "console=" CONFIG_DEFAULT_CONSOLE "\0"\
> +       "console=" CONFIG_DEFAULT_CONSOLE \
>         "meminfo=mem=80M mem=256M at 0x40000000 mem=128M at 0x50000000\0" \
>         "loaduimage=ext4load mmc ${mmcdev}:${mmcbootpart} 0x30007FC0 uImage\0" \
>         "mmcdev=0\0" \
> diff --git a/include/configs/s5pc210_universal.h b/include/configs/s5pc210_universal.h
> index 999bdd1676..832032da18 100644
> --- a/include/configs/s5pc210_universal.h
> +++ b/include/configs/s5pc210_universal.h
> @@ -27,7 +27,7 @@
>  /* Console configuration */
>
>  #define CONFIG_BOOTCOMMAND             "run mmcboot"
> -#define CONFIG_DEFAULT_CONSOLE         "ttySAC1,115200n8"
> +#define CONFIG_DEFAULT_CONSOLE         "console=ttySAC1,115200n8\0"
>
>  #define CONFIG_SYS_INIT_SP_ADDR        (CONFIG_SYS_LOAD_ADDR \
>                                         - GENERATED_GBL_DATA_SIZE)
> @@ -108,7 +108,7 @@
>         "mmcoops=mmc read 0 0x40000000 0x40 8; md 0x40000000 0x400\0" \
>         "verify=n\0" \
>         "rootfstype=ext4\0" \
> -       "console=" CONFIG_DEFAULT_CONSOLE "\0" \
> +       "console=" CONFIG_DEFAULT_CONSOLE \
>         "mtdparts=" CONFIG_MTDPARTS_DEFAULT \
>         "mbrparts=" MBRPARTS_DEFAULT \
>         "meminfo=crashkernel=32M at 0x50000000\0" \
> diff --git a/include/configs/trats.h b/include/configs/trats.h
> index 223fce49a7..af8e8ce3b6 100644
> --- a/include/configs/trats.h
> +++ b/include/configs/trats.h
> @@ -36,7 +36,7 @@
>  #define CONFIG_MACH_TYPE               MACH_TYPE_TRATS
>
>  #define CONFIG_BOOTCOMMAND             "run autoboot"
> -#define CONFIG_DEFAULT_CONSOLE         "ttySAC2,115200n8"
> +#define CONFIG_DEFAULT_CONSOLE         "console=ttySAC2,115200n8\0"
>
>  #define CONFIG_SYS_INIT_SP_ADDR        (CONFIG_SYS_LOAD_ADDR \
>                                         - GENERATED_GBL_DATA_SIZE)
> @@ -120,7 +120,7 @@
>         "mmcoops=mmc read 0 0x40000000 0x40 8; md 0x40000000 0x400\0" \
>         "verify=n\0" \
>         "rootfstype=ext4\0" \
> -       "console=" CONFIG_DEFAULT_CONSOLE "\0" \
> +       "console=" CONFIG_DEFAULT_CONSOLE \
>         "meminfo=crashkernel=32M at 0x50000000\0" \
>         "nfsroot=/nfsroot/arm\0" \
>         "bootblock=" CONFIG_BOOTBLOCK "\0" \
> diff --git a/include/configs/trats2.h b/include/configs/trats2.h
> index f1e4cbad30..9c6b2bbc8d 100644
> --- a/include/configs/trats2.h
> +++ b/include/configs/trats2.h
> @@ -34,7 +34,7 @@
>  /* Console configuration */
>
>  #define CONFIG_BOOTCOMMAND             "run autoboot"
> -#define CONFIG_DEFAULT_CONSOLE         "ttySAC2,115200n8"
> +#define CONFIG_DEFAULT_CONSOLE         "console=ttySAC2,115200n8\0"
>
>  #define CONFIG_SYS_INIT_SP_ADDR        (CONFIG_SYS_LOAD_ADDR \
>                                         - GENERATED_GBL_DATA_SIZE)
> @@ -103,7 +103,7 @@
>         "boottrace=setenv opts initcall_debug; run bootcmd\0" \
>         "verify=n\0" \
>         "rootfstype=ext4\0" \
> -       "console=" CONFIG_DEFAULT_CONSOLE "\0" \
> +       "console=" CONFIG_DEFAULT_CONSOLE \
>         "kernelname=uImage\0" \
>         "loaduimage=ext4load mmc ${mmcdev}:${mmcbootpart} 0x40007FC0 " \
>                 "${kernelname}\0" \
> --
> 2.19.1

This revert commit breaks the booting of the Odroid U3 using latest
u-boot on ArchLinux.
I have just tried to install archlinux on sdcard and update the latest kernel
after compiling the latest u-boot and upgrade.
It fails to boot up at-least their is not console output on the serial port.

[0] https://www.spinics.net/lists/arm-kernel/msg713918.html

I shared the my input on above link.
Can your verify this from your side.

Best Regards
-Anand


More information about the U-Boot mailing list