On Tue, 2021-07-27 at 13:16 +0200, David Müller (ELSOFT AG) wrote: > Iwona Winiarska wrote: > > > +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