Commit 7e97e7bd authored by Lars-Peter Clausen's avatar Lars-Peter Clausen Committed by Jonathan Cameron

iio:apds9300: Use new event config interface

Switch the apds9300 driver to the new IIO event config interface as the old one
is going to be removed.
Signed-off-by: default avatarLars-Peter Clausen <lars@metafoo.de>
Cc: Oleksandr Kravchenko <o.v.kravchenko@globallogic.com>
Signed-off-by: default avatarJonathan Cameron <jic23@kernel.org>
parent 6d59747e
...@@ -273,12 +273,14 @@ static int apds9300_read_raw(struct iio_dev *indio_dev, ...@@ -273,12 +273,14 @@ static int apds9300_read_raw(struct iio_dev *indio_dev,
return ret; return ret;
} }
static int apds9300_read_thresh(struct iio_dev *indio_dev, u64 event_code, static int apds9300_read_thresh(struct iio_dev *indio_dev,
int *val) const struct iio_chan_spec *chan, enum iio_event_type type,
enum iio_event_direction dir, enum iio_event_info info,
int *val, int *val2)
{ {
struct apds9300_data *data = iio_priv(indio_dev); struct apds9300_data *data = iio_priv(indio_dev);
switch (IIO_EVENT_CODE_EXTRACT_DIR(event_code)) { switch (dir) {
case IIO_EV_DIR_RISING: case IIO_EV_DIR_RISING:
*val = data->thresh_hi; *val = data->thresh_hi;
break; break;
...@@ -289,17 +291,19 @@ static int apds9300_read_thresh(struct iio_dev *indio_dev, u64 event_code, ...@@ -289,17 +291,19 @@ static int apds9300_read_thresh(struct iio_dev *indio_dev, u64 event_code,
return -EINVAL; return -EINVAL;
} }
return 0; return IIO_VAL_INT;
} }
static int apds9300_write_thresh(struct iio_dev *indio_dev, u64 event_code, static int apds9300_write_thresh(struct iio_dev *indio_dev,
int val) const struct iio_chan_spec *chan, enum iio_event_type type,
enum iio_event_direction dir, enum iio_event_info info, int val,
int val2)
{ {
struct apds9300_data *data = iio_priv(indio_dev); struct apds9300_data *data = iio_priv(indio_dev);
int ret; int ret;
mutex_lock(&data->mutex); mutex_lock(&data->mutex);
if (IIO_EVENT_CODE_EXTRACT_DIR(event_code) == IIO_EV_DIR_RISING) if (dir == IIO_EV_DIR_RISING)
ret = apds9300_set_thresh_hi(data, val); ret = apds9300_set_thresh_hi(data, val);
else else
ret = apds9300_set_thresh_low(data, val); ret = apds9300_set_thresh_low(data, val);
...@@ -309,7 +313,9 @@ static int apds9300_write_thresh(struct iio_dev *indio_dev, u64 event_code, ...@@ -309,7 +313,9 @@ static int apds9300_write_thresh(struct iio_dev *indio_dev, u64 event_code,
} }
static int apds9300_read_interrupt_config(struct iio_dev *indio_dev, static int apds9300_read_interrupt_config(struct iio_dev *indio_dev,
u64 event_code) const struct iio_chan_spec *chan,
enum iio_event_type type,
enum iio_event_direction dir)
{ {
struct apds9300_data *data = iio_priv(indio_dev); struct apds9300_data *data = iio_priv(indio_dev);
...@@ -317,7 +323,8 @@ static int apds9300_read_interrupt_config(struct iio_dev *indio_dev, ...@@ -317,7 +323,8 @@ static int apds9300_read_interrupt_config(struct iio_dev *indio_dev,
} }
static int apds9300_write_interrupt_config(struct iio_dev *indio_dev, static int apds9300_write_interrupt_config(struct iio_dev *indio_dev,
u64 event_code, int state) const struct iio_chan_spec *chan, enum iio_event_type type,
enum iio_event_direction dir, int state)
{ {
struct apds9300_data *data = iio_priv(indio_dev); struct apds9300_data *data = iio_priv(indio_dev);
int ret; int ret;
...@@ -337,10 +344,24 @@ static const struct iio_info apds9300_info_no_irq = { ...@@ -337,10 +344,24 @@ static const struct iio_info apds9300_info_no_irq = {
static const struct iio_info apds9300_info = { static const struct iio_info apds9300_info = {
.driver_module = THIS_MODULE, .driver_module = THIS_MODULE,
.read_raw = apds9300_read_raw, .read_raw = apds9300_read_raw,
.read_event_value = apds9300_read_thresh, .read_event_value_new = apds9300_read_thresh,
.write_event_value = apds9300_write_thresh, .write_event_value_new = apds9300_write_thresh,
.read_event_config = apds9300_read_interrupt_config, .read_event_config_new = apds9300_read_interrupt_config,
.write_event_config = apds9300_write_interrupt_config, .write_event_config_new = apds9300_write_interrupt_config,
};
static const struct iio_event_spec apds9300_event_spec[] = {
{
.type = IIO_EV_TYPE_THRESH,
.dir = IIO_EV_DIR_RISING,
.mask_separate = BIT(IIO_EV_INFO_VALUE) |
BIT(IIO_EV_INFO_ENABLE),
}, {
.type = IIO_EV_TYPE_THRESH,
.dir = IIO_EV_DIR_FALLING,
.mask_separate = BIT(IIO_EV_INFO_VALUE) |
BIT(IIO_EV_INFO_ENABLE),
},
}; };
static const struct iio_chan_spec apds9300_channels[] = { static const struct iio_chan_spec apds9300_channels[] = {
...@@ -355,10 +376,8 @@ static const struct iio_chan_spec apds9300_channels[] = { ...@@ -355,10 +376,8 @@ static const struct iio_chan_spec apds9300_channels[] = {
.channel2 = IIO_MOD_LIGHT_BOTH, .channel2 = IIO_MOD_LIGHT_BOTH,
.indexed = true, .indexed = true,
.info_mask_separate = BIT(IIO_CHAN_INFO_RAW), .info_mask_separate = BIT(IIO_CHAN_INFO_RAW),
.event_mask = (IIO_EV_BIT(IIO_EV_TYPE_THRESH, .event_spec = apds9300_event_spec,
IIO_EV_DIR_RISING) | .num_event_specs = ARRAY_SIZE(apds9300_event_spec),
IIO_EV_BIT(IIO_EV_TYPE_THRESH,
IIO_EV_DIR_FALLING)),
}, { }, {
.type = IIO_INTENSITY, .type = IIO_INTENSITY,
.channel = 1, .channel = 1,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment