Skip to content

Conversation

@soburi
Copy link
Member

@soburi soburi commented Dec 28, 2024

I found that gpio_basic_api testing had not been performed in rpi_pico, so I added this setting and performed testing.
As a result, GPIO_DISCONNECTED was being mishandled, so I have fixed this.

For rpi_pico2, due to an errata for RP2350-E9, INPUT and PULLDOWN cannot be enabled
simultaneously on the RP2350.

Set CONFIG_SKIP_PULL_TEST to avoid the test that causes the issue.

@soburi soburi force-pushed the rpi_pico_gpio_disconnected branch from 7399159 to 8748061 Compare December 28, 2024 02:32
@soburi soburi marked this pull request as ready for review December 28, 2024 03:43
@soburi soburi added the bug The issue is a bug, or the PR is fixing a bug label Dec 28, 2024
@zephyrbot zephyrbot added platform: Raspberry Pi Pico Raspberry Pi Pico (RPi Pico) area: GPIO labels Dec 28, 2024
If GPIO_DISCONNECTED is requested with gpio_pin_configure,
it will return -ENOTSUP since rpi_pico does not support it.

Signed-off-by: TOKITA Hiroshi <[email protected]>
@soburi soburi force-pushed the rpi_pico_gpio_disconnected branch from 8748061 to 55dfd2c Compare January 31, 2025 12:22
@zephyrbot zephyrbot requested a review from ThreeEights January 31, 2025 12:22
@soburi soburi force-pushed the rpi_pico_gpio_disconnected branch from 55dfd2c to cb6058d Compare January 31, 2025 12:25
Copy link
Member Author

Choose a reason for hiding this comment

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

@henrikbrixandersen

I addressed it based on your comments received elsewhere.

#84548 (comment)

@soburi
Copy link
Member Author

soburi commented Jan 31, 2025

@ThreeEights
Could you take a look, please?

Copy link
Contributor

@ThreeEights ThreeEights left a comment

Choose a reason for hiding this comment

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

Approved.

Add an overlay for `rpi_pico` and `rpi_pico2` to run this test.

Signed-off-by: TOKITA Hiroshi <[email protected]>
@soburi
Copy link
Member Author

soburi commented Jan 31, 2025

@henrikbrixandersen @ThreeEights

I have only changed the GPIO pin numbers a little.
Sorry, but please re-approve.

@soburi
Copy link
Member Author

soburi commented Feb 3, 2025

@kartben @fabiobaltieri
Can you please join me for the review for a bit?

@kartben kartben merged commit e335127 into zephyrproject-rtos:main Feb 3, 2025
24 checks passed
@soburi soburi deleted the rpi_pico_gpio_disconnected branch February 3, 2025 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: GPIO bug The issue is a bug, or the PR is fixing a bug platform: Raspberry Pi Pico Raspberry Pi Pico (RPi Pico)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants