thermal/core: Drop valid pointer check for type

prabhakarlad · dlezcano · commit c71d8035f1b7 · 2022-10-04T11:21:30.000+02:00
Drop the valid pointer check for type in thermal_zone_device_register_with_trips() as we already have it confirmed for != NULL from the previous if block. Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Link: https://lore.kernel.org/r/20220909181322.10933-1-prabhakar.mahadev-lad.rj@bp.renesas.com Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c
@@ -1186,7 +1186,7 @@ thermal_zone_device_register_with_trips(const char *type, struct thermal_trip *t
 		return ERR_PTR(-EINVAL);
 	}
 
-	if (type && strlen(type) >= THERMAL_NAME_LENGTH) {
+	if (strlen(type) >= THERMAL_NAME_LENGTH) {
 		pr_err("Thermal zone name (%s) too long, should be under %d chars\n",
 		       type, THERMAL_NAME_LENGTH);
 		return ERR_PTR(-EINVAL);

Original file line number	Diff line number	Diff line change
`@@ -1186,7 +1186,7 @@ thermal_zone_device_register_with_trips(const char type, struct thermal_trip t`
`1186`	`1186`	`return ERR_PTR(-EINVAL);`
`1187`	`1187`	`}`
`1188`	`1188`
`1189`		`- if (type && strlen(type) >= THERMAL_NAME_LENGTH) {`
	`1189`	`+ if (strlen(type) >= THERMAL_NAME_LENGTH) {`
`1190`	`1190`	`pr_err("Thermal zone name (%s) too long, should be under %d chars\n",`
`1191`	`1191`	`type, THERMAL_NAME_LENGTH);`
`1192`	`1192`	`return ERR_PTR(-EINVAL);`