[PATCH v2 041/169] Correct SPL uses of CMD_DHCP

Tom Rini trini at konsulko.com
Thu Feb 9 16:55:05 CET 2023


On Sun, Feb 05, 2023 at 03:36:27PM -0700, Simon Glass wrote:

> This converts 7 usages of this option to the non-SPL form, since there is
> no SPL_CMD_DHCP defined in Kconfig
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> 
> (no changes since v1)
> 
>  include/configs/am335x_evm.h         | 2 +-
>  include/configs/imx8mm-cl-iot-gate.h | 2 +-
>  include/configs/imx8mp_rsb3720.h     | 2 +-
>  include/configs/j721e_evm.h          | 2 +-
>  include/configs/qemu-arm.h           | 2 +-
>  include/configs/rockchip-common.h    | 2 +-
>  include/configs/rpi.h                | 2 +-
>  7 files changed, 7 insertions(+), 7 deletions(-)

So this, and a few other CMD cases are special. We have in SPL,
ENV_NOWHERE for various good and valid use cases. So we want to limit
the size of the default environment. We also use the generic distro
bootcmd framework on these boards. So CONFIG_IS_ENABLED(CMD_FOO) gets
used to nop out the boot targets we don't need taking up space. It's not
a fatal problem to change these, but I'm not going to pick these changes
up just yet.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230209/46d88d7c/attachment.sig>


More information about the U-Boot mailing list