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 CFECAC43458 for ; Wed, 1 Jul 2026 08:02:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type: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=CWNDIk7fyUz4D/2m8OAHIVTblCwgE7WjSABB2eON9n8=; b=ZxHRworj3w/EdcLsX2M1LHt78m nP05nWFfSJjCufoj8wKV0gUhGCfh7WbLEZFqPNr8a1nqyhMgVwmVLhobFWOIis2L+ne1cWCVmq/PQ vWnnePja6+JgVwoGmM5DnYSr07x/AiVvaHG95bBoksjxaZ+qlgtMhFg4yg8PhQw/UTlNoyvNqCe/t F7U4EdE4rFZBh+sk5tk2StsqcFmArHpsbQ3zeZTYn5g16IC4Eply/g4dB4tJAlk8Cb2Cs3/1Sne4g qFxyR4B4719aEjM8yyPJgYV1OMtBWK/jrWWfBgMUGnDlUZLkm9Jl/1zZx6AKj89rpr46X90MVHgv1 kDYBck0Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wepuC-00000000yfv-1cj3; 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> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit 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-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=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";