hwmon: (aquacomputer_d5next) Rename AQC_TEMP_SENSOR_DISCONNECTED
authorAleksa Savic <savicaleksa83@gmail.com>
Sat, 20 May 2023 09:54:46 +0000 (11:54 +0200)
committerGuenter Roeck <linux@roeck-us.net>
Thu, 8 Jun 2023 13:41:18 +0000 (06:41 -0700)
Rename the macro in question to AQC_SENSOR_NA because
more than just temperature sensors use this value to
indicate that they don't have a reading. Implemented by
Noah Bergbauer [1].

[1] https://github.com/aleksamagicka/aquacomputer_d5next-hwmon/pull/41

Originally-from: Noah Bergbauer <main@ehvag.de>
Signed-off-by: Aleksa Savic <savicaleksa83@gmail.com>
Link: https://lore.kernel.org/r/20230520095447.509287-2-savicaleksa83@gmail.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/aquacomputer_d5next.c

index a4fcd4e..834d011 100644 (file)
@@ -93,7 +93,7 @@ static u8 aquaero_secondary_ctrl_report[] = {
 #define AQC_FIRMWARE_VERSION           0xD
 
 #define AQC_SENSOR_SIZE                        0x02
-#define AQC_TEMP_SENSOR_DISCONNECTED   0x7FFF
+#define AQC_SENSOR_NA                  0x7FFF
 #define AQC_FAN_PERCENT_OFFSET         0x00
 #define AQC_FAN_VOLTAGE_OFFSET         0x02
 #define AQC_FAN_CURRENT_OFFSET         0x04
@@ -1224,7 +1224,7 @@ static int aqc_raw_event(struct hid_device *hdev, struct hid_report *report, u8
                sensor_value = get_unaligned_be16(data +
                                                  priv->temp_sensor_start_offset +
                                                  i * AQC_SENSOR_SIZE);
-               if (sensor_value == AQC_TEMP_SENSOR_DISCONNECTED)
+               if (sensor_value == AQC_SENSOR_NA)
                        priv->temp_input[i] = -ENODATA;
                else
                        priv->temp_input[i] = sensor_value * 10;
@@ -1235,7 +1235,7 @@ static int aqc_raw_event(struct hid_device *hdev, struct hid_report *report, u8
                sensor_value = get_unaligned_be16(data +
                                                  priv->virtual_temp_sensor_start_offset +
                                                  j * AQC_SENSOR_SIZE);
-               if (sensor_value == AQC_TEMP_SENSOR_DISCONNECTED)
+               if (sensor_value == AQC_SENSOR_NA)
                        priv->temp_input[i] = -ENODATA;
                else
                        priv->temp_input[i] = sensor_value * 10;
@@ -1277,7 +1277,7 @@ static int aqc_raw_event(struct hid_device *hdev, struct hid_report *report, u8
                        sensor_value = get_unaligned_be16(data +
                                        priv->calc_virt_temp_sensor_start_offset +
                                        j * AQC_SENSOR_SIZE);
-                       if (sensor_value == AQC_TEMP_SENSOR_DISCONNECTED)
+                       if (sensor_value == AQC_SENSOR_NA)
                                priv->temp_input[i] = -ENODATA;
                        else
                                priv->temp_input[i] = sensor_value * 10;