[PATCH v2 11/12] arm64: dts: rockchip: Add rk3576 SoC base DT

[Alexey Charkov]

> On 25 Aug 2024, at 18:10, Chukun Pan <amadeus at jmu.edu.cn> wrote:
> 
> Hi,
> 
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/rockchip/rk3576.dtsi
>> ...
>> +        opp-1416000000 {
>> +            opp-hz = /bits/ 64 <1416000000>;
>> +            opp-microvolt = <725000 725000 950000>;
>> +            opp-microvolt-L1 = <712500 712500 950000>;
>> +            opp-microvolt-L2 = <700000 700000 950000>;
>> +            opp-microvolt-L3 = <700000 700000 950000>;
>> +            opp-microvolt-L4 = <700000 700000 950000>;
>> +            opp-microvolt-L5 = <700000 700000 950000>;
>> +            clock-latency-ns = <40000>;
>> +        };
>> ...
> 
> I'm curious if these frequencies work properly. On the bsp kernel,
> 'opp-microvolt-L<name>' is used by the PVTM driver, I don't know
> if it works on the upstream kernel. Sorry but have you tested it
> with mhz (https://github.com/wtarreau/mhz)?

These are for voltage-frequency binning, and they at least require reading and interpreting ‘leakage’ values from efuse, which the mainline generic cpufreq-dt driver does not. So maybe drop the (.+)-L[0-9]+ values until we have something in the mainline tree to test them with?

Best regards,
Alexey
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20240825/db8f6104/attachment-0001.htm>