Skip to content

Comments

mcux: hal_nxp.cmake: Include DAC12 HAL driver#520

Closed
FPlohl wants to merge 1 commit intozephyrproject-rtos:masterfrom
norikdoo:WIP/FPlohl/driver_dac12
Closed

mcux: hal_nxp.cmake: Include DAC12 HAL driver#520
FPlohl wants to merge 1 commit intozephyrproject-rtos:masterfrom
norikdoo:WIP/FPlohl/driver_dac12

Conversation

@FPlohl
Copy link

@FPlohl FPlohl commented Mar 5, 2025

Include DAC12 HAL driver in case CONFIG_DAC_MCUX_DAC12 is enabled.

Include DAC12 HAL driver in case CONFIG_DAC_MCUX_DAC12 is enabled.

Signed-off-by: Florijan Plohl <florijan.plohl@norik.com>
@dnltz
Copy link

dnltz commented Mar 13, 2025

@mmahadevan108 can you please take a look at this?

@dnltz
Copy link

dnltz commented Mar 21, 2025

@lucien-nxp and @ZhaoxiangJin , can you take a look at this? This is blocking a Zephyr pull-request...

Copy link
Member

@dleach02 dleach02 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@FPlohl We just updated the NXP HAL to pull in the SDK-NG structure we have. We also moved the glue code to zephyr proper. Please move this change over to the zephyr/modules/hal_nxp/mcux//mcux-sdk-ng

@dleach02
Copy link
Member

@FPlohl will you be able to realign this PR to the SDK-NG structure?

@EmilioCBen
Copy link
Contributor

Changes are addressed in the Zephyr PR.

@EmilioCBen EmilioCBen closed this May 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants