Commit aeb58c86 authored by Srinivas Pandruvada's avatar Srinivas Pandruvada Committed by Rafael J. Wysocki

thermal/drivers/int340x: processor_thermal: Suppot 64 bit RFIM responses

Some of the RFIM mail box command returns 64 bit values. So enhance
mailbox interface to return 64 bit values and use them for RFIM
commands.
Signed-off-by: default avatarSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Fixes: 5d6fbc96 ("thermal/drivers/int340x: processor_thermal: Export additional attributes")
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent f73cd9c9
...@@ -80,7 +80,7 @@ void proc_thermal_rfim_remove(struct pci_dev *pdev); ...@@ -80,7 +80,7 @@ void proc_thermal_rfim_remove(struct pci_dev *pdev);
int proc_thermal_mbox_add(struct pci_dev *pdev, struct proc_thermal_device *proc_priv); int proc_thermal_mbox_add(struct pci_dev *pdev, struct proc_thermal_device *proc_priv);
void proc_thermal_mbox_remove(struct pci_dev *pdev); void proc_thermal_mbox_remove(struct pci_dev *pdev);
int processor_thermal_send_mbox_cmd(struct pci_dev *pdev, u16 cmd_id, u32 cmd_data, u32 *cmd_resp); int processor_thermal_send_mbox_cmd(struct pci_dev *pdev, u16 cmd_id, u32 cmd_data, u64 *cmd_resp);
int proc_thermal_add(struct device *dev, struct proc_thermal_device *priv); int proc_thermal_add(struct device *dev, struct proc_thermal_device *priv);
void proc_thermal_remove(struct proc_thermal_device *proc_priv); void proc_thermal_remove(struct proc_thermal_device *proc_priv);
int proc_thermal_suspend(struct device *dev); int proc_thermal_suspend(struct device *dev);
......
...@@ -23,7 +23,7 @@ ...@@ -23,7 +23,7 @@
static DEFINE_MUTEX(mbox_lock); static DEFINE_MUTEX(mbox_lock);
static int send_mbox_cmd(struct pci_dev *pdev, u16 cmd_id, u32 cmd_data, u32 *cmd_resp) static int send_mbox_cmd(struct pci_dev *pdev, u16 cmd_id, u32 cmd_data, u64 *cmd_resp)
{ {
struct proc_thermal_device *proc_priv; struct proc_thermal_device *proc_priv;
u32 retries, data; u32 retries, data;
...@@ -68,12 +68,16 @@ static int send_mbox_cmd(struct pci_dev *pdev, u16 cmd_id, u32 cmd_data, u32 *cm ...@@ -68,12 +68,16 @@ static int send_mbox_cmd(struct pci_dev *pdev, u16 cmd_id, u32 cmd_data, u32 *cm
goto unlock_mbox; goto unlock_mbox;
} }
if (cmd_id == MBOX_CMD_WORKLOAD_TYPE_READ) {
data = readl((void __iomem *) (proc_priv->mmio_base + MBOX_OFFSET_DATA));
*cmd_resp = data & 0xff;
}
ret = 0; ret = 0;
if (!cmd_resp)
break;
if (cmd_id == MBOX_CMD_WORKLOAD_TYPE_READ)
*cmd_resp = readl((void __iomem *) (proc_priv->mmio_base + MBOX_OFFSET_DATA));
else
*cmd_resp = readq((void __iomem *) (proc_priv->mmio_base + MBOX_OFFSET_DATA));
break; break;
} while (--retries); } while (--retries);
...@@ -82,7 +86,7 @@ static int send_mbox_cmd(struct pci_dev *pdev, u16 cmd_id, u32 cmd_data, u32 *cm ...@@ -82,7 +86,7 @@ static int send_mbox_cmd(struct pci_dev *pdev, u16 cmd_id, u32 cmd_data, u32 *cm
return ret; return ret;
} }
int processor_thermal_send_mbox_cmd(struct pci_dev *pdev, u16 cmd_id, u32 cmd_data, u32 *cmd_resp) int processor_thermal_send_mbox_cmd(struct pci_dev *pdev, u16 cmd_id, u32 cmd_data, u64 *cmd_resp)
{ {
return send_mbox_cmd(pdev, cmd_id, cmd_data, cmd_resp); return send_mbox_cmd(pdev, cmd_id, cmd_data, cmd_resp);
} }
...@@ -153,7 +157,7 @@ static ssize_t workload_type_show(struct device *dev, ...@@ -153,7 +157,7 @@ static ssize_t workload_type_show(struct device *dev,
char *buf) char *buf)
{ {
struct pci_dev *pdev = to_pci_dev(dev); struct pci_dev *pdev = to_pci_dev(dev);
u32 cmd_resp; u64 cmd_resp;
int ret; int ret;
ret = send_mbox_cmd(pdev, MBOX_CMD_WORKLOAD_TYPE_READ, 0, &cmd_resp); ret = send_mbox_cmd(pdev, MBOX_CMD_WORKLOAD_TYPE_READ, 0, &cmd_resp);
...@@ -187,7 +191,7 @@ static bool workload_req_created; ...@@ -187,7 +191,7 @@ static bool workload_req_created;
int proc_thermal_mbox_add(struct pci_dev *pdev, struct proc_thermal_device *proc_priv) int proc_thermal_mbox_add(struct pci_dev *pdev, struct proc_thermal_device *proc_priv)
{ {
u32 cmd_resp; u64 cmd_resp;
int ret; int ret;
/* Check if there is a mailbox support, if fails return success */ /* Check if there is a mailbox support, if fails return success */
......
...@@ -195,7 +195,7 @@ static ssize_t rfi_restriction_store(struct device *dev, ...@@ -195,7 +195,7 @@ static ssize_t rfi_restriction_store(struct device *dev,
const char *buf, size_t count) const char *buf, size_t count)
{ {
u16 cmd_id = 0x0008; u16 cmd_id = 0x0008;
u32 cmd_resp; u64 cmd_resp;
u32 input; u32 input;
int ret; int ret;
...@@ -215,14 +215,14 @@ static ssize_t rfi_restriction_show(struct device *dev, ...@@ -215,14 +215,14 @@ static ssize_t rfi_restriction_show(struct device *dev,
char *buf) char *buf)
{ {
u16 cmd_id = 0x0007; u16 cmd_id = 0x0007;
u32 cmd_resp; u64 cmd_resp;
int ret; int ret;
ret = processor_thermal_send_mbox_cmd(to_pci_dev(dev), cmd_id, 0, &cmd_resp); ret = processor_thermal_send_mbox_cmd(to_pci_dev(dev), cmd_id, 0, &cmd_resp);
if (ret) if (ret)
return ret; return ret;
return sprintf(buf, "%u\n", cmd_resp); return sprintf(buf, "%llu\n", cmd_resp);
} }
static ssize_t ddr_data_rate_show(struct device *dev, static ssize_t ddr_data_rate_show(struct device *dev,
...@@ -230,14 +230,14 @@ static ssize_t ddr_data_rate_show(struct device *dev, ...@@ -230,14 +230,14 @@ static ssize_t ddr_data_rate_show(struct device *dev,
char *buf) char *buf)
{ {
u16 cmd_id = 0x0107; u16 cmd_id = 0x0107;
u32 cmd_resp; u64 cmd_resp;
int ret; int ret;
ret = processor_thermal_send_mbox_cmd(to_pci_dev(dev), cmd_id, 0, &cmd_resp); ret = processor_thermal_send_mbox_cmd(to_pci_dev(dev), cmd_id, 0, &cmd_resp);
if (ret) if (ret)
return ret; return ret;
return sprintf(buf, "%u\n", cmd_resp); return sprintf(buf, "%llu\n", cmd_resp);
} }
static DEVICE_ATTR_RW(rfi_restriction); static DEVICE_ATTR_RW(rfi_restriction);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment