From: Abel Vesa <abel.vesa@xxxxxxxxxx>
commit d26c4ad3fa53e76a602a9974ade171c8399f2a29 upstream.
Allowing the usb_2 controller GDSC to be turned off during system suspend
renders the controller unable to resume.
So use PWRSTS_RET_ON instead in order to make sure this the GDSC doesn't
go down.
Fixes: 161b7c401f4b ("clk: qcom: Add Global Clock controller (GCC) driver for X1E80100")
Cc: stable@xxxxxxxxxxxxxxx # 6.8
Signed-off-by: Abel Vesa <abel.vesa@xxxxxxxxxx>
Reviewed-by: Johan Hovold <johan+linaro@xxxxxxxxxx>
Tested-by: Johan Hovold <johan+linaro@xxxxxxxxxx>
Link: https://lore.kernel.org/r/20250107-x1e80100-clk-gcc-fix-usb2-gdsc-pwrsts-v1-1-e15d1a5e7d80@xxxxxxxxxx
Signed-off-by: Bjorn Andersson <andersson@xxxxxxxxxx>
Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
---
drivers/clk/qcom/gcc-x1e80100.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- a/drivers/clk/qcom/gcc-x1e80100.c
+++ b/drivers/clk/qcom/gcc-x1e80100.c
@@ -6083,7 +6083,7 @@ static struct gdsc gcc_usb20_prim_gdsc =
.pd = {
.name = "gcc_usb20_prim_gdsc",
},
- .pwrsts = PWRSTS_OFF_ON,
+ .pwrsts = PWRSTS_RET_ON,
.flags = POLL_CFG_GDSCR | RETAIN_FF_ENABLE,
};
Patches currently in stable-queue which might be from abel.vesa@xxxxxxxxxx are
queue-6.12/dt-bindings-clock-add-i.mx91-clock-support.patch
queue-6.12/dt-bindings-clock-imx93-drop-imx93_clk_end-macro-def.patch
queue-6.12/clk-imx93-add-imx93_clk_spdif_ipg-clock.patch
queue-6.12/clk-imx8mp-fix-clkout1-2-support.patch
queue-6.12/clk-imx93-move-imx93_clk_end-macro-to-clk-driver.patch
queue-6.12/clk-qcom-gcc-x1e80100-do-not-turn-off-usb_2-controller-gdsc.patch
queue-6.12/arm64-dts-imx93-use-imx93_clk_spdif_ipg-as-spdif-ipg.patch
queue-6.12/clk-imx-add-i.mx91-clk.patch
queue-6.12/clk-imx-apply-some-clks-only-for-i.mx93.patch
queue-6.12/dt-bindings-clock-imx93-add-spdif-ipg-clk.patch
