Skip to content

Conversation

@extinguish
Copy link
Contributor

Summary

When we support the watchdog interrupt on the Armv7-A platform with the TEE enabled, the watchdog interrupt needs to be configured as a FIQ to avoid the impact of interrupt disabling in the AP. In this case, the drivertest_watchdog_api testcase cannot pass the test in such a scenario.

This is because the drivertest_watchdog_api itself requires calling a specified callback after the watchdog interrupt is triggered, instead of directly dumping the AP's context and then asserting the system.

Therefore, when both CONFIG_ARCH_ARMV7A and CONFIG_ARCH_HAVE_TRUSTZONE are enabled, we need to skip the current drivertest_watchdog_api testcase.

Impact

  1. Maintains test‑suite reliability – Skips the drivertest_watchdog_api test only when both CONFIG_ARCH_ARMV7A and CONFIG_ARCH_HAVE_TRUSTZONE are set, ensuring the test does not fail due to hardware/security constraints.
  2. Preserves watchdog testing coverage – All other watchdog tests (feeding, interrupts, loop operation) continue to run, providing meaningful driver validation.
  3. No effect on other platforms – The test remains fully enabled on all non‑Armv7‑A‑with‑TrustZone configurations.

Testing

Verified that on Armv7‑A + TrustZone platforms, the watchdog test suite compiles and runs without the drivertest_watchdog_api test.

When we support the watchdog interrupt on the Armv7-A platform with
the TEE enabled, the watchdog interrupt needs to be configured as
a FIQ to avoid the impact of interrupt disabling in the AP.
In this case, the drivertest_watchdog_api testcase cannot pass the
test in such a scenario.

This is because the drivertest_watchdog_api itself requires
calling a specified callback after the watchdog interrupt is
triggered, instead of directly dumping the AP's context and
then asserting the system.

Therefore, when both CONFIG_ARCH_ARMV7A and CONFIG_ARCH_HAVE_TRUSTZONE
are enabled, we need to skip the current drivertest_watchdog_api
testcase.

Signed-off-by: guoshichao <guoshichao@xiaomi.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant