Re: [PATCH v4 3/3] pwm: argon-fan-hat: Add Argon40 Fan HAT support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 6/23/25 11:11 AM, Uwe Kleine-König wrote:
Hello,

Hello Uwe,

when I replied to v3 this v4 was already on the list which I missed. My
concern applies here, too, though.

On Sat, Jun 21, 2025 at 07:19:56PM +0200, Marek Vasut wrote:
+static void argon_fan_hat_i2c_shutdown(struct i2c_client *i2c)
+{
+	argon_fan_hat_write(i2c, 100);
+}

If you drop this, I'm willing to apply.

Dropping this would make the hardware which uses this device more susceptible to thermal damage, e.g. in case it gets stuck during reboot and does not boot Linux afterward. I don't want to risk such thermal damage.

--
Best regards,
Marek Vasut




[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux