{"id":2228751,"url":"http://patchwork.ozlabs.org/api/patches/2228751/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/patch/20260427100832.581887-5-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-5-ilias.apalodimas@linaro.org>","list_archive_url":null,"date":"2026-04-27T10:08:25","name":"[v3,4/6] common: move ram_base calculation to independent INITCALL()","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"e3b30cb4b0be8d6ce4f42fe67b437eeab2fb17e0","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-5-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/2228751/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2228751/checks/","tags":{},"related":[],"headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","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=DpkTP2T+;\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=\"DpkTP2T+\";\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 4g3znC4NNxz1xvV\n\tfor <incoming@patchwork.ozlabs.org>; Mon, 27 Apr 2026 20:09:19 +1000 (AEST)","from h2850616.stratoserver.net (localhost [IPv6:::1])\n\tby phobos.denx.de (Postfix) with ESMTP id 900EF841DF;\n\tMon, 27 Apr 2026 12:08:59 +0200 (CEST)","by phobos.denx.de (Postfix, from userid 109)\n id F2F968423D; Mon, 27 Apr 2026 12:08:58 +0200 (CEST)","from mail-lj1-x22f.google.com (mail-lj1-x22f.google.com\n [IPv6:2a00:1450:4864:20::22f])\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 ED302839D9\n for <u-boot@lists.denx.de>; Mon, 27 Apr 2026 12:08:54 +0200 (CEST)","by mail-lj1-x22f.google.com with SMTP id\n 38308e7fff4ca-38a01c80c34so97665171fa.0\n for <u-boot@lists.denx.de>; Mon, 27 Apr 2026 03:08:54 -0700 (PDT)","from hera ([94.131.143.12]) by smtp.gmail.com with ESMTPSA id\n 38308e7fff4ca-38ecb6f0b88sm68554791fa.27.2026.04.27.03.08.52\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Mon, 27 Apr 2026 03:08:53 -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=1777284534; x=1777889334; 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=981fWpuAbK2bvrnCZZVrwADu5/ky6PA26Iw6kXj799M=;\n b=DpkTP2T+PSsn6QALxoPN4NVGU3WiB/3RATJ5Cy7QEnXpZpBm3uP3PjMRNc9WPyRbK5\n Pqx5b07rPuGT5gIXkN8CXVI7JoD9ffNdN4xUYbW96EhYFCVHb6UrzIX/tr14W5Wf6yEY\n zqOdE69yfGTTm4t9l3mDzWNw6WcCbobnx7RX54z0LMweUJzLabYGglUwlrr+XE/xEK7l\n H0YLKkUWCGJGm22BOmFtghM0+PDsYL6HikEkg9Lmxez7hYBykNjgt2Ji0IVeXlgHn/4I\n xa18YHu8ja+DVL2ob7d8TZdz3u5THOeZ6wTrUffxb8P6c2FNnnpAZwwssEp7IcovodmC\n aB/Q==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20251104; t=1777284534; x=1777889334;\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=981fWpuAbK2bvrnCZZVrwADu5/ky6PA26Iw6kXj799M=;\n b=BIwM8ZyOoVK9OMyg++do6PRkhxyQGd2gViKq5SJtEOURES+3AnOJM7U5WeK93j22OB\n f0pDZfgC+63BMrC6QDd9tpWx9jsxI+7Wy9wJ4C0NFAtvG8UxR/K2D5d0IxclAwybA4qN\n 7AezYhb5ZbUTuxVoAYU8c2xOKQXtG7acAvSs5QDIzRuUNEBSVKCbuevd2zHPn7rQHbQA\n BU3DrzdF2STG9kze7nuNfEHDx4Io3Z8uhZw/wIVFTehvKtolWMu1Rr77rAIlaxEKGx2B\n uwEN+oMk3jhRIRKOaEwdh/npZUCSZsAn63EhzWwkZxZ/RfPlLXkFWf3Ka+NuWDP+PaeB\n 8ZBg==","X-Forwarded-Encrypted":"i=1;\n AFNElJ9xPB9LDgQ5C+WjjqbVejcXIm4sPDr7dDSpvzE2+JgGR5fT158rTH0bhJAATLfhQ1cy8hMDLGA=@lists.denx.de","X-Gm-Message-State":"AOJu0Yx8KuvegeQqrKX61hnOR1FTs664y3bMpheA8QgBhLgcIAz6S9Xa\n l5p+rin2bxZwzLdIOk8anF70vPhSqSs806NzQyvKlFY8wKCbtwX9MO1v7Qx1AOIHuVU=","X-Gm-Gg":"AeBDietrUvfPBmYW8bifKlC/B3haMX4c2djbeOrwSqz2yChSsfj6vvMHniP308e1845\n XQtCLJ8FG9wncWQSemp+qwHuZi8uvRty84wl1/FRmdicWwWdojQMDYpLKvO8QtKAz4wo+Eh7ZFW\n 2eqoPsgnA5tfEMLVel9UKrTvaqGsSevkeR3oHgfHebZpNgq2wXzDHVWJUE8xOtXSaIT4Tx4l9aA\n +C2n6BqYPIFh2hKX3GUFtt8KqDOBoU0gwPxnrNK87LFIPDo7tHOxC5vYpLPXibkAvdRfp4EWThM\n +u97UkCPX0Fo74ZfyySU7/0z68fMqCvQiK8DtoWeR4HQW8GAxRwjO7oTuYTAsyuxFW5EdsKmn0S\n lJpJTHIQLrclaelkpe+po9N1WEElJ/CpZLJt77PFh6JfNEo5UukFXL/Yov4bUjBq4GzeKHFBYHM\n ySKaFWWzvceV57J49Q+f0T/wVnENKuVbKUIrn5IABUmEmepyb22mgQaAuYe2mAnuRNVByIP77IV\n 90F1bAU2Y86xcS/Xy7MUarv3LomgSG2RmLshQIs3Svzdt9HZAT6M0DBAAGbrHrIcewlKn9ccXgi\n P2JO/l7xc6P7/52WS+NGkRurH646Lb0kj2ZfmUPlAxLTVI8y7R52H1l4xaY2rXt76WIzax66MWD\n VaAus3U2sodFG1C5VahtyYTds4kiH2rS2Jf9KfgMHjECVdaZJc+5PcFPBXVPSTyEAwkG+/n3SqI\n Ak8jx9Rvzb+bQ1PO7LjtpORhF1Ani4Y5xuwgXSuiD6I+oyp8sBfbvwxqx/MK5OrCsXStVwp4y8L\n e1bSoiIOIDVKl3fv6JylZQm/yg/V2+kXQ/y","X-Received":"by 2002:a05:651c:b29:b0:38e:a181:9e41 with SMTP id\n 38308e7fff4ca-38ec78353b0mr146692181fa.12.1777284533984;\n Mon, 27 Apr 2026 03:08:53 -0700 (PDT)","From":"Ilias Apalodimas <ilias.apalodimas@linaro.org>","To":"trini@konsulko.com","Cc":"marek.vasut+renesas@mailbox.org, jonas@kwiboo.se, anshuld@ti.com,\n sjg@chromium.org, Ilias Apalodimas <ilias.apalodimas@linaro.org>,\n Michal Simek <michal.simek@amd.com>, u-boot@lists.denx.de","Subject":"[PATCH v3 4/6] common: move ram_base calculation to independent\n INITCALL()","Date":"Mon, 27 Apr 2026 13:08:25 +0300","Message-ID":"<20260427100832.581887-5-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 <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n <mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<https://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n <mailto:u-boot-request@lists.denx.de?subject=subscribe>","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>","X-Virus-Scanned":"clamav-milter 0.103.8 at phobos.denx.de","X-Virus-Status":"Clean"},"content":"Currently, ram_base is calculated within setup_dest_addr().\nHowever, upcoming patches that enable U-Boot relocation to the highest\nDRAM bank require ram_base to be initialized earlier.\n\nThe default dram_init_banksize() definition relies on ram_base\nto calculate the start of the first bank. But following patches\nwill move that function to execute immediately before setup_dest_addr().\n\nSo let's split the ram_base initialization in its own INITCALL.\n\nReviewed-by: Simon Glass <sjg@chromium.org>\nTested-by: Anshul Dalal <anshuld@ti.com>\nTested-by: Michal Simek <michal.simek@amd.com> # Versal Gen 2 Vek385\nReviewed-by: Marek Vasut <marek.vasut+renesas@mailbox.org>\nSigned-off-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>\n---\n common/board_f.c | 12 +++++++++---\n 1 file changed, 9 insertions(+), 3 deletions(-)\n\n--\n2.53.0","diff":"diff --git a/common/board_f.c b/common/board_f.c\nindex f0d07cb6f593..ac4dbf89a2e4 100644\n--- a/common/board_f.c\n+++ b/common/board_f.c\n@@ -328,6 +328,14 @@ __weak int arch_setup_dest_addr(void)\n \treturn 0;\n }\n\n+static int setup_ram_base(void)\n+{\n+#ifdef CFG_SYS_SDRAM_BASE\n+\tgd->ram_base = CFG_SYS_SDRAM_BASE;\n+#endif\n+\treturn 0;\n+}\n+\n static int setup_dest_addr(void)\n {\n \tdebug(\"Monitor len: %08x\\n\", gd->mon_len);\n@@ -347,9 +355,6 @@ static int setup_dest_addr(void)\n \t * get fixed.\n \t */\n \tgd->ram_size -= CONFIG_SYS_MEM_TOP_HIDE;\n-#endif\n-#ifdef CFG_SYS_SDRAM_BASE\n-\tgd->ram_base = CFG_SYS_SDRAM_BASE;\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@@ -968,6 +973,7 @@ static void initcall_run_f(void)\n \t *  - monitor code\n \t *  - board info struct\n \t */\n+\tINITCALL(setup_ram_base);\n \tINITCALL(setup_dest_addr);\n #if CONFIG_IS_ENABLED(OF_BOARD_FIXUP) && \\\n     !CONFIG_IS_ENABLED(OF_INITIAL_DTB_READONLY)\n","prefixes":["v3","4/6"]}