From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Sun, 24 Feb 2019 22:32:53 +0100 Subject: [U-Boot] [PATCH 4/4] arm: exynos: Remove duplicated "boardname" env setting In-Reply-To: <20190222183643.5995-4-krzk@kernel.org> References: <20190222183643.5995-1-krzk@kernel.org> <20190222183643.5995-4-krzk@kernel.org> Message-ID: <20190224223253.09e145f1@jawa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Fri, 22 Feb 2019 19:36:43 +0100 Krzysztof Kozlowski wrote: > Various places in the code set "boardname" env property. It was used > for booting from ITB images and choosing proper DTB file name. > Instead of duplicating it, use existing U-Boot wide - "board_name". > > Signed-off-by: Krzysztof Kozlowski > > --- > > Tested only on Odroid XU3. > Tests on other boards appreciated. > --- > board/samsung/common/bootscripts/autoboot.cmd | 10 +++++----- > board/samsung/common/misc.c | 2 +- > include/configs/odroid.h | 2 +- > include/configs/odroid_xu3.h | 8 ++++++-- > 4 files changed, 13 insertions(+), 9 deletions(-) > > diff --git a/board/samsung/common/bootscripts/autoboot.cmd > b/board/samsung/common/bootscripts/autoboot.cmd index > 11c724c4e095..d66bcccf5d1c 100644 --- > a/board/samsung/common/bootscripts/autoboot.cmd +++ > b/board/samsung/common/bootscripts/autoboot.cmd @@ -3,7 +3,7 @@ > # ./tools/mkimage -c none -A arm -T script -d autoboot.cmd boot.scr > # > # It requires a list of environment variables to be defined before > load: -# platform dependent: boardname, fdtfile, console > +# platform dependent: board_name, fdtfile, console > # system dependent: mmcbootdev, mmcbootpart, mmcrootdev, > mmcrootpart, rootfstype # > setenv fdtaddr "40800000" > @@ -35,17 +35,17 @@ else > setenv initrd_addr -; > fi;" > > -#### Routine: boot_fit - check that env $boardname is set and boot > proper config of ITB image +#### Routine: boot_fit - check that env > $board_name is set and boot proper config of ITB image setenv > setboot_fit " -if test -e '${boardname}'; then > +if test -e '${board_name}'; then > setenv fdt_addr ; > setenv initrd_addr ; > setenv kerneladdr 0x42000000; > setenv kernelname Image.itb; > - setenv itbcfg "\"#${boardname}\""; > + setenv itbcfg "\"#${board_name}\""; > setenv imgbootcmd bootm; > else > - echo Warning! Variable: \$boardname is undefined!; > + echo Warning! Variable: \$board_name is undefined!; > fi" > > #### Routine: setboot_uimg - prepare env to boot uImage > diff --git a/board/samsung/common/misc.c b/board/samsung/common/misc.c > index 05243fc89659..53cd1b290703 100644 > --- a/board/samsung/common/misc.c > +++ b/board/samsung/common/misc.c > @@ -101,7 +101,7 @@ void set_board_info(void) > bdtype = ""; > > sprintf(info, "%s%s", bdname, bdtype); > - env_set("boardname", info); > + env_set("board_name", info); > #endif > snprintf(info, ARRAY_SIZE(info), "%s%x-%s%s.dtb", > CONFIG_SYS_SOC, s5p_cpu_id, bdname, bdtype); > diff --git a/include/configs/odroid.h b/include/configs/odroid.h > index b8809c8dcc7c..9f2d43e3fa38 100644 > --- a/include/configs/odroid.h > +++ b/include/configs/odroid.h > @@ -132,7 +132,7 @@ > "setenv kernelname Image.itb;" \ > "run loadkernel;" \ > "run kernel_args;" \ > - "bootm ${kernel_addr_r}#${boardname}\0" \ > + "bootm ${kernel_addr_r}#${board_name}\0" \ > "boot_uimg=" \ > "setenv kernelname uImage;" \ > "run check_dtb;" \ > diff --git a/include/configs/odroid_xu3.h > b/include/configs/odroid_xu3.h index 5e765a2b2b02..af6004eaf511 100644 > --- a/include/configs/odroid_xu3.h > +++ b/include/configs/odroid_xu3.h > @@ -84,10 +84,14 @@ > #define CONFIG_SET_DFU_ALT_INFO > #define CONFIG_SET_DFU_ALT_BUF_LEN (SZ_1K) > > -/* Set soc_rev, soc_id, board_rev, boardname, fdtfile */ > +/* Set soc_rev, soc_id, board_rev, board_name, fdtfile */ > #define CONFIG_ODROID_REV_AIN 9 > #define CONFIG_REVISION_TAG > > +/* > + * Need to override existing one (smdk5420) with odroid so > set_board_info will > + * use proper prefix when creating full board_name (SYS_BOARD + type) > + */ > #undef CONFIG_SYS_BOARD > #define CONFIG_SYS_BOARD "odroid" > > @@ -101,7 +105,7 @@ > "rootfstype=ext4\0" \ > "console=" CONFIG_DEFAULT_CONSOLE \ > "fdtfile=exynos5422-odroidxu3.dtb\0" \ > - "boardname=odroidxu3\0" \ > + "board_name=odroidxu3\0" \ > "mmcbootdev=0\0" \ > "mmcrootdev=0\0" \ > "mmcbootpart=1\0" \ Reviewed-by: Lukasz Majewski Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 488 bytes Desc: OpenPGP digital signature URL: