Re: [PATCH v5 13/13] arm64: dts: freescale: sl28: enable fan support
From: Linus Walleij
Date: Wed Jul 08 2020 - 03:40:01 EST
Hi Michael,
just a drive-by-comment:
On Mon, Jul 6, 2020 at 7:57 PM Michael Walle <michael@xxxxxxxx> wrote:
> Add a pwm-fan mapped to the PWM channel 0 which is connected to the
> fan connector of the carrier.
>
> Signed-off-by: Michael Walle <michael@xxxxxxxx>
If you have a cooling device like this, do you also have a temperature
sensor? In that case it makes sense to add a thermal zone and a
policy, such as I did for a device in
6e97f0aaca4ca778905dd1dc667cbf379f4cae15
Yours,
Linus Walleij