thermal: Add braces around suspect code
PATCH 1/1 thermal: Add braces around suspect code
It looks like this code is missing braces, otherwise the if statement shouldn't have been indented. Fix it.
Signed-off-by Stephen Boyd <[email protected]>
--- drivers/thermal/of-thermal.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/thermal/of-thermal.c +2 -1
@@ -712,11 +712,12 @@ thermal_of_build_thermal_zone(struct device_node *np) } i =0 ; -for_each_child_of_node (child, gchild) +for_each_child_of_node (child, gchild) { ret =thermal_of_populate_bind_params (gchild, &tz->tbps[i++], tz->trips, tz->ntrips);if (ret) goto free_tbps; + } finish:of_node_put (child);
Signature · 2 lines
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation
Quoted 16 lines · Stephen Boyd
It looks like this code is missing braces, otherwise the if statement shouldn't have been indented. Fix it.
Signed-off-by Stephen Boyd <[email protected]>--- drivers/thermal/of-thermal.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)drivers/thermal/of-thermal.c +0
-0@@ -712,11 +712,12 @@ thermal_of_build_thermal_zone(struct device_node *np) } i =0 ;
drivers/thermal/of-thermal.c thermal_of_build_thermal_zone(struct device_node *np) · line 723
Quoted 10 lines
drivers/thermal/of-thermal.c +2
-1-for_each_child_of_node (child, gchild) +for_each_child_of_node (child, gchild) { ret =thermal_of_populate_bind_params (gchild, &tz->tbps[i++], tz->trips, tz->ntrips);if (ret) goto free_tbps; + } finish:of_node_put (child);
Looks good to me.
Reviewed-by Punit Agrawal <[email protected]>