search

zephyrproject-rtos / zephyr

Primary Git Repository for the Zephyr Project. Zephyr is a new generation, scalable, optimized, secure RTOS for multiple hardware architectures.
https://docs.zephyrproject.org
Apache License 2.0
10.34k stars 6.33k forks source link

Any app setting CONFIG_PM or CONFIG_POWER_OFF will fail to build during cmake step on esp32c6_devkitc #75878

Closed aescolar closed 1 month ago

aescolar commented 1 month ago

Describe the bug

or Any test/sample which sets CONFIG_PM or CONFIG_POWER_OFF will fail to build for esp32c6_devkitc

To Reproduce Steps to reproduce the behavior:

  1. twister -p esp32c6_devkitc -T tests/lib/cpp/cxx/
  2. See error

OR just

  1. cmake -GNinja -DBOARD=esp32c6_devkitc ../samples/hello_world/ -DCONFIG_PM=y
  2. See error

Expected behavior No build failures. Tests pass

Impact Failures in main CI

Logs and console output https://github.com/zephyrproject-rtos/zephyr/runs/27431582117

Cmake build failure
Loading Zephyr default modules (Zephyr base).
...
CMake Error at /__w/zephyr/zephyr/cmake/modules/extensions.cmake:428 (add_library):
  Cannot find source file:

    /__w/zephyr/zephyr/soc/espressif/esp32c6/power.c

Environment (please complete the following information):

Additional context Failures in CI bisected to 65f63622e273c19883b866d71a13e8f18b7d5c6c

70403

Underlying SOC issue present since its introduction in b3523c9bfaa6653b2f1bd60480a0febd5ceb8dd1

raffarost commented 1 month ago

Hi @aescolar, The first Zephyr support of ESP32C6 indeed ended up mistakenly including HAS_PM and HAS_POWEROFF. We have a PR open for review that will add support to PM and fix these issues:

https://github.com/zephyrproject-rtos/zephyr/pull/75278

aescolar commented 1 month ago

Marking as blocker, so we can have a green CI for 3.7.0 after merging the fix.