-
Notifications
You must be signed in to change notification settings - Fork 6k
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
soc: espressif: refactor clock and RTC subsystems #72701
soc: espressif: refactor clock and RTC subsystems #72701
Conversation
The following west manifest projects have been modified in this Pull Request:
Note: This message is automatically posted and updated by the Manifest GitHub Action. |
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.
all the default values of properties in the DT bindings need justifications for why they are the default value in the descriptions
6e8d85c
to
caf880e
Compare
@decsny there is no need indeed for this default values, it can depend mostly on the application and power requirements. I removed the default values and added |
caf880e
to
727ce09
Compare
727ce09
to
b0a825d
Compare
b0a825d
to
5be9e43
Compare
Add changes to comply with rtc refactor. Signed-off-by: Lucas Tamborrino <lucas.tamborrino@espressif.com>
The RTC subsystem in espressif's SOCs, among other tasks is responsible for clock selection for CPU and for low power domain clocks such as RTC_SLOW and RTC_FAST. This commit allows for proper clock source and rate selection for CPU, using the espressif,riscv and espressif,xtensa-lx6/7 bindings. It also enables clock selection for RTC_FAST and RTC_SLOW, that impacts some peripherals, such as rtc_timer. Signed-off-by: Lucas Tamborrino <lucas.tamborrino@espressif.com>
Changes for making the clock control api tests generic for any board. All the device subsys definitions were moved to its own folder according to the clock compatible. Also if the clock's async feature is not supported by the target, the test is skipped instead of failed. Signed-off-by: Lucas Tamborrino <lucas.tamborrino@espressif.com>
Add support for rtc timer node in the test. Signed-off-by: Lucas Tamborrino <lucas.tamborrino@espressif.com>
Add tests for the rtc clk subsystem. Signed-off-by: Lucas Tamborrino <lucas.tamborrino@espressif.com>
5be9e43
to
9bf47c3
Compare
@nashif, can you take a look please? |
The RTC subsystem in Espressif's SOCs, among other tasks, is responsible for clock selection for CPU and for low power domain clocks such as RTC_SLOW and RTC_FAST.
This commit allows for proper clock source and rate selection for CPU, using the espressif,riscv and espressif,xtensa-lx6/7 bindings.
It also enables clock selection for RTC_FAST and RTC_SLOW, that impacts some peripherals, such as rtc_timer.
It adds also changes for making the clock control api tests generic for any board.
All the device subsys definitions were moved to its own folder according to the clock compatible.
If the clock's async feature is not supported by the target, the test is skipped instead of failed.
Finally, it add tests for the rtc clk subsystem itself.