From: Mark Kettenis <mark.kettenis@xs4all.nl>
To: Simon Glass <sjg@chromium.org>
Cc: u-boot@lists.denx.de, trini@konsulko.com, hl@rock-chips.com,
jeffy.chen@rock-chips.com, sjg@chromium.org,
kever.yang@rock-chips.com, philipp.tomsich@theobroma-systems.com,
vagrant@debian.org
Subject: Re: [PATCH v6 09/11] xilinx: Disable CONFIG_BOOTSTD_DEFAULTS for some xilinx boards
Date: Sun, 02 Apr 2023 00:31:27 +0200 [thread overview]
Message-ID: <87wn2vb5fk.fsf@bloch.sibelius.xs4all.nl> (raw)
In-Reply-To: <20230402094532.v6.9.Icff8ccb474b62940b44121eca53b0c4a8bb00866@changeid> (message from Simon Glass on Sun, 2 Apr 2023 09:45:33 +1200)
> From: Simon Glass <sjg@chromium.org>
> Date: Sun, 2 Apr 2023 09:45:33 +1200
>
> Since CONFIG_BOOTSTD_DEFAULTS now selects ENV_VARS_UBOOT_CONFIG we cannot
> enable it for some Xilinx boards which have a very small environment.
>
> Disable it.
Does that mean these configs lose distroboot support? That would be a
shame!
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>
> Changes in v6:
> - Add new patch to disable CONFIG_BOOTSTD_DEFAULTS for some xilinx boards
>
> configs/xilinx_versal_mini_emmc0_defconfig | 1 +
> configs/xilinx_versal_mini_emmc1_defconfig | 1 +
> configs/xilinx_zynqmp_mini_emmc0_defconfig | 1 +
> configs/xilinx_zynqmp_mini_emmc1_defconfig | 1 +
> 4 files changed, 4 insertions(+)
>
> diff --git a/configs/xilinx_versal_mini_emmc0_defconfig b/configs/xilinx_versal_mini_emmc0_defconfig
> index 31b3c02f7389..8aadc7f6805b 100644
> --- a/configs/xilinx_versal_mini_emmc0_defconfig
> +++ b/configs/xilinx_versal_mini_emmc0_defconfig
> @@ -16,6 +16,7 @@ CONFIG_SYS_PROMPT="Versal> "
> CONFIG_SYS_LOAD_ADDR=0x8000000
> # CONFIG_EXPERT is not set
> CONFIG_REMAKE_ELF=y
> +# CONFIG_BOOTSTD_DEFAULTS is not set
> # CONFIG_AUTOBOOT is not set
> CONFIG_USE_BOOTCOMMAND=y
> CONFIG_BOOTCOMMAND="run distro_bootcmd"
> diff --git a/configs/xilinx_versal_mini_emmc1_defconfig b/configs/xilinx_versal_mini_emmc1_defconfig
> index 5480cf1d9cc4..5ccc20048212 100644
> --- a/configs/xilinx_versal_mini_emmc1_defconfig
> +++ b/configs/xilinx_versal_mini_emmc1_defconfig
> @@ -16,6 +16,7 @@ CONFIG_SYS_PROMPT="Versal> "
> CONFIG_SYS_LOAD_ADDR=0x8000000
> # CONFIG_EXPERT is not set
> CONFIG_REMAKE_ELF=y
> +# CONFIG_BOOTSTD_DEFAULTS is not set
> # CONFIG_AUTOBOOT is not set
> CONFIG_USE_BOOTCOMMAND=y
> CONFIG_BOOTCOMMAND="run distro_bootcmd"
> diff --git a/configs/xilinx_zynqmp_mini_emmc0_defconfig b/configs/xilinx_zynqmp_mini_emmc0_defconfig
> index cf7c7eda8c94..47eab450c90a 100644
> --- a/configs/xilinx_zynqmp_mini_emmc0_defconfig
> +++ b/configs/xilinx_zynqmp_mini_emmc0_defconfig
> @@ -17,6 +17,7 @@ CONFIG_SYS_LOAD_ADDR=0x8000000
> CONFIG_REMAKE_ELF=y
> # CONFIG_MP is not set
> CONFIG_FIT=y
> +# CONFIG_BOOTSTD_DEFAULTS is not set
> CONFIG_SUPPORT_RAW_INITRD=y
> # CONFIG_AUTOBOOT is not set
> # CONFIG_DISPLAY_CPUINFO is not set
> diff --git a/configs/xilinx_zynqmp_mini_emmc1_defconfig b/configs/xilinx_zynqmp_mini_emmc1_defconfig
> index a4a43b58a85e..da1e2d699caa 100644
> --- a/configs/xilinx_zynqmp_mini_emmc1_defconfig
> +++ b/configs/xilinx_zynqmp_mini_emmc1_defconfig
> @@ -18,6 +18,7 @@ CONFIG_REMAKE_ELF=y
> # CONFIG_MP is not set
> CONFIG_FIT=y
> CONFIG_SUPPORT_RAW_INITRD=y
> +# CONFIG_BOOTSTD_DEFAULTS is not set
> # CONFIG_AUTOBOOT is not set
> # CONFIG_DISPLAY_CPUINFO is not set
> CONFIG_BOARD_EARLY_INIT_R=y
> --
> 2.40.0.348.gf938b09366-goog
>
>
next prev parent reply other threads:[~2023-04-01 22:31 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-01 21:45 [PATCH v6 01/11] bootstd: Tweak bootflow logic for device tree Simon Glass
2023-04-01 21:45 ` [PATCH v6 02/11] virtio: Ensure PCI is set up first Simon Glass
2023-04-01 21:45 ` [PATCH v6 03/11] bootstd: Support booting EFI where multiple options exist Simon Glass
2023-04-01 21:45 ` [PATCH v6 04/11] bootstd: Report missing labels only when asked Simon Glass
2023-04-01 21:45 ` [PATCH v6 05/11] bootstd: Show a message sometimes if no bootflows are found Simon Glass
2023-04-01 21:45 ` [PATCH v6 06/11] rockchip: Move to standard boot Simon Glass
2023-04-01 21:45 ` [PATCH v6 07/11] rockchip: Use the same boot_targets for all boards Simon Glass
2023-04-01 21:45 ` [PATCH v6 08/11] bootstd: Disable BOOTSTD for boards with custom commands Simon Glass
2023-04-01 21:45 ` [PATCH v6 09/11] xilinx: Disable CONFIG_BOOTSTD_DEFAULTS for some xilinx boards Simon Glass
2023-04-01 22:31 ` Mark Kettenis [this message]
2023-04-02 2:41 ` Simon Glass
2023-04-02 14:42 ` Tom Rini
2023-04-02 19:16 ` Simon Glass
2023-04-03 6:47 ` Michal Simek
2023-04-01 22:29 ` [PATCH v6 01/11] bootstd: Tweak bootflow logic for device tree Mark Kettenis
[not found] ` <20230402094532.v6.11.I41a5e66644a895a311889b009116245245c741a5@changeid>
2023-04-01 23:28 ` [PATCH v6 11/11] bootstd: Enable BOOTSTD_DEFAULTS by default Tom Rini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87wn2vb5fk.fsf@bloch.sibelius.xs4all.nl \
--to=mark.kettenis@xs4all.nl \
--cc=hl@rock-chips.com \
--cc=jeffy.chen@rock-chips.com \
--cc=kever.yang@rock-chips.com \
--cc=philipp.tomsich@theobroma-systems.com \
--cc=sjg@chromium.org \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
--cc=vagrant@debian.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).