Hi, On Fri, Jul 24, 2020 at 2:33 PM Stephen Boyd <swboyd@xxxxxxxxxxxx> wrote: > > This is an unsigned value, actually it's a u8 but regmap doesn't handle > that easily. Let's make it unsigned throughout so that we don't need to > worry about signed vs. unsigned comparison behavior. > > Cc: Gwendal Grignou <gwendal@xxxxxxxxxxxx> > Cc: Daniel Campello <campello@xxxxxxxxxxxx> > Cc: Hartmut Knaack <knaack.h@xxxxxx> > Cc: Lars-Peter Clausen <lars@xxxxxxxxxx> > Cc: Peter Meerwald-Stadler <pmeerw@xxxxxxxxxx> > Cc: Douglas Anderson <dianders@xxxxxxxxxxxx> > Fixes: 72ad02b15d63 ("iio: Add SEMTECH SX9310/9311 sensor driver") > Signed-off-by: Stephen Boyd <swboyd@xxxxxxxxxxxx> > --- > drivers/iio/proximity/sx9310.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>