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 75F90C4332F for ; Thu, 14 Dec 2023 13:36: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: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=J1wub/0dV3pmE+5tSdv4Aq4xAb3jRwTN83LBI+rSB/o=; b=vjka7TZtp8b9nJ FMFe4yz7+h1JdYRahQermodinsmOMdDRV8VAssRsFpJFeu19Cz7+iBcWRvhuLq1kmO+TxBDKpKnFx +d9dYPhJkvORYq3nHcm6Nw6D+wzaxDYbRO2o1BQ8xUIT0q6t/BzWoRxm+ImnaheRuV8iVmdkjyBYF Yo57uYr6tRz613rh1I1Qd9R6UQGCuAqBmsHCcMzyxKF+9s7gAfj2Is286m7BpvS5+/0Cl1gz9+vRa fj5s3HXlDFXBhELHGcyuqlTRayiUxJNtlfvZdQyTAPz8gZiuaEOjG5uhhsQlpUD0sPDBzLj1fEhSg vTP+X1/8Dz5ArN6pHa3Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rDlsg-000Ql7-2a; Thu, 14 Dec 2023 13:36:02 +0000 Received: from mx1.tq-group.com ([93.104.207.81]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rDlsd-000Qke-2E for linux-arm-kernel@lists.infradead.org; Thu, 14 Dec 2023 13:36:01 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1702560959; x=1734096959; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=RdULjyNo4x0HQt1Yc4rzvZrYNht6UL/1jhIj0E0HlN8=; b=cFjqobWoAvolJEe185FMI7q/+4ZbBpEM8evoSshUyBWhmHPSrIWHrlNT ofki4qpd+fiF3c91OXHphnGqaBGVmZT1R/mQ2VdecavSThbGNxnIkJBOj Ts19bf38Qlirf+wjh19JbXEJiBTU8j7Cpa8VUvvdGB1r2DDdk/7WH4TEP c1UYU8HzC3Il5jHggdAWtKgndwsFucfRZ8QYfj4RQAb9JLHVx95kHAlVS 9kNQ7Ctwoglq2/9rqWjleRNJk9DB74NNXKN/PrL+VRCbwc/Pq6fqnO++O K6NZvWDRcE7S5rv0ielo7v6mH9MGqcF7fpxZS0vxEfyL2j8d9UYAwNQSo w==; X-IronPort-AV: E=Sophos;i="6.04,275,1695679200"; d="scan'208";a="34510600" Received: from vtuxmail01.tq-net.de ([10.115.0.20]) by mx1.tq-group.com with ESMTP; 14 Dec 2023 14:35:57 +0100 Received: from steina-w.localnet (steina-w.tq-net.de [10.123.53.18]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by vtuxmail01.tq-net.de (Postfix) with ESMTPSA id 2E694280075; Thu, 14 Dec 2023 14:35:57 +0100 (CET) From: Alexander Stein To: robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, linux-arm-kernel@lists.infradead.org Cc: linux-imx@nxp.com, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Xu Yang Subject: Re: [PATCH 3/4] arm64: dts: imx93: add usb nodes Date: Thu, 14 Dec 2023 14:35:57 +0100 Message-ID: <8309719.T7Z3S40VBb@steina-w> Organization: TQ-Systems GmbH In-Reply-To: <20231214112442.2412079-3-xu.yang_2@nxp.com> References: <20231214112442.2412079-1-xu.yang_2@nxp.com> <20231214112442.2412079-3-xu.yang_2@nxp.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231214_053600_121210_6AA8D15F X-CRM114-Status: GOOD ( 12.93 ) 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="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, thanks for the patch. Am Donnerstag, 14. Dezember 2023, 12:24:41 CET schrieb Xu Yang: > There are 2 USB controllers on i.MX93. Add them. > = > Signed-off-by: Xu Yang > --- > arch/arm64/boot/dts/freescale/imx93.dtsi | 58 ++++++++++++++++++++++++ > 1 file changed, 58 insertions(+) > = > diff --git a/arch/arm64/boot/dts/freescale/imx93.dtsi > b/arch/arm64/boot/dts/freescale/imx93.dtsi index 34c0540276d1..bcf4861cfe= de > 100644 > --- a/arch/arm64/boot/dts/freescale/imx93.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx93.dtsi > @@ -171,6 +171,20 @@ cm33: remoteproc-cm33 { > status =3D "disabled"; > }; > = > + usbphynop1: usbphynop1 { > + #phy-cells =3D <0>; > + compatible =3D "usb-nop-xceiv"; compatible first. > + clocks =3D <&clk IMX93_CLK_USB_PHY_BURUNIN>; > + clock-names =3D "main_clk"; > + }; > + > + usbphynop2: usbphynop2 { > + #phy-cells =3D <0>; > + compatible =3D "usb-nop-xceiv"; compatible first. > + clocks =3D <&clk IMX93_CLK_USB_PHY_BURUNIN>; > + clock-names =3D "main_clk"; > + }; > + > soc@0 { > compatible =3D "simple-bus"; > #address-cells =3D <1>; > @@ -1059,5 +1073,49 @@ ddr-pmu@4e300dc0 { > reg =3D <0x4e300dc0 0x200>; > interrupts =3D ; > }; > + > + usbotg1: usb@4c100000 { > + compatible =3D "fsl,imx8mm-usb", "fsl,imx7d-usb", = "fsl,imx27-usb"; > + reg =3D <0x4c100000 0x200>; > + interrupts =3D ; > + clocks =3D <&clk IMX93_CLK_USB_CONTROLLER_GATE>, > + <&clk IMX93_CLK_HSIO_32K_GATE>; > + clock-names =3D "usb_ctrl_root_clk", = "usb_wakeup_clk"; drivers/usb/chipidea/ci_hdrc_imx.c uses 3 clocks: "ipg", "ahb" and "per", s= ee = imx_get_clks(). How is this supposed to work? > + assigned-clocks =3D <&clk IMX93_CLK_HSIO>; > + assigned-clock-parents =3D <&clk = IMX93_CLK_SYS_PLL_PFD1_DIV2>; > + assigned-clock-rates =3D <133000000>; > + fsl,usbphy =3D <&usbphynop1>; > + fsl,usbmisc =3D <&usbmisc1 0>; > + status =3D "disabled"; > + }; > + > + usbmisc1: usbmisc@4c100200 { > + compatible =3D "fsl,imx8mm-usbmisc", "fsl,imx7d- usbmisc", > + "fsl,imx6q-usbmisc"; This means you are compatible to fsl,imx7d-usbmisc. Which means you use = register MX7D_USB_OTG_PHY_STATUS in USB misc (USBNC). But i.MX93 RM Rev. 2 = (04/2023) lacks this and other registers. > + #index-cells =3D <1>; > + reg =3D <0x4c100200 0x200>; reg after compatible. > + }; > + > + usbotg2: usb@4c200000 { > + compatible =3D "fsl,imx8mm-usb", "fsl,imx7d-usb", = "fsl,imx27-usb"; > + reg =3D <0x4c200000 0x200>; > + interrupts =3D ; > + clocks =3D <&clk IMX93_CLK_USB_CONTROLLER_GATE>, > + <&clk IMX93_CLK_HSIO_32K_GATE>; > + clock-names =3D "usb_ctrl_root_clk", = "usb_wakeup_clk"; The same as for usbotg1 applies here. > + assigned-clocks =3D <&clk IMX93_CLK_HSIO>; > + assigned-clock-parents =3D <&clk = IMX93_CLK_SYS_PLL_PFD1_DIV2>; > + assigned-clock-rates =3D <133000000>; > + fsl,usbphy =3D <&usbphynop2>; > + fsl,usbmisc =3D <&usbmisc2 0>; > + status =3D "disabled"; > + }; > + > + usbmisc2: usbmisc@4c200200 { > + compatible =3D "fsl,imx8mm-usbmisc", "fsl,imx7d- usbmisc", > + "fsl,imx6q-usbmisc"; > + #index-cells =3D <1>; > + reg =3D <0x4c200200 0x200>; The same as for usbmisc1 applies here. > + }; > }; > }; Best regards, Alexander -- = TQ-Systems GmbH | M=FChlstra=DFe 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht M=FCnchen, HRB 105018 Gesch=E4ftsf=FChrer: Detlef Schneider, R=FCdiger Stahl, Stefan Schneider http://www.tq-group.com/ _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel