Bad commit backported on the v5.15.y branch ?

From: Joel GUITTET
Date: Fri Oct 11 2024 - 04:48:56 EST


Hello,

I faced some issue related to scaling frequency on ZynqMP device using v5.15.167 kernel.
As an exemple setting the scaling frequency below show it's not properly set:

cat /sys/devices/system/cpu/cpufreq/policy0/scaling_available_frequencies
299999 399999 599999 1199999

echo 399999 > /sys/devices/system/cpu/cpufreq/policy0/scaling_setspeed

cat /sys/devices/system/cpu/cpufreq/policy0/scaling_cur_freq
399999

cat /sys/devices/system/cpu/cpufreq/policy0/cpuinfo_cur_freq
299999 ====> Should be 399999

After analysis of this issue with the help of Xilinx, it appears that a commit was backported on the 5.15.y branch, but probably it should not, or not as is. The commit is 9117fc44fd3a9538261e530ba5a022dfc9519620 modifying drivers/clk/zynqmp/divider.c.

Is anybody reading this message able to answer why it was backported ? The information I have until now is that it is intended recent kernel version. Dependencies for this modification is currently under clarification with Xilinx (maybe another commit to backport).

By the way, reverting this commit fix the issue shown above.

Thanks for the feedback,
Joel