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): MAX32657 bring up #1285

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Conversation

kevin-gillespie
Copy link
Contributor

@kevin-gillespie kevin-gillespie commented Dec 3, 2024

Description

Prevents unaligned memory access. Hard fault experienced without this build flag when using memcpy.

Adds UART one byte remaining interrupt.

Adding WLP_Bench board folder.

Checklist Before Requesting Review

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

@github-actions github-actions bot added the MAX32657 Related to the MAX32655 (ME30) label Dec 3, 2024
@kevin-gillespie kevin-gillespie changed the title fix(CMSIS): Adding build flag -mno-unaligned-access for MAX32567 fix(CMSIS): Fixing MAX32657 memory management and build flags Dec 3, 2024
@github-actions github-actions bot added the BLE Related to Bluetooth label Dec 4, 2024
@github-actions github-actions bot added the Register Change This issue or pull request involves a change to the MSDK registers. label Dec 5, 2024
@kevin-gillespie
Copy link
Contributor Author

/clang-format-run

@kevin-gillespie kevin-gillespie changed the title fix(CMSIS): Fixing MAX32657 memory management and build flags fix(CMSIS): MAX32657 bring up Dec 10, 2024
Comment on lines +271 to +272
#define MXC_F_UART_REVB_INT_FL_TX_OR_POS 5 /**< INT_FL_TX_OR Position */
#define MXC_F_UART_REVB_INT_FL_TX_OR ((uint32_t)(0x1UL << MXC_F_UART_REVB_INT_FL_TX_OR_POS)) /**< INT_FL_TX_OR Mask */
Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm, I guess I didn't regenerate the UART RevB register files after adding the UART one byte remaining interrupt field. The SVD and the ME30 user guide use the TX_OB name:

 #define MXC_F_UART_REVB_INT_EN_TX_OB_POS               5 /**< INT_EN_TX_OB Position */
 #define MXC_F_UART_REVB_INT_EN_TX_OB                   ((uint32_t)(0x1UL << MXC_F_UART_REVB_INT_EN_TX_OB_POS)) /**< INT_EN_TX_OB Mask */
 #define MXC_F_UART_REVB_INT_FL_TX_OB_POS               5 /**< INT_FL_TX_OB Position */
 #define MXC_F_UART_REVB_INT_FL_TX_OB                   ((uint32_t)(0x1UL << MXC_F_UART_REVB_INT_FL_TX_OB_POS)) /**< INT_FL_TX_OB Mask */

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BLE Related to Bluetooth MAX32657 Related to the MAX32655 (ME30) Register Change This issue or pull request involves a change to the MSDK registers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants