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 2DB17C433FE for ; Fri, 18 Nov 2022 13:49:01 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 0B10885258; Fri, 18 Nov 2022 14:48:59 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=gmail.com 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=gmail.com header.i=@gmail.com header.b="plYFl6Y9"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 1A12A85264; Fri, 18 Nov 2022 14:48:56 +0100 (CET) Received: from mail-ed1-x52e.google.com (mail-ed1-x52e.google.com [IPv6:2a00:1450:4864:20::52e]) (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 7A74985254 for ; Fri, 18 Nov 2022 14:48:31 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=oliver.graute@gmail.com Received: by mail-ed1-x52e.google.com with SMTP id y24so2202813edi.10 for ; Fri, 18 Nov 2022 05:48:31 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=user-agent:in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date:from:to:cc :subject:date:message-id:reply-to; bh=NpLHl2miysbqyRuPDSRPFkedTtaqVVM9DNfKgT3+ecE=; b=plYFl6Y9BbnZvnYofmDqlonJn3FpYkfENzCu7le2BmiG+AQER2BgcjK9oMzWyTL6Av X+k6e0pBHFK9o3mmFQ2+0ndPd0rtgNxWGaE528MFqg4YYr8rT+DCfdkH5mCPjlVLZKGk f42Zj8F8qaH3+XmMhAyKEorQusZEvXTnuvVo7DBUaOYscQY/n5syfefXbF+jq4fE9iJr t4DIO2BuVtwht6WDpwpU/6CfdgvrMpr1C5Zd78fmGrXXYF4mZw3/rCzC+7b8x+rnkDz7 n1sEhFAztY1SA0Bp0roX7NZNmSwaxfyL4tgWS/ffIROgv0L8Kn5Ln1jtBejFA2eAAwPJ RYGg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=user-agent:in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=NpLHl2miysbqyRuPDSRPFkedTtaqVVM9DNfKgT3+ecE=; b=RCvndBiz8CsyQdHpiPovng/7xOzowHHJMWmqFOFubXnr7OJdxU54lScaExG6AsZVUx GPmGn/HfIbzhC/CcG9ZY8A6Ow3fN5Pie3K0PlsTdVs2eP+0twOHbZdeIOsArvm8qGmed 9tBjKwcqdcbzmBGx3gnG6fq0pIIVwWvASXVuo9Q3oJSs+eDXKRuk8lFUI4/82KgEnkeY CDOIxlhESXzVs5+4Jz8/byw1fcjnJeY6hj9uWT6m4bGgKtgo8BufCcSdfmqfH9dkA8vb YeKa24bOronSr79xrkg8uG9LUQRLirCcigMrM5Uof82Y8GHdAQVHEiGa/DoHUZeLe0B6 mUug== X-Gm-Message-State: ANoB5pmlYkRur/CyARQBqVOX7BUk+T/ve/Mf/FITvJhJNP6SgBPUEKxh TGXvaTRaOu2IJckwexbZv1Q= X-Google-Smtp-Source: AA0mqf5tOsYsTHHu91A2uZrADLhquAREeDSgCLI3RuRWJbCLPWKCgfjEZS6bfHNDFzrSTOx7VvOLeA== X-Received: by 2002:a05:6402:5d5:b0:464:fa1:9ece with SMTP id n21-20020a05640205d500b004640fa19ecemr6482690edx.262.1668779307822; Fri, 18 Nov 2022 05:48:27 -0800 (PST) Received: from localhost (business-90-187-74-145.pool2.vodafone-ip.de. [90.187.74.145]) by smtp.gmail.com with ESMTPSA id ku22-20020a170907789600b007ad98918743sm1704744ejc.1.2022.11.18.05.48.26 (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Fri, 18 Nov 2022 05:48:27 -0800 (PST) Date: Fri, 18 Nov 2022 14:41:51 +0100 From: Oliver Graute To: Fabio Estevam Cc: Stefano Babic , Marcel Ziswiler , Ye Li , peng.fan@nxp.com, uboot-imx@nxp.com, u-boot@lists.denx.de, Simon Glass , alpernebiyasak@gmail.com Subject: Re: [PATCH v2 0/8] imx8: switch missing boards to binman Message-ID: <20221118134151.GC2534@optiplex> Mail-Followup-To: Fabio Estevam , Stefano Babic , Marcel Ziswiler , Ye Li , peng.fan@nxp.com, uboot-imx@nxp.com, u-boot@lists.denx.de, Simon Glass , alpernebiyasak@gmail.com References: <20221104150345.23853-1-oliver.graute@kococonnector.com> <20221109161927.GB17674@optiplex> <20221111125303.GA2534@optiplex> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.4 (2018-02-28) 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 On 11/11/22, Fabio Estevam wrote: > On Fri, Nov 11, 2022 at 2:40 PM Fabio Estevam wrote: > > > I removed SPL support, which does not seems to be needed as the scufw > > handles DDR init. > > > > I don't have access to an imx8qm/qxp board here. > > > > Could you try removing SPL support from your board and see if it boots > > with binman support? > > Ok, let's SPL for now as this is a different topic for discussion. > > > With the change below against u-boot-imx master-next branch, the > imx8qm/qxp boards build fine without blobs: > > diff --git a/arch/arm/dts/imx8qm-u-boot.dtsi b/arch/arm/dts/imx8qm-u-boot.dtsi > index 3507489a813c..a3e0af48109b 100644 > --- a/arch/arm/dts/imx8qm-u-boot.dtsi > +++ b/arch/arm/dts/imx8qm-u-boot.dtsi > @@ -10,7 +10,7 @@ > }; > > &binman { > -#ifdef CONFIG_SPL > +#ifdef CONFIG_SPL_BUILD > u-boot-spl-ddr { > align = <4>; > align-size = <4>; > @@ -50,7 +50,7 @@ > arch = "arm64"; > compression = "none"; > description = "U-Boot (64-bit)"; > - load = ; > + load = ; > type = "standalone"; > > uboot-blob { > diff --git a/arch/arm/dts/imx8qxp-u-boot.dtsi b/arch/arm/dts/imx8qxp-u-boot.dtsi > index 01183f8ade63..7622c40906f1 100644 > --- a/arch/arm/dts/imx8qxp-u-boot.dtsi > +++ b/arch/arm/dts/imx8qxp-u-boot.dtsi > @@ -10,7 +10,7 @@ > }; > > &binman { > -#ifdef CONFIG_SPL > +#ifdef CONFIG_SPL_BUILD > u-boot-spl-ddr { > align = <4>; > align-size = <4>; > @@ -50,7 +50,7 @@ > arch = "arm64"; > compression = "none"; > description = "U-Boot (64-bit)"; > - load = ; > + load = ; > type = "standalone"; > > uboot-blob { > > Could you please test it on your boards? Ok, I can confirm it builds without the binary blobs with this changes. I'll change it in patches accordingly Best regards, Oliver