[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <a830f913-e5ea-4754-912f-612e178a7282@kernel.org>
Date: Sun, 8 Feb 2026 11:08:42 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Gaurav Kohli <gaurav.kohli@....qualcomm.com>
Cc: andersson@...nel.org, mathieu.poirier@...aro.org, robh@...nel.org,
krzk+dt@...nel.org, conor+dt@...nel.org, rui.zhang@...el.com,
lukasz.luba@....com, konradybcio@...nel.org, mani@...nel.org,
casey.connolly@...aro.org, amit.kucheria@....qualcomm.com,
linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-pm@...r.kernel.org,
manaf.pallikunhi@....qualcomm.com
Subject: Re: [PATCH v2 1/8] thermal: Add Remote Proc cooling driver
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
Powered by blists - more mailing lists