Skip to content

boards: silabs: Add support for xg22_ek2710a board, additional xG22 SoCs #92808

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 5 commits into from
Aug 7, 2025

Conversation

loopj
Copy link
Contributor

@loopj loopj commented Jul 7, 2025

This PR adds support for:

  • Silicon Labs EFR32MG22 SoCs
  • Additional EFR32BG22 SoCs
  • The EFR32xG22E Explorer Kit board (BRD2710A / xG22-EK2710A)

Depends on zephyrproject-rtos/hal_silabs#116

Copy link

github-actions bot commented Jul 7, 2025

Hello @loopj, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

Copy link

github-actions bot commented Jul 7, 2025

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

Name Old Revision New Revision Diff
hal_silabs zephyrproject-rtos/hal_silabs@295caf3 zephyrproject-rtos/hal_silabs@13343bc (main) 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_silabs DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Jul 7, 2025
@tomi-font tomi-font removed their request for review July 8, 2025 10:26
Copy link
Contributor

@Martinhoff-maker Martinhoff-maker left a comment

Choose a reason for hiding this comment

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

Thanks for this addition @loopj!
For me, this PR is already really nice. I only have a few comments/clarifications.
Make sure to add commit bodies to all of your commits as required by the Zephyr contribution guidelines (https://docs.zephyrproject.org/latest/contribute/guidelines.html#commit-message-guidelines).
Additionally, in the first commit, I would suggest splitting the entropy driver changes into a separate commit.

@loopj
Copy link
Contributor Author

loopj commented Jul 8, 2025

Thanks for the quick reviews @Martinhoff-maker, @nordicjm!

I think I've addressed all review comments, let me know if I missed anything.

@loopj loopj requested a review from nordicjm July 10, 2025 03:13
nordicjm
nordicjm previously approved these changes Jul 14, 2025
Copy link
Contributor

@jerome-pouiller jerome-pouiller left a comment

Choose a reason for hiding this comment

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

Please add a picture of the board: https://silabs.scene7.com/is/image/siliconlabs/xg22-ek2710a-explorer-kit? (I belive you can even remove the background with https://www.adobe.com/express/feature/image/remove-background)

@loopj loopj dismissed stale reviews from Martinhoff-maker and jerome-pouiller via 17135ea July 28, 2025 17:23
@loopj loopj force-pushed the xg22-socs branch 3 times, most recently from 28b96a5 to 73d1758 Compare August 5, 2025 17:39
asmellby
asmellby previously approved these changes Aug 6, 2025
@jhedberg
Copy link
Member

jhedberg commented Aug 6, 2025

@loopj please update the west.yml reference now that the HAL tree PR has been merged

@loopj
Copy link
Contributor Author

loopj commented Aug 6, 2025

@loopj please update the west.yml reference now that the HAL tree PR has been merged

Done! Thanks everyone for the reviews

@asmellby
Copy link
Contributor

asmellby commented Aug 6, 2025

There's a merge conflict for west.yml, please rebase. The old merge base is also the reason for the (erroneous) binary blob tag that the bot just added.

loopj added 5 commits August 6, 2025 08:18
Adds SoC definitions and DTS files for SiLabs EFR32MG22 SoCs

Signed-off-by: James Smith <[email protected]>
Use CRYPTO_ACC_GECKO_TRNG for entropy on all xG22 SoCs.

Signed-off-by: James Smith <[email protected]>
Adds all known EFR32BG22 SoCs and associated DTS includes.

Signed-off-by: James Smith <[email protected]>
Adds support for the xg22_ek2710a (EFR32xG22E Explorer Kit) board.

Signed-off-by: James Smith <[email protected]>
Add a board image to the docs of the xg22_ek2710a

Signed-off-by: James Smith <[email protected]>
@loopj
Copy link
Contributor Author

loopj commented Aug 6, 2025

There's a merge conflict for west.yml, please rebase. The old merge base is also the reason for the (erroneous) binary blob tag that the bot just added.

Done, thanks @asmellby !

@github-actions github-actions bot removed DNM (manifest) This PR should not be merged (controlled by action-manifest) Binary Blobs Added labels Aug 6, 2025
Copy link

sonarqubecloud bot commented Aug 6, 2025

@cfriedt cfriedt merged commit 850050b into zephyrproject-rtos:main Aug 7, 2025
28 of 29 checks passed
@loopj loopj deleted the xg22-socs branch August 7, 2025 17:56
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