thermal/drivers/hisi: Remove pointless lock

The threaded interrupt inspect the sensors structure to look in the temp
threshold field, but this field is read-only in all the code, except in the
probe function before the threaded interrupt is set. In other words there
is not race window in the threaded interrupt when reading the field value.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Reviewed-by: Leo Yan <leo.yan@linaro.org>
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>

diff --git a/drivers/thermal/hisi_thermal.c b/drivers/thermal/hisi_thermal.c
index f5231974504c1d57cb3d82e2cb0e133a08b941a7..9ec5f290b850667ca08b52697cce54f986102c0a 100644 (file)
--- a/drivers/thermal/hisi_thermal.c
+++ b/drivers/thermal/hisi_thermal.c
@@ -221,14 +221,10 @@ static irqreturn_t hisi_thermal_alarm_irq(int irq, void *dev)
 static irqreturn_t hisi_thermal_alarm_irq_thread(int irq, void *dev)
 {
        struct hisi_thermal_data *data = dev;
-       struct hisi_thermal_sensor *sensor;
-
-       mutex_lock(&data->thermal_lock);
-       sensor = &data->sensors;
+       struct hisi_thermal_sensor *sensor = &data->sensors;
 
        dev_crit(&data->pdev->dev, "THERMAL ALARM: T > %d\n",
                 sensor->thres_temp);
-       mutex_unlock(&data->thermal_lock);
 
        thermal_zone_device_update(data->sensors.tzd,
                                   THERMAL_EVENT_UNSPECIFIED);