Re: [PATCH edk2-platforms v2 0/2] Platform/ARM/JunoPkg: Timer fixes

["Ard Biesheuvel" <ardb@kernel.org>]

On Tue, 21 Feb 2023 at 02:03, Rebecca Cran <rebecca@quicinc.com> wrote:
>
> The use of the EmbeddedPkg/MetronomeDxe driver on Juno can cause problems
> with drivers that use gBS->Stall, since it takes 10x longer than
> requested. For example requesting a timeout of 1 ms when doing a USB
> bulk transfer results in it taking 100 ms. Switching to the
> MdeModulePkg/Universal/Metronome driver fixes this since it assumes the
> timer clock ticks at least every 100 ns.
>
> While here, set the PCD value of the timer frequency to 0 so it gets
> read from the SoC instead of hard-coding it.
>
> Changes in v2:
>
> Dropped the patch to set PcdTimerPeriod.
>
> Rebecca Cran (2):
>   Platform/ARM/JunoPkg: Switch to MdeModulePkg/Universal/Metronome
>     driver
>   Platform/ARM/JunoPkg: Set PcdArmArchTimerFreqInHz to 0 to read from
>     CNTFRQ_EL0
>

Reviewed-by: Ard Biesheuvel <ardb@kernel.org>

Pushed as 81ec441723a0..e20ee6e3a65d

Thanks!

>  Platform/ARM/JunoPkg/ArmJuno.dsc | 6 +++---
>  Platform/ARM/JunoPkg/ArmJuno.fdf | 2 +-
>  2 files changed, 4 insertions(+), 4 deletions(-)
>
> --
> 2.30.2
>