From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.0 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A513EC2D0C9 for ; Fri, 13 Dec 2019 06:32:17 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 7B22522527 for ; Fri, 13 Dec 2019 06:32:17 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="evtfYWi+" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725770AbfLMGcQ (ORCPT ); Fri, 13 Dec 2019 01:32:16 -0500 Received: from mail-pl1-f196.google.com ([209.85.214.196]:36581 "EHLO mail-pl1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725468AbfLMGcQ (ORCPT ); Fri, 13 Dec 2019 01:32:16 -0500 Received: by mail-pl1-f196.google.com with SMTP id d15so821222pll.3; Thu, 12 Dec 2019 22:32:15 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=ebhv0NVcP8OMATWw26pK/5lGy3q6vqE36z5NvXTPjiw=; b=evtfYWi+sFuQDIJ8AohyuEo4h69m8fJV251pZPwJ63xBrHdeSW6/0/gPciVSRfeka4 LYPTvCYU+ymTPBuNMLEp+S6RhEXMzokyKqQJqaGL1eEe61bOk9+0Yqa/Thw5RD+3WCU7 MzYkma+toFnGSQ/fMr1RC8q9qeQzYlsMh4A8NxBipRq/y/Cs88FbOOE5dTo0M+Dwdpmv tkfYFaBMJnZjN82YCTnKhPkh7EPHWqQ+9B04q+WHpoMyxWqT0BWw2ZqOikHQKiuUaAUZ 5AgSxzeDTBKkrx7UW6je+ktS9wFECADzAsFbiYozj4XJjEdtoAqX5N4AvwKRgf361LLs oQlA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:subject:to:cc:references:from:message-id :date:user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=ebhv0NVcP8OMATWw26pK/5lGy3q6vqE36z5NvXTPjiw=; b=CLl9vZ8BBlyw27BKaI7RD5WMBZsWneSxiTeEBXAQ/x1FXpPdvsZ1fsxn6XokNdfJDa o4FWh7Yy7FXvk7hdfL5M+ufo8y2yDNqxV0+Y2ygVBNqrikeK+VZxmON5jLB4GrqvkPSK Rtzz9MvO4dFb6oq3Pl2wpiRH2hIAUrHg5S2+VnxUuvUPxCWtE6pqUp5g8F0xiLQ6uUaX dX+0CIjcNsSg80/VZayc40YPlfnyq1E7XC8HPtMamTvKK95ACykJQTjsHiy6YfFn0tfP I1HFVoNpXl/QwduG2fHlFliv+EHLkrBV1CyUi+aJwZZCtGfTniNMEaxDOKdoeADqtJ8U 98ww== X-Gm-Message-State: APjAAAUDMATaH2yXBrlybue7hET3NT9p4lddP/qu0hO9QWVZyipKAsCt C8ITcIB7hmfuliacuj3ipTw= X-Google-Smtp-Source: APXvYqz1ncVV9P6HghTrq7V9rQUNGx4ZUwqaC2viZd2Gqr5Q2w3JLGD+K1/7GK52nKzonqBYpbJ9cg== X-Received: by 2002:a17:902:9885:: with SMTP id s5mr14190488plp.217.1576218735351; Thu, 12 Dec 2019 22:32:15 -0800 (PST) Received: from server.roeck-us.net ([2600:1700:e321:62f0:329c:23ff:fee3:9d7c]) by smtp.gmail.com with ESMTPSA id r6sm9651452pfh.91.2019.12.12.22.32.12 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 12 Dec 2019 22:32:14 -0800 (PST) Subject: Re: [PATCH v11 14/14] hwmon: Add PECI dimmtemp driver To: Jae Hyun Yoo , Rob Herring , Greg Kroah-Hartman , Lee Jones , Jean Delvare , Mark Rutland , Joel Stanley , Andrew Jeffery , Jonathan Corbet , Gustavo Pimentel , Kishon Vijay Abraham I , Lorenzo Pieralisi , "Darrick J . Wong" , Eric Sandeen , Arnd Bergmann , Wu Hao , Tomohiro Kusumi , "Bryant G . Ly" , Frederic Barrat , "David S . Miller" , Mauro Carvalho Chehab , Andrew Morton , Randy Dunlap , Philippe Ombredanne , Vinod Koul , Stephen Boyd , David Kershner , Uwe Kleine-Konig , Sagar Dharia , Johan Hovold , Thomas Gleixner , Juergen Gross , Cyrille Pitchen , Tomer Maimon Cc: linux-hwmon@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-doc@vger.kernel.org, openbmc@lists.ozlabs.org, Alan Cox , Andy Shevchenko , Jason M Biils , Miguel Ojeda , Andrew Lunn , Stef van Os References: <20191211194624.2872-1-jae.hyun.yoo@linux.intel.com> <20191211194624.2872-15-jae.hyun.yoo@linux.intel.com> From: Guenter Roeck Message-ID: Date: Thu, 12 Dec 2019 22:32:11 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.2 MIME-Version: 1.0 In-Reply-To: <20191211194624.2872-15-jae.hyun.yoo@linux.intel.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-doc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-doc@vger.kernel.org On 12/11/19 11:46 AM, Jae Hyun Yoo wrote: > This commit adds PECI dimmtemp hwmon driver. > > Cc: Guenter Roeck > Cc: Jean Delvare > Cc: Alan Cox > Cc: Andrew Jeffery > Cc: Andy Shevchenko > Cc: Arnd Bergmann > Cc: Jason M Biils > Cc: Joel Stanley > Cc: Miguel Ojeda > Cc: Andrew Lunn > Cc: Stef van Os > Signed-off-by: Jae Hyun Yoo > Reviewed-by: Haiyue Wang > Reviewed-by: James Feist > Reviewed-by: Vernon Mauery > Acked-by: Guenter Roeck > --- > Changes since v10: > - Added Skylake Xeon D support. > - Added max and crit properties for temperature threshold checking. > - Fixed minor bugs and style issues. > > drivers/hwmon/Kconfig | 14 ++ > drivers/hwmon/Makefile | 1 + > drivers/hwmon/peci-dimmtemp.c | 393 ++++++++++++++++++++++++++++++++++ > 3 files changed, 408 insertions(+) > create mode 100644 drivers/hwmon/peci-dimmtemp.c > > diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig > index b6604759579c..d3370fbab40c 100644 > --- a/drivers/hwmon/Kconfig > +++ b/drivers/hwmon/Kconfig > @@ -1363,6 +1363,20 @@ config SENSORS_PECI_CPUTEMP > This driver can also be built as a module. If so, the module > will be called peci-cputemp. > > +config SENSORS_PECI_DIMMTEMP > + tristate "PECI DIMM temperature monitoring client" > + depends on PECI > + select MFD_INTEL_PECI_CLIENT > + help > + If you say yes here you get support for the generic Intel PECI hwmon > + driver which provides Digital Thermal Sensor (DTS) thermal readings of > + DIMM components that are accessible using the PECI Client Command > + Suite via the processor PECI client. > + Check for details. > + > + This driver can also be built as a module. If so, the module > + will be called peci-dimmtemp. > + > source "drivers/hwmon/pmbus/Kconfig" > > config SENSORS_PWM_FAN > diff --git a/drivers/hwmon/Makefile b/drivers/hwmon/Makefile > index d6fea48697af..4015c4b60bf4 100644 > --- a/drivers/hwmon/Makefile > +++ b/drivers/hwmon/Makefile > @@ -145,6 +145,7 @@ obj-$(CONFIG_SENSORS_PC87360) += pc87360.o > obj-$(CONFIG_SENSORS_PC87427) += pc87427.o > obj-$(CONFIG_SENSORS_PCF8591) += pcf8591.o > obj-$(CONFIG_SENSORS_PECI_CPUTEMP) += peci-cputemp.o > +obj-$(CONFIG_SENSORS_PECI_DIMMTEMP) += peci-dimmtemp.o > obj-$(CONFIG_SENSORS_POWR1220) += powr1220.o > obj-$(CONFIG_SENSORS_PWM_FAN) += pwm-fan.o > obj-$(CONFIG_SENSORS_RASPBERRYPI_HWMON) += raspberrypi-hwmon.o > diff --git a/drivers/hwmon/peci-dimmtemp.c b/drivers/hwmon/peci-dimmtemp.c > new file mode 100644 > index 000000000000..974f453f9366 > --- /dev/null > +++ b/drivers/hwmon/peci-dimmtemp.c > @@ -0,0 +1,393 @@ > +// SPDX-License-Identifier: GPL-2.0 > +// Copyright (c) 2018-2019 Intel Corporation > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include "peci-hwmon.h" > + > +#define DIMM_MASK_CHECK_DELAY_JIFFIES msecs_to_jiffies(5000) > +#define DIMM_MASK_CHECK_RETRY_MAX 60 /* 60 x 5 secs = 5 minutes */ > + > +struct peci_dimmtemp { > + struct peci_client_manager *mgr; > + struct device *dev; > + char name[PECI_NAME_SIZE]; > + const struct cpu_gen_info *gen_info; > + struct workqueue_struct *work_queue; > + struct delayed_work work_handler; > + struct peci_sensor_data temp[DIMM_NUMS_MAX]; > + long temp_max[DIMM_NUMS_MAX]; > + long temp_crit[DIMM_NUMS_MAX]; > + u32 dimm_mask; > + int retry_count; > + u32 temp_config[DIMM_NUMS_MAX + 1]; > + struct hwmon_channel_info temp_info; > + const struct hwmon_channel_info *info[2]; > + struct hwmon_chip_info chip; > +}; > + > +static const char *dimmtemp_label[CHAN_RANK_MAX][DIMM_IDX_MAX] = { > + { "DIMM A1", "DIMM A2", "DIMM A3" }, > + { "DIMM B1", "DIMM B2", "DIMM B3" }, > + { "DIMM C1", "DIMM C2", "DIMM C3" }, > + { "DIMM D1", "DIMM D2", "DIMM D3" }, > + { "DIMM E1", "DIMM E2", "DIMM E3" }, > + { "DIMM F1", "DIMM F2", "DIMM F3" }, > + { "DIMM G1", "DIMM G2", "DIMM G3" }, > + { "DIMM H1", "DIMM H2", "DIMM H3" }, > +}; > + > +static inline int read_ddr_dimm_temp_config(struct peci_dimmtemp *priv, > + int chan_rank, > + u8 *cfg_data) > +{ > + return peci_client_read_package_config(priv->mgr, > + PECI_MBX_INDEX_DDR_DIMM_TEMP, > + chan_rank, cfg_data); > +} > + > +static int get_dimm_temp(struct peci_dimmtemp *priv, int dimm_no) > +{ > + int dimm_order = dimm_no % priv->gen_info->dimm_idx_max; > + int chan_rank = dimm_no / priv->gen_info->dimm_idx_max; > + struct peci_rd_pci_cfg_local_msg rp_msg; > + u8 cfg_data[4]; > + int ret; > + > + if (!peci_sensor_need_update(&priv->temp[dimm_no])) > + return 0; > + > + ret = read_ddr_dimm_temp_config(priv, chan_rank, cfg_data); > + if (ret) > + return ret; > + > + priv->temp[dimm_no].value = cfg_data[dimm_order] * 1000; > + > + switch (priv->gen_info->model) { > + case INTEL_FAM6_SKYLAKE_X: > + rp_msg.addr = priv->mgr->client->addr; > + rp_msg.bus = 2; > + /* > + * Device 10, Function 2: IMC 0 channel 0 -> rank 0 > + * Device 10, Function 6: IMC 0 channel 1 -> rank 1 > + * Device 11, Function 2: IMC 0 channel 2 -> rank 2 > + * Device 12, Function 2: IMC 1 channel 0 -> rank 3 > + * Device 12, Function 6: IMC 1 channel 1 -> rank 4 > + * Device 13, Function 2: IMC 1 channel 2 -> rank 5 > + */ > + rp_msg.device = 10 + chan_rank / 3 * 2 + > + (chan_rank % 3 == 2 ? 1 : 0); > + rp_msg.function = chan_rank % 3 == 1 ? 6 : 2; > + rp_msg.reg = 0x120 + dimm_order * 4; > + rp_msg.rx_len = 4; > + > + ret = peci_command(priv->mgr->client->adapter, > + PECI_CMD_RD_PCI_CFG_LOCAL, &rp_msg); > + if (rp_msg.cc != PECI_DEV_CC_SUCCESS) > + ret = -EAGAIN; > + if (ret) > + return ret; > + > + priv->temp_max[dimm_no] = rp_msg.pci_config[1] * 1000; > + priv->temp_crit[dimm_no] = rp_msg.pci_config[2] * 1000; > + break; > + case INTEL_FAM6_SKYLAKE_XD: > + rp_msg.addr = priv->mgr->client->addr; > + rp_msg.bus = 2; > + /* > + * Device 10, Function 2: IMC 0 channel 0 -> rank 0 > + * Device 10, Function 6: IMC 0 channel 1 -> rank 1 > + * Device 12, Function 2: IMC 1 channel 0 -> rank 2 > + * Device 12, Function 6: IMC 1 channel 1 -> rank 3 > + */ > + rp_msg.device = 10 + chan_rank / 2 * 2; > + rp_msg.function = (chan_rank % 2) ? 6 : 2; > + rp_msg.reg = 0x120 + dimm_order * 4; > + rp_msg.rx_len = 4; > + > + ret = peci_command(priv->mgr->client->adapter, > + PECI_CMD_RD_PCI_CFG_LOCAL, &rp_msg); > + if (rp_msg.cc != PECI_DEV_CC_SUCCESS) > + ret = -EAGAIN; > + if (ret) > + return ret; > + > + priv->temp_max[dimm_no] = rp_msg.pci_config[1] * 1000; > + priv->temp_crit[dimm_no] = rp_msg.pci_config[2] * 1000; > + break; > + case INTEL_FAM6_HASWELL_X: > + case INTEL_FAM6_BROADWELL_X: > + rp_msg.addr = priv->mgr->client->addr; > + rp_msg.bus = 1; > + /* > + * Device 20, Function 0: IMC 0 channel 0 -> rank 0 > + * Device 20, Function 1: IMC 0 channel 1 -> rank 1 > + * Device 21, Function 0: IMC 0 channel 2 -> rank 2 > + * Device 21, Function 1: IMC 0 channel 3 -> rank 3 > + * Device 23, Function 0: IMC 1 channel 0 -> rank 4 > + * Device 23, Function 1: IMC 1 channel 1 -> rank 5 > + * Device 24, Function 0: IMC 1 channel 2 -> rank 6 > + * Device 24, Function 1: IMC 1 channel 3 -> rank 7 > + */ > + rp_msg.device = 20 + chan_rank / 2 + chan_rank / 4; > + rp_msg.function = chan_rank % 2; > + rp_msg.reg = 0x120 + dimm_order * 4; > + rp_msg.rx_len = 4; > + > + ret = peci_command(priv->mgr->client->adapter, > + PECI_CMD_RD_PCI_CFG_LOCAL, &rp_msg); > + if (rp_msg.cc != PECI_DEV_CC_SUCCESS) > + ret = -EAGAIN; > + if (ret) > + return ret; > + > + priv->temp_max[dimm_no] = rp_msg.pci_config[1] * 1000; > + priv->temp_crit[dimm_no] = rp_msg.pci_config[2] * 1000; > + break; > + default: > + return -EOPNOTSUPP; It looks like the sensors are created even on unsupported platforms, which would generate error messages whenever someone tries to read the attributes. There should be some code early on checking this, and the driver should not even instantiate if the CPU model is not supported. > + } > + > + peci_sensor_mark_updated(&priv->temp[dimm_no]); > + > + return 0; > +} > + > +static int dimmtemp_read_string(struct device *dev, > + enum hwmon_sensor_types type, > + u32 attr, int channel, const char **str) > +{ > + struct peci_dimmtemp *priv = dev_get_drvdata(dev); > + u32 dimm_idx_max = priv->gen_info->dimm_idx_max; > + int chan_rank, dimm_idx; > + > + if (attr != hwmon_temp_label) > + return -EOPNOTSUPP; > + > + chan_rank = channel / dimm_idx_max; > + dimm_idx = channel % dimm_idx_max; > + *str = dimmtemp_label[chan_rank][dimm_idx]; Similar to the other patch, I am concerned that this can end up setting *str to NULL at some point in the future. > + > + return 0; > +} > + > +static int dimmtemp_read(struct device *dev, enum hwmon_sensor_types type, > + u32 attr, int channel, long *val) > +{ > + struct peci_dimmtemp *priv = dev_get_drvdata(dev); > + int ret; > + > + ret = get_dimm_temp(priv, channel); > + if (ret) > + return ret; > + > + switch (attr) { > + case hwmon_temp_input: > + *val = priv->temp[channel].value; > + break; > + case hwmon_temp_max: > + *val = priv->temp_max[channel]; > + break; > + case hwmon_temp_crit: > + *val = priv->temp_crit[channel]; > + break; > + default: > + ret = -EOPNOTSUPP; > + break; > + } > + > + return ret; > +} > + > +static umode_t dimmtemp_is_visible(const void *data, > + enum hwmon_sensor_types type, > + u32 attr, int channel) > +{ > + const struct peci_dimmtemp *priv = data; > + > + if (priv->temp_config[channel] & BIT(attr) && > + priv->dimm_mask & BIT(channel)) > + return 0444; > + > + return 0; > +} > + > +static const struct hwmon_ops dimmtemp_ops = { > + .is_visible = dimmtemp_is_visible, > + .read_string = dimmtemp_read_string, > + .read = dimmtemp_read, > +}; > + > +static int check_populated_dimms(struct peci_dimmtemp *priv) > +{ > + u32 chan_rank_max = priv->gen_info->chan_rank_max; > + u32 dimm_idx_max = priv->gen_info->dimm_idx_max; > + int chan_rank, dimm_idx; > + u8 cfg_data[4]; > + > + for (chan_rank = 0; chan_rank < chan_rank_max; chan_rank++) { > + int ret; > + > + ret = read_ddr_dimm_temp_config(priv, chan_rank, cfg_data); > + if (ret) { > + priv->dimm_mask = 0; > + return ret; > + } > + > + for (dimm_idx = 0; dimm_idx < dimm_idx_max; dimm_idx++) > + if (cfg_data[dimm_idx]) > + priv->dimm_mask |= BIT(chan_rank * > + dimm_idx_max + > + dimm_idx); > + } > + > + if (!priv->dimm_mask) > + return -EAGAIN; > + > + dev_dbg(priv->dev, "Scanned populated DIMMs: 0x%x\n", priv->dimm_mask); > + > + return 0; > +} > + > +static int create_dimm_temp_info(struct peci_dimmtemp *priv) > +{ > + int ret, i, config_idx, channels; > + struct device *hwmon_dev; > + > + ret = check_populated_dimms(priv); > + if (ret) { > + if (ret == -EAGAIN) { > + if (priv->retry_count < DIMM_MASK_CHECK_RETRY_MAX) { > + queue_delayed_work(priv->work_queue, > + &priv->work_handler, > + DIMM_MASK_CHECK_DELAY_JIFFIES); > + priv->retry_count++; > + dev_dbg(priv->dev, > + "Deferred DIMM temp info creation\n"); > + } else { > + dev_err(priv->dev, > + "Timeout DIMM temp info creation\n"); > + ret = -ETIMEDOUT; > + } > + } > + > + return ret; > + } > + > + channels = priv->gen_info->chan_rank_max * > + priv->gen_info->dimm_idx_max; > + for (i = 0, config_idx = 0; i < channels; i++) > + if (priv->dimm_mask & BIT(i)) > + while (i >= config_idx) > + priv->temp_config[config_idx++] = > + HWMON_T_LABEL | HWMON_T_INPUT | > + HWMON_T_MAX | HWMON_T_CRIT; > + > + priv->chip.ops = &dimmtemp_ops; > + priv->chip.info = priv->info; > + > + priv->info[0] = &priv->temp_info; > + > + priv->temp_info.type = hwmon_temp; > + priv->temp_info.config = priv->temp_config; > + > + hwmon_dev = devm_hwmon_device_register_with_info(priv->dev, > + priv->name, > + priv, > + &priv->chip, > + NULL); > + ret = PTR_ERR_OR_ZERO(hwmon_dev); > + if (!ret) > + dev_dbg(priv->dev, "%s: sensor '%s'\n", > + dev_name(hwmon_dev), priv->name); > + Any chance to make this consistent with the other driver ? > + return ret; > +} > + > +static void create_dimm_temp_info_delayed(struct work_struct *work) > +{ > + struct delayed_work *dwork = to_delayed_work(work); > + struct peci_dimmtemp *priv = container_of(dwork, struct peci_dimmtemp, > + work_handler); > + int ret; > + > + ret = create_dimm_temp_info(priv); > + if (ret && ret != -EAGAIN) > + dev_dbg(priv->dev, "Failed to create DIMM temp info\n"); > +} > + > +static int peci_dimmtemp_probe(struct platform_device *pdev) > +{ > + struct peci_client_manager *mgr = dev_get_drvdata(pdev->dev.parent); > + struct device *dev = &pdev->dev; > + struct peci_dimmtemp *priv; > + int ret; > + > + if ((mgr->client->adapter->cmd_mask & > + (BIT(PECI_CMD_GET_TEMP) | BIT(PECI_CMD_RD_PKG_CFG))) != > + (BIT(PECI_CMD_GET_TEMP) | BIT(PECI_CMD_RD_PKG_CFG))) > + return -ENODEV; > + > + priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL); > + if (!priv) > + return -ENOMEM; > + > + dev_set_drvdata(dev, priv); > + priv->mgr = mgr; > + priv->dev = dev; > + priv->gen_info = mgr->gen_info; > + > + snprintf(priv->name, PECI_NAME_SIZE, "peci_dimmtemp.cpu%d", > + priv->mgr->client->addr - PECI_BASE_ADDR); > + > + priv->work_queue = alloc_ordered_workqueue(priv->name, 0); > + if (!priv->work_queue) > + return -ENOMEM; > + > + INIT_DELAYED_WORK(&priv->work_handler, create_dimm_temp_info_delayed); > + > + ret = create_dimm_temp_info(priv); > + if (ret && ret != -EAGAIN) { > + dev_err(dev, "Failed to create DIMM temp info\n"); Does this generate error messages if there are no DIMMS ? > + goto err_free_wq; > + } > + > + return 0; > + > +err_free_wq: > + destroy_workqueue(priv->work_queue); > + return ret; > +} > + > +static int peci_dimmtemp_remove(struct platform_device *pdev) > +{ > + struct peci_dimmtemp *priv = dev_get_drvdata(&pdev->dev); > + > + cancel_delayed_work_sync(&priv->work_handler); > + destroy_workqueue(priv->work_queue); > + > + return 0; > +} > + > +static const struct platform_device_id peci_dimmtemp_ids[] = { > + { .name = "peci-dimmtemp", .driver_data = 0 }, > + { } > +}; > +MODULE_DEVICE_TABLE(platform, peci_dimmtemp_ids); > + > +static struct platform_driver peci_dimmtemp_driver = { > + .probe = peci_dimmtemp_probe, > + .remove = peci_dimmtemp_remove, > + .id_table = peci_dimmtemp_ids, > + .driver = { .name = KBUILD_MODNAME, }, > +}; > +module_platform_driver(peci_dimmtemp_driver); > + > +MODULE_AUTHOR("Jae Hyun Yoo "); > +MODULE_DESCRIPTION("PECI dimmtemp driver"); > +MODULE_LICENSE("GPL v2"); >