Re: [PATCH v7 4/4] configs: sama5d2: enable option CONFIG_ATMEL_TCB_TIMER

["Clément Léger" <clement.leger@bootlin.com>]

Le Tue, 22 Mar 2022 11:18:39 +0000,
<Eugen.Hristev@microchip.com> a écrit :

> On 3/11/22 11:35 AM, Clément Léger 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.  
> 
> Hello Clement,
> 
> We have to make sure that the PIT_TIMER was not mandatory for SPL 
> operations, and if it was, the TCB_TIMER successfully replaces it.
> 
> 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 :
> 
> RomBOOT
> ;4▒ls▒K{▒KOp▒C▒▒
> <debug_uart>
> 
> U-Boot SPL 2022.04-rc4-00204-g5f7086d8a8-dirty (Mar 22 2022 - 13:10:58 
> +0200)
> SPL: Unsupported Boot Device!
> SPL: failed to boot from all boot devices
> 
> 
> (well, it fails to boot because I removed the MMC drivers to have it 
> smaller than 64k, but it boots)
> 
> 
> However, with your patches, even if I add the pre-reloc property to the 
> tcb nodes, it fails to boot:
> 
> RomBOOT
> ;4▒Ls▒k{▒kOp▒C▒▒
> <debug_uart>
> Could not initialize timer (err -22)
> 
> Could not initialize timer (err -22)
> 
> Could not initialize timer (err -22)
> 
> Could not initialize timer (err -22)
> 
> ...
> 
> 
> 
> Could you investigate this please ?

Hi Eugen,

Ok, I'll try to debug that !

> 
> Otherwise we cannot switch to using the TCB if it does not fully replace 
> the PIT_TIMER in all the use cases.

Yes, that is completely understandable.

> 
> 
> Thanks,
> 
> Eugen


-- 
Clément Léger,
Embedded Linux and Kernel engineer at Bootlin
https://bootlin.com