hwmon: (k8temp) Fix wrong sensor selection for AMD K8 RevF/RevG CPUs
authorAndreas Herrmann <andreas.herrmann3@amd.com>
Thu, 15 Jan 2009 21:27:47 +0000 (22:27 +0100)
committerJean Delvare <khali@linux-fr.org>
Thu, 15 Jan 2009 21:27:47 +0000 (22:27 +0100)
Meaning of ThermSenseCoreSel bit was inverted beginning with K8 RevF.
That means with current driver temp1/temp2 belong to core 1 and
temp3/temp4 belong to core 0 on a K8 RevF/RevG CPU.

This patch ensures that temp1/temp2 always belong to core 0 and
temp3/temp4 to core 1 for all K8 revisions.

Cc: Rudolf Marek <r.marek@assembler.cz>
Signed-off-by: Andreas Herrmann <andreas.herrmann3@amd.com>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
drivers/hwmon/k8temp.c

index ca56f2e26fd16e692fc963215a66911e5709a07c..a6381bc9189cdf127316c3900da1918708df5a1e 100644 (file)
@@ -48,6 +48,7 @@ struct k8temp_data {
        /* registers values */
        u8 sensorsp;            /* sensor presence bits - SEL_CORE & SEL_PLACE */
        u32 temp[2][2];         /* core, place */
+       u8 swap_core_select;    /* meaning of SEL_CORE is inverted */
 };
 
 static struct k8temp_data *k8temp_update_device(struct device *dev)
@@ -117,6 +118,9 @@ static ssize_t show_temp(struct device *dev,
        int place = attr->index;
        struct k8temp_data *data = k8temp_update_device(dev);
 
+       if (data->swap_core_select)
+               core = core ? 0 : 1;
+
        return sprintf(buf, "%d\n",
                       TEMP_FROM_REG(data->temp[core][place]));
 }
@@ -162,7 +166,12 @@ static int __devinit k8temp_probe(struct pci_dev *pdev,
                        goto exit_free;
                }
 
+               /*
+                * AMD NPT family 0fh, i.e. RevF and RevG:
+                * meaning of SEL_CORE bit is inverted
+                */
                if (model >= 0x40) {
+                       data->swap_core_select = 1;
                        dev_warn(&pdev->dev, "Temperature readouts might be "
                                 "wrong - check erratum #141\n");
                }