thermal: intel: intel_soc_dts_iosf: Remove redundant check

zhang-rui · rafaeljw · commit ee188ee6ed38 · 2023-08-22T22:07:10.000+02:00
Remove the redundant check in remove_dts_thermal_zone() because all of
its existing callers pass a valid pointer as the argument.

Signed-off-by: Zhang Rui &lt;rui.zhang@intel.com&gt;
Signed-off-by: Rafael J. Wysocki &lt;rafael.j.wysocki@intel.com&gt;
diff --git a/drivers/thermal/intel/intel_soc_dts_iosf.c b/drivers/thermal/intel/intel_soc_dts_iosf.c
@@ -212,11 +212,9 @@ static int soc_dts_enable(int id)
 
 static void remove_dts_thermal_zone(struct intel_soc_dts_sensor_entry *dts)
 {
-	if (dts) {
-		iosf_mbi_write(BT_MBI_UNIT_PMC, MBI_REG_WRITE,
-			       SOC_DTS_OFFSET_ENABLE, dts->store_status);
-		thermal_zone_device_unregister(dts->tzone);
-	}
+	iosf_mbi_write(BT_MBI_UNIT_PMC, MBI_REG_WRITE,
+		       SOC_DTS_OFFSET_ENABLE, dts->store_status);
+	thermal_zone_device_unregister(dts->tzone);
 }
 
 static int add_dts_thermal_zone(int id, struct intel_soc_dts_sensor_entry *dts,

Original file line number	Diff line number	Diff line change
`@@ -212,11 +212,9 @@ static int soc_dts_enable(int id)`
`212`	`212`
`213`	`213`	`static void remove_dts_thermal_zone(struct intel_soc_dts_sensor_entry *dts)`
`214`	`214`	`{`
`215`		`- if (dts) {`
`216`		`- iosf_mbi_write(BT_MBI_UNIT_PMC, MBI_REG_WRITE,`
`217`		`- SOC_DTS_OFFSET_ENABLE, dts->store_status);`
`218`		`- thermal_zone_device_unregister(dts->tzone);`
`219`		`- }`
	`215`	`+ iosf_mbi_write(BT_MBI_UNIT_PMC, MBI_REG_WRITE,`
	`216`	`+ SOC_DTS_OFFSET_ENABLE, dts->store_status);`
	`217`	`+ thermal_zone_device_unregister(dts->tzone);`
`220`	`218`	`}`
`221`	`219`
`222`	`220`	`static int add_dts_thermal_zone(int id, struct intel_soc_dts_sensor_entry *dts,`