Use gdsc_synced_poweroff_disable() callback for USB PHY GDSCs on Glymur - #1184
Open
kona-jagadeesh wants to merge 2 commits into
Open
kona-jagadeesh wants to merge 2 commits into
kona-jagadeesh wants to merge 2 commits into
Conversation
USB and PCIe GDSCs must remain enabled during system suspend for USB host mode and PCIe non-D3cold use cases. These GDSCs currently use PWRSTS_RET_ON in the GDSC driver, which prevents the gdsc driver from disabling them in hardware after the first enable. Consequently, they remain enabled indefinitely, even after their consumers become inactive, preventing the GDSCs from being powered off when no longer needed. Instead, use the GenPDs synced_poweroff flag to control whether a GDSC is disabled during suspend. USB and PCIe drivers can leave the flag unset when the GDSC must remain enabled, and set it when the GDSC should be powered off. This allows consumer drivers to have more dynamic control over the GDSC power state based on their current use case. The existing gdsc_gx_gdsc() callback also disables the GDSC based on synced_poweroff flag but it polls for GDSC status during disable if synced poweroff flag is set. But few PCIE GDSCs have a VOTABLE flag and should not be polled for status during disable. Hence introduce a new gdsc_synced_poweroff_disable() callback that disables the GDSC based on GenPD synced_poweroff flag but won't poll for status if GDSC has VOTABLE flag. Change-Id: Ic57554e741a9d04729078fa8647f8e10952d6639 Signed-off-by: Jagadeesh Kona <jagadeesh.kona@oss.qualcomm.com>
Test Matrix
|
Test jobs on 640c294 |
kona-jagadeesh
force-pushed
the
glymur-usb-phy-gdsc
branch
2 times, most recently
from
September 25, 2026 06:46
03de3c1 to
d2ed939
Compare
Use the gdsc_synced_poweroff_disable() as power_off callback for USB GDSCs on Glymur platform. This allows the GDSCs to be disabled during suspend only when the consumer driver explicitly calls dev_pm_genpd_synced_poweroff() before suspend. Consumer drivers can therefore keep the GDSCs powered on during system suspend by not calling dev_pm_genpd_synced_poweroff() before suspend. Change-Id: Ifbf55774c2f4f83a42437051d724e060144e3221 Signed-off-by: Jagadeesh Kona <jagadeesh.kona@oss.qualcomm.com>
kona-jagadeesh
force-pushed
the
glymur-usb-phy-gdsc
branch
from
September 25, 2026 06:47
d2ed939 to
8259207
Compare
Test Matrix
|
Test jobs on d2ed939 |
Test Matrix
|
Test jobs on 03de3c1 |
Test Matrix
|
Test jobs on 8259207 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This series includes changes to use gdsc_synced_poweroff_disable() callback for gcc_usb_0_phy_gdsc and gcc_usb_1_phy_gdsc on Glymur platform. With this change, these GDSCs get disabled only when USB drivers explicitly invoke dev_pm_genpd_synced_poweroff() before suspend, else GDSCs stay ON during suspend.