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

fix(CMSIS): Checking for definition of MXC_DMA1. #1270

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

kevin-gillespie
Copy link
Contributor

@kevin-gillespie kevin-gillespie commented Nov 14, 2024

Description

Solves a build error when building without secure execution.

+++ Compiling: ../../../MAX32657/source/pal_bb_drv.c
In file included from ../../../../../Libraries/PeriphDrivers/Include/MAX32657/mxc_device.h:22,
                 from ../../../MAX32657/include/dbb_registers.h:47,
                 from ../../../MAX32657/source/pal_bb_drv.c:54:
../../../../../Libraries/CMSIS/Device/Maxim/MAX32657/Include/max32657.h:708:5: error: "MXC_DMA1" is not defined, evaluates to 0 [-Werror=undef]
  708 | #if MXC_DMA1

Checklist Before Requesting Review

  • PR Title follows correct guidelines.
  • Description of changes and all other relevant information.

@kevin-gillespie kevin-gillespie merged commit 51f8b11 into main Nov 14, 2024
13 of 14 checks passed
@kevin-gillespie kevin-gillespie deleted the fix/dma1-non-secure branch November 14, 2024 16:57
sihyung-maxim pushed a commit to analogdevicesinc/hal_adi that referenced this pull request Nov 14, 2024
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.

2 participants