From bffc3e6a201b04b72c578872bfc3be7bc61d64f9 Mon Sep 17 00:00:00 2001 From: Manaf Meethalavalappu Pallikunhi Date: Wed, 16 Feb 2022 16:09:45 +0530 Subject: [PATCH] drivers: thermal: qmi_cooling: update cooling device name length The thermal framework doesn't have any cooling device name length limit check. So use the same qmi device name length for qmi cooling device as well so that both qmi device name and thermal framework cooling device name will be used the same name. Change-Id: I373107c5eb17852dd98d0fa787839e65284690fa Signed-off-by: Manaf Meethalavalappu Pallikunhi --- drivers/thermal/qcom/qmi_cooling.c | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/thermal/qcom/qmi_cooling.c b/drivers/thermal/qcom/qmi_cooling.c index 46e81de4dd7c..4911ff58c99a 100644 --- a/drivers/thermal/qcom/qmi_cooling.c +++ b/drivers/thermal/qcom/qmi_cooling.c @@ -19,12 +19,11 @@ #define QMI_CDEV_DRIVER "qmi-cooling-device" #define QMI_TMD_RESP_TOUT msecs_to_jiffies(100) -#define QMI_CLIENT_NAME_LENGTH 40 struct qmi_cooling_device { struct device_node *np; - char cdev_name[THERMAL_NAME_LENGTH]; - char qmi_name[QMI_CLIENT_NAME_LENGTH]; + char cdev_name[QMI_TMD_MITIGATION_DEV_ID_LENGTH_MAX_V01]; + char qmi_name[QMI_TMD_MITIGATION_DEV_ID_LENGTH_MAX_V01]; bool connection_active; struct list_head qmi_node; struct thermal_cooling_device *cdev; @@ -45,7 +44,7 @@ struct qmi_tmd_instance { static struct qmi_tmd_instance *tmd_instances; static int tmd_inst_cnt; -static char device_clients[][QMI_CLIENT_NAME_LENGTH] = { +static char device_clients[][QMI_TMD_MITIGATION_DEV_ID_LENGTH_MAX_V01] = { {"pa"}, {"pa_fr1"}, {"cx_vdd_limit"}, @@ -493,13 +492,13 @@ static int of_get_qmi_tmd_platform_data(struct device *dev) } strscpy(qmi_cdev->cdev_name, cdev_np->name, - THERMAL_NAME_LENGTH); + QMI_TMD_MITIGATION_DEV_ID_LENGTH_MAX_V01); if (!of_property_read_string(cdev_np, "qcom,qmi-dev-name", &qmi_name)) { strscpy(qmi_cdev->qmi_name, qmi_name, - QMI_CLIENT_NAME_LENGTH); + QMI_TMD_MITIGATION_DEV_ID_LENGTH_MAX_V01); } else { dev_err(dev, "Fail to parse dev name for %s\n", cdev_np->name);