{"id":2228752,"url":"http://patchwork.ozlabs.org/api/patches/2228752/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/patch/20260427100832.581887-6-ilias.apalodimas@linaro.org/","project":{"id":18,"url":"http://patchwork.ozlabs.org/api/projects/18/?format=json","name":"U-Boot","link_name":"uboot","list_id":"u-boot.lists.denx.de","list_email":"u-boot@lists.denx.de","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20260427100832.581887-6-ilias.apalodimas@linaro.org>","list_archive_url":null,"date":"2026-04-27T10:08:26","name":"[v3,5/6] common: Clean up setup_dest_addr()","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"14c41af0cf3ab4441ee9a34ef391c81992a8de33","submitter":{"id":74147,"url":"http://patchwork.ozlabs.org/api/people/74147/?format=json","name":"Ilias Apalodimas","email":"ilias.apalodimas@linaro.org"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/uboot/patch/20260427100832.581887-6-ilias.apalodimas@linaro.org/mbox/","series":[{"id":501617,"url":"http://patchwork.ozlabs.org/api/series/501617/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/list/?series=501617","date":"2026-04-27T10:08:21","name":"Relocate U-Boot in the last bank","version":3,"mbox":"http://patchwork.ozlabs.org/series/501617/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/2228752/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2228752/checks/","tags":{},"related":[],"headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=linaro.org header.i=@linaro.org header.a=rsa-sha256\n header.s=google header.b=SWgAcmUG;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de\n (client-ip=85.214.62.61; helo=phobos.denx.de;\n envelope-from=u-boot-bounces@lists.denx.de; receiver=patchwork.ozlabs.org)","phobos.denx.de;\n dmarc=pass (p=none dis=none) header.from=linaro.org","phobos.denx.de;\n spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de","phobos.denx.de;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=linaro.org header.i=@linaro.org header.b=\"SWgAcmUG\";\n\tdkim-atps=neutral","phobos.denx.de;\n dmarc=pass (p=none dis=none) header.from=linaro.org","phobos.denx.de;\n spf=pass smtp.mailfrom=ilias.apalodimas@linaro.org"],"Received":["from phobos.denx.de (phobos.denx.de [85.214.62.61])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4g3znN3QG6z1xvV\n\tfor ; Mon, 27 Apr 2026 20:09:27 +1000 (AEST)","from h2850616.stratoserver.net (localhost [IPv6:::1])\n\tby phobos.denx.de (Postfix) with ESMTP id 10D608421D;\n\tMon, 27 Apr 2026 12:09:08 +0200 (CEST)","by phobos.denx.de (Postfix, from userid 109)\n id 25E918407D; Mon, 27 Apr 2026 12:08:59 +0200 (CEST)","from mail-lj1-x22e.google.com (mail-lj1-x22e.google.com\n [IPv6:2a00:1450:4864:20::22e])\n (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits))\n (No client certificate requested)\n by phobos.denx.de (Postfix) with ESMTPS id 03A0A80433\n for ; Mon, 27 Apr 2026 12:08:57 +0200 (CEST)","by mail-lj1-x22e.google.com with SMTP id\n 38308e7fff4ca-3870778358aso79620011fa.1\n for ; Mon, 27 Apr 2026 03:08:56 -0700 (PDT)","from hera ([94.131.143.12]) by smtp.gmail.com with ESMTPSA id\n 38308e7fff4ca-38ecb6f0b88sm68554791fa.27.2026.04.27.03.08.54\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Mon, 27 Apr 2026 03:08:55 -0700 (PDT)"],"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,\n DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,\n SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.2","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=linaro.org; s=google; t=1777284536; x=1777889336; darn=lists.denx.de;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:from:to:cc:subject:date\n :message-id:reply-to;\n bh=R25wZyva9LfChnOjNMe8Tjuk8Tz3C++SIWoDQsrw6Gg=;\n b=SWgAcmUGL5Zi1E1fiKfuYV2+O+omIW+Ut1hyDKriJfN9WoBHwVU47T1mTsD/7D6RYj\n dw8cTDqDoZ93Vp9t60OU588MSDhQH75SIK5KM3h0RawZx6gRvWImZMaY9evTfRZRZcXw\n wrjr1yNSCLeuObb9bbEnrxFS8ZnrccEnKPEdD2GsR0MmjcEAjZPYSZQLUZ+IMQSwfHqZ\n Um+qSnLamrr1ozF6kHTq57PDx8ZsBQ1sf+oDNa+ySafMuYzYvVn+11xtdd+kzM+Oz/2y\n RGTCj0RiQvULX2lCE1ZtHdF3ZG25jz7b5j09/kzrSBVbUDUV9+pg1CRbFYS2yMtQV0bN\n g6Vg==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20251104; t=1777284536; x=1777889336;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from\n :to:cc:subject:date:message-id:reply-to;\n bh=R25wZyva9LfChnOjNMe8Tjuk8Tz3C++SIWoDQsrw6Gg=;\n b=LKJivlhcePLLIkwPhiiRni3fCmAbaIxPoxQkV7P04eJXsXqZ/0N8mVkMBXSqHEOVz/\n kunp5tTFLad6UgyxaPSX/8tqQA7mzI/+HtiCjzW2L2yfdKNBW9sdX36YnopTL/MJWr/U\n Ot+IVKG/502NGZjfEl3fwaz7KV0UscXp1kpgH3aIZto1VeLvo77WuSNFN9jxE9YTje7m\n dcfIcGgTqv9iLxxzE/0LdFw5KcTvTgv5XDKplO8dt0hUtQJ1N7+qhoVRguesZeNqMsD2\n GfO6tX5AKzxyx5ZdMy6oegQlUA/OJyTr6+DAVt/PnlIX8x9EhU/Zx4GjmSyG0JMDAoMR\n zjXg==","X-Forwarded-Encrypted":"i=1;\n AFNElJ9wPJw+cezM6fJBAKp/lPd2rvn/fUzSsMdujECf+Y9yWGaChwSzlS06BjbhzvBzyYzriiCH9vA=@lists.denx.de","X-Gm-Message-State":"AOJu0Yz/N7HXssSRpreNCOZ9NcVOihphQn08pa/14GF3OrAct5DPvz//\n c9xVgzD9hpFi+QmDtHFLSQ55ADgkJHS2PruPz4ouTc+rRoCN1GrOm2LeUMfBzzUPC1I=","X-Gm-Gg":"AeBDieuYbTWKb7F+XKoWlyNlI9VEC9u61cxQjT5Y4vvY3NQv9wONgalBTbOOPreu0ac\n bAKkApjQDBepBz244AqkVfSdpm/7kwfCRwB672y4IhNHoEroj1QNR+0iGfPP0cPNDZGDInqFS+v\n RKp4hXKmcdzRNQlurkFBd4Y+NvZcA8auXHyGbBqwB1CqtaobA/h6O+XDXEPBHfgLIKF0R2MLaav\n 2+YBGB2HNgUSJ2FiJ4dbfrCcwkemiC4wn0ywIpklMYk2hz3OzNnbLH8NZBf7hGIX+TzQ1AKd61o\n 22tDExhn4rtQ2WPqi1+apzkY0Ugt7ERPImbtxUkr+4+crzrtOVplXYjgeXc9SYTvpD9wtTE4AZF\n NRrgZ7NnlPrto58IyEt9jMK93tln+H0XUsN02z/3DJ2V8ns6YJUy85e+r/b9P2JqYYO2LD7Uwfd\n fuqFG4RgppbNzZcclRUST7wpHRe2ifSBxxiSjjdqkMq60MwHmQF8MFKeqgmhrMtWIczsOeZBv5W\n D0qB32W6AfEVP+iofvMf4aVbW7jsJOiTDDRQq60TfWtZtqgOWDBBpSg1B/AZE3CL6KAUjLKeNA0\n mfqJj881Zd5nGivKeQGF30+nKsLDNSNFc1Zmr4NWsqewTdiT++leWiUF0B+8DGe04K10OqVmQSz\n nhpZd6/B/XoBzbaswiYlJxmUznsY5X39Q9mkHSjxTzqxD4mYbnZ4n0tGO3sS1x6WeQGRxGR6HMd\n 0iord2kaFdtKN+Ni3TjT3tSLpWSyXnXZgcxKQY69/rXedtmKfjYT8GhdFOtO3vgdGr/vaQZGEaS\n docyFM9M2hMBCDD+ObqlNUBP9Z0rl4N0+D+","X-Received":"by 2002:a05:651c:1a0c:b0:38e:26bf:e5f3 with SMTP id\n 38308e7fff4ca-38ec7818e74mr140368781fa.8.1777284536144;\n Mon, 27 Apr 2026 03:08:56 -0700 (PDT)","From":"Ilias Apalodimas ","To":"trini@konsulko.com","Cc":"marek.vasut+renesas@mailbox.org, jonas@kwiboo.se, anshuld@ti.com,\n sjg@chromium.org, Ilias Apalodimas ,\n Michal Simek , u-boot@lists.denx.de","Subject":"[PATCH v3 5/6] common: Clean up setup_dest_addr()","Date":"Mon, 27 Apr 2026 13:08:26 +0300","Message-ID":"<20260427100832.581887-6-ilias.apalodimas@linaro.org>","X-Mailer":"git-send-email 2.53.0","In-Reply-To":"<20260427100832.581887-1-ilias.apalodimas@linaro.org>","References":"<20260427100832.581887-1-ilias.apalodimas@linaro.org>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.39","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n ","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n ","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"},"content":"Right now the function does\n- Re-adjust the ram_size based on Kconfig options\n- Set ram_top\n- Set the relocation address\n\nIt also does not set the ram_size in case ram_top grew\nfrom it's initial value. But ram_top and ram_size should\nalways be changed together.\n\nSo let's make things a bit cleaner and move the ram calculations\nin their own INITCALL\n\nReviewed-by: Simon Glass \nTested-by: Anshul Dalal \nTested-by: Michal Simek # Versal Gen 2 Vek385\nReviewed-by: Marek Vasut \nSigned-off-by: Ilias Apalodimas \n---\n common/board_f.c | 18 ++++++++++++------\n 1 file changed, 12 insertions(+), 6 deletions(-)\n\n--\n2.53.0","diff":"diff --git a/common/board_f.c b/common/board_f.c\nindex ac4dbf89a2e4..f58061026f8f 100644\n--- a/common/board_f.c\n+++ b/common/board_f.c\n@@ -336,13 +336,9 @@ static int setup_ram_base(void)\n \treturn 0;\n }\n\n-static int setup_dest_addr(void)\n+static int setup_ram_config(void)\n {\n \tdebug(\"Monitor len: %08x\\n\", gd->mon_len);\n-\t/*\n-\t * Ram is setup, size stored in gd !!\n-\t */\n-\tdebug(\"Ram size: %08llX\\n\", (unsigned long long)gd->ram_size);\n #if CONFIG_VAL(SYS_MEM_TOP_HIDE)\n \t/*\n \t * Subtract specified amount of memory to hide so that it won't\n@@ -358,8 +354,17 @@ static int setup_dest_addr(void)\n #endif\n \tgd->ram_top = gd->ram_base + get_effective_memsize();\n \tgd->ram_top = board_get_usable_ram_top(gd->mon_len);\n+\n+\tdebug(\"Ram top: %08llx\\n\", (unsigned long long)gd->ram_top);\n+\tdebug(\"Ram size: %08llx\\n\", (unsigned long long)gd->ram_size);\n+\n+\treturn 0;\n+}\n+\n+static int setup_dest_addr(void)\n+{\n \tgd->relocaddr = gd->ram_top;\n-\tdebug(\"Ram top: %08llX\\n\", (unsigned long long)gd->ram_top);\n+\tdebug(\"Reloc addr: %08llX\\n\", (unsigned long long)gd->relocaddr);\n\n \treturn arch_setup_dest_addr();\n }\n@@ -974,6 +979,7 @@ static void initcall_run_f(void)\n \t * - board info struct\n \t */\n \tINITCALL(setup_ram_base);\n+\tINITCALL(setup_ram_config);\n \tINITCALL(setup_dest_addr);\n #if CONFIG_IS_ENABLED(OF_BOARD_FIXUP) && \\\n !CONFIG_IS_ENABLED(OF_INITIAL_DTB_READONLY)\n","prefixes":["v3","5/6"]}