Skip to content

soc : ti: Soc layer power domain and clock nodes init #91914

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

soc : ti: Soc layer power domain and clock nodes init #91914

wants to merge 1 commit into from
+73 −2

Conversation

Kronosblaster
Copy link
Contributor

@Kronosblaster Kronosblaster commented Jun 19, 2025 •
edited
Loading

Power domains are added to the device tree.
The enabled nodes with power domains assigned to them have them turn on in the SOC layer.
Init for enabled clock nodes.
This PR waits on #90216 to be merged.

@github-actions github-actions bot added the platform: TI K3 Texas Instruments Keystone 3 Processors label Jun 19, 2025
@bjarki-andreasen
Copy link
Contributor

Using PM_DEVICE_RUNTIME, power domains are automatically resumed (pm_device_runtime_get()) when a device on the power domain is resumed, so if you pm_device_runtime_get(DEVICE_DT_GET(DT_NODELABEL(i2c0))); its power domain i2c0_pd will be resumed automatically, the SYS_INIT should not be neccesary, unless its an intermediary solution until drivers for devices like i2c0 implement PM_DEVICE maybe?

@Kronosblaster
Copy link
Contributor Author

Using PM_DEVICE_RUNTIME, power domains are automatically resumed (pm_device_runtime_get()) when a device on the power domain is resumed, so if you pm_device_runtime_get(DEVICE_DT_GET(DT_NODELABEL(i2c0))); its power domain i2c0_pd will be resumed automatically, the SYS_INIT should not be neccesary, unless its an intermediary solution until drivers for devices like i2c0 implement PM_DEVICE maybe?

Yeah. There are some drivers that are shared between vendors. This is intended such that those don't have to be modified. All the devices with status = "okay" have their power domains turned on in the SOC layer with this method.

@Kronosblaster Kronosblaster force-pushed the sysInit branch 5 times, most recently from 7207f98 to 7ba0f78 Compare June 20, 2025 06:28
bjarki-andreasen
bjarki-andreasen reviewed Jun 20, 2025
View reviewed changes
dnltz
dnltz requested changes Jun 20, 2025
View reviewed changes
@Kronosblaster Kronosblaster force-pushed the sysInit branch 11 times, most recently from aa773e5 to fb714fe Compare June 24, 2025 04:55
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

dnltz
dnltz reviewed Jun 24, 2025
View reviewed changes
@Kronosblaster Kronosblaster requested a review from dnltz August 11, 2025 10:50
dnltz
dnltz reviewed Aug 11, 2025
View reviewed changes
@Kronosblaster Kronosblaster force-pushed the sysInit branch 2 times, most recently from e414cf2 to bb2c0eb Compare August 12, 2025 10:12
@Kronosblaster Kronosblaster requested a review from dnltz August 12, 2025 10:27
@Kronosblaster Kronosblaster force-pushed the sysInit branch 2 times, most recently from 4772327 to e32d2a9 Compare August 13, 2025 09:56
@Kronosblaster
soc: ti: Performs init for PDs and clocks in soc layer
efc6b86 
Turns on all the power domains for devices with a power-domain device
tree property in the soc layer.
Sets clock frequency for enabled devices.
Supported for the R5f0_0 core on the
AM243x EVM.
Makes dmsc node in the power domain driver static.

Signed-off-by: Dave Joseph <[email protected]>
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@Kronosblaster Kronosblaster changed the title soc : ti: Power domain support and soc layer power domain init soc : ti: Soc layer power domain and clock nodes init Aug 19, 2025
@glneo glneo mentioned this pull request Sep 2, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bjarki-andreasen bjarki-andreasen bjarki-andreasen left review comments

@glneo glneo Awaiting requested review from glneo

@gramsay0 gramsay0 Awaiting requested review from gramsay0

@vaishnavachath vaishnavachath Awaiting requested review from vaishnavachath

@dnltz dnltz Awaiting requested review from dnltz

@ceolin ceolin Awaiting requested review from ceolin

@JordanYates JordanYates Awaiting requested review from JordanYates

@nashif nashif Awaiting requested review from nashif

@teburd teburd Awaiting requested review from teburd

@tmleman tmleman Awaiting requested review from tmleman

Requested changes must be addressed to merge this pull request.

Assignees

@vaishnavachath vaishnavachath

Labels
area: Power Management platform: TI K3 Texas Instruments Keystone 3 Processors
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Kronosblaster @bjarki-andreasen @dnltz @zephyrbot @vaishnavachath