diff options
author | Andrew F. Davis <afd@ti.com> | 2019-08-12 22:59:54 +0300 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2019-08-20 18:46:11 +0300 |
commit | d2986a9bd8c06df52483e244f5c381e3cb899876 (patch) | |
tree | 218adb2b649aaebdc44c346bbc608b4eba7f0adf /include/configs/ti_armv7_common.h | |
parent | ee53b59511a60d2e12c17b245ad754ac893c8cb6 (diff) | |
download | u-boot-d2986a9bd8c06df52483e244f5c381e3cb899876.tar.xz |
configs: Rename environment variable fit_loadaddr to addr_fit
This is the first part of a larger effort I would like to propose to
unify and simplify the default set of environment variables.
When many early environment variables were named there were fewer images
being loaded, usually just a kernel. At this time names like 'loadaddr'
would suffice. Now we have more images and many more commands that act on
them, often re-using the same variable for several different uses. The
contents of a variable are also not immediately known causing one to have
to look up a chain of variables to understand what a command is actually
doing. I suggest the following.
To start, all variables containing names should be prefixed with name_
and addresses with addr_. This is like how K2 already does things and
allows for simple universal commands like:
get_fdt_nfs=nfs ${addr_fdt} /boot/${name_fdt}
Which is very clear on what is intended here and would work across all
board that using the this naming convention.
We can do this one variable at a time, start here with addr_fit.
Signed-off-by: Andrew F. Davis <afd@ti.com>
Acked-by: Andreas Dannenberg <dannenberg@ti.com>
Diffstat (limited to 'include/configs/ti_armv7_common.h')
-rw-r--r-- | include/configs/ti_armv7_common.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/configs/ti_armv7_common.h b/include/configs/ti_armv7_common.h index 828fb1b2a5..ece329fc7d 100644 --- a/include/configs/ti_armv7_common.h +++ b/include/configs/ti_armv7_common.h @@ -52,9 +52,9 @@ #define DEFAULT_FIT_TI_ARGS \ "boot_fit=0\0" \ - "fit_loadaddr=0x90000000\0" \ + "addr_fit=0x90000000\0" \ "fit_bootfile=fitImage\0" \ - "update_to_fit=setenv loadaddr ${fit_loadaddr}; setenv bootfile ${fit_bootfile}\0" \ + "update_to_fit=setenv loadaddr ${addr_fit}; setenv bootfile ${fit_bootfile}\0" \ "loadfit=run args_mmc; bootm ${loadaddr}#${fdtfile};\0" \ /* |