All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rex-BC Chen <rex-bc.chen@mediatek.com>
To: <matthias.bgg@gmail.com>
Cc: <jia-wei.chang@mediatek.com>, <roger.lu@mediatek.com>,
	<hsinyi@google.com>, <khilman@baylibre.com>,
	<angelogioacchino.delregno@collabora.com>,
	<linux-pm@vger.kernel.org>, <devicetree@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-mediatek@lists.infradead.org>,
	<Project_Global_Chrome_Upstream_Group@mediatek.com>,
	"Andrew-sh . Cheng" <andrew-sh.cheng@mediatek.com>,
	<rafael@kernel.org>, <viresh.kumar@linaro.org>,
	<robh+dt@kernel.org>, <krzk+dt@kernel.org>
Subject: Re: [PATCH v6 08/10] arm64: dts: mediatek: Add opp table and clock property for MT8183 cpufreq
Date: Thu, 12 May 2022 14:11:42 +0800	[thread overview]
Message-ID: <ea3b53f234d68f993bc7cd540b33832434cf931c.camel@mediatek.com> (raw)
In-Reply-To: <20220505115226.20130-9-rex-bc.chen@mediatek.com>

On Thu, 2022-05-05 at 19:52 +0800, Rex-BC Chen wrote:
> - Add cpufreq opp table.
> - Add MediaTek cci opp table.
> - Add property of opp table and clock fro cpufreq.
> 
> Signed-off-by: Andrew-sh.Cheng <andrew-sh.cheng@mediatek.com>
> Signed-off-by: Rex-BC Chen <rex-bc.chen@mediatek.com>
> Reviewed-by: AngeloGioacchino Del Regno <
> angelogioacchino.delregno@collabora.com>
> ---
>  arch/arm64/boot/dts/mediatek/mt8183-evb.dts |  32 +++
>  arch/arm64/boot/dts/mediatek/mt8183.dtsi    | 270
> ++++++++++++++++++++
>  2 files changed, 302 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
> b/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
> index f3fd3cca23e9..8953dbf84f3e 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
> @@ -412,6 +412,38 @@
>  
>  };
>  
> +&cpu0 {
> +	proc-supply = <&mt6358_vproc12_reg>;
> +};
> +
> +&cpu1 {
> +	proc-supply = <&mt6358_vproc12_reg>;
> +};
> +
> +&cpu2 {
> +	proc-supply = <&mt6358_vproc12_reg>;
> +};
> +
> +&cpu3 {
> +	proc-supply = <&mt6358_vproc12_reg>;
> +};
> +
> +&cpu4 {
> +	proc-supply = <&mt6358_vproc11_reg>;
> +};
> +
> +&cpu5 {
> +	proc-supply = <&mt6358_vproc11_reg>;
> +};
> +
> +&cpu6 {
> +	proc-supply = <&mt6358_vproc11_reg>;
> +};
> +
> +&cpu7 {
> +	proc-supply = <&mt6358_vproc11_reg>;
> +};
> +
>  &uart0 {
>  	status = "okay";
>  };
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> index 01e650251928..cecf96b628b7 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> @@ -42,6 +42,244 @@
>  		rdma1 = &rdma1;
>  	};
>  
> +	cluster0_opp: opp-table-cluster0 {
> +		compatible = "operating-points-v2";
> +		opp-shared;
> +		opp0-793000000 {
> +			opp-hz = /bits/ 64 <793000000>;
> +			opp-microvolt = <650000>;
> +			required-opps = <&opp2_00>;
> +		};
> +		opp0-910000000 {
> +			opp-hz = /bits/ 64 <910000000>;
> +			opp-microvolt = <687500>;
> +			required-opps = <&opp2_01>;
> +		};
> +		opp0-1014000000 {
> +			opp-hz = /bits/ 64 <1014000000>;
> +			opp-microvolt = <718750>;
> +			required-opps = <&opp2_02>;
> +		};
> +		opp0-1131000000 {
> +			opp-hz = /bits/ 64 <1131000000>;
> +			opp-microvolt = <756250>;
> +			required-opps = <&opp2_03>;
> +		};
> +		opp0-1248000000 {
> +			opp-hz = /bits/ 64 <1248000000>;
> +			opp-microvolt = <800000>;
> +			required-opps = <&opp2_04>;
> +		};
> +		opp0-1326000000 {
> +			opp-hz = /bits/ 64 <1326000000>;
> +			opp-microvolt = <818750>;
> +			required-opps = <&opp2_05>;
> +		};
> +		opp0-1417000000 {
> +			opp-hz = /bits/ 64 <1417000000>;
> +			opp-microvolt = <850000>;
> +			required-opps = <&opp2_06>;
> +		};
> +		opp0-1508000000 {
> +			opp-hz = /bits/ 64 <1508000000>;
> +			opp-microvolt = <868750>;
> +			required-opps = <&opp2_07>;
> +		};
> +		opp0-1586000000 {
> +			opp-hz = /bits/ 64 <1586000000>;
> +			opp-microvolt = <893750>;
> +			required-opps = <&opp2_08>;
> +		};
> +		opp0-1625000000 {
> +			opp-hz = /bits/ 64 <1625000000>;
> +			opp-microvolt = <906250>;
> +			required-opps = <&opp2_09>;
> +		};
> +		opp0-1677000000 {
> +			opp-hz = /bits/ 64 <1677000000>;
> +			opp-microvolt = <931250>;
> +			required-opps = <&opp2_10>;
> +		};
> +		opp0-1716000000 {
> +			opp-hz = /bits/ 64 <1716000000>;
> +			opp-microvolt = <943750>;
> +			required-opps = <&opp2_11>;
> +		};
> +		opp0-1781000000 {
> +			opp-hz = /bits/ 64 <1781000000>;
> +			opp-microvolt = <975000>;
> +			required-opps = <&opp2_12>;
> +		};
> +		opp0-1846000000 {
> +			opp-hz = /bits/ 64 <1846000000>;
> +			opp-microvolt = <1000000>;
> +			required-opps = <&opp2_13>;
> +		};
> +		opp0-1924000000 {
> +			opp-hz = /bits/ 64 <1924000000>;
> +			opp-microvolt = <1025000>;
> +			required-opps = <&opp2_14>;
> +		};
> +		opp0-1989000000 {
> +			opp-hz = /bits/ 64 <1989000000>;
> +			opp-microvolt = <1050000>;
> +			required-opps = <&opp2_15>;
> +		};	};
> +
> +	cluster1_opp: opp-table-cluster1 {
> +		compatible = "operating-points-v2";
> +		opp-shared;
> +		opp1-793000000 {
> +			opp-hz = /bits/ 64 <793000000>;
> +			opp-microvolt = <700000>;
> +			required-opps = <&opp2_00>;
> +		};
> +		opp1-910000000 {
> +			opp-hz = /bits/ 64 <910000000>;
> +			opp-microvolt = <725000>;
> +			required-opps = <&opp2_01>;
> +		};
> +		opp1-1014000000 {
> +			opp-hz = /bits/ 64 <1014000000>;
> +			opp-microvolt = <750000>;
> +			required-opps = <&opp2_02>;
> +		};
> +		opp1-1131000000 {
> +			opp-hz = /bits/ 64 <1131000000>;
> +			opp-microvolt = <775000>;
> +			required-opps = <&opp2_03>;
> +		};
> +		opp1-1248000000 {
> +			opp-hz = /bits/ 64 <1248000000>;
> +			opp-microvolt = <800000>;
> +			required-opps = <&opp2_04>;
> +		};
> +		opp1-1326000000 {
> +			opp-hz = /bits/ 64 <1326000000>;
> +			opp-microvolt = <825000>;
> +			required-opps = <&opp2_05>;
> +		};
> +		opp1-1417000000 {
> +			opp-hz = /bits/ 64 <1417000000>;
> +			opp-microvolt = <850000>;
> +			required-opps = <&opp2_06>;
> +		};
> +		opp1-1508000000 {
> +			opp-hz = /bits/ 64 <1508000000>;
> +			opp-microvolt = <875000>;
> +			required-opps = <&opp2_07>;
> +		};
> +		opp1-1586000000 {
> +			opp-hz = /bits/ 64 <1586000000>;
> +			opp-microvolt = <900000>;
> +			required-opps = <&opp2_08>;
> +		};
> +		opp1-1625000000 {
> +			opp-hz = /bits/ 64 <1625000000>;
> +			opp-microvolt = <912500>;
> +			required-opps = <&opp2_09>;
> +		};
> +		opp1-1677000000 {
> +			opp-hz = /bits/ 64 <1677000000>;
> +			opp-microvolt = <931250>;
> +			required-opps = <&opp2_10>;
> +		};
> +		opp1-1716000000 {
> +			opp-hz = /bits/ 64 <1716000000>;
> +			opp-microvolt = <950000>;
> +			required-opps = <&opp2_11>;
> +		};
> +		opp1-1781000000 {
> +			opp-hz = /bits/ 64 <1781000000>;
> +			opp-microvolt = <975000>;
> +			required-opps = <&opp2_12>;
> +		};
> +		opp1-1846000000 {
> +			opp-hz = /bits/ 64 <1846000000>;
> +			opp-microvolt = <1000000>;
> +			required-opps = <&opp2_13>;
> +		};
> +		opp1-1924000000 {
> +			opp-hz = /bits/ 64 <1924000000>;
> +			opp-microvolt = <1025000>;
> +			required-opps = <&opp2_14>;
> +		};
> +		opp1-1989000000 {
> +			opp-hz = /bits/ 64 <1989000000>;
> +			opp-microvolt = <1050000>;
> +			required-opps = <&opp2_15>;
> +		};
> +	};
> +
> +	cci_opp: opp-table-cci {
> +		compatible = "operating-points-v2";
> +		opp-shared;
> +		opp2_00: opp-273000000 {
> +			opp-hz = /bits/ 64 <273000000>;
> +			opp-microvolt = <650000>;
> +		};
> +		opp2_01: opp-338000000 {
> +			opp-hz = /bits/ 64 <338000000>;
> +			opp-microvolt = <687500>;
> +		};
> +		opp2_02: opp-403000000 {
> +			opp-hz = /bits/ 64 <403000000>;
> +			opp-microvolt = <718750>;
> +		};
> +		opp2_03: opp-463000000 {
> +			opp-hz = /bits/ 64 <463000000>;
> +			opp-microvolt = <756250>;
> +		};
> +		opp2_04: opp-546000000 {
> +			opp-hz = /bits/ 64 <546000000>;
> +			opp-microvolt = <800000>;
> +		};
> +		opp2_05: opp-624000000 {
> +			opp-hz = /bits/ 64 <624000000>;
> +			opp-microvolt = <818750>;
> +		};
> +		opp2_06: opp-689000000 {
> +			opp-hz = /bits/ 64 <689000000>;
> +			opp-microvolt = <850000>;
> +		};
> +		opp2_07: opp-767000000 {
> +			opp-hz = /bits/ 64 <767000000>;
> +			opp-microvolt = <868750>;
> +		};
> +		opp2_08: opp-845000000 {
> +			opp-hz = /bits/ 64 <845000000>;
> +			opp-microvolt = <893750>;
> +		};
> +		opp2_09: opp-871000000 {
> +			opp-hz = /bits/ 64 <871000000>;
> +			opp-microvolt = <906250>;
> +		};
> +		opp2_10: opp-923000000 {
> +			opp-hz = /bits/ 64 <923000000>;
> +			opp-microvolt = <931250>;
> +		};
> +		opp2_11: opp-962000000 {
> +			opp-hz = /bits/ 64 <962000000>;
> +			opp-microvolt = <943750>;
> +		};
> +		opp2_12: opp-1027000000 {
> +			opp-hz = /bits/ 64 <1027000000>;
> +			opp-microvolt = <975000>;
> +		};
> +		opp2_13: opp-1092000000 {
> +			opp-hz = /bits/ 64 <1092000000>;
> +			opp-microvolt = <1000000>;
> +		};
> +		opp2_14: opp-1144000000 {
> +			opp-hz = /bits/ 64 <1144000000>;
> +			opp-microvolt = <1025000>;
> +		};
> +		opp2_15: opp-1196000000 {
> +			opp-hz = /bits/ 64 <1196000000>;
> +			opp-microvolt = <1050000>;
> +		};
> +	};
> +
>  	cpus {
>  		#address-cells = <1>;
>  		#size-cells = <0>;
> @@ -85,6 +323,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <741>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP0>;
> +			clocks = <&mcucfg CLK_MCU_MP0_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster0_opp>;
>  			dynamic-power-coefficient = <84>;
>  			#cooling-cells = <2>;
>  		};
> @@ -96,6 +338,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <741>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP0>;
> +			clocks = <&mcucfg CLK_MCU_MP0_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster0_opp>;
>  			dynamic-power-coefficient = <84>;
>  			#cooling-cells = <2>;
>  		};
> @@ -107,6 +353,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <741>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP0>;
> +			clocks = <&mcucfg CLK_MCU_MP0_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster0_opp>;
>  			dynamic-power-coefficient = <84>;
>  			#cooling-cells = <2>;
>  		};
> @@ -118,6 +368,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <741>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP0>;
> +			clocks = <&mcucfg CLK_MCU_MP0_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster0_opp>;
>  			dynamic-power-coefficient = <84>;
>  			#cooling-cells = <2>;
>  		};
> @@ -129,6 +383,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <1024>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP1>;
> +			clocks = <&mcucfg CLK_MCU_MP2_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster1_opp>;
>  			dynamic-power-coefficient = <211>;
>  			#cooling-cells = <2>;
>  		};
> @@ -140,6 +398,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <1024>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP1>;
> +			clocks = <&mcucfg CLK_MCU_MP2_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster1_opp>;
>  			dynamic-power-coefficient = <211>;
>  			#cooling-cells = <2>;
>  		};
> @@ -151,6 +413,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <1024>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP1>;
> +			clocks = <&mcucfg CLK_MCU_MP2_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster1_opp>;
>  			dynamic-power-coefficient = <211>;
>  			#cooling-cells = <2>;
>  		};
> @@ -162,6 +428,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <1024>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP1>;
> +			clocks = <&mcucfg CLK_MCU_MP2_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster1_opp>;
>  			dynamic-power-coefficient = <211>;
>  			#cooling-cells = <2>;
>  		};

Hello Matthias,

Could you give us some advice or ack for these patches of mt8183 dts?
The drviers and dt-binding of this series are picked by Viresh, and he
also wants to pick these three dts patches.

Thanks!

Viresh's branch: 
https://git.kernel.org/pub/scm/linux/kernel/git/vireshk/pm.git/log/?h=cpufreq/arm/linux-next

BRs,
Rex


WARNING: multiple messages have this Message-ID (diff)
From: Rex-BC Chen <rex-bc.chen@mediatek.com>
To: <matthias.bgg@gmail.com>
Cc: <jia-wei.chang@mediatek.com>, <roger.lu@mediatek.com>,
	<hsinyi@google.com>, <khilman@baylibre.com>,
	<angelogioacchino.delregno@collabora.com>,
	<linux-pm@vger.kernel.org>, <devicetree@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-mediatek@lists.infradead.org>,
	 <Project_Global_Chrome_Upstream_Group@mediatek.com>,
	"Andrew-sh . Cheng" <andrew-sh.cheng@mediatek.com>,
	<rafael@kernel.org>, <viresh.kumar@linaro.org>,
	<robh+dt@kernel.org>, <krzk+dt@kernel.org>
Subject: Re: [PATCH v6 08/10] arm64: dts: mediatek: Add opp table and clock property for MT8183 cpufreq
Date: Thu, 12 May 2022 14:11:42 +0800	[thread overview]
Message-ID: <ea3b53f234d68f993bc7cd540b33832434cf931c.camel@mediatek.com> (raw)
In-Reply-To: <20220505115226.20130-9-rex-bc.chen@mediatek.com>

On Thu, 2022-05-05 at 19:52 +0800, Rex-BC Chen wrote:
> - Add cpufreq opp table.
> - Add MediaTek cci opp table.
> - Add property of opp table and clock fro cpufreq.
> 
> Signed-off-by: Andrew-sh.Cheng <andrew-sh.cheng@mediatek.com>
> Signed-off-by: Rex-BC Chen <rex-bc.chen@mediatek.com>
> Reviewed-by: AngeloGioacchino Del Regno <
> angelogioacchino.delregno@collabora.com>
> ---
>  arch/arm64/boot/dts/mediatek/mt8183-evb.dts |  32 +++
>  arch/arm64/boot/dts/mediatek/mt8183.dtsi    | 270
> ++++++++++++++++++++
>  2 files changed, 302 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
> b/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
> index f3fd3cca23e9..8953dbf84f3e 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
> @@ -412,6 +412,38 @@
>  
>  };
>  
> +&cpu0 {
> +	proc-supply = <&mt6358_vproc12_reg>;
> +};
> +
> +&cpu1 {
> +	proc-supply = <&mt6358_vproc12_reg>;
> +};
> +
> +&cpu2 {
> +	proc-supply = <&mt6358_vproc12_reg>;
> +};
> +
> +&cpu3 {
> +	proc-supply = <&mt6358_vproc12_reg>;
> +};
> +
> +&cpu4 {
> +	proc-supply = <&mt6358_vproc11_reg>;
> +};
> +
> +&cpu5 {
> +	proc-supply = <&mt6358_vproc11_reg>;
> +};
> +
> +&cpu6 {
> +	proc-supply = <&mt6358_vproc11_reg>;
> +};
> +
> +&cpu7 {
> +	proc-supply = <&mt6358_vproc11_reg>;
> +};
> +
>  &uart0 {
>  	status = "okay";
>  };
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> index 01e650251928..cecf96b628b7 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> @@ -42,6 +42,244 @@
>  		rdma1 = &rdma1;
>  	};
>  
> +	cluster0_opp: opp-table-cluster0 {
> +		compatible = "operating-points-v2";
> +		opp-shared;
> +		opp0-793000000 {
> +			opp-hz = /bits/ 64 <793000000>;
> +			opp-microvolt = <650000>;
> +			required-opps = <&opp2_00>;
> +		};
> +		opp0-910000000 {
> +			opp-hz = /bits/ 64 <910000000>;
> +			opp-microvolt = <687500>;
> +			required-opps = <&opp2_01>;
> +		};
> +		opp0-1014000000 {
> +			opp-hz = /bits/ 64 <1014000000>;
> +			opp-microvolt = <718750>;
> +			required-opps = <&opp2_02>;
> +		};
> +		opp0-1131000000 {
> +			opp-hz = /bits/ 64 <1131000000>;
> +			opp-microvolt = <756250>;
> +			required-opps = <&opp2_03>;
> +		};
> +		opp0-1248000000 {
> +			opp-hz = /bits/ 64 <1248000000>;
> +			opp-microvolt = <800000>;
> +			required-opps = <&opp2_04>;
> +		};
> +		opp0-1326000000 {
> +			opp-hz = /bits/ 64 <1326000000>;
> +			opp-microvolt = <818750>;
> +			required-opps = <&opp2_05>;
> +		};
> +		opp0-1417000000 {
> +			opp-hz = /bits/ 64 <1417000000>;
> +			opp-microvolt = <850000>;
> +			required-opps = <&opp2_06>;
> +		};
> +		opp0-1508000000 {
> +			opp-hz = /bits/ 64 <1508000000>;
> +			opp-microvolt = <868750>;
> +			required-opps = <&opp2_07>;
> +		};
> +		opp0-1586000000 {
> +			opp-hz = /bits/ 64 <1586000000>;
> +			opp-microvolt = <893750>;
> +			required-opps = <&opp2_08>;
> +		};
> +		opp0-1625000000 {
> +			opp-hz = /bits/ 64 <1625000000>;
> +			opp-microvolt = <906250>;
> +			required-opps = <&opp2_09>;
> +		};
> +		opp0-1677000000 {
> +			opp-hz = /bits/ 64 <1677000000>;
> +			opp-microvolt = <931250>;
> +			required-opps = <&opp2_10>;
> +		};
> +		opp0-1716000000 {
> +			opp-hz = /bits/ 64 <1716000000>;
> +			opp-microvolt = <943750>;
> +			required-opps = <&opp2_11>;
> +		};
> +		opp0-1781000000 {
> +			opp-hz = /bits/ 64 <1781000000>;
> +			opp-microvolt = <975000>;
> +			required-opps = <&opp2_12>;
> +		};
> +		opp0-1846000000 {
> +			opp-hz = /bits/ 64 <1846000000>;
> +			opp-microvolt = <1000000>;
> +			required-opps = <&opp2_13>;
> +		};
> +		opp0-1924000000 {
> +			opp-hz = /bits/ 64 <1924000000>;
> +			opp-microvolt = <1025000>;
> +			required-opps = <&opp2_14>;
> +		};
> +		opp0-1989000000 {
> +			opp-hz = /bits/ 64 <1989000000>;
> +			opp-microvolt = <1050000>;
> +			required-opps = <&opp2_15>;
> +		};	};
> +
> +	cluster1_opp: opp-table-cluster1 {
> +		compatible = "operating-points-v2";
> +		opp-shared;
> +		opp1-793000000 {
> +			opp-hz = /bits/ 64 <793000000>;
> +			opp-microvolt = <700000>;
> +			required-opps = <&opp2_00>;
> +		};
> +		opp1-910000000 {
> +			opp-hz = /bits/ 64 <910000000>;
> +			opp-microvolt = <725000>;
> +			required-opps = <&opp2_01>;
> +		};
> +		opp1-1014000000 {
> +			opp-hz = /bits/ 64 <1014000000>;
> +			opp-microvolt = <750000>;
> +			required-opps = <&opp2_02>;
> +		};
> +		opp1-1131000000 {
> +			opp-hz = /bits/ 64 <1131000000>;
> +			opp-microvolt = <775000>;
> +			required-opps = <&opp2_03>;
> +		};
> +		opp1-1248000000 {
> +			opp-hz = /bits/ 64 <1248000000>;
> +			opp-microvolt = <800000>;
> +			required-opps = <&opp2_04>;
> +		};
> +		opp1-1326000000 {
> +			opp-hz = /bits/ 64 <1326000000>;
> +			opp-microvolt = <825000>;
> +			required-opps = <&opp2_05>;
> +		};
> +		opp1-1417000000 {
> +			opp-hz = /bits/ 64 <1417000000>;
> +			opp-microvolt = <850000>;
> +			required-opps = <&opp2_06>;
> +		};
> +		opp1-1508000000 {
> +			opp-hz = /bits/ 64 <1508000000>;
> +			opp-microvolt = <875000>;
> +			required-opps = <&opp2_07>;
> +		};
> +		opp1-1586000000 {
> +			opp-hz = /bits/ 64 <1586000000>;
> +			opp-microvolt = <900000>;
> +			required-opps = <&opp2_08>;
> +		};
> +		opp1-1625000000 {
> +			opp-hz = /bits/ 64 <1625000000>;
> +			opp-microvolt = <912500>;
> +			required-opps = <&opp2_09>;
> +		};
> +		opp1-1677000000 {
> +			opp-hz = /bits/ 64 <1677000000>;
> +			opp-microvolt = <931250>;
> +			required-opps = <&opp2_10>;
> +		};
> +		opp1-1716000000 {
> +			opp-hz = /bits/ 64 <1716000000>;
> +			opp-microvolt = <950000>;
> +			required-opps = <&opp2_11>;
> +		};
> +		opp1-1781000000 {
> +			opp-hz = /bits/ 64 <1781000000>;
> +			opp-microvolt = <975000>;
> +			required-opps = <&opp2_12>;
> +		};
> +		opp1-1846000000 {
> +			opp-hz = /bits/ 64 <1846000000>;
> +			opp-microvolt = <1000000>;
> +			required-opps = <&opp2_13>;
> +		};
> +		opp1-1924000000 {
> +			opp-hz = /bits/ 64 <1924000000>;
> +			opp-microvolt = <1025000>;
> +			required-opps = <&opp2_14>;
> +		};
> +		opp1-1989000000 {
> +			opp-hz = /bits/ 64 <1989000000>;
> +			opp-microvolt = <1050000>;
> +			required-opps = <&opp2_15>;
> +		};
> +	};
> +
> +	cci_opp: opp-table-cci {
> +		compatible = "operating-points-v2";
> +		opp-shared;
> +		opp2_00: opp-273000000 {
> +			opp-hz = /bits/ 64 <273000000>;
> +			opp-microvolt = <650000>;
> +		};
> +		opp2_01: opp-338000000 {
> +			opp-hz = /bits/ 64 <338000000>;
> +			opp-microvolt = <687500>;
> +		};
> +		opp2_02: opp-403000000 {
> +			opp-hz = /bits/ 64 <403000000>;
> +			opp-microvolt = <718750>;
> +		};
> +		opp2_03: opp-463000000 {
> +			opp-hz = /bits/ 64 <463000000>;
> +			opp-microvolt = <756250>;
> +		};
> +		opp2_04: opp-546000000 {
> +			opp-hz = /bits/ 64 <546000000>;
> +			opp-microvolt = <800000>;
> +		};
> +		opp2_05: opp-624000000 {
> +			opp-hz = /bits/ 64 <624000000>;
> +			opp-microvolt = <818750>;
> +		};
> +		opp2_06: opp-689000000 {
> +			opp-hz = /bits/ 64 <689000000>;
> +			opp-microvolt = <850000>;
> +		};
> +		opp2_07: opp-767000000 {
> +			opp-hz = /bits/ 64 <767000000>;
> +			opp-microvolt = <868750>;
> +		};
> +		opp2_08: opp-845000000 {
> +			opp-hz = /bits/ 64 <845000000>;
> +			opp-microvolt = <893750>;
> +		};
> +		opp2_09: opp-871000000 {
> +			opp-hz = /bits/ 64 <871000000>;
> +			opp-microvolt = <906250>;
> +		};
> +		opp2_10: opp-923000000 {
> +			opp-hz = /bits/ 64 <923000000>;
> +			opp-microvolt = <931250>;
> +		};
> +		opp2_11: opp-962000000 {
> +			opp-hz = /bits/ 64 <962000000>;
> +			opp-microvolt = <943750>;
> +		};
> +		opp2_12: opp-1027000000 {
> +			opp-hz = /bits/ 64 <1027000000>;
> +			opp-microvolt = <975000>;
> +		};
> +		opp2_13: opp-1092000000 {
> +			opp-hz = /bits/ 64 <1092000000>;
> +			opp-microvolt = <1000000>;
> +		};
> +		opp2_14: opp-1144000000 {
> +			opp-hz = /bits/ 64 <1144000000>;
> +			opp-microvolt = <1025000>;
> +		};
> +		opp2_15: opp-1196000000 {
> +			opp-hz = /bits/ 64 <1196000000>;
> +			opp-microvolt = <1050000>;
> +		};
> +	};
> +
>  	cpus {
>  		#address-cells = <1>;
>  		#size-cells = <0>;
> @@ -85,6 +323,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <741>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP0>;
> +			clocks = <&mcucfg CLK_MCU_MP0_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster0_opp>;
>  			dynamic-power-coefficient = <84>;
>  			#cooling-cells = <2>;
>  		};
> @@ -96,6 +338,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <741>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP0>;
> +			clocks = <&mcucfg CLK_MCU_MP0_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster0_opp>;
>  			dynamic-power-coefficient = <84>;
>  			#cooling-cells = <2>;
>  		};
> @@ -107,6 +353,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <741>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP0>;
> +			clocks = <&mcucfg CLK_MCU_MP0_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster0_opp>;
>  			dynamic-power-coefficient = <84>;
>  			#cooling-cells = <2>;
>  		};
> @@ -118,6 +368,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <741>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP0>;
> +			clocks = <&mcucfg CLK_MCU_MP0_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster0_opp>;
>  			dynamic-power-coefficient = <84>;
>  			#cooling-cells = <2>;
>  		};
> @@ -129,6 +383,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <1024>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP1>;
> +			clocks = <&mcucfg CLK_MCU_MP2_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster1_opp>;
>  			dynamic-power-coefficient = <211>;
>  			#cooling-cells = <2>;
>  		};
> @@ -140,6 +398,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <1024>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP1>;
> +			clocks = <&mcucfg CLK_MCU_MP2_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster1_opp>;
>  			dynamic-power-coefficient = <211>;
>  			#cooling-cells = <2>;
>  		};
> @@ -151,6 +413,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <1024>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP1>;
> +			clocks = <&mcucfg CLK_MCU_MP2_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster1_opp>;
>  			dynamic-power-coefficient = <211>;
>  			#cooling-cells = <2>;
>  		};
> @@ -162,6 +428,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <1024>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP1>;
> +			clocks = <&mcucfg CLK_MCU_MP2_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster1_opp>;
>  			dynamic-power-coefficient = <211>;
>  			#cooling-cells = <2>;
>  		};

Hello Matthias,

Could you give us some advice or ack for these patches of mt8183 dts?
The drviers and dt-binding of this series are picked by Viresh, and he
also wants to pick these three dts patches.

Thanks!

Viresh's branch: 
https://git.kernel.org/pub/scm/linux/kernel/git/vireshk/pm.git/log/?h=cpufreq/arm/linux-next

BRs,
Rex


_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

WARNING: multiple messages have this Message-ID (diff)
From: Rex-BC Chen <rex-bc.chen@mediatek.com>
To: <matthias.bgg@gmail.com>
Cc: <jia-wei.chang@mediatek.com>, <roger.lu@mediatek.com>,
	<hsinyi@google.com>, <khilman@baylibre.com>,
	<angelogioacchino.delregno@collabora.com>,
	<linux-pm@vger.kernel.org>, <devicetree@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-mediatek@lists.infradead.org>,
	 <Project_Global_Chrome_Upstream_Group@mediatek.com>,
	"Andrew-sh . Cheng" <andrew-sh.cheng@mediatek.com>,
	<rafael@kernel.org>, <viresh.kumar@linaro.org>,
	<robh+dt@kernel.org>, <krzk+dt@kernel.org>
Subject: Re: [PATCH v6 08/10] arm64: dts: mediatek: Add opp table and clock property for MT8183 cpufreq
Date: Thu, 12 May 2022 14:11:42 +0800	[thread overview]
Message-ID: <ea3b53f234d68f993bc7cd540b33832434cf931c.camel@mediatek.com> (raw)
In-Reply-To: <20220505115226.20130-9-rex-bc.chen@mediatek.com>

