16 Jun
2025
16 Jun
'25
8:18 a.m.
On Sun, 15 Jun 2025, "Nautiyal, Ankit K" ankit.k.nautiyal@intel.com wrote:
On 6/13/2025 3:06 PM, Jani Nikula wrote:
On Fri, 13 Jun 2025, Ankit Nautiyalankit.k.nautiyal@intel.com wrote:
*ana_cp_int = max(1, min(ana_cp_int_temp, 127));
Unrelated to this patch, but this should be:
*ana_cp_int = clamp(ana_cp_int_temp, 1, 127);
There's a similar issue with ana_cp_prop also in the file.
Agreed. Should there be a separate patch for this?
Yes. That's why I emphasized "unrelated to this patch". ;)
BR, Jani.
--
Jani Nikula, Intel