Messages in this thread | | | Date | Mon, 31 Aug 2020 16:34:08 +0530 | From | Viresh Kumar <> | Subject | Re: [PATCH 3/3] ARM: tegra: Pass multiple versions in opp-supported-hw property |
| |
On 31-08-20, 12:54, Dmitry Osipenko wrote: > It's not clear to me how it could be applicable to the Tegra CPU OPP > because Tegra depends on a combination of SPEEDO + PROCESS versions. > > It's not like all voltages are the same for all OPPs that have the same > PROCESS ID, otherwise it indeed would be nice to have > "opp-microvolt-process0", but unfortunately this variant is not suitable > for Tegra because some freqs have different voltages using the same > PROCESS ID and the same applies to the SPEEDO ID.
How exactly do you know what voltage belongs to a particular OPP ?
opp@216000000 { clock-latency-ns = <400000>; opp-supported-hw = <0x0F 0x0003>; opp-hz = /bits/ 64 <216000000>; opp-microvolt-fast-process0 = <750000 750000 1125000>; opp-microvolt-slow-process0 = <750000 850000 1125000>;
};
opp@312000000 { clock-latency-ns = <400000>; opp-supported-hw = <0x0F 0x0003>; opp-hz = /bits/ 64 <312000000>; opp-microvolt-fast-process0 = <750000 750000 1125000>; opp-microvolt-slow-process0 = <750000 850000 1125000>; };
You can make any combinations of such names that come from speedo, process, or something else. If you can get this done as a fixed formula then it is workable.
-- viresh
| |