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): Fix OWM Presence Detect Bit and GPIO Alternate Functions, Use VDDIOH by Default #811

Merged
merged 2 commits into from
Nov 30, 2023

Conversation

ozersa
Copy link
Contributor

@ozersa ozersa commented Nov 27, 2023

Pull Request Template

Description

  • MAX32655/ MAX78000/ MAX78002 presence detect bit is 7 not 5
  • MAX32655 OWM GPIO alternate function 2 not 1
  • MAX32655/MAX32690 OWM GPIO mapping B pins defined
  • Unneeded code which set voltage source deleted from drivers
  • Voltage source selected as VDDIOH to provide 3.3V.

Test output on MAX32655 with MAX31825 target
ROM ID read succeeded.
image

Checklist Before Requesting Review

  • PR Title follows correct guidelines.
  • Description of changes and all other relevant information.
  • (Optional) Link any related GitHub issues using a keyword
  • (Optional) Provide info on any relevant functional testing/validation. For API changes or significant features, this is not optional.

PS: It will be nice to update init function to it take map parameters to select related gpio mapping but I am not sure whether API should be changed now or not. Changing API will break existing customer build system but i am not sure whether is there any one customer to use OWM API or not.
See MAX32570 OWM init function

@github-actions github-actions bot added MAX32650 Related to the MAX32650 (ME10) MAX32655 Related to the MAX32655 (ME17) MAX32680 Related to the MAX32680 (ME20) MAX32690 Related to the MAX32690 (ME18) MAX78000 Related to the MAX78000 (AI85) MAX78002 Related to the MAX78002 (AI87) Register Change This issue or pull request involves a change to the MSDK registers. labels Nov 27, 2023
@ozersa ozersa force-pushed the fix/OWM_Driver_Issues branch 2 times, most recently from a325705 to fd15437 Compare November 27, 2023 18:24
- MAX32655/ MAX32650/ MAX78000/ MAX78002 presence detect bit is 7 not 5
- MAX32655 OWM GPIO alternate function 2 not 1
- MAX32655/MAX32690 OWM GPIO mapping B pins defined
- Unneeded code which set voltage source deleted from drivers
- Voltage source selected as VDDIOH to provide 3.3V.
- SVD files updated to indicated correct presence bit

Signed-off-by: Sadik Ozer <[email protected]>
Jake-Carter
Jake-Carter approved these changes Nov 28, 2023
@Jake-Carter
Copy link
Contributor

/generate-register-files

@Jake-Carter
Copy link
Contributor

Something seems wrong with the SVD workflow. @sihyung-maxim tagging you so you can look at this when you're back.

Will merge this now, the header files look correct to me

@Jake-Carter Jake-Carter changed the title fix(PeriphDrivers): OWM driver issue fix(PeriphDrivers): Fix OWM Presence Detect Bit and GPIO Alternate Functions, Use VDDIOH by Default Nov 30, 2023
@Jake-Carter Jake-Carter merged commit 06d693e into main Nov 30, 2023
11 of 12 checks passed
@Jake-Carter Jake-Carter deleted the fix/OWM_Driver_Issues branch November 30, 2023 03:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MAX32650 Related to the MAX32650 (ME10) MAX32655 Related to the MAX32655 (ME17) MAX32680 Related to the MAX32680 (ME20) MAX32690 Related to the MAX32690 (ME18) MAX78000 Related to the MAX78000 (AI85) MAX78002 Related to the MAX78002 (AI87) 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.

3 participants