iio: accel: mma9551: use bool for event state
authorJulien Stephan <jstephan@baylibre.com>
Thu, 31 Oct 2024 15:27:03 +0000 (16:27 +0100)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sun, 3 Nov 2024 20:33:44 +0000 (20:33 +0000)
Since the write_event_config callback now uses a bool for the state
parameter, update the signature of the function it calls accordingly,
and use a bool array for event_enabled.

Signed-off-by: Julien Stephan <jstephan@baylibre.com>
Link: https://patch.msgid.link/20241031-iio-fix-write-event-config-signature-v2-8-2bcacbb517a2@baylibre.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/accel/mma9551.c

index 605022f..6d73eec 100644 (file)
@@ -45,7 +45,7 @@ enum mma9551_tilt_axis {
 struct mma9551_data {
        struct i2c_client *client;
        struct mutex mutex;
-       int event_enabled[3];
+       bool event_enabled[3];
        int irqs[MMA9551_GPIO_COUNT];
 };
 
@@ -162,7 +162,7 @@ static int mma9551_read_event_config(struct iio_dev *indio_dev,
 
 static int mma9551_config_incli_event(struct iio_dev *indio_dev,
                                      enum iio_modifier axis,
-                                     int state)
+                                     bool state)
 {
        struct mma9551_data *data = iio_priv(indio_dev);
        enum mma9551_tilt_axis mma_axis;
@@ -174,7 +174,7 @@ static int mma9551_config_incli_event(struct iio_dev *indio_dev,
        if (data->event_enabled[mma_axis] == state)
                return 0;
 
-       if (state == 0) {
+       if (!state) {
                ret = mma9551_gpio_config(data->client,
                                          (enum mma9551_gpio_pin)mma_axis,
                                          MMA9551_APPID_NONE, 0, 0);