Skip to content

west: hal_rpi_pico: Update Pico-SDK to 2.0.0 #76986

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 4 commits into from
Oct 8, 2024

Conversation

soburi
Copy link
Member

@soburi soburi commented Aug 13, 2024

Update Pico-SDK to 2.0.0 release.

  • The directory structure has changed in 2.0.0, so we update it accordingly.
  • Following the GPIO interface changes in pico-sdk 2.0.0.
  • Some symbol names have been conflicted with introducing pico-sdk 2.0.0. Rename these.

@soburi soburi requested a review from yonsch as a code owner August 13, 2024 07:11
@soburi soburi added the DNM This PR should not be merged (Do Not Merge) label Aug 13, 2024
@zephyrbot
Copy link
Collaborator

zephyrbot commented Aug 13, 2024

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
hal_rpi_pico zephyrproject-rtos/hal_rpi_pico@fba7162 zephyrproject-rtos/hal_rpi_pico@79ee0f9 (zephyr) zephyrproject-rtos/hal_rpi_pico@fba7162c..79ee0f9e

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@soburi soburi changed the title west: hal_rpi_pico: Update Pico-SDK to v1.5.0 west: hal_rpi_pico: Update Pico-SDK to 2.0.0 Aug 13, 2024
@yonsch
Copy link
Collaborator

yonsch commented Aug 13, 2024

yonsch
yonsch previously approved these changes Aug 13, 2024
@de-nordic de-nordic removed their request for review August 20, 2024 15:11
@martinjaeger martinjaeger removed their request for review August 29, 2024 11:39
@ajf58
Copy link
Collaborator

ajf58 commented Sep 10, 2024

FWIW I've been using this PR in my work with no issues. I think this is good-to-go/LGTM etc. Is there anything I can do to help progress this PR?

soburi added 4 commits October 5, 2024 19:27
Update RaspberryPi Pico hal to 2.0.0 release

Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
The directory structure has changed in 2.0.0,
so we update it accordingly.

Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
Following the GPIO interface changes in pico-sdk 2.0.0.

Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
Some symbol names have been conflicted with introducing pico-sdk 2.0.0.
Rename these.

Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
@zephyrbot zephyrbot removed the DNM This PR should not be merged (Do Not Merge) label Oct 5, 2024
@soburi
Copy link
Member Author

soburi commented Oct 7, 2024

@henrikbrixandersen
Could you re-approve this please?

@soburi
Copy link
Member Author

soburi commented Oct 7, 2024

@yonsch
I know you're busy, but could you please approve it?

@dkalowsk
Copy link
Collaborator

dkalowsk commented Oct 7, 2024

@yonsch if/when you approve we can move this PR in and get the larger RPi PR in afterwards.

@nashif nashif merged commit 16c9b74 into zephyrproject-rtos:main Oct 8, 2024
25 of 26 checks passed
@soburi soburi deleted the pico-sdk-2.0.0 branch October 8, 2024 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants