From 377111171ccb15a86de1f0eaa3a40dc4cac7341e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Desbiens?= Date: Thu, 22 Oct 2026 13:59:37 -0400 Subject: [PATCH] Ran the Cortex-R52 FVP workflow on portable module manager changes The r52_fvp workflow runs the FVP module manager images, the only CI job that executes module manager code, but its push and pull_request path filters did not include common_modules/. A change to the portable module manager therefore never ran them, although the images link every source in common_modules/module_manager/src. Both path filters now include common_modules/**, and the header comment says why. The edited workflow parses as YAML with identical push and pull_request path lists. This pull request changes the workflow file itself, so its own CI runs the FVP suite. Assisted-by: Claude Code (Opus 5.5) --- .github/workflows/r52_fvp.yml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/r52_fvp.yml b/.github/workflows/r52_fvp.yml index 2293e1bdd..b7184abd9 100644 --- a/.github/workflows/r52_fvp.yml +++ b/.github/workflows/r52_fvp.yml @@ -22,7 +22,8 @@ name: r52_fvp # # - the default configuration -- the images gcc_check's CMake stage builds, # but run rather than merely linked, plus the FVP module manager example, -# whose two images are the module isolation tests; +# whose two images are the module isolation tests. They link every portable +# module manager source, so common_modules/ is among the trigger paths; # - the feature configuration -- the same images with VFP and the hard float # ABI, FIQ, IRQ nesting and FIQ nesting. Those four options gate whole # assembly blocks (the VMRS/VSTMDB/VLDMIA sequences, the FIQ context paths, @@ -44,6 +45,7 @@ on: - "CMakeLists.txt" - "cmake/cortex_r52*.cmake" - "common/**" + - "common_modules/**" - "ports/cortex_r52/**" - "ports_module/cortex_r52/**" pull_request: @@ -58,6 +60,7 @@ on: - "CMakeLists.txt" - "cmake/cortex_r52*.cmake" - "common/**" + - "common_modules/**" - "ports/cortex_r52/**" - "ports_module/cortex_r52/**"