Re: [PATCH v2 1/8] thermal: Add Remote Proc cooling driver

From: Krzysztof Kozlowski

Date: Sun Feb 08 2026 - 05:09:02 EST


On 30/01/2026 07:39, Gaurav Kohli wrote:
>>> +
>>> +MODULE_LICENSE("GPL");
>>> +MODULE_DESCRIPTION("Remote Processor Cooling Device");
>> I do not see any driver here, just bunch of exported functions. I do not
>> see point in this abstraction/wrapping layer.
>>
>> Another abstraction layer, NAK.
>
>
> Thanks Krzysztof for review.
>
> We need this abstraction layer to provide a common interface that
> multiple vendors can rely on for

Why do you keep inserting blank lines inside sentences?

>
> their remote processor based cooling communication. If we use
> QMI-cooling driver only, then solution will

So you just ignored my comment and repeat the same mantra. You don't
even use any arguments, just push own position. This will get you nowhere.

Read the comment again.

NAK



Best regards,
Krzysztof