On Tue, 2021-07-27 at 13:16 +0200, David M?ller (ELSOFT AG) wrote:

Iwona Winiarska wrote:

quoted

+static const struct peci_device_id peci_cpu_device_ids[] = {
+???????{ /* Haswell Xeon */
+???????????????.family?= 6,
+???????????????.model??= INTEL_FAM6_HASWELL_X,
+???????????????.data???= "hsx",
+???????},
+???????{ /* Broadwell Xeon */
+???????????????.family?= 6,
+???????????????.model??= INTEL_FAM6_BROADWELL_X,
+???????????????.data???= "bdx",
+???????},
+???????{ /* Broadwell Xeon D */
+???????????????.family?= 6,
+???????????????.model??= INTEL_FAM6_BROADWELL_D,
+???????????????.data???= "skxd",

I think this should read "bdxd" as "skxd" does not exist in the
cputemp/dimmtemp drivers.

It should be "bdxd" - I'll fix it in v2.

Thank you
-Iwona