Skip to content

Conversation

@perkinsmg
Copy link
Contributor

@perkinsmg perkinsmg commented Nov 24, 2025

Description

Don't call MXC_GPIO_SetDriveStrength for the special GPIO ports that are set up via MCR registers. MXC_GPIO_SetDriveStrength calls MXC_GPIO_RevA_SetDriveStrength directly and this only configures pins that are handled via the usual GCR registers.

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.

@github-actions github-actions bot added 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) labels Nov 24, 2025
@perkinsmg perkinsmg changed the title fix(PeriphDrives): Fix for special GPIO ports fix(PeriphDriver): Fix for special GPIO ports Nov 24, 2025
@perkinsmg perkinsmg changed the title fix(PeriphDriver): Fix for special GPIO ports fix(PeriphDrivers): Fix for special GPIO ports Nov 24, 2025
@perkinsmg perkinsmg force-pushed the fix/gpio-port-condition-bug branch from c11d697 to 4a99107 Compare November 24, 2025 10:40
@perkinsmg perkinsmg requested a review from ttmut November 24, 2025 10:41
@perkinsmg perkinsmg force-pushed the fix/gpio-port-condition-bug branch from 4a99107 to d3d5474 Compare November 25, 2025 09:40
Don't call MXC_GPIO_SetDriveStrength for the special GPIO ports that are
set up via MCR registers. MXC_GPIO_SetDriveStrength calls
MXC_GPIO_RevA_SetDriveStrength directly and this only configures pins that
are handled via the usual GCR registers.

Signed-off-by: Michael Perkins <[email protected]>
@perkinsmg perkinsmg force-pushed the fix/gpio-port-condition-bug branch from d3d5474 to dacedbc Compare November 25, 2025 12:36
Copy link
Collaborator

@petejohanson-adi petejohanson-adi left a comment

Choose a reason for hiding this comment

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

LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants