[PATCH v2 8/8] arm64: dts: allwinner: h616: enable DVFS for all boards
Amit Singh Tomar
amitsinght at marvell.com
Tue Mar 19 12:27:22 PDT 2024
Hi,
>> Okay, it would be interesting to see results of some benchmark here.
>
> But why? This is not a performance optimisation, it's adding a missing
> feature, because the CPU was locked to 1 GHz before, for safety
> reasons, due to missing thermal and DVFS capability. Now it's able to
> run at up to 1.5 GHz, as specified.
I completely understand, it's not intended for performance optimization.
> If you are upset about the bold claim, I can just remove it from the
> commit message, it was just a heads up that we were leaving a lot of
> performance on the table at the moment.
I was merely curious about it, not upset. It could certainly enhance
aspects such as memory bandwidth, but perhaps not to the extent as
suggested. Therefore, rephrasing the commit message should suffice.
Thanks
-Amit
More information about the linux-arm-kernel
mailing list