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(PeriphDrivers): Add GPIO Chip Select Configuration for MAX32665 SPI driver #877

Merged
merged 2 commits into from
Jan 18, 2024

Conversation

Brandon-Hurst
Copy link
Contributor

Description

fixes #396 . Add Chip Select configuration to MXC_SPI_Init using pre-existing me14_pins gpio cfg structs. SPI Example tested showing successful transactions and proper SPI Chip Select Behavior.

image
2024-01-15-121307
Second image shows green LED1 on EVK, which indicates a successful run of the SPI example.

Types

fix

Scopes

PeriphDrivers

@github-actions github-actions bot added the MAX32665 Related to the MAX32665 (ME14) label Jan 15, 2024
Copy link
Contributor

@Jake-Carter Jake-Carter left a comment

Choose a reason for hiding this comment

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

Thanks for picking this one up, looks good!

@Jake-Carter
Copy link
Contributor

Hey @Brandon-Hurst just some formatter/linter errors in the auto checks. See Style Guide

@Brandon-Hurst
Copy link
Contributor Author

Hey @Brandon-Hurst just some formatter/linter errors in the auto checks. See Style Guide

Thanks Jake, I fixed these with a new commit. Just a line of extra whitespace caused it to complain. Let me know if I need to squash and I can do that later to get this in.

@Jake-Carter
Copy link
Contributor

Thanks Brandon, no worries on the squash. We squash and merge into main

@Jake-Carter Jake-Carter merged commit 9836e56 into analogdevicesinc:main Jan 18, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MAX32665 Related to the MAX32665 (ME14)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MAX32665 SPI driver does not configure slave select.
2 participants