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 6C441E784BC for ; Mon, 2 Oct 2023 15:19:49 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 714C686DC1; Mon, 2 Oct 2023 17:19:47 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=ti.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=ti.com header.i=@ti.com header.b="BeN4L7lS"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 26EE886DC6; Mon, 2 Oct 2023 17:19:46 +0200 (CEST) Received: from fllv0016.ext.ti.com (fllv0016.ext.ti.com [198.47.19.142]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id E3C2586DAD for ; Mon, 2 Oct 2023 17:19:43 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=ti.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=nm@ti.com Received: from lelv0265.itg.ti.com ([10.180.67.224]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 392FJXi8047653; Mon, 2 Oct 2023 10:19:33 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1696259973; bh=d9xFPjGCFNY15v0/CcBEKp21aJZ8361ikrYQke3bCmg=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=BeN4L7lSBR23TUSGsQ6QccufovtkGkganp8s46NBl9Q963RSPZa6pifb6uP4EB9cJ FdLonPr8bm+jIj7mMYPQVEO7B47TmEG0fkMDVHJZ15MJh5ilRSPeq7sxgpllgBbFzU NhcXnQsD2YtOmn9lZ/cFlXBotveVDFyb1KrDJJ6Y= Received: from DFLE111.ent.ti.com (dfle111.ent.ti.com [10.64.6.32]) by lelv0265.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 392FJXGs007136 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Mon, 2 Oct 2023 10:19:33 -0500 Received: from DFLE107.ent.ti.com (10.64.6.28) by DFLE111.ent.ti.com (10.64.6.32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Mon, 2 Oct 2023 10:19:33 -0500 Received: from lelv0326.itg.ti.com (10.180.67.84) by DFLE107.ent.ti.com (10.64.6.28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Mon, 2 Oct 2023 10:19:33 -0500 Received: from localhost (ileaxei01-snat2.itg.ti.com [10.180.69.6]) by lelv0326.itg.ti.com (8.15.2/8.15.2) with ESMTP id 392FJXY8009277; Mon, 2 Oct 2023 10:19:33 -0500 Date: Mon, 2 Oct 2023 10:19:33 -0500 From: Nishanth Menon To: Roger Quadros CC: , , , , , , , , , Subject: Re: [PATCH 1/6] arm: dts: k3-am64-evm: Fix boot Message-ID: <20231002151933.vxjpor26m6g4pgm3@compactor> References: <20230929134646.214781-1-rogerq@kernel.org> <20230929134646.214781-2-rogerq@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20230929134646.214781-2-rogerq@kernel.org> X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 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.8 at phobos.denx.de X-Virus-Status: Clean On 16:46-20230929, Roger Quadros wrote: > Since commit [1] A53 u-boot proper is broken. > This is because nodes marked as 'bootph-pre-ram' are > not available at u-boot proper before relocation. > > To fix this we mark all nodes in sk-u-boot.dtsi as > 'bootph-all'. > > Move vtt_supply and cbass_mcu node to -r5-evm.dts as > it is only required for R5 SPL. > > [1] > 9e644284ab812 ("dm: core: Report bootph-pre-ram/sram node as pre-reloc after relocation") > > Signed-off-by: Roger Quadros > --- > arch/arm/dts/k3-am642-evm-u-boot.dtsi | 56 ++++++++++++--------------- > arch/arm/dts/k3-am642-r5-evm.dts | 8 ++++ > 2 files changed, 32 insertions(+), 32 deletions(-) > > diff --git a/arch/arm/dts/k3-am642-evm-u-boot.dtsi b/arch/arm/dts/k3-am642-evm-u-boot.dtsi > index c85f57d40f..d06955dfbc 100644 > --- a/arch/arm/dts/k3-am642-evm-u-boot.dtsi > +++ b/arch/arm/dts/k3-am642-evm-u-boot.dtsi > @@ -11,109 +11,101 @@ > }; > > memory@80000000 { > - bootph-pre-ram; > + bootph-all; > }; > }; > > -&vtt_supply { > - bootph-pre-ram; > -}; > - > &cbass_main { > - bootph-pre-ram; > -}; > - > -&cbass_mcu { > - bootph-pre-ram; > + bootph-all; > }; > > &main_timer0 { > - bootph-pre-ram; > + bootph-all; > clock-frequency = <200000000>; > }; > > &main_conf { > - bootph-pre-ram; > + bootph-all; > chipid@14 { > - bootph-pre-ram; > + bootph-all; > }; > }; > > &main_pmx0 { > - bootph-pre-ram; > + bootph-all; > }; > > &main_i2c0_pins_default { > - bootph-pre-ram; > + bootph-all; > }; > > &main_i2c0 { > - bootph-pre-ram; > + bootph-all; > }; > > &main_uart0_pins_default { > - bootph-pre-ram; > + bootph-all; > }; > > &main_uart0 { > - bootph-pre-ram; > + bootph-all; > }; > > &usb0 { > dr_mode="peripheral"; > - bootph-pre-ram; > + bootph-all; > }; > > &usbss0 { > - bootph-pre-ram; > + bootph-all; > }; > > &main_mmc1_pins_default { > - bootph-pre-ram; > + bootph-all; > }; > > &main_usb0_pins_default { > - bootph-pre-ram; > + bootph-all; > }; > > &dmss { > - bootph-pre-ram; > + bootph-all; > }; > > &secure_proxy_main { > - bootph-pre-ram; > + bootph-all; > }; > > &dmsc { > - bootph-pre-ram; > + bootph-all; > k3_sysreset: sysreset-controller { > compatible = "ti,sci-sysreset"; > - bootph-pre-ram; > + bootph-all; > }; > }; > > &k3_pds { > - bootph-pre-ram; > + bootph-all; > }; > > &k3_clks { > - bootph-pre-ram; > + bootph-all; > }; > > &k3_reset { > - bootph-pre-ram; > + bootph-all; > }; > > &sdhci0 { > - bootph-pre-ram; > + bootph-all; > }; > > &sdhci1 { > - bootph-pre-ram; > + bootph-all; > }; > > &cpsw3g { > - bootph-pre-ram; > + bootph-all; > }; > > &cpsw_port2 { > diff --git a/arch/arm/dts/k3-am642-r5-evm.dts b/arch/arm/dts/k3-am642-r5-evm.dts > index 73461f8f6c..696735d8e2 100644 > --- a/arch/arm/dts/k3-am642-r5-evm.dts > +++ b/arch/arm/dts/k3-am642-r5-evm.dts > @@ -40,6 +40,10 @@ > }; > }; > > +&vtt_supply { > + bootph-pre-ram; > +}; > + > &cbass_main { > sysctrler: sysctrler { > compatible = "ti,am654-system-controller"; > @@ -53,6 +57,10 @@ > bootph-pre-ram; > }; > > +&cbass_mcu { > + bootph-pre-ram; > +}; A bit superfluous, but ok - we just need this atm for esm. > + > &mcu_esm { > bootph-pre-ram; > }; > -- > 2.34.1 > Reviewed-by: Nishanth Menon -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D