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.

Sign up for GitHub

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

Jump to bottom

frdm_kw41z: Current master fails compilation in drivers/pwm/pwm_mcux_tpm.c #30634

Closed
pfalcon opened this issue Dec 11, 2020 · 2 comments · Fixed by #30644
Closed

frdm_kw41z: Current master fails compilation in drivers/pwm/pwm_mcux_tpm.c #30634

pfalcon opened this issue Dec 11, 2020 · 2 comments · Fixed by #30644
Assignees
@galak
Labels
area: Devicetree bug The issue is a bug, or the PR is fixing a bug platform: NXP NXP priority: high High impact/importance bug

Comments

@pfalcon
Copy link
Contributor

pfalcon commented Dec 11, 2020

Build error messages:

00:13:06.112  2883 | #define DT_FOREACH_OKAY_INST_nxp_kinetis_tpm(fn) fn(0) fn(1)
00:13:06.112       |                                                        ^~
00:13:06.112 /home/buildslave/workspace/zephyr-upstream/zephyr/include/sys/util_internal.h:106:36: note: in expansion of macro 'DT_FOREACH_OKAY_INST_nxp_kinetis_tpm'
00:13:06.112   106 | #define UTIL_PRIMITIVE_CAT(a, ...) a##__VA_ARGS__
00:13:06.112       |                                    ^
00:13:06.112 /home/buildslave/workspace/zephyr-upstream/zephyr/drivers/pwm/pwm_mcux_tpm.c:203:1: note: in expansion of macro 'DT_INST_FOREACH_STATUS_OKAY'
00:13:06.112   203 | DT_INST_FOREACH_STATUS_OKAY(TPM_DEVICE)
00:13:06.112       | ^~~~~~~~~~~~~~~~~~~~~~~~~~~
00:13:06.112 /home/buildslave/workspace/zephyr-upstream/zephyr/include/device.h:186:2: error: type defaults to 'int' in declaration of 'DEVICE_DT_DEFINE' [-Werror=implicit-int]

Full log:
https://ci.linaro.org/view/lite-iot-ci/job/zephyr-upstream/5818/PLATFORM=frdm_kw41z,ZEPHYR_TOOLCHAIN_VARIANT=zephyr,label=docker-xenial-amd64-13/console

Specific commit built: d5072d1
@pfalcon pfalcon added bug The issue is a bug, or the PR is fixing a bug area: Devicetree platform: NXP NXP labels Dec 11, 2020
@pfalcon
Copy link
Contributor Author

pfalcon commented Dec 11, 2020

Posted to capture the issue and avoid possibility of duplicating effort. I didn't perform further triaging so far.

@agansari, @MaureenHelm : FYI

@galak
Copy link
Collaborator

galak commented Dec 11, 2020

Fixed by #30644

@galak galak added the priority: high High impact/importance bug label Dec 11, 2020
@galak galak self-assigned this Dec 11, 2020
@nashif nashif mentioned this issue Dec 11, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@galak galak

Labels
area: Devicetree bug The issue is a bug, or the PR is fixing a bug platform: NXP NXP priority: high High impact/importance bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

drivers: pwm: pwm_mcux_tpm: Fix build failure galak/zephyr
2 participants
@pfalcon @galak