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 A124CECAAD8 for ; Wed, 21 Sep 2022 14:22:35 +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:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=rSk9yDcqEwbl3gpR1FsuqGfoqJaMblFWOhfKvIUOaLM=; b=pwmvxD0VdZ6PZQ SDLKJ2BSDKv5G0G8U+XU9sXxe9052FV0WCLunlxjfqdEQwLZSN4hCW+eRueHuKg/V2NbjCkHqFTz7 TrGNMM7+J7X+tEkk+w7fj6NlqQChkuwV1rRRWp5NDzymW2CLImfTyV9y0xmc5zDdy/h8Ch6/OcWa6 7qOsYO9NGbMi7B7qRZ6yy1TH24KNAAXxjv4YzCgNLcmSlEvkAT7YSUmV1NrxhJB8B3jzbA9BSCy2H EZLbbEGekivj5rIgx477pUEO4eSuGg1PuSsK6AeJtvbSem7kXoZIpUclod4ACodTXYGQfAZKAhe0v LgaVV/mcrO0qT/M8In2A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1ob0bY-00BYai-58; Wed, 21 Sep 2022 14:21:36 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1ob0bU-00BYZm-VR for linux-arm-kernel@lists.infradead.org; Wed, 21 Sep 2022 14:21:34 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id D725962C12; Wed, 21 Sep 2022 14:21:31 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2E25EC433C1; Wed, 21 Sep 2022 14:21:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1663770091; bh=sZuUg6MHLptQb1xH+y+4v8/pAIwRKYIeNBbcXlslgbI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Db6adK9H1qWDNCoP3x5e6eq5kaSMb403SagtmMWCT5UbHySzfBZh+UDq4mMbRaqk0 NVM921BmggbjZ6ZX7LK4ooi4yLHKld+x+4GIyqX23E/LHm/Y4Hr9AnQR9S1MF1YO+b ohSPVFRe4wA/b8wCIR70IFxIAZQfd2NC6r6wd7+17sCQSBWND8Ivx7UP59nB2UkExa ggBWpRWwOBVIOlShZ9xqssPiNl16SiIm/+EW3LgSikxgYGLWITkHQke9/ZT1vD1oyh z6OAKZB3Xmjd/VhA1PiLqb8pgEDNwvHG7I2ZBbDuygbXf2+BRjuu8KVMyE8muyLofB crsfW2AkPNfnA== Date: Wed, 21 Sep 2022 22:21:23 +0800 From: Shawn Guo To: Li Jun , Abel Vesa Cc: robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, linux-imx@nxp.com, laurent.pinchart@ideasonboard.com, l.stach@pengutronix.de, marex@denx.de, peng.fan@nxp.com, paul.elder@ideasonboard.com, Markus.Niebel@ew.tq-group.com, aford173@gmail.com, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3] arm64: dts: imx8mp: correct usb clocks Message-ID: <20220921142123.GT1728671@dragon> References: <1663565197-3961-1-git-send-email-jun.li@nxp.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1663565197-3961-1-git-send-email-jun.li@nxp.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220921_072133_143532_C951185F X-CRM114-Status: GOOD ( 19.56 ) 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 Added Abel. On Mon, Sep 19, 2022 at 01:26:37PM +0800, Li Jun wrote: > After commit cf7f3f4fa9e5 ("clk: imx8mp: fix usb_root_clk parent"), > usb_root_clk is no longer for suspend clock so update dts accordingly > to use right bus clock and suspend clock. > > Fixes: fb8587a2c165 ("arm64: dtsi: imx8mp: add usb nodes") > Cc: stable@vger.kernel.org # v5.19+ > Reviewed-by: Peng Fan > Tested-by: Alexander Stein > Signed-off-by: Li Jun I just realized this patch has a hard/build dependency on define IMX8MP_CLK_USB_SUSP. So the patch needs to go upstream via the same tree that the define is applied. I guess that's Abel's i.MX clk tree? In that case: Acked-by: Shawn Guo Shawn > --- > change for v3: > - Add fixes tag, Alexander's T-b and Peng's R-b tages. > > change for v2: > - Add stable tag to apply v5.19+ > > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > index fe178b7d063c..2f18778a057f 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > @@ -1169,7 +1169,7 @@ usb3_0: usb@32f10100 { > reg = <0x32f10100 0x8>, > <0x381f0000 0x20>; > clocks = <&clk IMX8MP_CLK_HSIO_ROOT>, > - <&clk IMX8MP_CLK_USB_ROOT>; > + <&clk IMX8MP_CLK_USB_SUSP>; > clock-names = "hsio", "suspend"; > interrupts = ; > power-domains = <&hsio_blk_ctrl IMX8MP_HSIOBLK_PD_USB>; > @@ -1182,9 +1182,9 @@ usb3_0: usb@32f10100 { > usb_dwc3_0: usb@38100000 { > compatible = "snps,dwc3"; > reg = <0x38100000 0x10000>; > - clocks = <&clk IMX8MP_CLK_HSIO_AXI>, > + clocks = <&clk IMX8MP_CLK_USB_ROOT>, > <&clk IMX8MP_CLK_USB_CORE_REF>, > - <&clk IMX8MP_CLK_USB_ROOT>; > + <&clk IMX8MP_CLK_USB_SUSP>; > clock-names = "bus_early", "ref", "suspend"; > interrupts = ; > phys = <&usb3_phy0>, <&usb3_phy0>; > @@ -1211,7 +1211,7 @@ usb3_1: usb@32f10108 { > reg = <0x32f10108 0x8>, > <0x382f0000 0x20>; > clocks = <&clk IMX8MP_CLK_HSIO_ROOT>, > - <&clk IMX8MP_CLK_USB_ROOT>; > + <&clk IMX8MP_CLK_USB_SUSP>; > clock-names = "hsio", "suspend"; > interrupts = ; > power-domains = <&hsio_blk_ctrl IMX8MP_HSIOBLK_PD_USB>; > @@ -1224,9 +1224,9 @@ usb3_1: usb@32f10108 { > usb_dwc3_1: usb@38200000 { > compatible = "snps,dwc3"; > reg = <0x38200000 0x10000>; > - clocks = <&clk IMX8MP_CLK_HSIO_AXI>, > + clocks = <&clk IMX8MP_CLK_USB_ROOT>, > <&clk IMX8MP_CLK_USB_CORE_REF>, > - <&clk IMX8MP_CLK_USB_ROOT>; > + <&clk IMX8MP_CLK_USB_SUSP>; > clock-names = "bus_early", "ref", "suspend"; > interrupts = ; > phys = <&usb3_phy1>, <&usb3_phy1>; > -- > 2.34.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel