From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id EE530C43458 for ; Wed, 1 Jul 2026 08:02:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=UB5dgCsC3jDgx8bLG0K9iGWPIv5KE1Yjf39ylp0bs7M=; b=o8yNLZBvwvGAab GO8SStSb5MZGFoEq23ABSeIVuStA/SBOoGyYY2PIUPZWxPrYMACw2MGgHGj1HEJqtSr1P/cI8I7AV bLc4CWrQ2jhbmDxFSTB1QCFXtB6hIfEF4cHSB14gLfbE1vQN8xoNeAEQt6cTO1hClcCObChxkecho Wq3qwI5Hd1Id1i6Ms6hVlBlLLiz/yw0TwyL34a7hRKnhVq3Lmfdow9ehIhsm77puCAbbJN3QZzK7l kiG7OHdZ7RVaC37hZuuLowDc7MH3kKr15TMZJj3TqA5UhhS4lqgHi5aakwY5Y0DbdtKHRc8BkqXes QJOtjdQjDTuEGuKvdqLA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wepuC-00000000yfp-1HPA; Wed, 01 Jul 2026 08:02:48 +0000 Received: from sender4-pp-f112.zoho.com ([136.143.188.112]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wepu8-00000000yfH-3egi; Wed, 01 Jul 2026 08:02:46 +0000 ARC-Seal: i=1; a=rsa-sha256; t=1782892953; cv=none; d=zohomail.com; s=zohoarc; b=aTPEf3BqjKu6ldy2Xslnu+HCEWFE6vblrMkjVb5aeayVdGPijkyKG1LMZ6CZNLezp4ugzaVWMYco2vxL9LfHDXR2hdnbFzSDgPg8upjd9SJe1Rjm5Zsgvr4aAiXu7+iJqGDW0WJl8SIG1lR2U5JKH1fsYHzSxQW8Qfvy8d2mH8s= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1782892953; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=CWNDIk7fyUz4D/2m8OAHIVTblCwgE7WjSABB2eON9n8=; b=lamTyYNS9M6k0ALZ9lZlnurd7UG/ZXESPXWU00I5ivICDw4j4INNfn5LX3Qz3+BsPU8LTB3JEBwOk+a6HRM4V7bADRXU7UhppH+jbKrp/wCU829XSmn1tOa9BufYdJFyudT94cSNT1cfFuyD9w6AK+MciAzMUPHMLl4NQyu4VCM= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=michael.riesch@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1782892953; s=zohomail; d=collabora.com; i=michael.riesch@collabora.com; h=Message-ID:Date:Date:MIME-Version:Subject:Subject:To:To:Cc:Cc:References:From:From:In-Reply-To:Content-Type:Content-Transfer-Encoding:Message-Id:Reply-To; bh=CWNDIk7fyUz4D/2m8OAHIVTblCwgE7WjSABB2eON9n8=; b=F0yG8nQo3YbIjKWV9YHgILL1rxYz9mgW3xw/AfWH94ua+v/J8PH7nd3RXd24npaQ IQR5CFW6blT7eUGRjNoFo4TdbDDuGKkWvGSfKjMCC09bfsJAMtMR+F8FIfyglywBXBh wvcWiRoSQ2Xgy2B2JXi+Gxz6KCm+/vVsV2ha6qGc= Received: by mx.zohomail.com with SMTPS id 1782892951290681.0323311430374; Wed, 1 Jul 2026 01:02:31 -0700 (PDT) Message-ID: <39bf50d1-a54c-41c0-bfee-ef73c293b990@collabora.com> Date: Wed, 1 Jul 2026 10:02:26 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 2/2] arm64: dts: rockchip: Fix devices suspend freeze on RK3568/RK3566 To: Jonas Karlman , Heiko Stuebner , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Peter Geis Cc: Diederik de Haas , Greg Kroah-Hartman , devicetree@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-usb@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <20260624192726.781864-1-jonas@kwiboo.se> <20260624192726.781864-3-jonas@kwiboo.se> Content-Language: en-US From: Michael Riesch In-Reply-To: <20260624192726.781864-3-jonas@kwiboo.se> X-ZohoMailClient: External X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260701_010244_970845_0793B7C4 X-CRM114-Status: GOOD ( 17.28 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org Hi Jonas, On 6/24/26 21:27, Jonas Karlman wrote: > The EHCI/OHCI controller depends on clk_usbphy1_480m being enabled, or > the system may freeze when registers are accessed, i.e. during suspend > in ohci_suspend(). > > Add the missing clk_usbphy1_480m clocks reference to EHCI/OHCI > controllers to ensure the clock is enabled when ECHI/OHCI registers are > accessed to prevent a system freeze. > > Fixes suspend pm_test issue with EHCI/OHCI devices due to the missing > clk_usbphy1_480m reference and makes following pm_test modes work: > > echo N > /sys/module/printk/parameters/console_suspend > > echo devices > /sys/power/pm_test > echo platform > /sys/power/pm_test > echo processors > /sys/power/pm_test > echo core > /sys/power/pm_test > > echo mem > /sys/power/state > > Fixes: 91c4c3e06a25 ("arm64: dts: rockchip: add usb2 nodes to rk3568 device tree") > Fixes: 78f7186095db ("arm64: dts: rockchip: rename and sort the rk356x usb2 phy handles") > Tested-by: Diederik de Haas > Signed-off-by: Jonas Karlman Acked-by: Michael Riesch Thanks and best regards, Michael > --- > v2: Collect t-b tag > --- > arch/arm64/boot/dts/rockchip/rk356x-base.dtsi | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/arch/arm64/boot/dts/rockchip/rk356x-base.dtsi b/arch/arm64/boot/dts/rockchip/rk356x-base.dtsi > index a5832895bd39..c930a6fd6ea0 100644 > --- a/arch/arm64/boot/dts/rockchip/rk356x-base.dtsi > +++ b/arch/arm64/boot/dts/rockchip/rk356x-base.dtsi > @@ -321,44 +321,44 @@ its: msi-controller@fd440000 { > }; > > usb_host0_ehci: usb@fd800000 { > - compatible = "generic-ehci"; > + compatible = "rockchip,rk3568-ehci", "generic-ehci"; > reg = <0x0 0xfd800000 0x0 0x40000>; > interrupts = ; > clocks = <&cru HCLK_USB2HOST0>, <&cru HCLK_USB2HOST0_ARB>, > - <&cru PCLK_USB>; > + <&cru PCLK_USB>, <&usb2phy1>; > phys = <&usb2phy1_otg>; > phy-names = "usb"; > status = "disabled"; > }; > > usb_host0_ohci: usb@fd840000 { > - compatible = "generic-ohci"; > + compatible = "rockchip,rk3568-ohci", "generic-ohci"; > reg = <0x0 0xfd840000 0x0 0x40000>; > interrupts = ; > clocks = <&cru HCLK_USB2HOST0>, <&cru HCLK_USB2HOST0_ARB>, > - <&cru PCLK_USB>; > + <&cru PCLK_USB>, <&usb2phy1>; > phys = <&usb2phy1_otg>; > phy-names = "usb"; > status = "disabled"; > }; > > usb_host1_ehci: usb@fd880000 { > - compatible = "generic-ehci"; > + compatible = "rockchip,rk3568-ehci", "generic-ehci"; > reg = <0x0 0xfd880000 0x0 0x40000>; > interrupts = ; > clocks = <&cru HCLK_USB2HOST1>, <&cru HCLK_USB2HOST1_ARB>, > - <&cru PCLK_USB>; > + <&cru PCLK_USB>, <&usb2phy1>; > phys = <&usb2phy1_host>; > phy-names = "usb"; > status = "disabled"; > }; > > usb_host1_ohci: usb@fd8c0000 { > - compatible = "generic-ohci"; > + compatible = "rockchip,rk3568-ohci", "generic-ohci"; > reg = <0x0 0xfd8c0000 0x0 0x40000>; > interrupts = ; > clocks = <&cru HCLK_USB2HOST1>, <&cru HCLK_USB2HOST1_ARB>, > - <&cru PCLK_USB>; > + <&cru PCLK_USB>, <&usb2phy1>; > phys = <&usb2phy1_host>; > phy-names = "usb"; > status = "disabled"; _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip