Re: [PATCH 10/14] peci: Add peci-cpu driver

From: David Müller (ELSOFT AG)
Date: Tue Jul 27 2021 - 07:26:06 EST


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.