cpufreq: mediatek: avoid redundant conditions

LiaoYuanhong-vivo · vireshk · commit 3e681899cc6e · 2025-09-04T10:20:44.000+05:30
While 'if (i &lt;= 0) ... else if (i &gt; 0) ...' is technically equivalent to
'if (i &lt;= 0) ... else ...', the latter is vastly easier to read because
it avoids writing out a condition that is unnecessary. Let's drop such
unnecessary conditions.

Signed-off-by: Liao Yuanhong &lt;liaoyuanhong@vivo.com&gt;
Signed-off-by: Viresh Kumar &lt;viresh.kumar@linaro.org&gt;
diff --git a/drivers/cpufreq/mediatek-cpufreq.c b/drivers/cpufreq/mediatek-cpufreq.c
@@ -123,7 +123,7 @@ static int mtk_cpufreq_voltage_tracking(struct mtk_cpu_dvfs_info *info,
 						      soc_data->sram_max_volt);
 				return ret;
 			}
-		} else if (pre_vproc > new_vproc) {
+		} else {
 			vproc = max(new_vproc,
 				    pre_vsram - soc_data->max_volt_shift);
 			ret = regulator_set_voltage(proc_reg, vproc,

Original file line number	Diff line number	Diff line change
`@@ -123,7 +123,7 @@ static int mtk_cpufreq_voltage_tracking(struct mtk_cpu_dvfs_info *info,`
`123`	`123`	`soc_data->sram_max_volt);`
`124`	`124`	`return ret;`
`125`	`125`	`}`
`126`		`- } else if (pre_vproc > new_vproc) {`
	`126`	`+ } else {`
`127`	`127`	`vproc = max(new_vproc,`
`128`	`128`	`pre_vsram - soc_data->max_volt_shift);`
`129`	`129`	`ret = regulator_set_voltage(proc_reg, vproc,`