thermal: int340x: processor_thermal: Move mailbox code to common module
authorSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Tue, 29 Aug 2023 00:23:40 +0000 (17:23 -0700)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 14 Sep 2023 19:52:19 +0000 (21:52 +0200)
The processor thermal mailbox is used for workload type request and
also in the processor thermal RFIM module. So, move the workload type
request code to its own module from the current processor thermal
mailbox module.

processor_thermal_mailbox.c contains only mailbox read/write related
source code. The source related to workload_types requests is moved to
a module processor_thermal_wt_req.c.

In addition
 - Rename PROC_THERMAL_FEATURE_MBOX to PROC_THERMAL_FEATURE_WT_REQ.
 - proc_thermal_mbox_add(), which adds workload type sysfs attribute group
   is renamed to proc_thermal_wt_req_add().
 - proc_thermal_mbox_remove() is renamed to proc_thermal_wt_req_remove().

While here, resolve check patch warnings for 100 columns for only modified
lines.

No functional changes are expected.

Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Reviewed-by: Zhang Rui <rui.zhang@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/thermal/intel/int340x_thermal/Makefile
drivers/thermal/intel/int340x_thermal/processor_thermal_device.c
drivers/thermal/intel/int340x_thermal/processor_thermal_device.h
drivers/thermal/intel/int340x_thermal/processor_thermal_device_pci.c
drivers/thermal/intel/int340x_thermal/processor_thermal_device_pci_legacy.c
drivers/thermal/intel/int340x_thermal/processor_thermal_mbox.c
drivers/thermal/intel/int340x_thermal/processor_thermal_wt_req.c [new file with mode: 0644]

index 4e852ce4a5d5a5eb332c25693c22d637aa64cb4e..7e40aebb05569a26ea8a4e3b553901c2629da3ac 100644 (file)
@@ -10,5 +10,6 @@ obj-$(CONFIG_INT340X_THERMAL) += processor_thermal_device_pci.o
 obj-$(CONFIG_PROC_THERMAL_MMIO_RAPL) += processor_thermal_rapl.o
 obj-$(CONFIG_INT340X_THERMAL)  += processor_thermal_rfim.o
 obj-$(CONFIG_INT340X_THERMAL)  += processor_thermal_mbox.o
+obj-$(CONFIG_INT340X_THERMAL)  += processor_thermal_wt_req.o
 obj-$(CONFIG_INT3406_THERMAL)  += int3406_thermal.o
 obj-$(CONFIG_ACPI_THERMAL_REL) += acpi_thermal_rel.o
index 3ca0a2f5937f2624ff6e73ab0a006fb7a49361e4..3f0eac6cb4da031b8e3258c5d04001d3ccb05a99 100644 (file)
@@ -346,8 +346,8 @@ int proc_thermal_mmio_add(struct pci_dev *pdev,
                }
        }
 
