From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 8EB6DC761AF for ; Wed, 29 Mar 2023 14:15:42 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id CF5E385906; Wed, 29 Mar 2023 16:15:37 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=konsulko.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=konsulko.com header.i=@konsulko.com header.b="gEp6StQa"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 8486F804DB; Wed, 29 Mar 2023 16:15:33 +0200 (CEST) Received: from mail-yw1-x112b.google.com (mail-yw1-x112b.google.com [IPv6:2607:f8b0:4864:20::112b]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 2987E85D10 for ; Wed, 29 Mar 2023 16:15:28 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=konsulko.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=trini@konsulko.com Received: by mail-yw1-x112b.google.com with SMTP id 00721157ae682-544f7c176easo295191617b3.9 for ; Wed, 29 Mar 2023 07:15:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=konsulko.com; s=google; t=1680099326; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=NU0qGCt6ofnrzdOfgI1iM5PXM2oS21ulCuGeoD1B3yY=; b=gEp6StQaVXgC5sWXQv6lOQVG6/HnyWY7ic3t0hrpO4HfeNUL0E2HgE7UW5IcudUaIh Ohjm6Dh3fSMWS8LRrd2Hhr5AKrQRm18q6sZIq14FrAhHsq4/ulYA6/NwPjRQy6M5wv9I u+y6vY0XF6ox9xVz1fioHHLHGMVK+1Wye/UeI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1680099326; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=NU0qGCt6ofnrzdOfgI1iM5PXM2oS21ulCuGeoD1B3yY=; b=htIvgUzd1WO5DOogjGOJmmtCMdg9cqmzUXI3gbLnQCB1ggO9wS4GUMLKOVqtUwXx0R w4GF8iYRdSn4x/Cdf2awiRpIIm5aTDJ+FL1Df8IL8GYH2hTVsGoboTE1PF6Fiy9mJCcp j9MVFbcfOn6WQz2624Pa6snZnaqz2bqn+wQ83vYFeJBFDUiZEBMxXYt9YUZ+L7OB8KlN +EENX4zAr7k7vjyVDAcZuH8iX2PXczG196tlwCnFEQ6wLqNTCZ5jaDvCBudmD6Zz8tmx zBibMwfv3ZkdHoX61S1XXqI/E+42dhAp3Q5sXCTxFICA6AP5SoXtnFu2c5Jp/j45RHkq AgKw== X-Gm-Message-State: AAQBX9e26nQNcIDsOglE7/t3wMZ0Uc9YEJa0AqQdxtILOqDMg+bOEXi8 Ff34BUdwjQFHObtB6Y2R7utuNA== X-Google-Smtp-Source: AKy350ac3i3LaSoCsW50t+Omm+nnP3wOPV1T6Uc6+kfVn/88mWmlEmvRMLZ1zyaCHRoTeq+mRi7KDA== X-Received: by 2002:a81:b409:0:b0:538:49a4:b166 with SMTP id h9-20020a81b409000000b0053849a4b166mr20014683ywi.11.1680099326503; Wed, 29 Mar 2023 07:15:26 -0700 (PDT) Received: from bill-the-cat (2603-6081-7b00-6400-aab4-31cf-c563-6345.res6.spectrum.com. [2603:6081:7b00:6400:aab4:31cf:c563:6345]) by smtp.gmail.com with ESMTPSA id l186-20020a8157c3000000b00545a0818494sm2991195ywb.36.2023.03.29.07.15.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 29 Mar 2023 07:15:26 -0700 (PDT) Date: Wed, 29 Mar 2023 10:15:24 -0400 From: Tom Rini To: Simon Glass Cc: U-Boot Mailing List , huang lin , Jeffy Chen , Kever Yang , Philipp Tomsich , Vagrant Cascadian , "Albert ARIBAUD (3ADEV)" , Eugen Hristev , Greg Gallagher , Heiko Schocher , Michal Simek , Nicolas Ferre , Philippe Reynes , Tony Dinh Subject: Re: [PATCH v4 1/4] bootstd: Enable BOOTSTD_DEFAULTS by default Message-ID: <20230329141524.GB6083@bill-the-cat> References: <20230329201631.v4.1.I41a5e66644a895a311889b009116245245c741a5@changeid> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="8SGlhxA7MfC104aG" Content-Disposition: inline In-Reply-To: <20230329201631.v4.1.I41a5e66644a895a311889b009116245245c741a5@changeid> X-Clacks-Overhead: GNU Terry Pratchett X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.8 at phobos.denx.de X-Virus-Status: Clean --8SGlhxA7MfC104aG Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Mar 29, 2023 at 08:16:36PM +1300, Simon Glass wrote: > This is needed to enable the boot command used to start standard boot. > Enable it by default. This brings in quite a few features, mostly in > common with DISTRO_DEFAULTS >=20 > Disable this option for boards which don't have enough space. >=20 > Disable CONFIG_ENV_VARS_UBOOT_CONFIG for some Xilinx boards which have > a very small environment. Disable BOOTSTD_DEFAULTS for smartweb since > it is too close to its limit. >=20 > Signed-off-by: Simon Glass > --- >=20 > Changes in v4: > - Rebase to -next >=20 > boot/Kconfig | 1 + > configs/bk4r1_defconfig | 1 + > configs/sama5d27_giantboard_defconfig | 1 + > configs/sama5d27_som1_ek_mmc1_defconfig | 1 + > configs/sama5d27_som1_ek_mmc_defconfig | 1 + > configs/sama5d27_som1_ek_qspiflash_defconfig | 1 + > configs/sama5d27_wlsom1_ek_mmc_defconfig | 1 + > configs/sama5d2_icp_mmc_defconfig | 1 + > configs/sama5d2_xplained_emmc_defconfig | 1 + > configs/sama5d2_xplained_mmc_defconfig | 1 + > configs/sama5d2_xplained_qspiflash_defconfig | 1 + > configs/sheevaplug_defconfig | 1 + > configs/smartweb_defconfig | 1 + > configs/taurus_defconfig | 1 + > configs/tools-only_defconfig | 2 +- > configs/xilinx_versal_mini_emmc0_defconfig | 1 + > configs/xilinx_versal_mini_emmc1_defconfig | 1 + > configs/xilinx_zynqmp_mini_emmc0_defconfig | 2 +- > configs/xilinx_zynqmp_mini_emmc1_defconfig | 2 +- > 19 files changed, 19 insertions(+), 3 deletions(-) >=20 > diff --git a/boot/Kconfig b/boot/Kconfig > index d95a2a702665..61ebc2750154 100644 > --- a/boot/Kconfig > +++ b/boot/Kconfig > @@ -435,6 +435,7 @@ if BOOTSTD > config BOOTSTD_DEFAULTS > bool "Select some common defaults for standard boot" > depends on BOOTSTD > + default y > imply USE_BOOTCOMMAND > select BOOT_DEFAULTS I intentionally did not do this before, and I don't think it's right, certainly right now. This fixes some boards (such as some mediatek SoCs for example) that have boostd, have no bootcmd in environment and so don't autoboot before bootstd, but do now. But it also bloats all of the platforms that don't want bootstd because they have a real bootcmd that's not some variant of "prepare to and then run distro_bootcmd". Those should get bootstd disabled, first. Then we can evaluate if this ends up being the right option. --=20 Tom --8SGlhxA7MfC104aG Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGzBAABCgAdFiEEGjx/cOCPqxcHgJu/FHw5/5Y0tywFAmQkR/kACgkQFHw5/5Y0 tyw58wwAht8rhjESjHgFoJgp0JvsBwtIpUXzrmHALsSH+noZE9Dr4wmosje9qaZI tjgKRQ+DLH4+ay9b9vwHCRHvsvQLstOqK4AtWbBCzvKWq+G5sbErgEKa0m/+7ugk kans5i7VJZw9TU7D+vdtVcHsmlq52qfpVlYbFjPF3j8vg27xkWQ3Ky69qsiOINcY WmX33nehdqCbjrM6NnRDRqs493bCOAh9ICEdar8YILyaMa6lh+YUqotp0AyAmINB a2+LnNIEipT7gkkstZD9EL0p9xE4QgJk3V4iVgfX2UVVDjTLUB5HoQtacaG5un8a 3h+4Kx88OkZZ5xm0ERxugqBpg71u6KtPSxvqtpPX41T0JexP9DrUe01bTTUwA1QY QMf+B5fzhRAWoSUsQTU5T+bkHT9YC0wuyhp+hDvni3dex/kALFoCjq6EOOrTabv6 hIJUegnRQbRd8Ws7MMV/YcmBCEc2mvdmmkGNqk3QMf0uap59tyY2ZVSavEb2TnVM s/+5yh2R =+yuB -----END PGP SIGNATURE----- --8SGlhxA7MfC104aG--