dev(timer): timer tests cleanup and rework#390
Merged
Conversation
Contributor
SrikarJosyula
commented
May 6, 2026
SrikarJosyula
commented
- Resolves: Add ACS Test Coverage for RS_L5TI_01 (System Counter 1GHz Requirement) #252
- CNTFRQ_EL0 cannot be relied on to determine the base frequency
- Drop timer tests t006 and t007 along with their declarations and entry IDs.
- Remove the related rule-to-test mappings and direct execution hooks from BSA/SBSA paths.
- Add new timer test t006 for S_L5TI_01 to verify that the OS-visible generic counter reports a 1GHz frequency through CNTFRQ_EL0
- Add a platform time API for an independent timer source and use it as a sanity check against CNTPCT_EL0 progression
- Update the testcase checklist and test scenario documents
- CNTFRQ_EL0 cannot be relied on to determine the base frequency - Drop timer tests t006 and t007 along with their declarations and entry IDs. - Remove the related rule-to-test mappings and direct execution hooks from BSA/SBSA paths. - Clean up UEFI/CMake build inputs so the deleted timer tests are no longer referenced. - Update the testcase checklist and test scenario documents Signed-off-by: Srikar Josyula <srikar.josyula@arm.com> Change-Id: Ib0a1822db4d569829132d2adf68998770b0c47ef
- Add timer test t006 for S_L5TI_01 to verify that the OS-visible generic counter reports a 1GHz frequency through CNTFRQ_EL0 - Add a platform time API for an independent timer source and use it as a sanity check against CNTPCT_EL0 progression - Include the new test into SBSA rule metadata, timer execution, UEFI build files, CMake test lists, and the SBSA checklist - Add UEFI ACPI/DT implementations using GetTime and baremetal stubs for RD platforms Signed-off-by: Srikar Josyula <srikar.josyula@arm.com> Change-Id: I553fbdde5804312ba556601d4d4dce3421d2068f
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.