-
Notifications
You must be signed in to change notification settings - Fork 7.9k
boards: silabs: Add xg27 radio boards #96296
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
base: main
Are you sure you want to change the base?
boards: silabs: Add xg27 radio boards #96296
Conversation
Set #address-cells and #size-cells on usart0. Since usarts can be used for both UART and SPI, this property shouldn't be set in SoC-level DTS, since it makes it harder to switch compatible. Signed-off-by: Aksel Skauge Mellbye <aksel.mellbye@silabs.com>
Reformat devicetree files: * Sort nodes by unit address or name * Sort properties by category and name Add missing properties to existing nodes. Signed-off-by: Aksel Skauge Mellbye <aksel.mellbye@silabs.com>
Add missing peripheral and clock nodes for xg27. Signed-off-by: Aksel Skauge Mellbye <aksel.mellbye@silabs.com>
Add efr32bg27 and efr32mg27 socs. Signed-off-by: Aksel Skauge Mellbye <aksel.mellbye@silabs.com>
Use cryptoacc variant on all xg27 socs. Signed-off-by: Aksel Skauge Mellbye <aksel.mellbye@silabs.com>
The following west manifest projects have changed revision in this Pull Request:
⛔ DNM label due to: 1 project with PR revision Note: This message is automatically posted and updated by the Manifest GitHub Action. |
board_runner_args(jlink "--device=EFR32MG27CxxxF768") | ||
else() | ||
board_runner_args(jlink "--device=EFR32BG27CxxxF768") | ||
endif() |
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.
Maybe:
string(TOUPPER ${CONFIG_SOC} JLINK_DEVICE)
string(SUBSTRING 0 17 ${JLINK_DEVICE} JLINK_DEVICE)
board_runner_args(jlink "--device=${JLINK_DEVICE}")
... can be done in another PR.
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.
Won't have xxx
in the middle...
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.
What is the risk to have a future rb4119b that does not match with this definition? That's said, I have not better proposal.
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.
Very low. But if it happens, it doesn't need to include the generic .dtsi file.
Add 3 xg27 radio boards: * bg27_rb4110b * bg27_rb4111b * xg27_rb4194a Signed-off-by: Aksel Skauge Mellbye <aksel.mellbye@silabs.com>
949a94a
to
c085794
Compare
|
Reformat xg27 soc devicetree files according to https://docs.zephyrproject.org/latest/contribute/style/devicetree.html and https://docs.kernel.org/devicetree/bindings/dts-coding-style.html:
Add support for all socs in the xg27 family.
Add three xg27 radio boards:
bg27_rb4110b
,bg27_rb4111b
andxg27_rb4194a
.