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