Skip to content

Conversation

ajf58
Copy link
Contributor

@ajf58 ajf58 commented Aug 7, 2025

Update the Raspberry Pi Pico HAL to be based on the latest release of the upstream SDK (v2.2.0).

This PR also changes the default delay multiplier for the external oscillator to match the SDK's default, which first changed in SDK v2.1.1.

#87513 is superseded by this PR.

Copy link

github-actions bot commented Aug 7, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_rpi_pico zephyrproject-rtos/hal_rpi_pico@b547a36 zephyrproject-rtos/hal_rpi_pico@09e9575 (zephyr) zephyrproject-rtos/[email protected]

All manifest checks OK

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

@github-actions github-actions bot added manifest manifest-hal_rpi_pico DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Aug 7, 2025
@ajf58 ajf58 force-pushed the rpi-pico-update-sdk-2.2.0 branch from 298fffd to 7a55cc6 Compare August 8, 2025 19:15
Copy link

sonarqubecloud bot commented Aug 8, 2025

soburi
soburi previously approved these changes Sep 10, 2025
@soburi
Copy link
Member

soburi commented Sep 10, 2025

@ThreeEights

Could you take a look, please?

@soburi
Copy link
Member

soburi commented Sep 10, 2025

@ajf58 Please rebase.

Sort some entries alphabetically to improve readability and
maintainability. This commit just reorders existing entries and does not
change any include paths.

Signed-off-by: Andrew Featherstone <[email protected]>
Update the Raspberry Pi Pico HAL to be based on the latest release of
the upstream SDK (v2.2.0).

SDK v2.2.0 has new include directory structure, so update the relevant
CMakeLists.txt files accordingly.

Signed-off-by: Andrew Featherstone <[email protected]>
v2.1.1 onwards of the SDK increases the default. Update the binding to
match. See raspberrypi/pico-sdk#2245

Signed-off-by: Andrew Featherstone <[email protected]>
@ajf58 ajf58 force-pushed the rpi-pico-update-sdk-2.2.0 branch from 7a55cc6 to 9e1a72c Compare September 21, 2025 19:29
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Sep 21, 2025
@ajf58
Copy link
Contributor Author

ajf58 commented Sep 21, 2025

@ajf58 Please rebase.

Done. N.b. if we get this PR approved and merged then #96331 isn't required.

Copy link

@soburi
Copy link
Member

soburi commented Sep 22, 2025

@ajf58 Please rebase.

Done. N.b. if we get this PR approved and merged then #96331 isn't required.

I forgot that the zephyr side hasn't finished merging yet. Understood.

@soburi
Copy link
Member

soburi commented Sep 22, 2025

@ThreeEights
Could you take a look, please?

@soburi
Copy link
Member

soburi commented Oct 11, 2025

@dsseng

Could you take a look, please?

@soburi soburi requested a review from dsseng October 11, 2025 06:10
${rp2_common_dir}/hardware_sync/include
${rp2_common_dir}/hardware_timer/include
${rp2_common_dir}/hardware_pll/include
${rp2_common_dir}/hardware_rcp/include
Copy link
Member

Choose a reason for hiding this comment

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

Do we use/want to use RCP? Non-blocking, I think it's better to have it findable for the case anyone wants to use

Copy link
Member

Choose a reason for hiding this comment

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

bootrom.c depends on it. This is required.

@soburi
Copy link
Member

soburi commented Oct 11, 2025

@carlescufi @fabiobaltieri

I wanted to have the approval completed by a person with Collaborator privileges, but I am unable to contact them, so I would like to ask the two people I asked to approve the request in the past again.

I apologize for bothering you during your busy schedule, but I would appreciate your help.

@soburi soburi added this to the v4.3.0 milestone Oct 14, 2025
@soburi
Copy link
Member

soburi commented Oct 15, 2025

@ThreeEights

Could you take a look?

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.

4 participants