From patchwork Thu Sep 30 06:41:41 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ilias Apalodimas X-Patchwork-Id: 514879 Delivered-To: patch@linaro.org Received: by 2002:a05:6638:4087:0:0:0:0 with SMTP id m7csp779710jam; Wed, 29 Sep 2021 23:42:38 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyfYCqnHOyCvTJQjs3nm3IJTOdt6fGVDh2Cr33hebggmXnYuIuuZl5QxPVp0Y7fRSQuzKLy X-Received: by 2002:a17:906:2b07:: with SMTP id a7mr5089803ejg.284.1632984157889; Wed, 29 Sep 2021 23:42:37 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1632984157; cv=none; d=google.com; s=arc-20160816; b=Q8oE1HlBDmKpC000nm8wQH+FgbxdR9r2zsCXaGvUnj+gyEVvIBMa4M4HycHIfHQzuw HUMOsV+FE4x8fb8nlt98IrY1ldaOUjlXegBFQKIWhitZQ5b8N8/+C2bSNFhozaAZ6CKa J6Q48YdneHwLjP0ssNu0dgfLHY2cW1DehXyStWSt3tKWscMCyGKZbo4JTS/VfOLsIRfY PoBJTOc+iGwfz/WS2hFWGQPHJC3nw+vxc/euzMI9g8nOBRh8UfydWSagoOpzPWSYzNSe kOjbsUslNfxi6m2SJmljXy1luTjP3lOoZYClzu4v5npsyu8PYz7fkBTzlnjb2RtMhzRn WQUA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=K0VEt7OtbSJSDIb7ggVA3N4pGaW3pUyIwpOHJrjnsSg=; b=CPDtnSOZA+WEnfHTFVfgsMYNNcKTCCwO9ayEz8rL3OjN2x6PW+aatzqDJVlK/R530W iy792DS2zwtqJjDSQL+ymvbPP2sFWEO6M66bwiDtLLhc+l908otJrsmar2Vb08ZpLyIO h/LwDZKUfyJm8f0bwSv3XhPr1yF5Gk8VeTZuh5BnYz72eEQFfRA0MErt2dwlJ/n4RPoM 98SkvhRj33uEQNjnWYWW+OkEX5G/WwiNSoClEcny5fa/dHv3J+RCToyKoIZnlDshOUXl KrS7fH3xxs/B9+3dXXe2LTzpdX2TA7BdfI4bvKIIdSIC6J1RkxfKxcPHEiHdkn6Rcwvz 24hQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=OewUhE0+; spf=pass (google.com: domain of u-boot-bounces@lists.denx.de designates 2a01:238:438b:c500:173d:9f52:ddab:ee01 as permitted sender) smtp.mailfrom=u-boot-bounces@lists.denx.de; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from phobos.denx.de (phobos.denx.de. [2a01:238:438b:c500:173d:9f52:ddab:ee01]) by mx.google.com with ESMTPS id cf6si2397759edb.49.2021.09.29.23.42.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 29 Sep 2021 23:42:37 -0700 (PDT) Received-SPF: pass (google.com: domain of u-boot-bounces@lists.denx.de designates 2a01:238:438b:c500:173d:9f52:ddab:ee01 as permitted sender) client-ip=2a01:238:438b:c500:173d:9f52:ddab:ee01; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=OewUhE0+; spf=pass (google.com: domain of u-boot-bounces@lists.denx.de designates 2a01:238:438b:c500:173d:9f52:ddab:ee01 as permitted sender) smtp.mailfrom=u-boot-bounces@lists.denx.de; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 42FBB8128D; Thu, 30 Sep 2021 08:42:28 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.b="OewUhE0+"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 985378111F; Thu, 30 Sep 2021 08:42:12 +0200 (CEST) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de X-Spam-Level: X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.2 Received: from mail-ed1-x52f.google.com (mail-ed1-x52f.google.com [IPv6:2a00:1450:4864:20::52f]) (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 9B1D880F1A for ; Thu, 30 Sep 2021 08:42:04 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=ilias.apalodimas@linaro.org Received: by mail-ed1-x52f.google.com with SMTP id x7so17727684edd.6 for ; Wed, 29 Sep 2021 23:42:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=K0VEt7OtbSJSDIb7ggVA3N4pGaW3pUyIwpOHJrjnsSg=; b=OewUhE0+94iYsCXmmNhexoGHKwe+xiYJM5dLP6rU+ucB2KMpGaCYqMaNXdryqnH6UZ ZvxqFbO4x88ZvE3obPtdJOGuLoZtab+OTHpOGMNVEEp8M+mxdcRA9IDeKKmRAj36HtO7 QpbbTijDuvIUmK29QibYLAUW91fPiQiEReXgwsA9UsMiqLgieUNe0IOnqFp48LYMSokO NZ/FK/0UaQwdX/r3eG+LYdzlJXYlgLJHbDNrUmF8q3fToqhiFCs37X4OAqmKiS6acghq 1tzVIyBQp5sPa96xUkuzxy8UzF6sYUprkpbWxIvi6W5G34EklxW+tYM9Xk1QYbTErRZJ aUEg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=K0VEt7OtbSJSDIb7ggVA3N4pGaW3pUyIwpOHJrjnsSg=; b=Y5qiFQqdoFa5ZQI/+6C2TeGdzsunzCPPuCGMKNaA/VmnMBhqRlJjsTltM6r8cs9J02 9m4aqVcv3h9BF6PWy+n4bwRYdJHnwNd68gdjK3vtNPc/n2WQpAnwACNlbI7T7px5fmw7 iRPQxhNTFn3D+KTe+pQJ13GeqrtdtOfoVtSdjv1wVtrJa/s75uTDyAFi3k1GNdWzOZyf DzyOMksO1t3XtS8WPUX+CTw1d3GRcSD0yUjEcDb1W7pPCPSWHtr1fSkvFn9Hcz9KylTw 2JdySUJE4ewgXwe9TEm2ZmrrPlOVYaRYtR6IkMGZmSOm7b7qM1MAnRjNaGmtDvVg/C6M XVoQ== X-Gm-Message-State: AOAM533JLixYXQOUrNZz7f1D7ZXlLHHlNyS2Vx0ipyFt2WFdOUbfukME zZAb54ZPuWefhFDglTPdlaYAtg== X-Received: by 2002:a50:da42:: with SMTP id a2mr4967811edk.361.1632984123660; Wed, 29 Sep 2021 23:42:03 -0700 (PDT) Received: from apalos.home ([2a02:587:4672:866:2e56:dcff:fe9a:8f06]) by smtp.gmail.com with ESMTPSA id d7sm438991ejt.13.2021.09.29.23.42.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 29 Sep 2021 23:42:03 -0700 (PDT) From: Ilias Apalodimas To: trini@konsulko.com Cc: mark.kettenis@xs4all.nl, Ilias Apalodimas , Simon Glass , Bharat Gooty , Rayagonda Kokatanur , Rick Chen , Leo , Thomas Fitzsimmons , Bin Meng , =?utf-8?q?Marek?= =?utf-8?q?_Beh=C3=BAn?= , Green Wan , Brad Kim , Lukas Auer , Heinrich Schuchardt , David Abdurachmanov , Dimitri John Ledkov , u-boot@lists.denx.de Subject: [PATCH 3/3] treewide: Remove OF_PRIOR_STAGE Date: Thu, 30 Sep 2021 09:41:41 +0300 Message-Id: <20210930064143.432963-3-ilias.apalodimas@linaro.org> X-Mailer: git-send-email 2.33.0 In-Reply-To: <20210930064143.432963-1-ilias.apalodimas@linaro.org> References: <20210930064143.432963-1-ilias.apalodimas@linaro.org> MIME-Version: 1.0 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 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.2 at phobos.denx.de X-Virus-Status: Clean The previous patches removed OF_PRIOR_STAGE from the last consumers of the Kconfig option. Cleanup any references to it in documentation, code and configuration options. Signed-off-by: Ilias Apalodimas Reviewed-by: Simon Glass --- Changes since v1: - none dts/Kconfig | 11 ++--------- include/fdtdec.h | 4 ---- lib/fdtdec.c | 2 -- tools/binman/binman.rst | 16 ++++++---------- 4 files changed, 8 insertions(+), 25 deletions(-) -- 2.33.0 diff --git a/dts/Kconfig b/dts/Kconfig index 39270b47f9f0..100769017e12 100644 --- a/dts/Kconfig +++ b/dts/Kconfig @@ -22,7 +22,7 @@ config BINMAN config BINMAN_STANDALONE_FDT bool depends on BINMAN - default y if OF_BOARD || OF_PRIOR_STAGE + default y if OF_BOARD help This option tells U-Boot build system that a standalone device tree source is explicitly required when using binman to package U-Boot. @@ -32,7 +32,7 @@ config BINMAN_STANDALONE_FDT directory for a specific board. Such device tree sources are built for OF_SEPARATE or OF_EMBED. However for a scenario like the board device tree blob is not provided in the U-Boot build tree, but fed to U-Boot - in the runtime, e.g.: in the OF_PRIOR_STAGE case that it is passed by + in the runtime, e.g.: in the OF_BOARD case that it is passed by a prior stage bootloader. For such scenario, a standalone device tree blob containing binman node to describe how to package U-Boot should be provided explicitly. @@ -122,13 +122,6 @@ config OF_HOSTFILE This is only useful for Sandbox. Use the -d flag to U-Boot to specify the file to read. -config OF_PRIOR_STAGE - bool "Prior stage bootloader DTB for DT control" - help - If this option is enabled, the device tree used for DT - control will be read from a device tree binary, at a memory - location passed to U-Boot by the prior stage bootloader. - endchoice config DEFAULT_DEVICE_TREE diff --git a/include/fdtdec.h b/include/fdtdec.h index 8ac20c9a64f7..d0e13fc18313 100644 --- a/include/fdtdec.h +++ b/include/fdtdec.h @@ -55,10 +55,6 @@ struct bd_info; #define SPL_BUILD 0 #endif -#ifdef CONFIG_OF_PRIOR_STAGE -extern phys_addr_t prior_stage_fdt_address; -#endif - /* * Information about a resource. start is the first address of the resource * and end is the last address (inclusive). The length of the resource will diff --git a/lib/fdtdec.c b/lib/fdtdec.c index 7358cb6dd168..7b379564600d 100644 --- a/lib/fdtdec.c +++ b/lib/fdtdec.c @@ -1580,8 +1580,6 @@ int fdtdec_setup(void) puts("Failed to read control FDT\n"); return -1; } -# elif defined(CONFIG_OF_PRIOR_STAGE) - gd->fdt_blob = (void *)(uintptr_t)prior_stage_fdt_address; # endif # ifndef CONFIG_SPL_BUILD /* Allow the early environment to override the fdt address */ diff --git a/tools/binman/binman.rst b/tools/binman/binman.rst index 09e7b5719825..614df541c5ac 100644 --- a/tools/binman/binman.rst +++ b/tools/binman/binman.rst @@ -232,18 +232,18 @@ You can use other, more specific CONFIG options - see 'Automatic .dtsi inclusion' below. -Using binman with OF_BOARD or OF_PRIOR_STAGE +Using binman with OF_BOARD -------------------------------------------- Normally binman is used with a board configured with OF_SEPARATE or OF_EMBED. This is a typical scenario where a device tree source that contains the binman node is provided in the arch//dts directory for a specific board. -However for a board configured with OF_BOARD or OF_PRIOR_STAGE, no device tree -blob is provided in the U-Boot build phase hence the binman node information -is not available. In order to support such use case, a new Kconfig option -BINMAN_STANDALONE_FDT is introduced, to tell the build system that a standalone -device tree blob containing binman node is explicitly required. +However for a board configured with OF_BOARD, no device tree blob is provided +in the U-Boot build phase hence the binman node information is not available. +In order to support such use case, a new Kconfig option BINMAN_STANDALONE_FDT +is introduced, to tell the build system that a standalone device tree blob +containing binman node is explicitly required. Note there is a Kconfig option BINMAN_FDT which enables U-Boot run time to access information about binman entries, stored in the device tree in a binman @@ -252,10 +252,6 @@ For the other OF_CONTROL methods, it's quite possible binman node is not available as binman is invoked during the build phase, thus this option is not turned on by default for these OF_CONTROL methods. -See qemu-riscv64_spl_defconfig for an example of how binman is used with -OF_PRIOR_STAGE to generate u-boot.itb image. - - Access to binman entry offsets at run time (symbols) ----------------------------------------------------