[PATCH v2 0/4] phy: rockchip: inno-usb2: ensure PHY is running when clock is requested
WARM1d
From: Sebastian Reichel <hidden>
Date: 2026-10-01 13:13:12
Also in:
linux-phy, linux-rockchip, lkml
This was noticed on RK3588 EVB1 when resuming from system suspend. This is technically a fix, but its unclear when the bug was introduced and system suspend is broken on RK3588 for quite a while and not just due to this problem. So I think this fix can be merged the normal way via linux-next. Changes in v2: - Link to v1: https://patch.msgid.link/20260908-phy-rockchip-inno-usb2-clock-fix-v1-1-f7d59c31b908@collabora.com - Move suspend exit code from rockchip_usb2phy_power_on into a separate patch and reuse it in the clock prepare function; move is done in a separate patch - Updated commit messages with proper rationale - Added one more patch dropping useless assignment of rport->suspended directly after calling rockchip_usb2phy_power_on/off - Add one more patch unifying and simplifying the debug messages for phy init/power_on/power_off/exit --- To: Vinod Koul <vkoul@kernel.org> To: Neil Armstrong <neil.armstrong@linaro.org> To: Manivannan Sadhasivam <mani@kernel.org> To: Heiko Stuebner <heiko@sntech.de> Cc: linux-phy@lists.infradead.org Cc: linux-arm-kernel@lists.infradead.org Cc: linux-rockchip@lists.infradead.org Cc: linux-kernel@vger.kernel.org Cc: Igor Paunovic <redacted> Cc: kernel@collabora.com Signed-off-by: Sebastian Reichel <redacted> --- Sebastian Reichel (4): phy: rockchip: inno-usb2: simplify and unify PHY op debug messages phy: rockchip: inno-usb2: drop duplicated update of rport->suspended phy: rockchip: inno-usb2: move suspend handling into new function phy: rockchip: inno-usb2: move suspend handling into clock prepare/unprepare for RK3588/RK3576 drivers/phy/rockchip/phy-rockchip-inno-usb2.c | 85 +++++++++++++++++++-------- 1 file changed, 59 insertions(+), 26 deletions(-) --- base-commit: cee9395acd8043be0644b25c34bfa86623f2b935 change-id: 20260908-phy-rockchip-inno-usb2-clock-fix-edd65b57f884 Best regards, -- Sebastian Reichel [off-list ref]