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 <gwen...@chromium.org>
Cc: Daniel Campello <campe...@chromium.org>
Cc: Hartmut Knaack <knaac...@gmx.de>
Cc: Lars-Peter Clausen <l...@metafoo.de>
Cc: Peter Meerwald-Stadler <pme...@pmeerw.net>
Cc: Douglas Anderson <diand...@chromium.org>
Fixes: 72ad02b15d63 ("iio: Add SEMTECH SX9310/9311 sensor driver")
Signed-off-by: Stephen Boyd <swb...@chromium.org>
---
 drivers/iio/proximity/sx9310.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/iio/proximity/sx9310.c b/drivers/iio/proximity/sx9310.c
index 84c3c9ae80dc..fca871ad82ba 100644
--- a/drivers/iio/proximity/sx9310.c
+++ b/drivers/iio/proximity/sx9310.c
@@ -144,7 +144,7 @@ struct sx9310_data {
        struct completion completion;
        unsigned int chan_read, chan_event;
        int channel_users[SX9310_NUM_CHANNELS];
-       int whoami;
+       unsigned int whoami;
 };
 
 static const struct iio_event_spec sx9310_events[] = {
@@ -864,7 +864,8 @@ static int sx9310_init_device(struct iio_dev *indio_dev)
 
 static int sx9310_set_indio_dev_name(struct device *dev,
                                     struct iio_dev *indio_dev,
-                                    const struct i2c_device_id *id, int whoami)
+                                    const struct i2c_device_id *id,
+                                    unsigned int whoami)
 {
        const struct acpi_device_id *acpi_id;
 
-- 
Sent by a computer, using git, on the internet

Reply via email to