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.9k stars 6.64k forks source link

west update fails on `hal_espressif` #72207

Closed rexut closed 6 months ago

rexut commented 6 months ago

Describe the bug

Actually, with the detached SHA1 e554857 from Zephyr's west manifest, west update runs into the following error:

ERROR: update failed for project hal_espressif

Isn't that the wrong SHA1 https://github.com/zephyrproject-rtos/hal_espressif/commit/e554857b07d54e2152c6f1a9b39da6d6c13fe7f7 ?:

https://github.com/zephyrproject-rtos/zephyr/blob/ea995f6378f7f6a6f8b6b1f7df29e0af7437f7a1/west.yml#L155

In the hal_espressif branch zephyr on head is SHA1 https://github.com/zephyrproject-rtos/hal_espressif/commit/c49581173bab8625d1f08d5c41d5a186afc240b3. Both SHA1 refer to the commit:

driver: sdhc: ESP32 pinctrl config for SDHC

Additional LL files for SDHC driver, as well as pinctrl configuration

Signed-off-by: Raffael Rostagno <raffael.rostagno@espressif.com>

To Reproduce

Steps to reproduce the behavior:

  1. have a out-of-tree or downstream project with Zephyr as module
  2. have a local west workspace for at least unchanged since 29. April
  3. run west update
  4. See error

Expected behavior

west update without any error. Sane update the references in hal_espressif.

Impact

Can't participate from last fixes / changes on Espressif HAL.

Environment (please complete the following information):

Additional context

Simple change the SHA1 in Zephyr's west manifest from e554857b07d54e2152c6f1a9b39da6d6c13fe7f7 to c49581173bab8625d1f08d5c41d5a186afc240b3 that was committed yesterday.

jvermillard commented 6 months ago

duplicate of #72196