Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

compiler warning when i3c device and CONFIG_NULL_POINTER_EXCEPTION_DETECTION_MPU=y #83448

Open
mertekren opened this issue Dec 30, 2024 · 1 comment
Assignees
Labels
area: I3C bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug

Comments

@mertekren
Copy link
Contributor

Describe the bug

Following compiler warning occurs when I3C device with 9 byte of reg address to dts, and DCONFIG_NULL_POINTER_EXCEPTION_DETECTION_MPU=y.
C:/repos/ADI-zephyr/zephyr/arch/arm/core/mpu/arm_mpu.c: In function 'z_arm_mpu_init':
C:/repos/ADI-zephyr/zephyr/arch/arm/core/mpu/arm_mpu.c:481:8: warning: integer constant is too large for its type
481 | (!DT_NULL_PAGE_DETECT_NODE_EXIST)

To Reproduce

  1. Build a board with M33 with DCONFIG_NULL_POINTER_EXCEPTION_DETECTION_MPU=y with a tests/*:
    example:
    west build -b mimxrt685_evk/mimxrt685s/cm33 .\tests\drivers\uart\uart_basic_api\ -p always -DCONFIG_NULL_POINTER_EXCEPTION_DETECTION_MPU=y
  2. See the compiler warning:
    warning: integer constant is too large for its type
    481 | (!DT_NULL_PAGE_DETECT_NODE_EXIST)

Expected behavior

Compiler warning shall not occur.
Impact

CI/CD requires -Werror option, thus this fails CI/CD procedure in out-of-tree zephyr pipeline.
Logs and console output

Environment (please complete the following information):

  • Zephyr 4.00+ (macro DT_NULL_PAGE_DETECT_NODE_EXIST added here )
  • OS: all
  • Toolchain Zephyr SDK
  • Commit SHA or Version used
@mertekren mertekren added the bug The issue is a bug, or the PR is fixing a bug label Dec 30, 2024
Copy link

Hi @mertekren! We appreciate you submitting your first issue for our open-source project. 🌟

Even though I'm a bot, I can assure you that the whole community is genuinely grateful for your time and effort. 🤖💙

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: I3C bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Development

No branches or pull requests

3 participants