From: Guenter Roeck <linux@roeck-us.net>
Date: Tue, 9 Jul 2024 22:41:07 +0000 (-0700)
Subject: hwmon: (g762) Initialize fans after configuring clock
X-Git-Tag: v6.11-rc1~212^2~4
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=f4a180247e06c69e7d8fac4eb14d172c3a5d43a3;p=thirdparty%2Fkernel%2Flinux.git

hwmon: (g762) Initialize fans after configuring clock

Adding support for G761 included adding support for an internal clock.
Enabling the internal clock requires setting a bit in the FAN_CMD2
register. This is implemented in g762_fan_init(). However, g762_fan_init()
is called before clock support is selected, and the flag indicating that
the internal clock should be used is not yet set.

Initialize the clock before initializing the fan to solve the problem.
While at it, also add "g7621" to the i2c_device_id array.

Cc: Christian Marangi <ansuelsmth@gmail.com>
Fixes: 6ce402327a6f ("hwmon: g672: add support for g761")
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---

diff --git a/drivers/hwmon/g762.c b/drivers/hwmon/g762.c
index a00cf3245eec2..4fa3aa1271daa 100644
--- a/drivers/hwmon/g762.c
+++ b/drivers/hwmon/g762.c
@@ -44,6 +44,7 @@
 #define DRVNAME "g762"
 
 static const struct i2c_device_id g762_id[] = {
+	{ "g761" },
 	{ "g762" },
 	{ "g763" },
 	{ }
@@ -1083,15 +1084,16 @@ static int g762_probe(struct i2c_client *client)
 	data->client = client;
 	mutex_init(&data->update_lock);
 
-	/* Enable fan failure detection and fan out of control protection */
-	ret = g762_fan_init(dev);
+	/* Get configuration via DT ... */
+	ret = g762_of_clock_enable(client);
 	if (ret)
 		return ret;
 
-	/* Get configuration via DT ... */
-	ret = g762_of_clock_enable(client);
+	/* Enable fan failure detection and fan out of control protection */
+	ret = g762_fan_init(dev);
 	if (ret)
 		return ret;
+
 	ret = g762_of_prop_import(client);
 	if (ret)
 		return ret;