On Thu, 20 Jun 2024 at 16:33, Leif Lindholm <[email protected]> wrote:
>
> Related to https://github.com/tianocore/edk2/pull/5797
>
> PcdArmArchTimerFreqInHz is about to be removed, as it is now obsolete.
> Its functionality has been partially broken, and mostly ignored, for
> all AArch64 platforms since December 2020.
>
> This set cleans up some broken line endings in .dsc* files, then
> drops all non-invasive references to the Pcd:
> - .dsc* files setting it to 0 (which is the default in the definition,
>   and means "just read it from the system register instead")
> - .inf files declaring a dependency that is in fact not there in
>   current code.
>
> Finally, it drops the setting of the Pcd for platforms that set it to
> non-0. This has *never* done the right thing on these platforms since
> they are all AArch64, but it may affect timer timeout, so deserves
> deeper testing.
>
> Cc: Ard Biesheuvel <[email protected]>
> Cc: Chuong Tran <[email protected]>
> Cc: Graeme Gregory <[email protected]>
> Cc: Marcin Juszkiewicz <[email protected]>
> Cc: Marcin Wojtas <[email protected]>
> Cc: Meenakshi Aggarwal <[email protected]>
> Cc: Narinder Dhillon <[email protected]>
> Cc: Nhi Pham <[email protected]>
> Cc: Rebecca Cran <[email protected]>
> Cc: Sami Mujawar <[email protected]>
> Cc: Thomas Abraham <[email protected]>
> Cc: Wenyi Xie <[email protected]>
>
> Leif Lindholm (5):
>   Platform/SbsaQemu: fix .dsc line endings
>   Platform,Silicon: drop redundant uses of PcdArmArchTimerFreqInHz
>   Platform/ARM: drop use of PcdArmArchTimerFreqInHz
>   Platform/Hisilicon: drop D05 use of PcdArmArchTimerFreqInHz
>   Silicon/Marvell: drop use of PcdArmArchTimerFreqInHz
>

Looks fine to me

Reviewed-by: Ard Biesheuvel <[email protected]>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#119660): https://edk2.groups.io/g/devel/message/119660
Mute This Topic: https://groups.io/mt/106780872/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to