Commit 94ff50f4 authored by Thorsten Blum's avatar Thorsten Blum Committed by Daniel Lezcano
Browse files

thermal/drivers/sprd: Use min instead of clamp in sprd_thm_temp_to_rawdata 



Clamping 'val' to itself is unnecessary and the expression can be
simplified by using min() instead. Casting SPRD_THM_RAW_DATA_HIGH to u32
is also redundant and can be removed.

Reviewed-by: default avatarBaolin Wang <baolin.wang@linux.alibaba.com>
Reviewed-by: default avatarLukasz Luba <lukasz.luba@arm.com>
Signed-off-by: default avatarThorsten Blum <thorsten.blum@linux.dev>
Link: https://patch.msgid.link/20260310102523.201722-3-thorsten.blum@linux.dev


Signed-off-by: default avatarDaniel Lezcano <daniel.lezcano@kernel.org>
parent a53a67b1

drivers/thermal/sprd_thermal.c

+1 −1
Original line number Diff line number Diff line
@@ -201,7 +201,7 @@ static int sprd_thm_temp_to_rawdata(int temp, struct sprd_thermal_sensor *sen) 
	 */

	val = (temp + sen->cal_offset) / sen->cal_slope;



	return clamp(val, val, (u32)(SPRD_THM_RAW_DATA_HIGH - 1));

	return min(val, SPRD_THM_RAW_DATA_HIGH - 1);

}



static int sprd_thm_read_temp(struct thermal_zone_device *tz, int *temp)