Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
- PCIe tests to not run if no ECAM
- For UEFI PCIe tests 840 and 601 , presence of
  ECAM is mandatory for this rule from compliance
  perspective

Signed-off-by: Sujana M <sujana.murali@arm.com>
  • Loading branch information
Sujana-M committed Nov 11, 2024
1 parent 20afb64 commit ef4013d
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions val/sbsa/src/sbsa_execute_test.c
Original file line number Diff line number Diff line change
Expand Up @@ -278,6 +278,14 @@ val_sbsa_pcie_execute_tests(uint32_t level, uint32_t num_pe)
status |= p009_entry(num_pe); /* This covers GIC rule */
#endif

ecam_status = p001_entry(num_pe);
if (ecam_status == ACS_STATUS_FAIL) {
val_print(ACS_PRINT_WARN, "\n *** Skipping remaining PCIE tests ***\n", 0);
return status;
}

status |= ecam_status;

#ifndef TARGET_LINUX
if (((level > 2) && (g_sbsa_only_level == 0)) || (g_sbsa_only_level == 3))
status |= p040_entry(num_pe);
Expand All @@ -289,13 +297,6 @@ val_sbsa_pcie_execute_tests(uint32_t level, uint32_t num_pe)
#endif

if (((level > 5) && (g_sbsa_only_level == 0)) || (g_sbsa_only_level == 6)) {
ecam_status = p001_entry(num_pe);
if (ecam_status == ACS_STATUS_FAIL) {
val_print(ACS_PRINT_WARN, "\n *** Skipping remaining PCIE tests ***\n", 0);
return status;
}

status |= ecam_status;
#if defined(TARGET_LINUX) || defined(TARGET_EMULATION)
status |= p005_entry(num_pe);
#endif
Expand Down

0 comments on commit ef4013d

Please sign in to comment.