Skip to content

fix(PeriphDrivers): MSDK-1245: Re-order VSSEL selection to avoid possible damages on pads #880

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

Merged
merged 1 commit into from
Jan 16, 2024

Conversation

ahmetalincak
Copy link
Contributor

Description

Voltage Rail selection should be made before padding configuration. A hesitation came from Clayton:

For the MXC_GPIO_Config function, the last thing done is to select the voltage rail (gpio_me18.c). This creates a situation where damage to the part can occur. The GPIO on parts with multiple rails defaults to VDDIO (1.8V) with pullup/pulldown disabled. If a customer has 3.0V on a GPIO pin (say I2C pullups), and the code turns on the pullup before selecting the VDDIOH rail, the 3.0V back powers VDDIO, potentially causing damage to the pads.

@github-actions github-actions bot added MAX32520 Related to the MAX32520 (ES17) MAX32570 Related to the MAX32570 (ME13) MAX32650 Related to the MAX32650 (ME10) MAX32655 Related to the MAX32655 (ME17) MAX32660 Related to the MAX32660 (ME11) MAX32665 Related to the MAX32665 (ME14) MAX32680 Related to the MAX32680 (ME20) MAX32690 Related to the MAX32690 (ME18) MAX78000 Related to the MAX78000 (AI85) MAX78002 Related to the MAX78002 (AI87) MAX32662 Related to the MAX32662 (ME12) labels Jan 16, 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.

This makes sense. Thanks @cwarer1 and @ahmetalincak

@Jake-Carter Jake-Carter merged commit fa0500e into main Jan 16, 2024
@Jake-Carter Jake-Carter deleted the fix/gpio-config-set-vssel-first branch January 16, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MAX32520 Related to the MAX32520 (ES17) MAX32570 Related to the MAX32570 (ME13) MAX32650 Related to the MAX32650 (ME10) MAX32655 Related to the MAX32655 (ME17) MAX32660 Related to the MAX32660 (ME11) MAX32662 Related to the MAX32662 (ME12) MAX32665 Related to the MAX32665 (ME14) 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.

3 participants