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 DA8D6C433EF for ; Tue, 22 Mar 2022 12:36:37 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 5E53B839C7; Tue, 22 Mar 2022 13:36:35 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=bootlin.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=bootlin.com header.i=@bootlin.com header.b="ajyiK2lG"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 0547983A34; Tue, 22 Mar 2022 13:36:33 +0100 (CET) Received: from relay2-d.mail.gandi.net (relay2-d.mail.gandi.net [IPv6:2001:4b98:dc4:8::222]) (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 790D78395F for ; Tue, 22 Mar 2022 13:36:29 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=clement.leger@bootlin.com Received: (Authenticated sender: clement.leger@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id A886E40004; Tue, 22 Mar 2022 12:36:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1647952589; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=88E4V+JutDbS1mz0Q0PPiAa+N34cqypCHT+STVRpBjA=; b=ajyiK2lGM9g6a7MOQBGlZo9gRH71YK+28+GVWJ0AKuToZokLwpCjn1SgQeSAG0AikYlLwH XKxbB1pDapuxe9IjI9hw5tF4+TuFLILGboXwBGQD/MgB6T07HC/FIw3ubLivs6JpbxK0At r7RrNKl0UC9Vrl8lHK8YEyhyEHwEEBh8GB3yQi1LPFgSwIKiNI/LS9XhVZsaKcM/20iZhu aR3KjWmGKQpdtL9h0OBJrqWmcXAmURulxKP+6pHYO7We/VCN2rks1F2rN8X/jh5dRc3ZoM xNhwQwpj77ej7cNKcmh6LcnkxSfcnhMJXOulVfo3H6mknuV20vyeDz28pZ+gNA== Date: Tue, 22 Mar 2022 13:35:08 +0100 From: =?UTF-8?B?Q2zDqW1lbnQgTMOpZ2Vy?= To: Cc: , , , , Subject: Re: [PATCH v7 4/4] configs: sama5d2: enable option CONFIG_ATMEL_TCB_TIMER Message-ID: <20220322133508.31630f38@fixe.home> In-Reply-To: References: <20220311093539.72669-1-clement.leger@bootlin.com> <20220311093539.72669-5-clement.leger@bootlin.com> Organization: Bootlin X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.31; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable 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.5 at phobos.denx.de X-Virus-Status: Clean Le Tue, 22 Mar 2022 11:18:39 +0000, a =C3=A9crit : > On 3/11/22 11:35 AM, Cl=C3=A9ment L=C3=A9ger wrote: > > Enable CONFIG_ATMEL_TCB_TIMER and disable CONFIG_ATMEL_PIT_TIMER. > > This will allow using the TCB timer instead of the PIT one when running > > under OP-TEE. =20 >=20 > Hello Clement, >=20 > We have to make sure that the PIT_TIMER was not mandatory for SPL=20 > operations, and if it was, the TCB_TIMER successfully replaces it. >=20 > If we have the PIT_TIMER with the u-boot property > u-boot,dm-pre-reloc; > then the PIT_TIMER is available in the SPL and the SPL works : >=20 > RomBOOT > ;4=E2=96=92ls=E2=96=92K{=E2=96=92KOp=E2=96=92C=E2=96=92=E2=96=92 > >=20 > U-Boot SPL 2022.04-rc4-00204-g5f7086d8a8-dirty (Mar 22 2022 - 13:10:58=20 > +0200) > SPL: Unsupported Boot Device! > SPL: failed to boot from all boot devices >=20 >=20 > (well, it fails to boot because I removed the MMC drivers to have it=20 > smaller than 64k, but it boots) >=20 >=20 > However, with your patches, even if I add the pre-reloc property to the=20 > tcb nodes, it fails to boot: >=20 > RomBOOT > ;4=E2=96=92Ls=E2=96=92k{=E2=96=92kOp=E2=96=92C=E2=96=92=E2=96=92 > > Could not initialize timer (err -22) >=20 > Could not initialize timer (err -22) >=20 > Could not initialize timer (err -22) >=20 > Could not initialize timer (err -22) >=20 > ... >=20 >=20 >=20 > Could you investigate this please ? Hi Eugen, Ok, I'll try to debug that ! >=20 > Otherwise we cannot switch to using the TCB if it does not fully replace= =20 > the PIT_TIMER in all the use cases. Yes, that is completely understandable. >=20 >=20 > Thanks, >=20 > Eugen --=20 Cl=C3=A9ment L=C3=A9ger, Embedded Linux and Kernel engineer at Bootlin https://bootlin.com