-
Notifications
You must be signed in to change notification settings - Fork 65
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
simplicity_sdk: Import EFM32 PG23/PG23 platform files #95
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! Thanks for this contribution. There are a few things that will need to be changed for this to be merged:
- The commit that adds the files from Simplicity SDK must have a section describing the origin of the content in the commit body, see https://docs.zephyrproject.org/latest/contribute/guidelines.html#identifying-contribution-origin for guidelines and ab08d35 for an example.
- The commit that adds the files from Simplicity SDK should be a separate commit from the one that changes the scripts.
- The commit that patches the SecureFault_IRQn must start with "simplicity_sdk: Patch [...]" to work with an upcoming script that automates the application of Simplicity SDK updates (revert patches/update/reapply patches).
- See additional comments on the contents of the scripts themselves.
In general, pg23 should be considered an instance of xg23, and pg28 should be considered an instance of xg28. Please take inspiration from the introduction of xg29 in the main tree when you do corresponding changes there. We are in the process of changing the directory layout of dts/arm/silabs/
and soc/silabs/silabs_s2/
to better group related content together. This means that moving the existing efr32xg23.dtsi
into the xg23
directory and renaming it to xg23.dtsi
would be appreciated as part of adding support for pg23, since that file can then be reused for both.
43ab7b5
to
dba5535
Compare
6a2bd24
to
7697eea
Compare
Origin: Simplicity SDK License: Zlib URL: https://github.com/SiliconLabs/simplicity_sdk Commit: 8627f8482564dc6d94b56512740a39d6f409a0eb Version: 2024.12.0 Purpose: Import EFM32PG23/PG28 and EFR32FG28 device files to unlock BRD2504a and BRD2506a DevKits Signed-off-by: Christoph Jans <jans.christoph@gmail.com>
Adding xg28 to devices to generate xg28-clock.h. Import pg23 and pg28/fg28 platform files. Signed-off-by: Christoph Jans <jans.christoph@gmail.com>
0dedee0
to
a0796eb
Compare
Hopefully now all request are incorporated in the changes. Will start preparing a board file PR on the primary github zephyr repo with your information on the xg23 as a common ancestor dtsi file. |
Add SecureFault_IRQn enum member if ARM_SECURE_FIRMWARE is set. Signed-off-by: Christoph Jans <jans.christoph@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks a lot!
We'll hold off on merging this until the corresponding main tree PR is opened, to allow CI to run on it. Please update west.yml to point to pull/95/head
for the hal_silabs module in that PR.
Import EFM32 platform files in preparation for upstream support of two EFM32 devKits, namely BRD2504a and BRD2506a. Adding two new groups pg23/pg28 for non-radio developer boards.