Skip to content

tests: drivers.pwm.loopback.pwm_loopback.capture_timeout fail to build #90802

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

Open
1 task done
hakehuang opened this issue May 29, 2025 · 1 comment
Open
1 task done
Assignees
Labels
bug The issue is a bug, or the PR is fixing a bug platform: NXP S32 NXP Semiconductors, S32

Comments

@hakehuang
Copy link
Collaborator

hakehuang commented May 29, 2025

Describe the bug

platforms:

  • s32z2xxdc2@D/s32z270/rtu1
  • s32z2xxdc2@B/s32z270/rtu1
  • s32z2xxdc2@D/s32z270/rtu0
  • s32z2xxdc2@B/s32z270/rtu0

git bisect found below commit introduce failure

git bisect sha: f715301

commit f7153012eff53b28dfa77b3ea2f76fb75d9fc812
Author: Fin Maaß <[email protected]>
Date:   Mon Apr 28 13:55:34 2025 +0200

    dts: enum_macros: make sure that they are lowercase
    
    DT_ENUM_HAS_VALUE_BY_IDX states, that the value
    must be lowercase-and-underscores, this makes sure,
    that they can match.
    
    Signed-off-by: Fin Maaß <[email protected]>

 drivers/comparator/comparator_nrf_comp.c   | 6 +++---
 drivers/comparator/comparator_nrf_lpcomp.c | 2 +-
 scripts/dts/gen_defines.py                 | 3 +++
 3 files changed, 7 insertions(+), 4 deletions(-)

Regression

  • This is a regression.

Steps to reproduce

scripts/twister -p s32z2xxdc2@D/s32z270/rtu1 -T tests/drivers/pwm --build-only

Relevant log output

/home/jenkins/agent/workspace/zephyr_downstream_build_daily/zephyr/drivers/pwm/pwm_nxp_s32_emios.c
In file included from /home/jenkins/agent/workspace/zephyr_downstream_build_daily/zephyr/include/zephyr/sys/util_macro.h:34,
                 from /home/jenkins/agent/workspace/zephyr_downstream_build_daily/zephyr/include/zephyr/irq_multilevel.h:16,
                 from /home/jenkins/agent/workspace/zephyr_downstream_build_daily/zephyr/include/zephyr/devicetree.h:20,
                 from /home/jenkins/agent/workspace/zephyr_downstream_build_daily/zephyr/include/zephyr/device.h:12,
                 from /home/jenkins/agent/workspace/zephyr_downstream_build_daily/zephyr/drivers/pwm/pwm_nxp_s32_emios.c:7:
/home/jenkins/agent/workspace/zephyr_downstream_build_daily/zephyr/drivers/pwm/pwm_nxp_s32_emios.c:842:49: error: 'EMIOS_PWM_IP_MODE_SAIC_FLAG' undeclared here (not in a function); did you mean 'EMIOS_PWM_IP_MODE_DAOC_FLAG'?
  842 | #define EMIOS_PWM_MODE(mode)            DT_CAT3(EMIOS_PWM_IP_MODE_, mode, _FLAG)
      |                                                 ^~~~~~~~~~~~~~~~~~
/home/jenkins/agent/workspace/zephyr_downstream_build_daily/zephyr/include/zephyr/sys/util_internal.h:72:26: note: in definition of macro '__DEBRACKET'

Impact

S32 PWM tests

Environment

OS: Ubuntu
Tool chain: Zephyr SDK
Version: nxp-v4.1.0-19362-g779c8d5e74b2

@hakehuang hakehuang added the bug The issue is a bug, or the PR is fixing a bug label May 29, 2025
@Dat-NguyenDuy Dat-NguyenDuy added the platform: NXP S32 NXP Semiconductors, S32 label May 30, 2025
@manuargue
Copy link
Member

also affecting mr_canhubk3 board

@manuargue manuargue changed the title tests: drivers.pwm.loopback.pwm_loopback.capture_timeout: NXP S32 platform need update dts due to driver change tests: drivers.pwm.loopback.pwm_loopback.capture_timeout fail to build Jun 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The issue is a bug, or the PR is fixing a bug platform: NXP S32 NXP Semiconductors, S32
Projects
None yet
Development

No branches or pull requests

3 participants