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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D4081C433EF for ; Mon, 1 Nov 2021 11:42:36 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id C1673610CF for ; Mon, 1 Nov 2021 11:42:35 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org C1673610CF Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=denx.de Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 8291A83323; Mon, 1 Nov 2021 12:42:33 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=denx.de Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1635766953; bh=CGv7g1pLM4Ei3Rjbsk3HW47/kFoULJSaeOxqJSgPs7A=; h=Date:From:To:Cc:Subject:In-Reply-To:References:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=EZhkyTmgvpKPA2PpstNhQeRNB07UEPOWLhiMYRAvQ6K2o9IQkIrLuLIzD/TYXArtv Dd7hQbgSRPGCfMA9VQNQsFj+QkBN0FmI0qo04D0nZKIiE7ucNR9hfga/4SPurxpnOg kVuweTSCE/05MjIj6EayOj7qJV0FKYj9Tybbm0zVxSbxSnfR3sPu/aRGRXsSwO/7+J PXcBguiqscV+IJpMQ5NOpnCVp8h9VdpKrcCD5/jT24rXRh1aWyvRZuRJYa75r7pHub 1dhQ4UNyLs+vTQoyDmvH/SZc7csk6BOlOzYJ2mx9LcYBvqe/SkRau3UY3SwXW8CoLr wWxRxMBOyY5qA== Received: from ktm (85-222-111-42.dynamic.chello.pl [85.222.111.42]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) (Authenticated sender: lukma@denx.de) by phobos.denx.de (Postfix) with ESMTPSA id 4F821831C9; Mon, 1 Nov 2021 12:42:31 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1635766951; bh=CGv7g1pLM4Ei3Rjbsk3HW47/kFoULJSaeOxqJSgPs7A=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=EblSYSI/3FutUMHAI/i3Ju/wMdPHgIMeSxGK94le9yAOCn6nKKdaB2Ltqn/GWuWMf tA+a0F7SxyZ33xgsJR0lCUR1NgPt0A7n9oPGFhXxIZywqk4RKzufCjVvKRYhcVortL 0nkCmVFr8QuyC73QafL1cv/t2fElTDVXVjodICsshMwRkaqbJclNunNM3OLFf/DXbg XtG6Hr1oJ2k8MmFcBiquHABdRaX1YS3srXf2Z/3UQjCWR3cxZQm5cXxl6jldxKAe0h zSKZMbhR6oCy+5g2OCc9xMoBEvrmLCoCcenxkbP8uX5qfv+88wCxVOY1bGq16mSE8S wiyJ3dNxeHdmQ== Date: Mon, 1 Nov 2021 12:42:30 +0100 From: Lukasz Majewski To: Simon Glass Cc: u-boot@lists.denx.de, Stefano Babic , Simon Glass , Tom Rini Subject: Re: [PATCH v1] arm: xea: Modify board code to generate single binary u-boot Message-ID: <20211101124153.0ff53072@ktm> In-Reply-To: References: <20211023150343.26260-1-lukma@denx.de> Organization: denx.de X-Mailer: Claws Mail 3.18.0 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; boundary="Sig_/PlbQLYQtWLLb3XC6/VmZzV4"; protocol="application/pgp-signature" 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 --Sig_/PlbQLYQtWLLb3XC6/VmZzV4 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Hi Simon, > Hi Lukasz, >=20 > On Sat, 23 Oct 2021 at 09:04, Lukasz Majewski wrote: > > > > This change provides the possibility to build XEA (imx287 based) > > board U-Boot as a single binary (without support for > > CONFIG_SPL_FRAMEWORK). > > > > The generated u-boot.sb can be used in the factory environment to > > for example perform initial setup or HW testing. > > > > It can be used with 'uuu' utility > > (SDPS: boot -f /srv/tftp/xea/u-boot.sb) > > > > In the configs/imx28_xea_defconfig one needs to disable following > > configs: =20 >=20 > Something is missing here. I've posted the v2 of this patch with this information added. >=20 > > > > The board_init_ll() is used in arch/arm/cpu/arm926ejs/mxs/start.S, > > which is utilized when CONFIG_SPL_FRAMEWORK is disabled. > > > > However, when it is enabled the arch/arm/cpu/arm926ejs/start.S is > > used, which requires the lowlevel_init() function. > > > > Signed-off-by: Lukasz Majewski > > > > --- > > > > board/liebherr/xea/spl_xea.c | 7 +++++++ > > board/liebherr/xea/xea.c | 2 +- > > 2 files changed, 8 insertions(+), 1 deletion(-) =20 >=20 > Could you add a Kconifig to control this feature and add something to > the board docs? This don't need any special Kconfig option. You just undefine the CONFIG_SPL_FRAMEWORK, and that's all. As fair as I can tell - the CONFIG_SPL_FRAMEWORK option is used with for example imx28evk board. The issue here is that imx28 SoC needs u-boot.sb to boot up when USB is used to debrick it (this is somewhat the legacy approach with a single binary u-boot). >=20 > Regards, > Simon Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de --Sig_/PlbQLYQtWLLb3XC6/VmZzV4 Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQEzBAEBCgAdFiEEgAyFJ+N6uu6+XupJAR8vZIA0zr0FAmF/0qYACgkQAR8vZIA0 zr29Zwf/Q3poRgv9XuXOdX1IL/ZvOYNriWJb8/T0S1nIfDWZ8488XxGq9YUmKPkX jz9Hba8wA8xK0o9VKF8vkFmwqYQ7jUeOZXuFksYAotdqrfcDEr8mi17DNIgQa7rx gkeGlW2UZuw+LOR3j32w8olR0MjrDHGjx1El4lt24KJMOAhR7841xQz6mWOnOa3M p9fDHwFEJDK5yUVSebZg1Ldd4B/5frB2QUiHzaIFgTQXeH7FErLu6nPozZcw4H5T BprGqFJnbgCgvdauaqvn7s2jRBz0CPmsL2/A0aLiWW9Ob4NvL2TBkfbnCSAJUyqj gZvaXjPCnoCYsc4nSg2t7AyyP3+J5A== =5jJa -----END PGP SIGNATURE----- --Sig_/PlbQLYQtWLLb3XC6/VmZzV4--