[PATCH 3/4] hwmon: (msi_mystic_light) Make sensor read function board specific

[Guenter Roeck <linux@roeck-us.net>]

Mystic Light controllers on different generations of MSI motherboards
use different HID communication mechanisms to retrieve voltage sensor
data. While 800-series motherboards expose sensor readings directly via
a synchronous HID_REQ_GET_REPORT request on feature report 0xf0, older
500- and 600-series motherboards require requesting sensor data via an
output report and receiving the response asynchronously as an input
event.

Add a read_report() callback to struct msi_ml_board, rename the existing
implementation to msi_ml_read_report_7e61() for MS-7E61, and turn
msi_ml_read_report() into a dispatch wrapper so probe and hwmon read
paths remain board-independent.

Cc: Tomasz Bojanowski <tomasz@bojanowski.me>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---
 drivers/hwmon/msi_mystic_light.c | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/drivers/hwmon/msi_mystic_light.c b/drivers/hwmon/msi_mystic_light.c
index 4b4e07f15aa8..098b4ecd0596 100644
--- a/drivers/hwmon/msi_mystic_light.c
+++ b/drivers/hwmon/msi_mystic_light.c
@@ -59,6 +59,7 @@ struct msi_ml_board {
 	const char *const *in_label;
 	/* little endian u16 in mV, offset at most MSI_ML_SENSOR_REPORT_SIZE - 2 */
 	const u8 *in_offset;
+	int (*read_report)(struct msi_ml_data *priv);
 };
 
 /*
@@ -67,7 +68,7 @@ struct msi_ml_board {
  * hwmon core serializes. No additional locking is needed for the buffer
  * and the cached values.
  */
-static int msi_ml_read_report(struct msi_ml_data *priv)
+static int msi_ml_read_report_7e61(struct msi_ml_data *priv)
 {
 	const struct msi_ml_board *board = priv->board;
 	u8 *buf = priv->buf;
@@ -98,6 +99,11 @@ static int msi_ml_read_report(struct msi_ml_data *priv)
 	return 0;
 }
 
+static int msi_ml_read_report(struct msi_ml_data *priv)
+{
+	return priv->board->read_report(priv);
+}
+
 static umode_t msi_ml_is_visible(const void *data,
 				 enum hwmon_sensor_types type, u32 attr,
 				 int channel)
@@ -265,6 +271,7 @@ static const struct msi_ml_board msi_ml_7e61 = {
 	.num_in = ARRAY_SIZE(msi_ml_7e61_in_label),
 	.in_label = msi_ml_7e61_in_label,
 	.in_offset = msi_ml_7e61_in_offset,
+	.read_report = msi_ml_read_report_7e61,
 };
 
 static const struct dmi_system_id msi_ml_dmi_table[] __initconst = {
-- 
2.45.2