Messages in this thread | | | Date | Thu, 6 Jun 2019 07:24:53 -0700 | From | Eduardo Valentin <> | Subject | Re: [PATCH][next] drivers: thermal: tsens: remove redundant u32 comparison with less than zero |
| |
hey Colin,
On Fri, May 31, 2019 at 11:57:08AM +0100, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > The u32 variable hw_id is unsigned and cannot be less than zero so > the comparison with less than zero is always false and hence is redundant > and can be removed. > > Addresses-Coverity: ("Unsigned compared against 0") > Signed-off-by: Colin Ian King <colin.king@canonical.com>
Thanks for the patch but the commit that introduced this has been reverted. I pushed an update on my -next branch to silence this in the linux-next tree too.
> --- > drivers/thermal/qcom/tsens-common.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/thermal/qcom/tsens-common.c b/drivers/thermal/qcom/tsens-common.c > index 928e8e81ba69..f4419f45025d 100644 > --- a/drivers/thermal/qcom/tsens-common.c > +++ b/drivers/thermal/qcom/tsens-common.c > @@ -69,7 +69,7 @@ bool is_sensor_enabled(struct tsens_priv *priv, u32 hw_id) > u32 val; > int ret; > > - if ((hw_id > (priv->num_sensors - 1)) || (hw_id < 0)) > + if (hw_id > (priv->num_sensors - 1)) > return -EINVAL; > ret = regmap_field_read(priv->rf[SENSOR_EN], &val); > if (ret)
| |