Skip to content

Conversation

cfriedt
Copy link
Member

@cfriedt cfriedt commented Oct 13, 2025

TIMER_MAX is part of POSIX and defined in limits.h.

To avoid conflict with the standard, namespace the local definition with a NPM13XX_ prefix.

This fixes an issue that appeared in weekly CI:

error: "TIMER_MAX" redefined [-Werror]

https://github.com/zephyrproject-rtos/zephyr/actions/runs/18437938778/job/52533996345

`TIMER_MAX` is part of POSIX and defined in limits.h.

To avoid conflict with the standard, namespace the local definition with a
`NPM13XX_` prefix.

This fixes an issue that appeared in weekly CI:

https://github.com/zephyrproject-rtos/zephyr/actions/runs/18437938778/\
job/52533996345

Signed-off-by: Chris Friedt <[email protected]>
Copy link

@cfriedt cfriedt added the Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. label Oct 14, 2025
@cfriedt
Copy link
Member Author

cfriedt commented Oct 14, 2025

Added the Hotfix label (although this is only fixing a failure in weekly CI).

@cfriedt cfriedt merged commit 2870082 into zephyrproject-rtos:main Oct 14, 2025
33 checks passed
@cfriedt cfriedt deleted the namespace-timer-max-in-drivers-mfd-npm13xx-c branch October 14, 2025 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: MFD Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants