Re: [PATCH v2 44/48] ARM: tegra: Add OPP tables and power domains to Tegra30 device-tree

On 17-12-20, 21:06, Dmitry Osipenko wrote:

diff --git a/arch/arm/boot/dts/tegra30-peripherals-opp.dtsi b/arch/arm/boot/dts/tegra30-peripherals-opp.dtsi index cbe84d25e726..983db1a06682 100644 --- a/arch/arm/boot/dts/tegra30-peripherals-opp.dtsi +++ b/arch/arm/boot/dts/tegra30-peripherals-opp.dtsi @@ -1,6 +1,56 @@ // SPDX-License-Identifier: GPL-2.0

/ {

• core_opp_table: core-power-domain-opp-table {

• compatible = "operating-points-v2";
  

• opp-shared;
  

• core_opp_950: opp@950000 {
  

• 	opp-microvolt = <950000 950000 1350000>;
  

• 	opp-level = <950000>;
  

Perhaps you don't need to exactly copy the voltage value into the level field. The level field can just be kept to 0, 1,2, 3, etc..

• };
  

• core_opp_1000: opp@1000000 {
  

• 	opp-microvolt = <1000000 1000000 1350000>;
  

• 	opp-level = <1000000>;
  

• };
  

• core_opp_1050: opp@1050000 {
  

• 	opp-microvolt = <1050000 1050000 1350000>;
  

• 	opp-level = <1050000>;
  

• };
  

• core_opp_1100: opp@1100000 {
  

• 	opp-microvolt = <1100000 1100000 1350000>;
  

• 	opp-level = <1100000>;
  

• };
  

• core_opp_1150: opp@1150000 {
  

• 	opp-microvolt = <1150000 1150000 1350000>;
  

• 	opp-level = <1150000>;
  

• };
  

• core_opp_1200: opp@1200000 {
  

• 	opp-microvolt = <1200000 1200000 1350000>;
  

• 	opp-level = <1200000>;
  

• };
  

• core_opp_1250: opp@1250000 {
  

• 	opp-microvolt = <1250000 1250000 1350000>;
  

• 	opp-level = <1250000>;
  

• };
  

• core_opp_1300: opp@1300000 {
  

• 	opp-microvolt = <1300000 1300000 1350000>;
  

• 	opp-level = <1300000>;
  

• };
  

• core_opp_1350: opp@1350000 {
  

• 	opp-microvolt = <1350000 1350000 1350000>;
  

• 	opp-level = <1350000>;
  

• };
  

• };