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.