On Wed, Aug 30, 2017 at 10:47:35AM +0200, Daniel Lezcano wrote: > There is no point to specify the temperature as long variable, the int is > enough. > > Replace all long variables to int, so making the code consistent. > > Signed-off-by: Daniel Lezcano <daniel.lezc...@linaro.org>
Reviewed-by: Leo Yan <leo....@linaro.org> > --- > drivers/thermal/hisi_thermal.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/thermal/hisi_thermal.c b/drivers/thermal/hisi_thermal.c > index 68b625c..9eee82b 100644 > --- a/drivers/thermal/hisi_thermal.c > +++ b/drivers/thermal/hisi_thermal.c > @@ -83,12 +83,12 @@ static inline int hisi_thermal_step_to_temp(int step) > return HISI_TEMP_BASE + (step * HISI_TEMP_STEP); > } > > -static inline long hisi_thermal_temp_to_step(long temp) > +static inline int hisi_thermal_temp_to_step(int temp) > { > return (temp - HISI_TEMP_BASE) / HISI_TEMP_STEP; > } > > -static inline long hisi_thermal_round_temp(int temp) > +static inline int hisi_thermal_round_temp(int temp) > { > return hisi_thermal_step_to_temp( > hisi_thermal_temp_to_step(temp)); > -- > 2.7.4 >