-       if (feature_mask & PROC_THERMAL_FEATURE_MBOX) {
-               ret = proc_thermal_mbox_add(pdev, proc_priv);
+       if (feature_mask & PROC_THERMAL_FEATURE_WT_REQ) {
+               ret = proc_thermal_wt_req_add(pdev, proc_priv);
                if (ret) {
                        dev_err(&pdev->dev, "failed to add MBOX interface\n");
                        goto err_rem_rfim;
@@ -374,8 +374,8 @@ void proc_thermal_mmio_remove(struct pci_dev *pdev, struct proc_thermal_device *
            proc_priv->mmio_feature_mask & PROC_THERMAL_FEATURE_DVFS)
                proc_thermal_rfim_remove(pdev);
 
-       if (proc_priv->mmio_feature_mask & PROC_THERMAL_FEATURE_MBOX)
-               proc_thermal_mbox_remove(pdev);
+       if (proc_priv->mmio_feature_mask & PROC_THERMAL_FEATURE_WT_REQ)
+               proc_thermal_wt_req_remove(pdev);
 }
 EXPORT_SYMBOL_GPL(proc_thermal_mmio_remove);
 
index 7acaa8f1b896f30a0440e8b33cb199c3b4da8ecc..bae6e3e4e22e09ee113547c227da4ec60098b54d 100644 (file)
@@ -59,7 +59,7 @@ struct rapl_mmio_regs {
 #define PROC_THERMAL_FEATURE_RAPL      0x01
 #define PROC_THERMAL_FEATURE_FIVR      0x02
 #define PROC_THERMAL_FEATURE_DVFS      0x04
-#define PROC_THERMAL_FEATURE_MBOX      0x08
+#define PROC_THERMAL_FEATURE_WT_REQ    0x08
 #define PROC_THERMAL_FEATURE_DLVR      0x10
 
 #if IS_ENABLED(CONFIG_PROC_THERMAL_MMIO_RAPL)
@@ -80,8 +80,14 @@ static void __maybe_unused proc_thermal_rapl_remove(void)
 int proc_thermal_rfim_add(struct pci_dev *pdev, struct proc_thermal_device *proc_priv);
 void proc_thermal_rfim_remove(struct pci_dev *pdev);
 
-int proc_thermal_mbox_add(struct pci_dev *pdev, struct proc_thermal_device *proc_priv);
-void proc_thermal_mbox_remove(struct pci_dev *pdev);
+int proc_thermal_wt_req_add(struct pci_dev *pdev, struct proc_thermal_device *proc_priv);
+void proc_thermal_wt_req_remove(struct pci_dev *pdev);
+
+#define MBOX_CMD_WORKLOAD_TYPE_READ    0x0E
+#define MBOX_CMD_WORKLOAD_TYPE_WRITE   0x0F
+
+#define MBOX_DATA_BIT_AC_DC            30
+#define MBOX_DATA_BIT_VALID            31
 
 int processor_thermal_send_mbox_read_cmd(struct pci_dev *pdev, u16 id, u64 *resp);
 int processor_thermal_send_mbox_write_cmd(struct pci_dev *pdev, u16 id, u32 data);
index 0d1e980072704e96c8b5d82db5161368d2def95b..314a726f35c778ed89750deb83357191b9490bc1 100644 (file)
@@ -350,9 +350,13 @@ static SIMPLE_DEV_PM_OPS(proc_thermal_pci_pm, proc_thermal_pci_suspend,
                         proc_thermal_pci_resume);
 
 static const struct pci_device_id proc_thermal_pci_ids[] = {
-       { PCI_DEVICE_DATA(INTEL, ADL_THERMAL, PROC_THERMAL_FEATURE_RAPL | PROC_THERMAL_FEATURE_FIVR | PROC_THERMAL_FEATURE_DVFS | PROC_THERMAL_FEATURE_MBOX) },
-       { PCI_DEVICE_DATA(INTEL, MTLP_THERMAL, PROC_THERMAL_FEATURE_RAPL | PROC_THERMAL_FEATURE_FIVR | PROC_THERMAL_FEATURE_DVFS | PROC_THERMAL_FEATURE_MBOX | PROC_THERMAL_FEATURE_DLVR) },
-       { PCI_DEVICE_DATA(INTEL, RPL_THERMAL, PROC_THERMAL_FEATURE_RAPL | PROC_THERMAL_FEATURE_FIVR | PROC_THERMAL_FEATURE_DVFS | PROC_THERMAL_FEATURE_MBOX) },
+       { PCI_DEVICE_DATA(INTEL, ADL_THERMAL, PROC_THERMAL_FEATURE_RAPL |
+         PROC_THERMAL_FEATURE_FIVR | PROC_THERMAL_FEATURE_DVFS | PROC_THERMAL_FEATURE_WT_REQ) },
+       { PCI_DEVICE_DATA(INTEL, MTLP_THERMAL, PROC_THERMAL_FEATURE_RAPL |
+         PROC_THERMAL_FEATURE_FIVR | PROC_THERMAL_FEATURE_DVFS | PROC_THERMAL_FEATURE_WT_REQ |
+         PROC_THERMAL_FEATURE_DLVR) },
+       { PCI_DEVICE_DATA(INTEL, RPL_THERMAL, PROC_THERMAL_FEATURE_RAPL |
+         PROC_THERMAL_FEATURE_FIVR | PROC_THERMAL_FEATURE_DVFS | PROC_THERMAL_FEATURE_WT_REQ) },
        { },
 };
 
index 16fd9df5f36d090e31feb5c0482ac94fce8e4e20..ccfdd2f9d9737ce5497611f7c7b070321edfaeea 100644 (file)
@@ -137,7 +137,8 @@ static const struct pci_device_id proc_thermal_pci_ids[] = {
        { PCI_DEVICE_DATA(INTEL, ICL_THERMAL, PROC_THERMAL_FEATURE_RAPL) },
        { PCI_DEVICE_DATA(INTEL, JSL_THERMAL, 0) },
        { PCI_DEVICE_DATA(INTEL, SKL_THERMAL, PROC_THERMAL_FEATURE_RAPL) },
-       { PCI_DEVICE_DATA(INTEL, TGL_THERMAL, PROC_THERMAL_FEATURE_RAPL | PROC_THERMAL_FEATURE_FIVR | PROC_THERMAL_FEATURE_MBOX) },
+       { PCI_DEVICE_DATA(INTEL, TGL_THERMAL, PROC_THERMAL_FEATURE_RAPL |
+         PROC_THERMAL_FEATURE_FIVR | PROC_THERMAL_FEATURE_WT_REQ) },
        { },
 };
 
index 0b89a4340ff4e12034efaf16507c812c7cb7a4b5..ec766c5615b733fe5c3faa43a0de2cd14b9ea4a9 100644 (file)
 #include <linux/io-64-nonatomic-lo-hi.h>
 #include "processor_thermal_device.h"
 
-#define MBOX_CMD_WORKLOAD_TYPE_READ    0x0E
-#define MBOX_CMD_WORKLOAD_TYPE_WRITE   0x0F
-
 #define MBOX_OFFSET_DATA               0x5810
 #define MBOX_OFFSET_INTERFACE          0x5818
 
 #define MBOX_BUSY_BIT                  31
 #define MBOX_RETRY_COUNT               100
 
-#define MBOX_DATA_BIT_VALID            31
-#define MBOX_DATA_BIT_AC_DC            30
-
 static DEFINE_MUTEX(mbox_lock);
 
 static int wait_for_mbox_ready(struct proc_thermal_device *proc_priv)
@@ -114,128 +108,4 @@ int processor_thermal_send_mbox_write_cmd(struct pci_dev *pdev, u16 id, u32 data
 }
 EXPORT_SYMBOL_NS_GPL(processor_thermal_send_mbox_write_cmd, INT340X_THERMAL);
 
-/* List of workload types */
-static const char * const workload_types[] = {
-       "none",
-       "idle",
-       "semi_active",
-       "bursty",
-       "sustained",
-       "battery_life",
-       NULL
-};
-
-static ssize_t workload_available_types_show(struct device *dev,
-                                              struct device_attribute *attr,
-                                              char *buf)
-{
-       int i = 0;
-       int ret = 0;
-
-       while (workload_types[i] != NULL)
-               ret += sprintf(&buf[ret], "%s ", workload_types[i++]);
-
-       ret += sprintf(&buf[ret], "\n");
-
-       return ret;
-}
-
-static DEVICE_ATTR_RO(workload_available_types);
-
-static ssize_t workload_type_store(struct device *dev,
-                                   struct device_attribute *attr,
-                                   const char *buf, size_t count)
-{
-       struct pci_dev *pdev = to_pci_dev(dev);
-       char str_preference[15];
-       u32 data = 0;
-       ssize_t ret;
-
-       ret = sscanf(buf, "%14s", str_preference);
-       if (ret != 1)
-               return -EINVAL;
-
-       ret = match_string(workload_types, -1, str_preference);
-       if (ret < 0)
-               return ret;
-
-       ret &= 0xff;
-
-       if (ret)
-               data = BIT(MBOX_DATA_BIT_VALID) | BIT(MBOX_DATA_BIT_AC_DC);
-
-       data |= ret;
-
-       ret = send_mbox_write_cmd(pdev, MBOX_CMD_WORKLOAD_TYPE_WRITE, data);
-       if (ret)
-               return false;
-
-       return count;
-}
-
-static ssize_t workload_type_show(struct device *dev,
-                                  struct device_attribute *attr,
-                                  char *buf)
-{
-       struct pci_dev *pdev = to_pci_dev(dev);
-       u64 cmd_resp;
-       int ret;
-
-       ret = send_mbox_read_cmd(pdev, MBOX_CMD_WORKLOAD_TYPE_READ, &cmd_resp);
-       if (ret)
-               return false;
-
-       cmd_resp &= 0xff;
-
-       if (cmd_resp > ARRAY_SIZE(workload_types) - 1)
-               return -EINVAL;
-
-       return sprintf(buf, "%s\n", workload_types[cmd_resp]);
-}
-
-static DEVICE_ATTR_RW(workload_type);
-
-static struct attribute *workload_req_attrs[] = {
-       &dev_attr_workload_available_types.attr,
-       &dev_attr_workload_type.attr,
-       NULL
-};
-
-static const struct attribute_group workload_req_attribute_group = {
-       .attrs = workload_req_attrs,
-       .name = "workload_request"
-};
-
-static bool workload_req_created;
-
-int proc_thermal_mbox_add(struct pci_dev *pdev, struct proc_thermal_device *proc_priv)
-{
-       u64 cmd_resp;
-       int ret;
-
-       /* Check if there is a mailbox support, if fails return success */
-       ret = send_mbox_read_cmd(pdev, MBOX_CMD_WORKLOAD_TYPE_READ, &cmd_resp);
-       if (ret)
-               return 0;
-
-       ret = sysfs_create_group(&pdev->dev.kobj, &workload_req_attribute_group);
-       if (ret)
-               return ret;
-
-       workload_req_created = true;
-
-       return 0;
-}
-EXPORT_SYMBOL_GPL(proc_thermal_mbox_add);
-
-void proc_thermal_mbox_remove(struct pci_dev *pdev)
-{
-       if (workload_req_created)
-               sysfs_remove_group(&pdev->dev.kobj, &workload_req_attribute_group);
-
-       workload_req_created = false;
-
-}
-EXPORT_SYMBOL_GPL(proc_thermal_mbox_remove);
-
 MODULE_LICENSE("GPL v2");
diff --git a/drivers/thermal/intel/int340x_thermal/processor_thermal_wt_req.c b/drivers/thermal/intel/int340x_thermal/processor_thermal_wt_req.c
new file mode 100644 (file)
index 0000000..711c4f7
--- /dev/null
@@ -0,0 +1,136 @@
+// SPDX-License-Identifier: GPL-2.0-only
+/*
+ * processor thermal device for Workload type hints
+ * update from user space
+ *
+ * Copyright (c) 2020-2023, Intel Corporation.
+ */
+
+#include <linux/pci.h>
+#include "processor_thermal_device.h"
+
+/* List of workload types */
+static const char * const workload_types[] = {
+       "none",
+       "idle",
+       "semi_active",
+       "bursty",
+       "sustained",
+       "battery_life",
+       NULL
+};
+
+static ssize_t workload_available_types_show(struct device *dev,
+                                            struct device_attribute *attr,
+                                            char *buf)
+{
+       int i = 0;
+       int ret = 0;
+
+       while (workload_types[i] != NULL)
+               ret += sprintf(&buf[ret], "%s ", workload_types[i++]);
+
+       ret += sprintf(&buf[ret], "\n");
+
+       return ret;
+}
+
+static DEVICE_ATTR_RO(workload_available_types);
+
+static ssize_t workload_type_store(struct device *dev,
+                                  struct device_attribute *attr,
+                                  const char *buf, size_t count)
+{
+       struct pci_dev *pdev = to_pci_dev(dev);
+       char str_preference[15];
+       u32 data = 0;
+       ssize_t ret;
+
+       ret = sscanf(buf, "%14s", str_preference);
+       if (ret != 1)
+               return -EINVAL;
+
+       ret = match_string(workload_types, -1, str_preference);
+       if (ret < 0)
+               return ret;
+
+       ret &= 0xff;
+
+       if (ret)
+               data = BIT(MBOX_DATA_BIT_VALID) | BIT(MBOX_DATA_BIT_AC_DC);
+
+       data |= ret;
+
+       ret = processor_thermal_send_mbox_write_cmd(pdev, MBOX_CMD_WORKLOAD_TYPE_WRITE, data);
+       if (ret)
+               return false;
+
+       return count;
+}
+
+static ssize_t workload_type_show(struct device *dev,
+                                 struct device_attribute *attr,
+                                 char *buf)
+{
+       struct pci_dev *pdev = to_pci_dev(dev);
+       u64 cmd_resp;
+       int ret;
+
+       ret = processor_thermal_send_mbox_read_cmd(pdev, MBOX_CMD_WORKLOAD_TYPE_READ, &cmd_resp);
+       if (ret)
+               return false;
+
+       cmd_resp &= 0xff;
+
+       if (cmd_resp > ARRAY_SIZE(workload_types) - 1)
+               return -EINVAL;
+
+       return sprintf(buf, "%s\n", workload_types[cmd_resp]);
+}
+
+static DEVICE_ATTR_RW(workload_type);
+
+static struct attribute *workload_req_attrs[] = {
+       &dev_attr_workload_available_types.attr,
+       &dev_attr_workload_type.attr,
+       NULL
+};
+
+static const struct attribute_group workload_req_attribute_group = {
+       .attrs = workload_req_attrs,
+       .name = "workload_request"
+};
+
+static bool workload_req_created;
+
+int proc_thermal_wt_req_add(struct pci_dev *pdev, struct proc_thermal_device *proc_priv)
+{
+       u64 cmd_resp;
+       int ret;
+
+       /* Check if there is a mailbox support, if fails return success */
+       ret = processor_thermal_send_mbox_read_cmd(pdev, MBOX_CMD_WORKLOAD_TYPE_READ, &cmd_resp);
+       if (ret)
+               return 0;
+
+       ret = sysfs_create_group(&pdev->dev.kobj, &workload_req_attribute_group);
+       if (ret)
+               return ret;
+
+       workload_req_created = true;
+
+       return 0;
+}
+EXPORT_SYMBOL_GPL(proc_thermal_wt_req_add);
+
+void proc_thermal_wt_req_remove(struct pci_dev *pdev)
+{
+       if (workload_req_created)
+               sysfs_remove_group(&pdev->dev.kobj, &workload_req_attribute_group);
+
+       workload_req_created = false;
+}
+EXPORT_SYMBOL_GPL(proc_thermal_wt_req_remove);
+
+MODULE_IMPORT_NS(INT340X_THERMAL);
+MODULE_LICENSE("GPL");