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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

loopj
Copy link

@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@190a144 (main) zephyrproject-rtos/hal_silabs#116 zephyrproject-rtos/hal_silabs#116/files

DNM label due to: 1 project with PR revision

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
Collaborator

@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 added 3 commits July 8, 2025 10:23
Adds SoC definitions and DTS files for SiLabs EFR32MG22 SoCs

Signed-off-by: James Smith <james@loopj.com>
Use CRYPTO_ACC_GECKO_TRNG for entropy on all xG22 SoCs.

Signed-off-by: James Smith <james@loopj.com>
Adds all known EFR32BG22 SoCs and associated DTS includes.

Signed-off-by: James Smith <james@loopj.com>
@loopj
Copy link
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
Adds support for the xg22_ek2710a (EFR32xG22E Explorer Kit) board.

Signed-off-by: James Smith <james@loopj.com>
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants