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 30C51C001E0 for ; Mon, 23 Oct 2023 13:23:49 +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=rfK8/AAjyFC1UgHo2+b9e/ZSqPsgt6JN9mNbo61MFNc=; b=UEaGIaJBX7bVsZ LhWrIgw3T8Uaycq/Z0dALAj+greVMmQIpeJP+8yzekAdxDZO54LNGsL2L5zxLPlE3ZQCIhITNMGuG uI7mCdK27xD0CmTTUGrpj9zqTzvy9sx8KtLIWebl/CicEzHU+8A5mSPdeYWHRmV+CwIUyn7xFDBO4 72Xfn+QeTCqWebmUe9WtTqBqxGK5OgsxNQY6yEmvMdQ1OPqSSwOJUxmDw2oMvGnQ1Rk4Pmm40zsMh aW5YHqeiQOdy4F6tqSjcCNbJB42LBvcrJ/uF/1l/vJAr2J/tBzZChoXuBakvgmZdOaGobY9w1wVEE EPkjGFH+cFKcsilQwXgg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1quutz-007QuI-09; Mon, 23 Oct 2023 13:23:27 +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 1quutv-007Qt4-1C for linux-arm-kernel@lists.infradead.org; Mon, 23 Oct 2023 13:23:25 +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=1698067403; x=1729603403; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=KraaAR9AJRx4szBcb69Mh9Dil932gzrCb3ff3bUUqeE=; b=AKQu1qM9ISxr/+OuGRMxyQv8FLQuh2xwwdh2esOg+j1j3kCftE2paKMs whEA37pT3wUd8UnLiNaVxkUekodgIBw597aLsfrCBF9IDGtQoPDUGAkR2 VxPgRgfOjs+ZmP/JcvDG6EH7wDKuOjdjhhtmemcKFfZyoQFuKZKJUzgcB JsQb2sWNAykivCcb4z1IN+5b6lme60UjUZNrbuJkQ4zt2Xikzvswa8npE yew6rLiro637pvdLhPI8O5KTDmhwIcYG8PYeeQBZoLgpd1gTt3AGWRjj9 ko7+2WQQBDkcWSP0f/exCyedbZviaCW1BYfX4hkYo79p7cXdVowDJihvt g==; X-IronPort-AV: E=Sophos;i="6.03,244,1694728800"; d="scan'208";a="33603121" Received: from vtuxmail01.tq-net.de ([10.115.0.20]) by mx1.tq-group.com with ESMTP; 23 Oct 2023 15:23:19 +0200 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 C6A0128007F; Mon, 23 Oct 2023 15:23:18 +0200 (CEST) From: Alexander Stein To: linux-arm-kernel@lists.infradead.org, Adam Ford Cc: aford@beaconembedded.com, Adam Ford , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH V2] arm64: dts: imx8mp: Add NPU Node Date: Mon, 23 Oct 2023 15:23:21 +0200 Message-ID: <9185912.CDJkKcVGEf@steina-w> Organization: TQ-Systems GmbH In-Reply-To: <20231022161911.10792-1-aford173@gmail.com> References: <20231022161911.10792-1-aford173@gmail.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231023_062323_791148_231D3C08 X-CRM114-Status: GOOD ( 17.05 ) 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 Adam, thanks for the update. Am Sonntag, 22. Oktober 2023, 18:19:10 CEST schrieb Adam Ford: > The NPU is based on the Vivante GC8000 and its power-domain > is controlled my pgc_mlmix. Since the power-domain uses > some of these clocks, setup the clock parent and rates > inside the power-domain, and add the NPU node. > = > The data sheet states the CLK_ML_AHB should be 300MHz for > nominal, but 800MHz clock will divide down to 266 instead. > Boards which operate in over-drive mode should update the > clocks on their boards accordingly. When the driver loads, > the NPU numerates as: > = > etnaviv-gpu 38500000.npu: model: GC8000, revision: 8002 There seems to be some race condition upon GPU/NPU detection. Sometimes I g= et = these messages: [ 1.338100] etnaviv-gpu 38000000.gpu: model: GC7000, revision: 6204 [ 1.344469] etnaviv-gpu 38008000.gpu: model: GC520, revision: 5341 [ 1.350748] etnaviv-gpu 38500000.npu: model: GC0, revision: 0 [ 1.356514] etnaviv-gpu 38500000.npu: Unknown GPU model Sometimes I see GC8000 being detected as you have written. Despite that the = patch itself looks good. Reviewed-by: Alexander Stein > = > Signed-off-by: Adam Ford > --- > = > V2: Move the clock parent and rate assignments to the ppc_mlmix node > since clock parents should be configured before they are used, > and pgc_mlmix uses them first. > Slow the clock rates down to confirm to nominal mode instead of > overdrive mode. > = > diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi > b/arch/arm64/boot/dts/freescale/imx8mp.dtsi index > c9a610ba4836..a18b9ba98ea2 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > @@ -887,6 +887,15 @@ pgc_mlmix: power-domain@24 { > clocks =3D <&clk = IMX8MP_CLK_ML_AXI>, > <&clk = IMX8MP_CLK_ML_AHB>, > <&clk = IMX8MP_CLK_NPU_ROOT>; > + assigned-clocks =3D = <&clk IMX8MP_CLK_ML_CORE>, > + = <&clk IMX8MP_CLK_ML_AXI>, > + = <&clk IMX8MP_CLK_ML_AHB>; > + assigned-clock- parents =3D <&clk IMX8MP_SYS_PLL1_800M>, > + = <&clk IMX8MP_SYS_PLL1_800M>, > + = <&clk IMX8MP_SYS_PLL1_800M>; > + assigned-clock- rates =3D <800000000>, > + = <800000000>, > + = <300000000>; > }; > }; > }; > @@ -2012,6 +2021,18 @@ vpumix_blk_ctrl: blk-ctrl@38330000 { > interconnect-names =3D "g1", "g2", "vc8000e"; > }; > = > + npu: npu@38500000 { > + compatible =3D "vivante,gc"; > + reg =3D <0x38500000 0x200000>; > + interrupts =3D ; > + clocks =3D <&clk IMX8MP_CLK_NPU_ROOT>, > + <&clk IMX8MP_CLK_NPU_ROOT>, > + <&clk IMX8MP_CLK_ML_AXI>, > + <&clk IMX8MP_CLK_ML_AHB>; > + clock-names =3D "core", "shader", "bus", "reg"; > + power-domains =3D <&pgc_mlmix>; > + }; > + > gic: interrupt-controller@38800000 { > compatible =3D "arm,gic-v3"; > reg =3D <0x38800000 0x10000>, -- = 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