diff mbox series

[v1] board: toradex: verdin-am62: Remove not needed env variables

Message ID 20240418131302.19693-1-francesco@dolcini.it
State Accepted
Delegated to: Tom Rini
Headers show
Series [v1] board: toradex: verdin-am62: Remove not needed env variables | expand

Commit Message

Francesco Dolcini April 18, 2024, 1:13 p.m. UTC 
From: Francesco Dolcini <francesco.dolcini@toradex.com>

Remove not needed variables from environment.

 - boot_scripts is not needed, the default value is just fine and
   already includes boot.scr
 - setup variable used to be executed from some bootscript, however
   it's not required and there is no point on having this small helper
   here

Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>
---
 include/configs/verdin-am62.h | 3 ---
 1 file changed, 3 deletions(-)

Comments

Tom Rini April 29, 2024, 11:04 p.m. UTC | #1 
On Thu, 18 Apr 2024 15:13:02 +0200, Francesco Dolcini wrote:

> Remove not needed variables from environment.
> 
>  - boot_scripts is not needed, the default value is just fine and
>    already includes boot.scr
>  - setup variable used to be executed from some bootscript, however
>    it's not required and there is no point on having this small helper
>    here
> 
> [...]

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/include/configs/verdin-am62.h b/include/configs/verdin-am62.h
index 9d2e37f2d96d..ac44809fdfa5 100644
--- a/include/configs/verdin-am62.h
+++ b/include/configs/verdin-am62.h
@@ -39,12 +39,9 @@ 
 #define CFG_EXTRA_ENV_SETTINGS \
 	BOOTENV \
 	MEM_LAYOUT_ENV_SETTINGS \
-	"boot_scripts=boot.scr\0" \
 	"boot_script_dhcp=boot.scr\0" \
 	"console=ttyS2\0" \
 	"fdt_board=dev\0" \
-	"setup=setenv setupargs console=tty1 console=${console},${baudrate} " \
-		"consoleblank=0 earlycon=ns16550a,mmio32,0x02800000\0" \
 	"update_tiboot3=askenv confirm Did you load tiboot3.bin (y/N)?; " \
 		"if test \"$confirm\" = \"y\"; then " \
 		"setexpr blkcnt ${filesize} + 0x1ff && setexpr blkcnt " \