Re: [PATCH v2] arm64: dts: renesas: r8a779g3: Add Argon40 fan HAT DTO to Retronix R-Car V4H Sparrow Hawk

[Wolfram Sang <wsa+renesas@sang-engineering.com>]

Hi Marek,

On Fri, Sep 05, 2025 at 04:00:57AM +0200, Marek Vasut wrote:
> Add DT overlay to bind Argon40 fan HAT, on Retronix R-Car V4H
> Sparrow Hawk board. Fan RPM control and full RPM on reboot has
> been tested.
> 
> Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>

Basically works for me as well, so:

Tested-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

> + * r8a779g3-sparrow-hawk$ grep -H . /sys/class/hwmon/hwmon?/name
> + * /sys/class/hwmon/hwmon0/name:sensor1_thermal
> + * /sys/class/hwmon/hwmon1/name:sensor2_thermal
> + * /sys/class/hwmon/hwmon2/name:sensor3_thermal
> + * /sys/class/hwmon/hwmon3/name:sensor4_thermal
> + * /sys/class/hwmon/hwmon4/name:pwmfan
> + *                       ^      ^^^^^^

For me, the output looks different, though:

/sys/class/hwmon/hwmon0/name:pwmfan
/sys/class/hwmon/hwmon1/name:sensor1_thermal
/sys/class/hwmon/hwmon2/name:sensor2_thermal
/sys/class/hwmon/hwmon3/name:sensor3_thermal
/sys/class/hwmon/hwmon4/name:sensor4_thermal
/sys/class/hwmon/hwmon5/name:pwmfan

hwmon0 is the Argon, hwmon5 the (unpopulated) on board connector.

I hope the naming is stable, but in any case, the docs need to be
reworked a little, I guess?

Thanks for your work!

   Wolfram