-
Notifications
You must be signed in to change notification settings - Fork 8.3k
soc: stm32f7: add power management support #99157
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?
Conversation
3faf095 to
80311ed
Compare
a03342c to
f9cd6d0
Compare
Adds low-power mode support for STM32F7 series Implements basic sleep and stop modes integrated with the power management subsystem, similar to the STM32F4 implementation. Added Mode: STOP low-power mode, exposed as Zephyr PM state "suspend-to-idle". Tested on: NUCLEO-F722ZE board. Signed-off-by: Roy Jamil <roy.jamil@ac6.fr>
f9cd6d0 to
b813643
Compare
|
| compatible = "arm,armv7m-mpu"; | ||
| reg = <0xe000ed90 0x40>; | ||
| }; | ||
| }; |
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.
This is suspicious...
| power-states { | ||
| stop: stop { | ||
| compatible = "zephyr,power-state"; | ||
| power-state-name = "suspend-to-idle"; | ||
| min-residency-us = <400>; | ||
| exit-latency-us = <300>; | ||
| }; |
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.
Indentation issue + missing };
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.
@erwango If there are missing }; the dts-linter would complain so I do not believe we have any missing };
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.
It's rather the indentation that is fuzzy. Indeed compiling this DTS is valid. But CI DTS linter compliance test obviously complains.
| #include <stm32f7xx_ll_bus.h> | ||
| #include <stm32f7xx_ll_cortex.h> | ||
| #include <stm32f7xx_ll_pwr.h> | ||
| #include <stm32f7xx.h> |
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.
See #99012
| #include <zephyr/drivers/clock_control/stm32_clock_control.h> | ||
| #include <zephyr/drivers/counter.h> | ||
| #include <zephyr/drivers/interrupt_controller/gic.h> | ||
| #include <zephyr/kernel.h> | ||
| #include <zephyr/logging/log.h> | ||
| #include <zephyr/pm/pm.h> | ||
| #include <zephyr/init.h> |
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.
Suggestion
-#include <zephyr/drivers/clock_control/stm32_clock_control.h>
-#include <zephyr/drivers/counter.h>
-#include <zephyr/drivers/interrupt_controller/gic.h>
#include <zephyr/kernel.h>
#include <zephyr/logging/log.h>
#include <zephyr/pm/pm.h>
-#include <zephyr/init.h>| power-states { | ||
| stop: stop { | ||
| compatible = "zephyr,power-state"; | ||
| power-state-name = "suspend-to-idle"; | ||
| min-residency-us = <400>; | ||
| exit-latency-us = <300>; | ||
| }; |
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.
It's rather the indentation that is fuzzy. Indeed compiling this DTS is valid. But CI DTS linter compliance test obviously complains.



Adds low-power mode support for STM32F7 series
Implements basic sleep and stop modes integrated
with the power management subsystem, similar to
the STM32F4 implementation.
Added Mode:
STOP low-power mode, exposed as Zephyr PM state
"suspend-to-idle".
Tested on: NUCLEO-F722ZE board.
Signed-off-by: Roy Jamil roy.jamil@ac6.fr