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

drivers: wifi: nxp: Fix the compilation error on the MIMXRT1170_EVK board with the NXP_88W8987_MURATA_1ZM_M2 WiFi module #489

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

CsBoBoNice
Copy link

drivers: wifi: nxp: Fix the compilation error on the MIMXRT1170_EVK board with the NXP_88W8987_MURATA_1ZM_M2 WiFi module

  • Fix the compilation error on the MIMXRT1170_EVK board with the NXP_88W8987_MURATA_1ZM_M2 WiFi module

#if CONFIG_ZEPHYR
(void)OSA_EventNotifyPost(wm_wifi.wifi_core_task_Handle);
#ifdef __ZEPHYR__
(void)OSA_TaskNotifyPost(wm_wifi.wifi_core_task_Handle);
Copy link
Collaborator

Choose a reason for hiding this comment

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

please remove the non-zephyr part and remove #ifdef __ZEPHYR__

Copy link

Choose a reason for hiding this comment

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

Guess, as this is the original MCUx SDK code, it may contain non-Zephyr bare-metal code.

@@ -45,6 +45,7 @@
#define sdio_d(...)
#endif /* CONFIG_WIFI_SDIO_DEBUG */

#ifndef __ZEPHYR__
Copy link
Collaborator

Choose a reason for hiding this comment

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

if it's not used for zephyr, you can remove it

…oard with the NXP_88W8987_MURATA_1ZM_M2 WiFi module

- Fix the compilation error on the MIMXRT1170_EVK board with the NXP_88W8987_MURATA_1ZM_M2 WiFi module

Signed-off-by: Chen Shu <[email protected]>
@CsBoBoNice CsBoBoNice reopened this Feb 10, 2025
@CsBoBoNice
Copy link
Author

I have removed the non-Zephyr parts and the #ifdef ZEPHYR as requested. Please review the changes.

@CsBoBoNice
Copy link
Author

This time, I've kept the changes to a minimum and removed all unrelated modifications from the previous commits. I look forward to your review and approval.

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.

3 participants