Skip to content

Conversation

sylvioalves
Copy link
Contributor

@sylvioalves sylvioalves commented Oct 6, 2025

Replace all <sys/param.h> to <zephyr/sys/utils.h> for
MIN/MAX macros in Espressif HAL.

Fixes #97074

@sylvioalves sylvioalves force-pushed the bugfix/hal-include-sys-param branch from b80d5f8 to 8d7884e Compare October 6, 2025 18:03
Copy link

github-actions bot commented Oct 6, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_espressif zephyrproject-rtos/hal_espressif@06d53f4 zephyrproject-rtos/hal_espressif@167cd71 (main) zephyrproject-rtos/[email protected]

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hal_espressif DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Oct 6, 2025
@zephyrbot zephyrbot added platform: ESP32 Espressif ESP32 size: XS A PR changing only a single line of code labels Oct 6, 2025
@sylvioalves sylvioalves closed this Oct 6, 2025
@sylvioalves sylvioalves reopened this Oct 6, 2025
Replace all <sys/param.h> to <zephyr/sys/utils.h> for
MIN/MAX macros in Espressif HAL.

Signed-off-by: Sylvio Alves <[email protected]>
@sylvioalves sylvioalves force-pushed the bugfix/hal-include-sys-param branch from 8d7884e to f81b6f3 Compare October 6, 2025 20:26
@sylvioalves sylvioalves changed the title west.yml: espressif: fix include header conflict west.yml: espressif: use Zephyr utils for MIN/MAX Oct 6, 2025
@sylvioalves sylvioalves requested a review from cfriedt October 6, 2025 20:27
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Oct 6, 2025
Copy link

sonarqubecloud bot commented Oct 6, 2025

@sylvioalves sylvioalves requested a review from nashif October 6, 2025 23:02
@nashif nashif added the Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. label Oct 6, 2025
@cfriedt cfriedt merged commit 6862856 into zephyrproject-rtos:main Oct 7, 2025
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. manifest manifest-hal_espressif platform: ESP32 Espressif ESP32 size: XS A PR changing only a single line of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

espressif: manual inclusion of sys/syslimits.h from sys/param.h in module breaks build
7 participants