Linaro UEFI

UEFI sets wrong values to CNTFRQ on Fast Models platforms

Bug #1319856 reported by simon inizan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Linaro UEFI
Fix Committed
Undecided
Ryan Harkin
Linaro UEFI 14.07

Bug Description

in all these files:
ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-RTSM-A15_MPCore.dsc: gArmTokenSpaceGuid.PcdArmArchTimerFreqInHz|60000000
ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-FVP-AArch64.dsc: gArmTokenSpaceGuid.PcdArmArchTimerFreqInHz|120000000
ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-RTSM-AEMv8Ax4.dsc: gArmTokenSpaceGuid.PcdArmArchTimerFreqInHz|120000000
ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-RTSM-AEMv8Ax4-foundation.dsc: gArmTokenSpaceGuid.PcdArmArchTimerFreqInHz|120000000
ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-RTSM-A15.dsc: gArmTokenSpaceGuid.PcdArmArchTimerFreqInHz|60000000

The ArchTimer value (which I believe ends up being written in CNTFRQ or CNTFRQ_EL1) is not consistent with the input clock on these models, which is always 100MHz.

Linux uses the value from the DTS, for example:
https://git.linaro.org/kernel/linux-linaro-tracking.git/blob/HEAD:/arch/arm64/boot/dts/foundation-v8.dts

Leif Lindholm (leif-lindholm)
Changed in linaro-uefi:
assignee: nobody → Ryan Harkin (ryanharkin)
Revision history for this message
Ilias Biris (ibiris) wrote :

expecting this to be fixed upstream - need to test possibly to confirm the fix.

Revision history for this message
Ryan Harkin (ryanharkin) wrote :

This is fixed upstream:
http://sourceforge.net/p/edk2/code/15555

Changed in linaro-uefi:
status: New → Fix Released
Ilias Biris (ibiris)
Changed in linaro-uefi:
status: Fix Released → Fix Committed
milestone: none → 14.07
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.