tree e6ec4ad011f9524cdc0f2cd4cc8e779bc4dccf7e
parent 1ab20c0e53fa2167357bd90b7f7f7019cad9daaa
author Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> 1589469905 +0200
committer Daniel Lezcano <daniel.lezcano@linaro.org> 1590174599 +0200

thermal: rcar_thermal: Clean up rcar_thermal_update_temp()

Moving the ctemp variable out of the private data structure made it
possible to clean up rcar_thermal_update_temp(). Initialize the local
ctemp to the error code to return if the reading fails and just return
it at the end of the function.

It's OK to change the datatype of old, new and ctemp to int as all
values are ANDed with CTEMP (0x3f) before being stored. While at it
change the datatype of the loop variable 'i' to to unsigned int.

Suggested-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Link: https://lore.kernel.org/r/20200514152505.1927634-1-niklas.soderlund+renesas@ragnatech.se