On Thu, 2022-05-05 at 19:52 +0800, Rex-BC Chen wrote:
> - Add cpufreq opp table.
> - Add MediaTek cci opp table.
> - Add property of opp table and clock fro cpufreq.
> 
> Signed-off-by: Andrew-sh.Cheng <andrew-sh.cheng@mediatek.com>
> Signed-off-by: Rex-BC Chen <rex-bc.chen@mediatek.com>
> Reviewed-by: AngeloGioacchino Del Regno <
> angelogioacchino.delregno@collabora.com>
> ---
>  arch/arm64/boot/dts/mediatek/mt8183-evb.dts |  32 +++
>  arch/arm64/boot/dts/mediatek/mt8183.dtsi    | 270
> ++++++++++++++++++++
>  2 files changed, 302 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
> b/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
> index f3fd3cca23e9..8953dbf84f3e 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8183-evb.dts
> @@ -412,6 +412,38 @@
>  
>  };
>  
> +&cpu0 {
> +	proc-supply = <&mt6358_vproc12_reg>;
> +};
> +
> +&cpu1 {
> +	proc-supply = <&mt6358_vproc12_reg>;
> +};
> +
> +&cpu2 {
> +	proc-supply = <&mt6358_vproc12_reg>;
> +};
> +
> +&cpu3 {
> +	proc-supply = <&mt6358_vproc12_reg>;
> +};
> +
> +&cpu4 {
> +	proc-supply = <&mt6358_vproc11_reg>;
> +};
> +
> +&cpu5 {
> +	proc-supply = <&mt6358_vproc11_reg>;
> +};
> +
> +&cpu6 {
> +	proc-supply = <&mt6358_vproc11_reg>;
> +};
> +
> +&cpu7 {
> +	proc-supply = <&mt6358_vproc11_reg>;
> +};
> +
>  &uart0 {
>  	status = "okay";
>  };
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> index 01e650251928..cecf96b628b7 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> @@ -42,6 +42,244 @@
>  		rdma1 = &rdma1;
>  	};
>  
> +	cluster0_opp: opp-table-cluster0 {
> +		compatible = "operating-points-v2";
> +		opp-shared;
> +		opp0-793000000 {
> +			opp-hz = /bits/ 64 <793000000>;
> +			opp-microvolt = <650000>;
> +			required-opps = <&opp2_00>;
> +		};
> +		opp0-910000000 {
> +			opp-hz = /bits/ 64 <910000000>;
> +			opp-microvolt = <687500>;
> +			required-opps = <&opp2_01>;
> +		};
> +		opp0-1014000000 {
> +			opp-hz = /bits/ 64 <1014000000>;
> +			opp-microvolt = <718750>;
> +			required-opps = <&opp2_02>;
> +		};
> +		opp0-1131000000 {
> +			opp-hz = /bits/ 64 <1131000000>;
> +			opp-microvolt = <756250>;
> +			required-opps = <&opp2_03>;
> +		};
> +		opp0-1248000000 {
> +			opp-hz = /bits/ 64 <1248000000>;
> +			opp-microvolt = <800000>;
> +			required-opps = <&opp2_04>;
> +		};
> +		opp0-1326000000 {
> +			opp-hz = /bits/ 64 <1326000000>;
> +			opp-microvolt = <818750>;
> +			required-opps = <&opp2_05>;
> +		};
> +		opp0-1417000000 {
> +			opp-hz = /bits/ 64 <1417000000>;
> +			opp-microvolt = <850000>;
> +			required-opps = <&opp2_06>;
> +		};
> +		opp0-1508000000 {
> +			opp-hz = /bits/ 64 <1508000000>;
> +			opp-microvolt = <868750>;
> +			required-opps = <&opp2_07>;
> +		};
> +		opp0-1586000000 {
> +			opp-hz = /bits/ 64 <1586000000>;
> +			opp-microvolt = <893750>;
> +			required-opps = <&opp2_08>;
> +		};
> +		opp0-1625000000 {
> +			opp-hz = /bits/ 64 <1625000000>;
> +			opp-microvolt = <906250>;
> +			required-opps = <&opp2_09>;
> +		};
> +		opp0-1677000000 {
> +			opp-hz = /bits/ 64 <1677000000>;
> +			opp-microvolt = <931250>;
> +			required-opps = <&opp2_10>;
> +		};
> +		opp0-1716000000 {
> +			opp-hz = /bits/ 64 <1716000000>;
> +			opp-microvolt = <943750>;
> +			required-opps = <&opp2_11>;
> +		};
> +		opp0-1781000000 {
> +			opp-hz = /bits/ 64 <1781000000>;
> +			opp-microvolt = <975000>;
> +			required-opps = <&opp2_12>;
> +		};
> +		opp0-1846000000 {
> +			opp-hz = /bits/ 64 <1846000000>;
> +			opp-microvolt = <1000000>;
> +			required-opps = <&opp2_13>;
> +		};
> +		opp0-1924000000 {
> +			opp-hz = /bits/ 64 <1924000000>;
> +			opp-microvolt = <1025000>;
> +			required-opps = <&opp2_14>;
> +		};
> +		opp0-1989000000 {
> +			opp-hz = /bits/ 64 <1989000000>;
> +			opp-microvolt = <1050000>;
> +			required-opps = <&opp2_15>;
> +		};	};
> +
> +	cluster1_opp: opp-table-cluster1 {
> +		compatible = "operating-points-v2";
> +		opp-shared;
> +		opp1-793000000 {
> +			opp-hz = /bits/ 64 <793000000>;
> +			opp-microvolt = <700000>;
> +			required-opps = <&opp2_00>;
> +		};
> +		opp1-910000000 {
> +			opp-hz = /bits/ 64 <910000000>;
> +			opp-microvolt = <725000>;
> +			required-opps = <&opp2_01>;
> +		};
> +		opp1-1014000000 {
> +			opp-hz = /bits/ 64 <1014000000>;
> +			opp-microvolt = <750000>;
> +			required-opps = <&opp2_02>;
> +		};
> +		opp1-1131000000 {
> +			opp-hz = /bits/ 64 <1131000000>;
> +			opp-microvolt = <775000>;
> +			required-opps = <&opp2_03>;
> +		};
> +		opp1-1248000000 {
> +			opp-hz = /bits/ 64 <1248000000>;
> +			opp-microvolt = <800000>;
> +			required-opps = <&opp2_04>;
> +		};
> +		opp1-1326000000 {
> +			opp-hz = /bits/ 64 <1326000000>;
> +			opp-microvolt = <825000>;
> +			required-opps = <&opp2_05>;
> +		};
> +		opp1-1417000000 {
> +			opp-hz = /bits/ 64 <1417000000>;
> +			opp-microvolt = <850000>;
> +			required-opps = <&opp2_06>;
> +		};
> +		opp1-1508000000 {
> +			opp-hz = /bits/ 64 <1508000000>;
> +			opp-microvolt = <875000>;
> +			required-opps = <&opp2_07>;
> +		};
> +		opp1-1586000000 {
> +			opp-hz = /bits/ 64 <1586000000>;
> +			opp-microvolt = <900000>;
> +			required-opps = <&opp2_08>;
> +		};
> +		opp1-1625000000 {
> +			opp-hz = /bits/ 64 <1625000000>;
> +			opp-microvolt = <912500>;
> +			required-opps = <&opp2_09>;
> +		};
> +		opp1-1677000000 {
> +			opp-hz = /bits/ 64 <1677000000>;
> +			opp-microvolt = <931250>;
> +			required-opps = <&opp2_10>;
> +		};
> +		opp1-1716000000 {
> +			opp-hz = /bits/ 64 <1716000000>;
> +			opp-microvolt = <950000>;
> +			required-opps = <&opp2_11>;
> +		};
> +		opp1-1781000000 {
> +			opp-hz = /bits/ 64 <1781000000>;
> +			opp-microvolt = <975000>;
> +			required-opps = <&opp2_12>;
> +		};
> +		opp1-1846000000 {
> +			opp-hz = /bits/ 64 <1846000000>;
> +			opp-microvolt = <1000000>;
> +			required-opps = <&opp2_13>;
> +		};
> +		opp1-1924000000 {
> +			opp-hz = /bits/ 64 <1924000000>;
> +			opp-microvolt = <1025000>;
> +			required-opps = <&opp2_14>;
> +		};
> +		opp1-1989000000 {
> +			opp-hz = /bits/ 64 <1989000000>;
> +			opp-microvolt = <1050000>;
> +			required-opps = <&opp2_15>;
> +		};
> +	};
> +
> +	cci_opp: opp-table-cci {
> +		compatible = "operating-points-v2";
> +		opp-shared;
> +		opp2_00: opp-273000000 {
> +			opp-hz = /bits/ 64 <273000000>;
> +			opp-microvolt = <650000>;
> +		};
> +		opp2_01: opp-338000000 {
> +			opp-hz = /bits/ 64 <338000000>;
> +			opp-microvolt = <687500>;
> +		};
> +		opp2_02: opp-403000000 {
> +			opp-hz = /bits/ 64 <403000000>;
> +			opp-microvolt = <718750>;
> +		};
> +		opp2_03: opp-463000000 {
> +			opp-hz = /bits/ 64 <463000000>;
> +			opp-microvolt = <756250>;
> +		};
> +		opp2_04: opp-546000000 {
> +			opp-hz = /bits/ 64 <546000000>;
> +			opp-microvolt = <800000>;
> +		};
> +		opp2_05: opp-624000000 {
> +			opp-hz = /bits/ 64 <624000000>;
> +			opp-microvolt = <818750>;
> +		};
> +		opp2_06: opp-689000000 {
> +			opp-hz = /bits/ 64 <689000000>;
> +			opp-microvolt = <850000>;
> +		};
> +		opp2_07: opp-767000000 {
> +			opp-hz = /bits/ 64 <767000000>;
> +			opp-microvolt = <868750>;
> +		};
> +		opp2_08: opp-845000000 {
> +			opp-hz = /bits/ 64 <845000000>;
> +			opp-microvolt = <893750>;
> +		};
> +		opp2_09: opp-871000000 {
> +			opp-hz = /bits/ 64 <871000000>;
> +			opp-microvolt = <906250>;
> +		};
> +		opp2_10: opp-923000000 {
> +			opp-hz = /bits/ 64 <923000000>;
> +			opp-microvolt = <931250>;
> +		};
> +		opp2_11: opp-962000000 {
> +			opp-hz = /bits/ 64 <962000000>;
> +			opp-microvolt = <943750>;
> +		};
> +		opp2_12: opp-1027000000 {
> +			opp-hz = /bits/ 64 <1027000000>;
> +			opp-microvolt = <975000>;
> +		};
> +		opp2_13: opp-1092000000 {
> +			opp-hz = /bits/ 64 <1092000000>;
> +			opp-microvolt = <1000000>;
> +		};
> +		opp2_14: opp-1144000000 {
> +			opp-hz = /bits/ 64 <1144000000>;
> +			opp-microvolt = <1025000>;
> +		};
> +		opp2_15: opp-1196000000 {
> +			opp-hz = /bits/ 64 <1196000000>;
> +			opp-microvolt = <1050000>;
> +		};
> +	};
> +
>  	cpus {
>  		#address-cells = <1>;
>  		#size-cells = <0>;
> @@ -85,6 +323,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <741>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP0>;
> +			clocks = <&mcucfg CLK_MCU_MP0_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster0_opp>;
>  			dynamic-power-coefficient = <84>;
>  			#cooling-cells = <2>;
>  		};
> @@ -96,6 +338,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <741>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP0>;
> +			clocks = <&mcucfg CLK_MCU_MP0_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster0_opp>;
>  			dynamic-power-coefficient = <84>;
>  			#cooling-cells = <2>;
>  		};
> @@ -107,6 +353,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <741>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP0>;
> +			clocks = <&mcucfg CLK_MCU_MP0_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster0_opp>;
>  			dynamic-power-coefficient = <84>;
>  			#cooling-cells = <2>;
>  		};
> @@ -118,6 +368,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <741>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP0>;
> +			clocks = <&mcucfg CLK_MCU_MP0_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster0_opp>;
>  			dynamic-power-coefficient = <84>;
>  			#cooling-cells = <2>;
>  		};
> @@ -129,6 +383,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <1024>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP1>;
> +			clocks = <&mcucfg CLK_MCU_MP2_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster1_opp>;
>  			dynamic-power-coefficient = <211>;
>  			#cooling-cells = <2>;
>  		};
> @@ -140,6 +398,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <1024>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP1>;
> +			clocks = <&mcucfg CLK_MCU_MP2_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster1_opp>;
>  			dynamic-power-coefficient = <211>;
>  			#cooling-cells = <2>;
>  		};
> @@ -151,6 +413,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <1024>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP1>;
> +			clocks = <&mcucfg CLK_MCU_MP2_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster1_opp>;
>  			dynamic-power-coefficient = <211>;
>  			#cooling-cells = <2>;
>  		};
> @@ -162,6 +428,10 @@
>  			enable-method = "psci";
>  			capacity-dmips-mhz = <1024>;
>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP1>;
> +			clocks = <&mcucfg CLK_MCU_MP2_SEL>,
> +				 <&topckgen CLK_TOP_ARMPLL_DIV_PLL1>;
> +			clock-names = "cpu", "intermediate";
> +			operating-points-v2 = <&cluster1_opp>;
>  			dynamic-power-coefficient = <211>;
>  			#cooling-cells = <2>;
>  		};

