Skip to content

drivers: gpio: nxp: mcux_igpio: fix mcux_igpio_port_get_raw() #100585

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.

Sign up for GitHub

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

Jump to bottom
Open
butok wants to merge 1 commit into
base: main
Choose a base branch
from
Open

drivers: gpio: nxp: mcux_igpio: fix mcux_igpio_port_get_raw() #100585

butok wants to merge 1 commit into from
+5 −1

Conversation

@butok
Copy link
Contributor

@butok butok commented Dec 5, 2025

  • Reads the Pad Status Register (PSR) to get the current GPIO input value, instead of the Data Register (DR).
    Following IMXRT1170RM.pdf "13.5.2 GPIO Write Mode".
  • Enables input buffer via Software Input On (SION) to get correct pin value in PSR for both input/output pins.
  • Issue has been discovered during Safety DIO self-tests.

@butok
drivers: gpio: nxp: mcux_igpio: fix mcux_igpio_port_get_raw()
eeef930 
- Reads the Pad Status Register (PSR) to get
  the current GPIO input value, instead of the Data Register (DR).
  Following IMXRT1170RM.pdf "13.5.2 GPIO Write Mode".
- Enables input buffer via Software Input On (SION)
  to get correct pin value in PSR for both input/output pins.
- Issue has been discovered during Safety DIO self-tests.

Signed-off-by: Andrej Butok <andrey.butok@nxp.com>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 5, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@zejiang0jason
Copy link
Contributor

zejiang0jason commented Dec 6, 2025

Hi @butok , could you please link the issue, or the test case name which can reproduce the issue? thanks.

@butok
Copy link
Contributor Author

butok commented Dec 8, 2025 •
edited
Loading

Hi @zejiang0jason
No link, as it was discovered during testing of the bare-metal NXP functional safety library with Zephyr, which is external code (not upstream).

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

Reviewers

@Holt-Sun Holt-Sun Holt-Sun approved these changes

@mmahadevan108 mmahadevan108 mmahadevan108 approved these changes

@zejiang0jason zejiang0jason zejiang0jason approved these changes

@dbaluta dbaluta Awaiting requested review from dbaluta

@henrikbrixandersen henrikbrixandersen Awaiting requested review from henrikbrixandersen

@manuargue manuargue Awaiting requested review from manuargue

Assignees

@mmahadevan108 mmahadevan108

Labels

area: GPIO platform: NXP NXP

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@butok @zejiang0jason @Holt-Sun @mmahadevan108 @zephyrbot