Skip to content

soc: silabs: siwx91x: Add siwx91x Power Manager driver #89761

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 4 commits into from
Jun 13, 2025

Conversation

fimohame
Copy link
Contributor

@fimohame fimohame commented May 10, 2025

This commit introduces Power Manager driver support for the siwx917 device, enabling both PS4 Standby and PS4 Sleep modes. Additionally, it enables runtime GPIO driver support for the siwx917 device.

@fimohame fimohame force-pushed the SiWx91x-PM-Driver branch 10 times, most recently from 8085b87 to c80f644 Compare May 16, 2025 15:07
Copy link

github-actions bot commented May 16, 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@411072b zephyrproject-rtos/hal_silabs@95fb8af (main) zephyrproject-rtos/hal_silabs@411072b6..95fb8af4

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 May 16, 2025
@fimohame fimohame closed this May 16, 2025
@fimohame fimohame reopened this May 16, 2025
@fimohame fimohame force-pushed the SiWx91x-PM-Driver branch from fd27103 to 4fe0e06 Compare May 19, 2025 05:21
@fimohame fimohame marked this pull request as ready for review May 19, 2025 08:50
@fimohame fimohame changed the title driver: powermanager: siwx917: Add siwx91x Power Manager driver soc: silabs: siwx91x: Add siwx91x Power Manager driver Jun 13, 2025
@fimohame fimohame force-pushed the SiWx91x-PM-Driver branch 2 times, most recently from a70c1c9 to fde200c Compare June 13, 2025 11:55
@fimohame fimohame requested a review from asmellby June 13, 2025 12:33
@fimohame fimohame force-pushed the SiWx91x-PM-Driver branch 2 times, most recently from bab7275 to a0b1deb Compare June 13, 2025 13:21
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Jun 13, 2025
${WISECONNECT_DIR}/components/device/silabs/si91x/mcu/core/chip/src/rsi_ps_ram_func.c
${WISECONNECT_DIR}/components/device/silabs/si91x/mcu/drivers/cmsis_driver/USART.c
${WISECONNECT_DIR}/components/device/silabs/si91x/mcu/drivers/cmsis_driver/UDMA.c
${WISECONNECT_DIR}/components/device/silabs/si91x/mcu/drivers/cmsis_driver/SAI.c
Copy link
Contributor

Choose a reason for hiding this comment

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

Until now we have have included most of the wiseconnect sources files without checking if they were really used by the configuration or not.

This practice does not allow to properly understand (or audit) the architecture of the code and raise concern for the maintenance of the code.

Hence, could you only include in the build the files required by the configuration selected by the user? (For the existing code, I will take of it later in the semester).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have addressed it.

asmellby
asmellby previously approved these changes Jun 13, 2025
jhedberg
jhedberg previously approved these changes Jun 13, 2025
@fimohame fimohame dismissed stale reviews from Martinhoff-maker, jhedberg, and asmellby via b1e477f June 13, 2025 14:34
@fimohame fimohame force-pushed the SiWx91x-PM-Driver branch from a0b1deb to b1e477f Compare June 13, 2025 14:34
fimohame added 4 commits June 13, 2025 20:12
This commit enables the Power Manager driver
support for the siwx91x device.

Signed-off-by: S Mohamed Fiaz <fiaz.mohamed@silabs.com>
This commit enables the device runtime driver support
for the siwx91x device.

Signed-off-by: S Mohamed Fiaz <fiaz.mohamed@silabs.com>
This commit updates the hal_silabs revision

Signed-off-by: S Mohamed Fiaz <fiaz.mohamed@silabs.com>
…ceTree

This commit adds support for configuring the power/performance
profile for the siwx91x device using a generic
'power-profile' property in DeviceTree.
The property is available for NWP nodes,
allowing flexible selection of power management
profiles per application or board via overlay.

Signed-off-by: S Mohamed Fiaz <fiaz.mohamed@silabs.com>
Copy link

@dkalowsk dkalowsk merged commit aaf21a4 into zephyrproject-rtos:main Jun 13, 2025
28 checks passed
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.

6 participants