modules: hal_ethos_u: fix failure to override ETHOSU_TARGET_NPU_CONFIG

[ccli8]

This fixes failure to override ETHOSU_TARGET_NPU_CONFIG in hal_ethos_u due to cmake policy CMP0126 not being NEW. ETHOSU_TARGET_NPU_CONFIG originally as directory variable will fail to override that in hal_ethos_u as cache variable.. This is fixed by passing also as cache variable.

See:
https://cmake.org/cmake/help/latest/policy/CMP0126.html#policy:CMP0126

[wearyzen]

could you update the PR with the test that failed or how you encountered the failure?

[ccli8]

This failure was found in the port of nuvoton's m55m1 ethos-u (#88941). On m55m1, its ethos-u is ethos-u55-256, not default ethos-u55-128:

zephyr/modules/hal_ethos_u/Kconfig

Lines 16 to 17 in 70759f1

	default ARM_ETHOS_U55_256 if SOC_SERIES_M55M1X
	default ARM_ETHOS_U55_128

Build on numaker_m55m1 board:

west -v build -b numaker_m55m1 zephyr/samples/modules/tflite-micro/tflm_ethosu

hal_ethos_u will dump config message, like:

*******************************************************
PROJECT_NAME                           : ethosu_core_driver
ETHOSU_TARGET_NPU_CONFIG               : ethos-u55-128
CMAKE_SYSTEM_PROCESSOR                 : arm
CMSIS_PATH                             : <SKIP>
ETHOSU_LOG_ENABLE                      : ON
ETHOSU_LOG_SEVERITY                    : warning
ETHOSU_INFERENCE_TIMEOUT               : Default (no timeout)
*******************************************************

On m55m1 target, ETHOSU_TARGET_NPU_CONFIG should be ethos-u55-256, not ethos-u55-128. This PR is addressing the issue.

[jonny-svaerd-arm]

@ccli8 Thanks! Considering that the Ethos-U Core Driver is added later in that CMake file, I think the FORCE option can be removed? Theoretically it's possible to set the CMake define on CLI, but if the entry has FORCE then that will override everything.

[ccli8]

Considering that the Ethos-U Core Driver is added later in that CMake file, I think the FORCE option can be removed? Theoretically it's possible to set the CMake define on CLI, but if the entry has FORCE then that will override everything.

Agree. Removed the FORCE option.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[jonny-svaerd-arm]

LGTM, but I don't have review rights. Could you approve this from your end please @kristofer-jonsson-arm