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 61C02C636CC for ; Tue, 31 Jan 2023 21:58:19 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 4545B85BA5; Tue, 31 Jan 2023 22:58:17 +0100 (CET) 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="jrSkN9v6"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 1B6D685B90; Tue, 31 Jan 2023 22:58:07 +0100 (CET) Received: from mail-qt1-x82f.google.com (mail-qt1-x82f.google.com [IPv6:2607:f8b0:4864:20::82f]) (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 0E18685B65 for ; Tue, 31 Jan 2023 22:58:02 +0100 (CET) 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-qt1-x82f.google.com with SMTP id g8so2048955qtq.13 for ; Tue, 31 Jan 2023 13:58:02 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=konsulko.com; s=google; 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=3ldIbRvzm5MZFM1qZAmK2nS6Q7cyqn3e/wH3exhN3Mg=; b=jrSkN9v6mUjxsMn997OEeLTSZK27KgSxKkbyn6zyv+Wuqj1Xc22xGdwfUd7FPO+ekF UImnJtRzVdAvAuOBybQ4ReEYw/ZVTytLjmUep20jpDBkppXAIhK6fM6LGDqnkpu5IV3w UiUWbJN324SlAeZdd8j1CmvN8tRxpgYe9+mzk= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=3ldIbRvzm5MZFM1qZAmK2nS6Q7cyqn3e/wH3exhN3Mg=; b=lW0a9Tv1dLuInE9BoF54dNmt9uYPoW7P0llCgtHB64N0eXhS58hVo9kNbaR/Erz5IY mRudfJVueXw2gPw2rkeFQrMM4AmNmpUtR4CCeQDaxjI3zuRy/0aE4yOdevM8rcvpyXKX /iuDGTc6uD3uexuCkr0wcRCd5b0Z5IZ0ncpgzKpFYZIwfAuzirmGRiwBdg3p9HRJt5Qx DE6T/InqE0MRziKqmeyH47jqWFMk5E5iWawfu4rVNOBjp2C0faYj8L9Zxly7OwEjJF8J s9tQsmUSXKFDGuScfqP8CQxBV7T/IV0A17qaGHbKuoz8SUTXfo4ZoNQFKW4s1tADUah0 ffNw== X-Gm-Message-State: AO0yUKV1PNe2iSLb77to8nb0SzQ1qLof9QF028i3A/sfluFAD/8nDwPQ 7TvGDJ155RM+zb0zqzfWi8s1VF4olZJOZuNLvhk= X-Google-Smtp-Source: AK7set/OuI/qev1quEap62gqhGRsyQRgRmIshOn13VDu6uPXYHnppqvOZFg8arzWnF7DeGRlKBGROw== X-Received: by 2002:a05:622a:58a:b0:3b8:685c:9cb with SMTP id c10-20020a05622a058a00b003b8685c09cbmr807968qtb.0.1675202280742; Tue, 31 Jan 2023 13:58:00 -0800 (PST) Received: from bill-the-cat (2603-6081-7b00-6400-d3bc-07cf-bf94-8c52.res6.spectrum.com. [2603:6081:7b00:6400:d3bc:7cf:bf94:8c52]) by smtp.gmail.com with ESMTPSA id j7-20020a37b907000000b00706c1fc62desm10768321qkf.112.2023.01.31.13.57.59 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 31 Jan 2023 13:58:00 -0800 (PST) Date: Tue, 31 Jan 2023 16:57:58 -0500 From: Tom Rini To: Simon Glass Cc: U-Boot Mailing List , Joe Hershberger , Masahiro Yamada Subject: Re: [PATCH 00/32] RFC: Migrate to split config Message-ID: References: <20230131152702.249197-1-sjg@chromium.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="qQgI39fpdGMFuYrB" Content-Disposition: inline In-Reply-To: <20230131152702.249197-1-sjg@chromium.org> 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.6 at phobos.denx.de X-Virus-Status: Clean --qQgI39fpdGMFuYrB Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jan 31, 2023 at 08:26:30AM -0700, Simon Glass wrote: > U-Boot uses an SPL prefix on CONFIG options to indicate when an option > relates to SPL. For example, while CONFIG_TEXT_BASE is the text base for > U-Boot proper, CONFIG_SPL_TEXT_BASE is the text base for SPL. >=20 > Within the code it is possible do things like CONFIG_VAL(TEXT_BASE) to > get that value. It returns the appropriate option, depending on the phase > being built. >=20 > The same applies for boolean options, like CONFIG_BLK and CONFIG_SPL_BLK > but in that case we use CONFIG_IS_ENABLED(BLK) to obtain the value. >=20 > For Makefiles we use an SPL_TPL_ macro which evalutes either to empty, > SPL, TPL or even VPL depending on the build phase. So we can do things > like: >=20 > obj-$(CONFIG_$(SPL_TPL_)CLK) +=3D clk-uclass.o >=20 > To complicate things a little, some options like CONFIG_ARCH_APPLE do not > have an SPL version, so people have to remember to use IS_ENABLED() > instead of CONFIG_IS_ENABLED(). If the latter is used, the value will be > false. >=20 > All of this is rather confusing, since: >=20 > - people have to know whether to use IS_ENABLED() or CONFIG_IS_ENABLED() > - the Makefiles are cluttered with $(SPL_TPL_) constructs > - plain use of a CONFIG option (e.g. #ifdef CONFIG_CLK) may be correct in > some cases but not in others >=20 > This series updates U-Boot to use separate config files for each phase of > the build. It drops use of CONFIG_IS_ENABLED() and SPL_TPL_ as well. >=20 > For now this series does not fully work due to inconsistencies in the > Kconfig options. Many of these have been fixed [1] [2a] [2b] but more > remain. >=20 > This series is available at u-boot-dm/splc-working >=20 > [1] https://patchwork.ozlabs.org/project/uboot/list/?series=3D339004 > [2a] https://patchwork.ozlabs.org/project/uboot/list/?series=3D339199&sta= te=3D* > [2b] https://patchwork.ozlabs.org/project/uboot/list/?series=3D339208&sta= te=3D* To summarize what I said on IRC, it's going to depend on what this looks like once every board is expected to build and boot, which isn't the case today, as there's more "dummy" or in other words, def_bool n symbols added, so that we don't change things before/after this series. --=20 Tom --qQgI39fpdGMFuYrB Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGzBAABCgAdFiEEGjx/cOCPqxcHgJu/FHw5/5Y0tywFAmPZjuYACgkQFHw5/5Y0 tywmDgv/Vt3RdS0xa8QG/N5Rji5czTNTswlqmn3VXJK4xKvOzg8KO6gDVqwPTHJO IjdzhoLSVeLHZxS9vP+Y9yJn6L359g1osOGuh1NWRK4To3TjcbzMQrV7keYKP66B BmJkr4mYjpzRKhlz3JRBAhAxp0SOTvg9DnERK4n9VTQrAt25aSur8Y2QV/DiblyJ rFB++PZnytxY0mTQgJYcC3lMWMCqHsUKuiX390hraPlMaZ9bxiSFbqPmVGjX6/+d /XV+kwXm64bQ4wOsaCiNIcF+bLzsjH1ZHAvVplTV29jz+g3WqO37rVLWxcaeh5ni umdtWKH31v1b/+WyH//Tx8FEOQZkr/M4gbDZuU6NtN9A9F8dYMqA2mbzRAjj7DXf UuZJpjmIgvd7S2JybIEL4mu1u11p8dvUZp/SrrFrq5DFpEWxYIZ6iWx9gvo3I/4G V0VHCBU17uBWpD1wjBB0Fb3XYEwdwFRKTBFSrkFsODXjOezEJVL7YuTfl2k5zVTT 9XOdfiUi =mFH7 -----END PGP SIGNATURE----- --qQgI39fpdGMFuYrB--