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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 716A9C43334 for ; Mon, 4 Jul 2022 13:51:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233830AbiGDNvP (ORCPT ); Mon, 4 Jul 2022 09:51:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57708 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233529AbiGDNvK (ORCPT ); Mon, 4 Jul 2022 09:51:10 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 63611655D; Mon, 4 Jul 2022 06:51:09 -0700 (PDT) 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 D295961594; Mon, 4 Jul 2022 13:51:08 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 085AAC341CB; Mon, 4 Jul 2022 13:51:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1656942668; bh=KvKeOxcHnEj4x6o/YxW2xXiwJTOrRcOqrpTbLSJnsnE=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=Wz6FlkeDtwnaVwEIaprnDp7s9qkYvSJSJyt4eeUq8wXV1DjeGOF5l41B1wwIyV/V1 CK+6rNOInqhxM3DbXlCCyvgszai80R0X+IcfzTWrNjT2k74wO2OCZhaoCI8OTBk40A 44CZY03Ha0b4SlHnqYSxqoioZs12vagHUC6M82iJUnISi2Q3DNOnXhjfI6qQBKacqs XX6asUZRfhJg6UBf9jIqq0A5teW0FrSUL8KtxxK2qtMNzKZgHnfMk6CbTUwkHizWav hi5qGBb71JeIk992Nwi8KKL8IUZgUXDCOBkBP5qWmxNHTUPEMdJMuvkuYrtCfv6tV9 v0Wk+JRCLcUTw== Message-ID: <36d8fbe2-b64d-4997-5836-b2c32e4d987f@kernel.org> Date: Mon, 4 Jul 2022 16:50:51 +0300 MIME-Version: 1.0 Subject: Re: [PATCH V3 10/10] arm64: dts: imx8mp: add NoC node Content-Language: en-US To: "Peng Fan (OSS)" , shawnguo@kernel.org, s.hauer@pengutronix.de, festevam@gmail.com, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, abel.vesa@nxp.com, abailon@baylibre.com, l.stach@pengutronix.de, laurent.pinchart@ideasonboard.com, marex@denx.de, paul.elder@ideasonboard.com, Markus.Niebel@ew.tq-group.com, aford173@gmail.com, cw00.choi@samsung.com, kyungmin.park@samsung.com, myungjoo.ham@samsung.com Cc: kernel@pengutronix.de, linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-imx@nxp.com, abelvesa@kernel.org, Peng Fan References: <20220703091132.1412063-1-peng.fan@oss.nxp.com> <20220703091132.1412063-11-peng.fan@oss.nxp.com> From: Georgi Djakov In-Reply-To: <20220703091132.1412063-11-peng.fan@oss.nxp.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Peng, On 3.07.22 12:11, Peng Fan (OSS) wrote: > From: Peng Fan > > Add i.MX8MP NoC node to make the interconnect i.MX8MP driver could work. > Currently dynamic frequency scaling of the i.MX8MP NoC has not been > supported, only NoC initial settings are configured by interconnect > driver. > > Signed-off-by: Peng Fan > --- > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 22 ++++++++++++++++++++++ > 1 file changed, 22 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > index eb2d516278eb..13a2ee77d3c6 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > @@ -4,6 +4,7 @@ > */ > > #include > +#include It's also possible to drop this line for now and add it in the next patches when we add the interconnect consumers. This will allow merging this DT patch independently. But it's up to Shawn. > #include > #include > #include > @@ -1019,6 +1020,27 @@ eqos: ethernet@30bf0000 { > }; > }; > > + noc: interconnect@32700000 { > + compatible = "fsl,imx8mp-noc", "fsl,imx8m-noc", "syscon"; I got the impression that we are dropping the syscon, or not? > + reg = <0x32700000 0x100000>; > + clocks = <&clk IMX8MP_CLK_NOC>; > + #interconnect-cells = <1>; > + > + operating-points-v2 = <&noc_opp_table>; > + > + noc_opp_table: opp-table { > + compatible = "operating-points-v2"; > + > + opp-200M { > + opp-hz = /bits/ 64 <200000000>; > + }; > + > + opp-1000M { > + opp-hz = /bits/ 64 <1000000000>; > + }; > + }; > + }; > + > aips4: bus@32c00000 { > compatible = "fsl,aips-bus", "simple-bus"; > reg = <0x32c00000 0x400000>; Patches 1-9 look good and I am planning to merge them. Thanks, Georgi 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 DBAB3C433EF for ; Mon, 4 Jul 2022 13:52:12 +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-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id: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=6+tY1+q5ct9PmiCIxxK1/00aSxZqk1gsk4qgpBP4uxg=; b=qr38j8VS474cqp kXmhA3Nq7gaoSxCIAqszaPq8T8ZGKBpOVA1FIEB2HMlTJyvU0+EZrpIU5jR9dSUn2Evnbg38CxipR S9YOKEqMfklLyO/ATjHlqi4qQXCR119dQUA+wfvZK2jv5PsaaETF1+PVYxaq37OOsBWC+rBMLeIUD qOuVloJJgpLEZMjBk2+1dWxNqbV6iD/ehhCT+KneGawoGwSdYwcnLBfeUTWeMB5btrFJlsJPALpsA UZ1+p9swhoJDpL+q2Dseh9gXlCsjUMtf6aGEzp4gu0gi2AUSXFoglh92D5/q9sp08Ikri+o2ZJK5g sfuUX2kb7vUqebyynkkg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o8MTp-009J4G-S3; Mon, 04 Jul 2022 13:51:13 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1o8MTm-009J1p-ET for linux-arm-kernel@lists.infradead.org; Mon, 04 Jul 2022 13:51:11 +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 CB3D16158B; Mon, 4 Jul 2022 13:51:08 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 085AAC341CB; Mon, 4 Jul 2022 13:51:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1656942668; bh=KvKeOxcHnEj4x6o/YxW2xXiwJTOrRcOqrpTbLSJnsnE=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=Wz6FlkeDtwnaVwEIaprnDp7s9qkYvSJSJyt4eeUq8wXV1DjeGOF5l41B1wwIyV/V1 CK+6rNOInqhxM3DbXlCCyvgszai80R0X+IcfzTWrNjT2k74wO2OCZhaoCI8OTBk40A 44CZY03Ha0b4SlHnqYSxqoioZs12vagHUC6M82iJUnISi2Q3DNOnXhjfI6qQBKacqs XX6asUZRfhJg6UBf9jIqq0A5teW0FrSUL8KtxxK2qtMNzKZgHnfMk6CbTUwkHizWav hi5qGBb71JeIk992Nwi8KKL8IUZgUXDCOBkBP5qWmxNHTUPEMdJMuvkuYrtCfv6tV9 v0Wk+JRCLcUTw== Message-ID: <36d8fbe2-b64d-4997-5836-b2c32e4d987f@kernel.org> Date: Mon, 4 Jul 2022 16:50:51 +0300 MIME-Version: 1.0 Subject: Re: [PATCH V3 10/10] arm64: dts: imx8mp: add NoC node Content-Language: en-US To: "Peng Fan (OSS)" , shawnguo@kernel.org, s.hauer@pengutronix.de, festevam@gmail.com, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, abel.vesa@nxp.com, abailon@baylibre.com, l.stach@pengutronix.de, laurent.pinchart@ideasonboard.com, marex@denx.de, paul.elder@ideasonboard.com, Markus.Niebel@ew.tq-group.com, aford173@gmail.com, cw00.choi@samsung.com, kyungmin.park@samsung.com, myungjoo.ham@samsung.com Cc: kernel@pengutronix.de, linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-imx@nxp.com, abelvesa@kernel.org, Peng Fan References: <20220703091132.1412063-1-peng.fan@oss.nxp.com> <20220703091132.1412063-11-peng.fan@oss.nxp.com> From: Georgi Djakov In-Reply-To: <20220703091132.1412063-11-peng.fan@oss.nxp.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220704_065110_621637_D92CF4B2 X-CRM114-Status: GOOD ( 18.49 ) 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Peng, On 3.07.22 12:11, Peng Fan (OSS) wrote: > From: Peng Fan > > Add i.MX8MP NoC node to make the interconnect i.MX8MP driver could work. > Currently dynamic frequency scaling of the i.MX8MP NoC has not been > supported, only NoC initial settings are configured by interconnect > driver. > > Signed-off-by: Peng Fan > --- > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 22 ++++++++++++++++++++++ > 1 file changed, 22 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > index eb2d516278eb..13a2ee77d3c6 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > @@ -4,6 +4,7 @@ > */ > > #include > +#include It's also possible to drop this line for now and add it in the next patches when we add the interconnect consumers. This will allow merging this DT patch independently. But it's up to Shawn. > #include > #include > #include > @@ -1019,6 +1020,27 @@ eqos: ethernet@30bf0000 { > }; > }; > > + noc: interconnect@32700000 { > + compatible = "fsl,imx8mp-noc", "fsl,imx8m-noc", "syscon"; I got the impression that we are dropping the syscon, or not? > + reg = <0x32700000 0x100000>; > + clocks = <&clk IMX8MP_CLK_NOC>; > + #interconnect-cells = <1>; > + > + operating-points-v2 = <&noc_opp_table>; > + > + noc_opp_table: opp-table { > + compatible = "operating-points-v2"; > + > + opp-200M { > + opp-hz = /bits/ 64 <200000000>; > + }; > + > + opp-1000M { > + opp-hz = /bits/ 64 <1000000000>; > + }; > + }; > + }; > + > aips4: bus@32c00000 { > compatible = "fsl,aips-bus", "simple-bus"; > reg = <0x32c00000 0x400000>; Patches 1-9 look good and I am planning to merge them. Thanks, Georgi _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel