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 F3E79C433EF for ; Sat, 9 Apr 2022 10:25:10 +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:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=M/r/iIBd8rw/uXd6xuVqljf24mcQ513n+wgZCPLTUqI=; b=W6bEY8FBNTPvgm RKidyJNqnJKgPVIz7lcZ5Zvr7iosjeDErKm8C1sBRVM15iNbT1TuX99yfws1W5Co+s/YyNgeBuZYt O5cqiizcL5fptEbTJz6zwCRjTq8b7idDuS205L8dwdQzwCCToyhF4ilwT5hcqx/8kLIfzNUyafQIj 0DfJVwzVeHpUyM2Y+x5ix4hvdMEEaxc0+ilX2AD1EBU4ZOp4tekTBgIb4hpEzIu+eTJvwIn7TcZ5J 60eQ67Ze71h7VeOf5Wyxu+3769HMCxRsnD/wOFlvb0VecH0G+5O9gffQAOavNk4pAoSBGuB+Ipvxn 6XpiZfwT96AAmph1kObg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nd8Ft-002niA-NU; Sat, 09 Apr 2022 10:23:45 +0000 Received: from gloria.sntech.de ([185.11.138.130]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nd8Fp-002ng7-FX; Sat, 09 Apr 2022 10:23:43 +0000 Received: from p508fd313.dip0.t-ipconnect.de ([80.143.211.19] helo=phil.localnet) by gloria.sntech.de with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1nd8Fg-0000g7-0v; Sat, 09 Apr 2022 12:23:32 +0200 From: Heiko Stuebner To: Frank Wunderlich , Frank Wunderlich Cc: linux-rockchip@lists.infradead.org, Rob Herring , Krzysztof Kozlowski , Peter Geis , Michael Riesch , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: Aw: [PATCH] arm64: dts: rockchip: Fix clocks for rk356x usb Date: Sat, 09 Apr 2022 12:23:31 +0200 Message-ID: <9183927.T7Z3S40VBb@phil> In-Reply-To: References: <20220409075147.136187-1-linux@fw-web.de> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220409_032341_568454_1DE006CC X-CRM114-Status: GOOD ( 14.01 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Am Samstag, 9. April 2022, 12:14:28 CEST schrieb Frank Wunderlich: > Hi, > > > Gesendet: Samstag, 09. April 2022 um 09:51 Uhr > > Von: "Frank Wunderlich" > > > - clock-names = "ref_clk", "suspend_clk", > > - "bus_clk"; > > + clock-names = "ref", "suspend_clk", > > + "bus_early"; > > dr_mode = "host"; > > phy_type = "utmi_wide"; > > power-domains = <&power RK3568_PD_PIPE>; > > @@ -280,8 +280,8 @@ usb_host1_xhci: usb@fd000000 { > > interrupts = ; > > clocks = <&cru CLK_USB3OTG1_REF>, <&cru CLK_USB3OTG1_SUSPEND>, > > <&cru ACLK_USB3OTG1>; > > - clock-names = "ref_clk", "suspend_clk", > > - "bus_clk"; > > + clock-names = "ref", "suspend_clk", > > + "bus_early"; > > dr_mode = "host"; > > this is the patch breaking it: > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=33fb697ec7e58c4f9b6a68d2786441189cd2df92 > > suspend clock needs to be renamed too from "suspend_clk" to "suspend" > > else i get this error on poweroff > > xhci-hcd xhci-hcd.1.auto: Host halt failed, -110 > > regards Frank ok, so do you want to send a v2 including that change? Alternatively I can also add this change when applying. Also for educational purposes, the format for referencing a commit you're fixing would be Fixes: 33fb697ec7e5 ("usb: dwc3: Get clocks individually") Signed-off-by: .... Heiko _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel