[U-Boot] [PATCHv2 2/5] env: Remove useless CONFIG_ENV_SIZE_REDUND and fix ENV_IS_REDUND check

Simon Glass sjg at chromium.org
Tue Nov 19 01:21:06 UTC 2019


On Mon, 18 Nov 2019 at 17:07, Tom Rini <trini at konsulko.com> wrote:
>
> We have CONFIG_ENV_SIZE_IS_REDUND but don't really use it.  We have one
> board where we can simply multiple CONFIG_ENV_SIZE by two for the same
> result.  The other place where we could but were not previously using
> this is for where env_internal.h checks for if we should set
> ENV_IS_EMBEDDED.  This seems like the most likely use, historically, of
> the variable, but it was not used.  Add logic to check for this now.
>
> Cc: Wolfgang Denk <wd at denx.de>
> Cc: Joe Hershberger <joe.hershberger at ni.com>
> Signed-off-by: Tom Rini <trini at konsulko.com>
> ---
>  env/Kconfig                                    | 7 -------
>  env/flash.c                                    | 5 -----
>  env/mmc.c                                      | 5 -----
>  env/nand.c                                     | 5 -----
>  env/sata.c                                     | 2 +-
>  include/configs/MPC8308RDB.h                   | 1 -
>  include/configs/MPC8313ERDB_NAND.h             | 1 -
>  include/configs/MPC8349EMDS.h                  | 1 -
>  include/configs/MPC8349EMDS_SDRAM.h            | 1 -
>  include/configs/MigoR.h                        | 1 -
>  include/configs/TQM834x.h                      | 1 -
>  include/configs/UCP1020.h                      | 2 --
>  include/configs/am335x_shc.h                   | 1 -
>  include/configs/apf27.h                        | 1 -
>  include/configs/apx4devkit.h                   | 1 -
>  include/configs/armadillo-800eva.h             | 1 -
>  include/configs/bk4r1.h                        | 1 -
>  include/configs/blanche.h                      | 1 -
>  include/configs/caddy2.h                       | 1 -
>  include/configs/condor.h                       | 1 -
>  include/configs/dh_imx6.h                      | 1 -
>  include/configs/display5.h                     | 1 -
>  include/configs/draco.h                        | 1 -
>  include/configs/eagle.h                        | 1 -
>  include/configs/edb93xx.h                      | 1 -
>  include/configs/etamin.h                       | 1 -
>  include/configs/flea3.h                        | 1 -
>  include/configs/gardena-smart-gateway-mt7688.h | 1 -
>  include/configs/gazerbeam.h                    | 1 -
>  include/configs/gw_ventana.h                   | 1 -
>  include/configs/hrcon.h                        | 1 -
>  include/configs/ids8313.h                      | 1 -
>  include/configs/imx27lite-common.h             | 1 -
>  include/configs/km/km-mpc83xx.h                | 1 -
>  include/configs/km/km_arm.h                    | 1 -
>  include/configs/kp_imx53.h                     | 1 -
>  include/configs/m53menlo.h                     | 1 -
>  include/configs/mccmon6.h                      | 1 -
>  include/configs/mpc8308_p1m.h                  | 1 -
>  include/configs/mx28evk.h                      | 1 -
>  include/configs/mx35pdk.h                      | 1 -
>  include/configs/novena.h                       | 1 -
>  include/configs/pcm052.h                       | 1 -
>  include/configs/pcm058.h                       | 1 -
>  include/configs/pfla02.h                       | 1 -
>  include/configs/platinum.h                     | 1 -
>  include/configs/rastaban.h                     | 1 -
>  include/configs/rcar-gen2-common.h             | 1 -
>  include/configs/rcar-gen3-common.h             | 1 -
>  include/configs/sbc8349.h                      | 1 -
>  include/configs/sh7752evb.h                    | 1 -
>  include/configs/sh7753evb.h                    | 1 -
>  include/configs/sh7757lcr.h                    | 1 -
>  include/configs/sh7763rdp.h                    | 1 -
>  include/configs/sksimx6.h                      | 1 -
>  include/configs/socfpga_vining_fpga.h          | 1 -
>  include/configs/socrates.h                     | 1 -
>  include/configs/strider.h                      | 1 -
>  include/configs/tam3517-common.h               | 1 -
>  include/configs/thuban.h                       | 1 -
>  include/configs/titanium.h                     | 1 -
>  include/configs/vcoreiii.h                     | 1 -
>  include/configs/vct.h                          | 1 -
>  include/configs/ve8313.h                       | 1 -
>  include/configs/vining_2000.h                  | 1 -
>  include/configs/vme8349.h                      | 1 -
>  include/configs/woodburn_common.h              | 1 -
>  include/configs/x600.h                         | 1 -
>  include/env_internal.h                         | 7 +++++--
>  scripts/config_whitelist.txt                   | 1 -
>  tools/envcrc.c                                 | 3 ---
>  71 files changed, 6 insertions(+), 93 deletions(-)

Reviewed-by: Simon Glass <sjg at chromium.org>


More information about the U-Boot mailing list