Re: [PATCH v4 0/5] Add driver for EC found on Qualcomm reference devices

From: Anvesh Jain P

Date: Mon Mar 16 2026 - 06:23:21 EST




On 3/14/2026 9:39 AM, Bryan O'Donoghue wrote:
> On 13/03/2026 17:17, Dmitry Baryshkov wrote:
>>> Fan Count == 0 || Thermistor == 0 => error and probe() fail.
>> WHy? EC is still there. QUite useless though.
>
> You want to register when zero fans are found, zero thermistors... ?
>
> That's a bit fruity.
>

We need to keep the driver loaded even if zero fans are reported, as it
is also responsible for sending suspend/resume notifications to the EC.
Failing the probe would prevent these PM notifications from reaching the EC.

Instead, I will add a dev_warn message suggesting a firmware upgrade
when zero fans are found. Let me know your thoughts.

> ---
> bod

--
Best Regards,
Anvesh