Hello Matthias,

Could you give us some advice or ack for these patches of mt8183 dts?
The drviers and dt-binding of this series are picked by Viresh, and he
also wants to pick these three dts patches.

Thanks!

Viresh's branch: 
https://git.kernel.org/pub/scm/linux/kernel/git/vireshk/pm.git/log/?h=cpufreq/arm/linux-next

BRs,
Rex


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2022-05-12  6:11 UTC|newest]

Thread overview: 96+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-05 11:52 [PATCH v6 00/10] cpufreq: mediatek: Cleanup and support MT8183 and MT8186 Rex-BC Chen
2022-05-05 11:52 ` Rex-BC Chen
2022-05-05 11:52 ` Rex-BC Chen
2022-05-05 11:52 ` [PATCH v6 01/10] dt-bindings: cpufreq: mediatek: Add MediaTek CCI property Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-11 14:37   ` Rob Herring
2022-05-11 14:37     ` Rob Herring
2022-05-11 14:37     ` Rob Herring
2022-05-05 11:52 ` [PATCH v6 02/10] cpufreq: mediatek: Add platform_device_unregister when driver exit Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-05 15:04   ` AngeloGioacchino Del Regno
2022-05-05 15:04     ` AngeloGioacchino Del Regno
2022-05-05 15:04     ` AngeloGioacchino Del Regno
2022-05-06  1:49     ` Rex-BC Chen
2022-05-06  1:49       ` Rex-BC Chen
2022-05-06  1:49       ` Rex-BC Chen
2022-05-06  4:09   ` Viresh Kumar
2022-05-06  4:09     ` Viresh Kumar
2022-05-06  4:09     ` Viresh Kumar
2022-05-05 11:52 ` [PATCH v6 03/10] cpufreq: mediatek: Move voltage limits to platform data Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-06  4:10   ` Viresh Kumar
2022-05-06  4:10     ` Viresh Kumar
2022-05-06  4:10     ` Viresh Kumar
2022-05-05 11:52 ` [PATCH v6 04/10] cpufreq: mediatek: Refine mtk_cpufreq_voltage_tracking() Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-05 11:52 ` [PATCH v6 05/10] cpufreq: mediatek: Add opp notification support Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-06  1:46   ` Rex-BC Chen
2022-05-06  1:46     ` Rex-BC Chen
2022-05-06  1:46     ` Rex-BC Chen
2022-05-06  3:22     ` Chen-Yu Tsai
2022-05-06  3:22       ` Chen-Yu Tsai
2022-05-06  3:22       ` Chen-Yu Tsai
2022-05-06  3:32       ` Viresh Kumar
2022-05-06  3:32         ` Viresh Kumar
2022-05-06  3:32         ` Viresh Kumar
2022-05-06  4:15   ` Viresh Kumar
2022-05-06  4:15     ` Viresh Kumar
2022-05-06  4:15     ` Viresh Kumar
2022-05-06  6:21     ` Rex-BC Chen
2022-05-06  6:21       ` Rex-BC Chen
2022-05-06  6:21       ` Rex-BC Chen
2022-05-05 11:52 ` [PATCH v6 06/10] cpufreq: mediatek: Link CCI device to CPU Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-05 11:52 ` [PATCH v6 07/10] cpufreq: mediatek: Add support for MT8186 Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-05 11:52 ` [PATCH v6 08/10] arm64: dts: mediatek: Add opp table and clock property for MT8183 cpufreq Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-12  6:11   ` Rex-BC Chen [this message]
2022-05-12  6:11     ` Rex-BC Chen
2022-05-12  6:11     ` Rex-BC Chen
2022-05-05 11:52 ` [PATCH v6 09/10] arm64: dts: mediatek: Add MediaTek CCI node for MT8183 Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-16  5:35   ` Chen-Yu Tsai
2022-05-16  5:35     ` Chen-Yu Tsai
2022-05-16  5:35     ` Chen-Yu Tsai
2022-05-16 10:41     ` Rex-BC Chen
2022-05-16 10:41       ` Rex-BC Chen
2022-05-16 10:41       ` Rex-BC Chen
2022-05-05 11:52 ` [PATCH v6 10/10] arm64: dts: mediatek: Add mediatek,cci property for MT8183 cpufreq Rex-BC Chen
2022-05-05 11:52   ` [PATCH v6 10/10] arm64: dts: mediatek: Add mediatek, cci " Rex-BC Chen
2022-05-05 11:52   ` Rex-BC Chen
2022-05-06  4:20 ` [PATCH v6 00/10] cpufreq: mediatek: Cleanup and support MT8183 and MT8186 Viresh Kumar
2022-05-06  4:20   ` Viresh Kumar
2022-05-06  4:20   ` Viresh Kumar
2022-05-06  6:32   ` Rex-BC Chen
2022-05-06  6:32     ` Rex-BC Chen
2022-05-06  6:32     ` Rex-BC Chen
2022-05-12  5:27     ` Viresh Kumar
2022-05-12  5:27       ` Viresh Kumar
2022-05-12  5:27       ` Viresh Kumar
2022-05-12  5:33       ` Rex-BC Chen
2022-05-12  5:33         ` Rex-BC Chen
2022-05-12  5:33         ` Rex-BC Chen
2022-05-12  5:48         ` Viresh Kumar
2022-05-12  5:48           ` Viresh Kumar
2022-05-12  5:48           ` Viresh Kumar
2022-05-12  6:05           ` Rex-BC Chen
2022-05-12  6:05             ` Rex-BC Chen
2022-05-12  6:05             ` Rex-BC Chen
2022-05-17  9:57             ` Matthias Brugger
2022-05-17  9:57               ` Matthias Brugger
2022-05-17  9:57               ` Matthias Brugger
2022-05-17 10:02               ` Rex-BC Chen
2022-05-17 10:02                 ` Rex-BC Chen
2022-05-17 10:02                 ` Rex-BC Chen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ea3b53f234d68f993bc7cd540b33832434cf931c.camel@mediatek.com \
    --to=rex-bc.chen@mediatek.com \
    --cc=Project_Global_Chrome_Upstream_Group@mediatek.com \
    --cc=andrew-sh.cheng@mediatek.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=devicetree@vger.kernel.org \
    --cc=hsinyi@google.com \
    --cc=jia-wei.chang@mediatek.com \
    --cc=khilman@baylibre.com \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=matthias.bgg@gmail.com \
    --cc=rafael@kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=roger.lu@mediatek.com \
    --cc=viresh.kumar@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.