Skip to contentSkip to conversation

thermal: sun8i: remove unused variable and unneeded macros

PATCH Applied Yangtao Li 2 participants 2 messages Jan 12, 2020

Subsystems (3)
Arm/Allwinner sunXi SoC support Thermal Allwinner thermal driver
lore
Share & tools
lore

Yangtao Li

author maintainer
#0 lore raw

PATCH 1/1 thermal: sun8i: remove unused variable and unneeded macros

The cp_ft_flag variable is not used after initialization, so delete it. After that, THS_EFUSE_CP_FT_MASK, THS_EFUSE_CP_FT_BIT and THS_CALIBRATION_IN_FT are not needed, so delete them.

Signed-off-by Yangtao Li <[email protected]>
---
 drivers/thermal/sun8i_thermal.c | 6 ------
 1 file changed, 6 deletions(-)
drivers/thermal/sun8i_thermal.c +0 -6
index 4bcde9491edb..bd7549f9ecba 100644
@@ -54,9 +54,6 @@
 #define SUN50I_H6_THS_DATA_IRQ_STS(x)		BIT(x)
 
 /* millidegree celsius */
-#define THS_EFUSE_CP_FT_MASK			0x3000
-#define THS_EFUSE_CP_FT_BIT			12
-#define THS_CALIBRATION_IN_FT			1
 
 struct tsensor {
 	struct ths_device		*tmdev;

@@ -88,7 +85,6 @@ struct ths_device {
 	struct clk				*bus_clk;
 	struct clk                              *mod_clk;
 	struct tsensor				sensor[MAX_SENSOR_NUM];
-	u32					cp_ft_flag;
 };
 
 /* Temp Unit: millidegree Celsius */

@@ -244,8 +240,6 @@ static int sun50i_h6_ths_calibrate(struct ths_device *tmdev,
 	 * register values and this will become a calibration offset.
 	 */
 	ft_temp = (caldata[0] & FT_TEMP_MASK) * 100;
-	tmdev->cp_ft_flag = (caldata[0] & THS_EFUSE_CP_FT_MASK)
-		>> THS_EFUSE_CP_FT_BIT;
 
 	for (i = 0; i < tmdev->chip->sensor_num; i++) {
 		int sensor_reg = caldata[i + 1];
Signature · 1 line
2.17.1
Applied
#1 lore raw
Quoted 6 lines · Yangtao Li

The cp_ft_flag variable is not used after initialization, so delete it. After that, THS_EFUSE_CP_FT_MASK, THS_EFUSE_CP_FT_BIT and THS_CALIBRATION_IN_FT are not needed, so delete them.

Signed-off-by Yangtao Li <[email protected]>

---

Applied, thanks

Signature · 5 lines
 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog