Sudip Mukherjee <sudipm.mukher...@gmail.com> writes: > state is an unsigned long and can never be less than 0. > > Signed-off-by: Sudip Mukherjee <su...@vectorindia.org> > --- > drivers/thermal/devfreq_cooling.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/thermal/devfreq_cooling.c > b/drivers/thermal/devfreq_cooling.c > index 01f0015..81631b1 100644 > --- a/drivers/thermal/devfreq_cooling.c > +++ b/drivers/thermal/devfreq_cooling.c > @@ -312,7 +312,7 @@ static int devfreq_cooling_state2power(struct > thermal_cooling_device *cdev, > unsigned long freq; > u32 static_power; > > - if (state < 0 || state >= dfc->freq_table_size) > + if (state >= dfc->freq_table_size) > return -EINVAL; > > freq = dfc->freq_table[state];
Makes sense. Acked-by: Punit Agrawal <punit.agra...@arm.com> Thanks for sending the fix.
