On 8/25/26 2:25 PM, Gaurav Kohli wrote:
>
>
> On 8/24/2026 7:49 PM, Konrad Dybcio wrote:
>> On 8/9/26 2:29 PM, Gaurav Kohli wrote:
>>> Unlike the CPU, the CDSP does not throttle its speed automatically
>>> when it reaches high temperatures in monaco.
>>
>> [...]
>>
>>> nsp-0-0-0-thermal {
>>> + polling-delay-passive = <200>;
>>
>> ..and here it's 200 - I really think there should be a single value
>> that makes sense, this seems no less than random
>
> Thanks for the review, Konrad.
>
> All targets added in this series use polling-delay-passive = <200> except
> Lemans, where the DTS already has polling-delay-passive = <10>. This series
> only adds the cooling maps there.
>
> As per internal profiling, 200 ms was chosen for remoteproc/QMI-based
> cooling. I can send a separate follow-up patch to normalize Lemans to 200 ms
> as well.
I'm fine with a profiled value that's different between platforms if
it's really beneficial, but it should be made explicit that it's
right. If you believe the current settings can be improved, I'm all
for it.
Konrad
