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>;
> + };
> + };
--
viresh
