From: Jonathan Cameron <jic23@kernel.org>
To: Lars-Peter Clausen <lars@metafoo.de>
Cc: linux-iio@vger.kernel.org
Subject: Re: [PATCH 2/2] iio: Remove support for the legacy event config interface
Date: Sun, 08 Dec 2013 13:21:03 +0000 [thread overview]
Message-ID: <52A4723F.10302@kernel.org> (raw)
In-Reply-To: <1386413120-13289-2-git-send-email-lars@metafoo.de>
On 12/07/13 10:45, Lars-Peter Clausen wrote:
> Now that all drivers have been converted to the new event config interface we
> can remove for the legacy event config interface. Also drop the '_new' suffix
> for the event config interface callbacks, since those are the only callbacks
> now.
>
> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Applied to the togreg branch of iio.git (pushed out as testing unti
build tests are complete).
Thanks for clearing up this loose end (and for the original
series obviously!)
Jonathan
> ---
> drivers/iio/adc/max1363.c | 8 +-
> drivers/iio/dac/ad5421.c | 6 +-
> drivers/iio/industrialio-event.c | 154 +++++------------------------
> drivers/iio/light/apds9300.c | 8 +-
> drivers/iio/light/cm36651.c | 8 +-
> drivers/iio/light/gp2ap020a00f.c | 8 +-
> drivers/iio/light/tsl2563.c | 8 +-
> drivers/staging/iio/accel/lis3l02dq_core.c | 8 +-
> drivers/staging/iio/accel/sca3000_core.c | 16 +--
> drivers/staging/iio/adc/ad7291.c | 8 +-
> drivers/staging/iio/adc/ad799x_core.c | 6 +-
> drivers/staging/iio/cdc/ad7150.c | 8 +-
> drivers/staging/iio/iio_simple_dummy.c | 8 +-
> drivers/staging/iio/light/tsl2x7x_core.c | 40 ++++----
> include/linux/iio/events.h | 4 -
> include/linux/iio/iio.h | 34 +------
> 16 files changed, 97 insertions(+), 235 deletions(-)
>
> diff --git a/drivers/iio/adc/max1363.c b/drivers/iio/adc/max1363.c
> index 6118dce..e283f2f 100644
> --- a/drivers/iio/adc/max1363.c
> +++ b/drivers/iio/adc/max1363.c
> @@ -1039,10 +1039,10 @@ static const struct iio_info max1238_info = {
> };
>
> static const struct iio_info max1363_info = {
> - .read_event_value_new = &max1363_read_thresh,
> - .write_event_value_new = &max1363_write_thresh,
> - .read_event_config_new = &max1363_read_event_config,
> - .write_event_config_new = &max1363_write_event_config,
> + .read_event_value = &max1363_read_thresh,
> + .write_event_value = &max1363_write_thresh,
> + .read_event_config = &max1363_read_event_config,
> + .write_event_config = &max1363_write_event_config,
> .read_raw = &max1363_read_raw,
> .update_scan_mode = &max1363_update_scan_mode,
> .driver_module = THIS_MODULE,
> diff --git a/drivers/iio/dac/ad5421.c b/drivers/iio/dac/ad5421.c
> index 59a0cb5..1cb15d8 100644
> --- a/drivers/iio/dac/ad5421.c
> +++ b/drivers/iio/dac/ad5421.c
> @@ -458,9 +458,9 @@ static int ad5421_read_event_value(struct iio_dev *indio_dev,
> static const struct iio_info ad5421_info = {
> .read_raw = ad5421_read_raw,
> .write_raw = ad5421_write_raw,
> - .read_event_config_new = ad5421_read_event_config,
> - .write_event_config_new = ad5421_write_event_config,
> - .read_event_value_new = ad5421_read_event_value,
> + .read_event_config = ad5421_read_event_config,
> + .write_event_config = ad5421_write_event_config,
> + .read_event_value = ad5421_read_event_value,
> .driver_module = THIS_MODULE,
> };
>
> diff --git a/drivers/iio/industrialio-event.c b/drivers/iio/industrialio-event.c
> index bc043fa..c9c1419 100644
> --- a/drivers/iio/industrialio-event.c
> +++ b/drivers/iio/industrialio-event.c
> @@ -242,13 +242,9 @@ static ssize_t iio_ev_state_store(struct device *dev,
> if (ret < 0)
> return ret;
>
> - if (indio_dev->info->write_event_config)
> - ret = indio_dev->info->write_event_config(indio_dev,
> - this_attr->address, val);
> - else
> - ret = indio_dev->info->write_event_config_new(indio_dev,
> - this_attr->c, iio_ev_attr_type(this_attr),
> - iio_ev_attr_dir(this_attr), val);
> + ret = indio_dev->info->write_event_config(indio_dev,
> + this_attr->c, iio_ev_attr_type(this_attr),
> + iio_ev_attr_dir(this_attr), val);
>
> return (ret < 0) ? ret : len;
> }
> @@ -261,13 +257,9 @@ static ssize_t iio_ev_state_show(struct device *dev,
> struct iio_dev_attr *this_attr = to_iio_dev_attr(attr);
> int val;
>
> - if (indio_dev->info->read_event_config)
> - val = indio_dev->info->read_event_config(indio_dev,
> - this_attr->address);
> - else
> - val = indio_dev->info->read_event_config_new(indio_dev,
> - this_attr->c, iio_ev_attr_type(this_attr),
> - iio_ev_attr_dir(this_attr));
> + val = indio_dev->info->read_event_config(indio_dev,
> + this_attr->c, iio_ev_attr_type(this_attr),
> + iio_ev_attr_dir(this_attr));
> if (val < 0)
> return val;
> else
> @@ -283,21 +275,13 @@ static ssize_t iio_ev_value_show(struct device *dev,
> int val, val2;
> int ret;
>
> - if (indio_dev->info->read_event_value) {
> - ret = indio_dev->info->read_event_value(indio_dev,
> - this_attr->address, &val);
> - if (ret < 0)
> - return ret;
> - return sprintf(buf, "%d\n", val);
> - } else {
> - ret = indio_dev->info->read_event_value_new(indio_dev,
> - this_attr->c, iio_ev_attr_type(this_attr),
> - iio_ev_attr_dir(this_attr), iio_ev_attr_info(this_attr),
> - &val, &val2);
> - if (ret < 0)
> - return ret;
> - return iio_format_value(buf, ret, val, val2);
> - }
> + ret = indio_dev->info->read_event_value(indio_dev,
> + this_attr->c, iio_ev_attr_type(this_attr),
> + iio_ev_attr_dir(this_attr), iio_ev_attr_info(this_attr),
> + &val, &val2);
> + if (ret < 0)
> + return ret;
> + return iio_format_value(buf, ret, val, val2);
> }
>
> static ssize_t iio_ev_value_store(struct device *dev,
> @@ -310,25 +294,16 @@ static ssize_t iio_ev_value_store(struct device *dev,
> int val, val2;
> int ret;
>
> - if (!indio_dev->info->write_event_value &&
> - !indio_dev->info->write_event_value_new)
> + if (!indio_dev->info->write_event_value)
> return -EINVAL;
>
> - if (indio_dev->info->write_event_value) {
> - ret = kstrtoint(buf, 10, &val);
> - if (ret)
> - return ret;
> - ret = indio_dev->info->write_event_value(indio_dev,
> - this_attr->address, val);
> - } else {
> - ret = iio_str_to_fixpoint(buf, 100000, &val, &val2);
> - if (ret)
> - return ret;
> - ret = indio_dev->info->write_event_value_new(indio_dev,
> - this_attr->c, iio_ev_attr_type(this_attr),
> - iio_ev_attr_dir(this_attr), iio_ev_attr_info(this_attr),
> - val, val2);
> - }
> + ret = iio_str_to_fixpoint(buf, 100000, &val, &val2);
> + if (ret)
> + return ret;
> + ret = indio_dev->info->write_event_value(indio_dev,
> + this_attr->c, iio_ev_attr_type(this_attr),
> + iio_ev_attr_dir(this_attr), iio_ev_attr_info(this_attr),
> + val, val2);
> if (ret < 0)
> return ret;
>
> @@ -377,7 +352,7 @@ static int iio_device_add_event(struct iio_dev *indio_dev,
> return attrcount;
> }
>
> -static int iio_device_add_event_sysfs_new(struct iio_dev *indio_dev,
> +static int iio_device_add_event_sysfs(struct iio_dev *indio_dev,
> struct iio_chan_spec const *chan)
> {
> int ret = 0, i, attrcount = 0;
> @@ -420,89 +395,6 @@ error_ret:
> return ret;
> }
>
> -static int iio_device_add_event_sysfs_old(struct iio_dev *indio_dev,
> - struct iio_chan_spec const *chan)
> -{
> - int ret = 0, i, attrcount = 0;
> - u64 mask = 0;
> - char *postfix;
> - if (!chan->event_mask)
> - return 0;
> -
> - for_each_set_bit(i, &chan->event_mask, sizeof(chan->event_mask)*8) {
> - postfix = kasprintf(GFP_KERNEL, "%s_%s_en",
> - iio_ev_type_text[i/IIO_EV_DIR_MAX],
> - iio_ev_dir_text[i%IIO_EV_DIR_MAX]);
> - if (postfix == NULL) {
> - ret = -ENOMEM;
> - goto error_ret;
> - }
> - if (chan->modified)
> - mask = IIO_MOD_EVENT_CODE(chan->type, 0, chan->channel2,
> - i/IIO_EV_DIR_MAX,
> - i%IIO_EV_DIR_MAX);
> - else if (chan->differential)
> - mask = IIO_EVENT_CODE(chan->type,
> - 0, 0,
> - i%IIO_EV_DIR_MAX,
> - i/IIO_EV_DIR_MAX,
> - 0,
> - chan->channel,
> - chan->channel2);
> - else
> - mask = IIO_UNMOD_EVENT_CODE(chan->type,
> - chan->channel,
> - i/IIO_EV_DIR_MAX,
> - i%IIO_EV_DIR_MAX);
> -
> - ret = __iio_add_chan_devattr(postfix,
> - chan,
> - &iio_ev_state_show,
> - iio_ev_state_store,
> - mask,
> - 0,
> - &indio_dev->dev,
> - &indio_dev->event_interface->
> - dev_attr_list);
> - kfree(postfix);
> - if (ret)
> - goto error_ret;
> - attrcount++;
> - postfix = kasprintf(GFP_KERNEL, "%s_%s_value",
> - iio_ev_type_text[i/IIO_EV_DIR_MAX],
> - iio_ev_dir_text[i%IIO_EV_DIR_MAX]);
> - if (postfix == NULL) {
> - ret = -ENOMEM;
> - goto error_ret;
> - }
> - ret = __iio_add_chan_devattr(postfix, chan,
> - iio_ev_value_show,
> - iio_ev_value_store,
> - mask,
> - 0,
> - &indio_dev->dev,
> - &indio_dev->event_interface->
> - dev_attr_list);
> - kfree(postfix);
> - if (ret)
> - goto error_ret;
> - attrcount++;
> - }
> - ret = attrcount;
> -error_ret:
> - return ret;
> -}
> -
> -
> -static int iio_device_add_event_sysfs(struct iio_dev *indio_dev,
> - struct iio_chan_spec const *chan)
> -{
> - if (chan->event_mask)
> - return iio_device_add_event_sysfs_old(indio_dev, chan);
> - else
> - return iio_device_add_event_sysfs_new(indio_dev, chan);
> -}
> -
> static inline int __iio_add_event_config_attrs(struct iio_dev *indio_dev)
> {
> int j, ret, attrcount = 0;
> @@ -523,8 +415,6 @@ static bool iio_check_for_dynamic_events(struct iio_dev *indio_dev)
> int j;
>
> for (j = 0; j < indio_dev->num_channels; j++) {
> - if (indio_dev->channels[j].event_mask != 0)
> - return true;
> if (indio_dev->channels[j].num_event_specs != 0)
> return true;
> }
> diff --git a/drivers/iio/light/apds9300.c b/drivers/iio/light/apds9300.c
> index 51097bb..9ddde0c 100644
> --- a/drivers/iio/light/apds9300.c
> +++ b/drivers/iio/light/apds9300.c
> @@ -344,10 +344,10 @@ static const struct iio_info apds9300_info_no_irq = {
> static const struct iio_info apds9300_info = {
> .driver_module = THIS_MODULE,
> .read_raw = apds9300_read_raw,
> - .read_event_value_new = apds9300_read_thresh,
> - .write_event_value_new = apds9300_write_thresh,
> - .read_event_config_new = apds9300_read_interrupt_config,
> - .write_event_config_new = apds9300_write_interrupt_config,
> + .read_event_value = apds9300_read_thresh,
> + .write_event_value = apds9300_write_thresh,
> + .read_event_config = apds9300_read_interrupt_config,
> + .write_event_config = apds9300_write_interrupt_config,
> };
>
> static const struct iio_event_spec apds9300_event_spec[] = {
> diff --git a/drivers/iio/light/cm36651.c b/drivers/iio/light/cm36651.c
> index 2a0eb8d..000cc8e 100644
> --- a/drivers/iio/light/cm36651.c
> +++ b/drivers/iio/light/cm36651.c
> @@ -614,10 +614,10 @@ static const struct iio_info cm36651_info = {
> .driver_module = THIS_MODULE,
> .read_raw = &cm36651_read_raw,
> .write_raw = &cm36651_write_raw,
> - .read_event_value_new = &cm36651_read_prox_thresh,
> - .write_event_value_new = &cm36651_write_prox_thresh,
> - .read_event_config_new = &cm36651_read_prox_event_config,
> - .write_event_config_new = &cm36651_write_prox_event_config,
> + .read_event_value = &cm36651_read_prox_thresh,
> + .write_event_value = &cm36651_write_prox_thresh,
> + .read_event_config = &cm36651_read_prox_event_config,
> + .write_event_config = &cm36651_write_prox_event_config,
> .attrs = &cm36651_attribute_group,
> };
>
> diff --git a/drivers/iio/light/gp2ap020a00f.c b/drivers/iio/light/gp2ap020a00f.c
> index dc79835..5ea4a03 100644
> --- a/drivers/iio/light/gp2ap020a00f.c
> +++ b/drivers/iio/light/gp2ap020a00f.c
> @@ -1388,10 +1388,10 @@ static const struct iio_chan_spec gp2ap020a00f_channels[] = {
>
> static const struct iio_info gp2ap020a00f_info = {
> .read_raw = &gp2ap020a00f_read_raw,
> - .read_event_value_new = &gp2ap020a00f_read_event_val,
> - .read_event_config_new = &gp2ap020a00f_read_event_config,
> - .write_event_value_new = &gp2ap020a00f_write_event_val,
> - .write_event_config_new = &gp2ap020a00f_write_event_config,
> + .read_event_value = &gp2ap020a00f_read_event_val,
> + .read_event_config = &gp2ap020a00f_read_event_config,
> + .write_event_value = &gp2ap020a00f_write_event_val,
> + .write_event_config = &gp2ap020a00f_write_event_config,
> .driver_module = THIS_MODULE,
> };
>
> diff --git a/drivers/iio/light/tsl2563.c b/drivers/iio/light/tsl2563.c
> index 0c6e459..3d81101 100644
> --- a/drivers/iio/light/tsl2563.c
> +++ b/drivers/iio/light/tsl2563.c
> @@ -702,10 +702,10 @@ static const struct iio_info tsl2563_info = {
> .driver_module = THIS_MODULE,
> .read_raw = &tsl2563_read_raw,
> .write_raw = &tsl2563_write_raw,
> - .read_event_value_new = &tsl2563_read_thresh,
> - .write_event_value_new = &tsl2563_write_thresh,
> - .read_event_config_new = &tsl2563_read_interrupt_config,
> - .write_event_config_new = &tsl2563_write_interrupt_config,
> + .read_event_value = &tsl2563_read_thresh,
> + .write_event_value = &tsl2563_write_thresh,
> + .read_event_config = &tsl2563_read_interrupt_config,
> + .write_event_config = &tsl2563_write_interrupt_config,
> };
>
> static int tsl2563_probe(struct i2c_client *client,
> diff --git a/drivers/staging/iio/accel/lis3l02dq_core.c b/drivers/staging/iio/accel/lis3l02dq_core.c
> index 735c0a3..898653c 100644
> --- a/drivers/staging/iio/accel/lis3l02dq_core.c
> +++ b/drivers/staging/iio/accel/lis3l02dq_core.c
> @@ -676,10 +676,10 @@ static const struct attribute_group lis3l02dq_attribute_group = {
> static const struct iio_info lis3l02dq_info = {
> .read_raw = &lis3l02dq_read_raw,
> .write_raw = &lis3l02dq_write_raw,
> - .read_event_value_new = &lis3l02dq_read_thresh,
> - .write_event_value_new = &lis3l02dq_write_thresh,
> - .write_event_config_new = &lis3l02dq_write_event_config,
> - .read_event_config_new = &lis3l02dq_read_event_config,
> + .read_event_value = &lis3l02dq_read_thresh,
> + .write_event_value = &lis3l02dq_write_thresh,
> + .write_event_config = &lis3l02dq_write_event_config,
> + .read_event_config = &lis3l02dq_read_event_config,
> .driver_module = THIS_MODULE,
> .attrs = &lis3l02dq_attribute_group,
> };
> diff --git a/drivers/staging/iio/accel/sca3000_core.c b/drivers/staging/iio/accel/sca3000_core.c
> index c49e6ef..7f6ccdf 100644
> --- a/drivers/staging/iio/accel/sca3000_core.c
> +++ b/drivers/staging/iio/accel/sca3000_core.c
> @@ -1126,20 +1126,20 @@ static const struct iio_info sca3000_info = {
> .attrs = &sca3000_attribute_group,
> .read_raw = &sca3000_read_raw,
> .event_attrs = &sca3000_event_attribute_group,
> - .read_event_value_new = &sca3000_read_thresh,
> - .write_event_value_new = &sca3000_write_thresh,
> - .read_event_config_new = &sca3000_read_event_config,
> - .write_event_config_new = &sca3000_write_event_config,
> + .read_event_value = &sca3000_read_thresh,
> + .write_event_value = &sca3000_write_thresh,
> + .read_event_config = &sca3000_read_event_config,
> + .write_event_config = &sca3000_write_event_config,
> .driver_module = THIS_MODULE,
> };
>
> static const struct iio_info sca3000_info_with_temp = {
> .attrs = &sca3000_attribute_group_with_temp,
> .read_raw = &sca3000_read_raw,
> - .read_event_value_new = &sca3000_read_thresh,
> - .write_event_value_new = &sca3000_write_thresh,
> - .read_event_config_new = &sca3000_read_event_config,
> - .write_event_config_new = &sca3000_write_event_config,
> + .read_event_value = &sca3000_read_thresh,
> + .write_event_value = &sca3000_write_thresh,
> + .read_event_config = &sca3000_read_event_config,
> + .write_event_config = &sca3000_write_event_config,
> .driver_module = THIS_MODULE,
> };
>
> diff --git a/drivers/staging/iio/adc/ad7291.c b/drivers/staging/iio/adc/ad7291.c
> index d13f8ae..357cef2 100644
> --- a/drivers/staging/iio/adc/ad7291.c
> +++ b/drivers/staging/iio/adc/ad7291.c
> @@ -452,10 +452,10 @@ static const struct iio_chan_spec ad7291_channels[] = {
>
> static const struct iio_info ad7291_info = {
> .read_raw = &ad7291_read_raw,
> - .read_event_config_new = &ad7291_read_event_config,
> - .write_event_config_new = &ad7291_write_event_config,
> - .read_event_value_new = &ad7291_read_event_value,
> - .write_event_value_new = &ad7291_write_event_value,
> + .read_event_config = &ad7291_read_event_config,
> + .write_event_config = &ad7291_write_event_config,
> + .read_event_value = &ad7291_read_event_value,
> + .write_event_value = &ad7291_write_event_value,
> .driver_module = THIS_MODULE,
> };
>
> diff --git a/drivers/staging/iio/adc/ad799x_core.c b/drivers/staging/iio/adc/ad799x_core.c
> index 9428be8..5ea3641 100644
> --- a/drivers/staging/iio/adc/ad799x_core.c
> +++ b/drivers/staging/iio/adc/ad799x_core.c
> @@ -377,9 +377,9 @@ static const struct iio_info ad7991_info = {
> static const struct iio_info ad7993_4_7_8_info = {
> .read_raw = &ad799x_read_raw,
> .event_attrs = &ad799x_event_attrs_group,
> - .read_event_config_new = &ad799x_read_event_config,
> - .read_event_value_new = &ad799x_read_event_value,
> - .write_event_value_new = &ad799x_write_event_value,
> + .read_event_config = &ad799x_read_event_config,
> + .read_event_value = &ad799x_read_event_value,
> + .write_event_value = &ad799x_write_event_value,
> .driver_module = THIS_MODULE,
> .update_scan_mode = ad7997_8_update_scan_mode,
> };
> diff --git a/drivers/staging/iio/cdc/ad7150.c b/drivers/staging/iio/cdc/ad7150.c
> index 7e7f989..047af23 100644
> --- a/drivers/staging/iio/cdc/ad7150.c
> +++ b/drivers/staging/iio/cdc/ad7150.c
> @@ -576,10 +576,10 @@ static const struct iio_info ad7150_info = {
> .event_attrs = &ad7150_event_attribute_group,
> .driver_module = THIS_MODULE,
> .read_raw = &ad7150_read_raw,
> - .read_event_config_new = &ad7150_read_event_config,
> - .write_event_config_new = &ad7150_write_event_config,
> - .read_event_value_new = &ad7150_read_event_value,
> - .write_event_value_new = &ad7150_write_event_value,
> + .read_event_config = &ad7150_read_event_config,
> + .write_event_config = &ad7150_write_event_config,
> + .read_event_value = &ad7150_read_event_value,
> + .write_event_value = &ad7150_write_event_value,
> };
>
> /*
> diff --git a/drivers/staging/iio/iio_simple_dummy.c b/drivers/staging/iio/iio_simple_dummy.c
> index 1fac989..fd334a0 100644
> --- a/drivers/staging/iio/iio_simple_dummy.c
> +++ b/drivers/staging/iio/iio_simple_dummy.c
> @@ -370,10 +370,10 @@ static const struct iio_info iio_dummy_info = {
> .read_raw = &iio_dummy_read_raw,
> .write_raw = &iio_dummy_write_raw,
> #ifdef CONFIG_IIO_SIMPLE_DUMMY_EVENTS
> - .read_event_config_new = &iio_simple_dummy_read_event_config,
> - .write_event_config_new = &iio_simple_dummy_write_event_config,
> - .read_event_value_new = &iio_simple_dummy_read_event_value,
> - .write_event_value_new = &iio_simple_dummy_write_event_value,
> + .read_event_config = &iio_simple_dummy_read_event_config,
> + .write_event_config = &iio_simple_dummy_write_event_config,
> + .read_event_value = &iio_simple_dummy_read_event_value,
> + .write_event_value = &iio_simple_dummy_write_event_value,
> #endif /* CONFIG_IIO_SIMPLE_DUMMY_EVENTS */
> };
>
> diff --git a/drivers/staging/iio/light/tsl2x7x_core.c b/drivers/staging/iio/light/tsl2x7x_core.c
> index 1880502..1e53808 100644
> --- a/drivers/staging/iio/light/tsl2x7x_core.c
> +++ b/drivers/staging/iio/light/tsl2x7x_core.c
> @@ -1672,10 +1672,10 @@ static const struct iio_info tsl2X7X_device_info[] = {
> .driver_module = THIS_MODULE,
> .read_raw = &tsl2x7x_read_raw,
> .write_raw = &tsl2x7x_write_raw,
> - .read_event_value_new = &tsl2x7x_read_thresh,
> - .write_event_value_new = &tsl2x7x_write_thresh,
> - .read_event_config_new = &tsl2x7x_read_interrupt_config,
> - .write_event_config_new = &tsl2x7x_write_interrupt_config,
> + .read_event_value = &tsl2x7x_read_thresh,
> + .write_event_value = &tsl2x7x_write_thresh,
> + .read_event_config = &tsl2x7x_read_interrupt_config,
> + .write_event_config = &tsl2x7x_write_interrupt_config,
> },
> [PRX] = {
> .attrs = &tsl2X7X_device_attr_group_tbl[PRX],
> @@ -1683,10 +1683,10 @@ static const struct iio_info tsl2X7X_device_info[] = {
> .driver_module = THIS_MODULE,
> .read_raw = &tsl2x7x_read_raw,
> .write_raw = &tsl2x7x_write_raw,
> - .read_event_value_new = &tsl2x7x_read_thresh,
> - .write_event_value_new = &tsl2x7x_write_thresh,
> - .read_event_config_new = &tsl2x7x_read_interrupt_config,
> - .write_event_config_new = &tsl2x7x_write_interrupt_config,
> + .read_event_value = &tsl2x7x_read_thresh,
> + .write_event_value = &tsl2x7x_write_thresh,
> + .read_event_config = &tsl2x7x_read_interrupt_config,
> + .write_event_config = &tsl2x7x_write_interrupt_config,
> },
> [ALSPRX] = {
> .attrs = &tsl2X7X_device_attr_group_tbl[ALSPRX],
> @@ -1694,10 +1694,10 @@ static const struct iio_info tsl2X7X_device_info[] = {
> .driver_module = THIS_MODULE,
> .read_raw = &tsl2x7x_read_raw,
> .write_raw = &tsl2x7x_write_raw,
> - .read_event_value_new = &tsl2x7x_read_thresh,
> - .write_event_value_new = &tsl2x7x_write_thresh,
> - .read_event_config_new = &tsl2x7x_read_interrupt_config,
> - .write_event_config_new = &tsl2x7x_write_interrupt_config,
> + .read_event_value = &tsl2x7x_read_thresh,
> + .write_event_value = &tsl2x7x_write_thresh,
> + .read_event_config = &tsl2x7x_read_interrupt_config,
> + .write_event_config = &tsl2x7x_write_interrupt_config,
> },
> [PRX2] = {
> .attrs = &tsl2X7X_device_attr_group_tbl[PRX2],
> @@ -1705,10 +1705,10 @@ static const struct iio_info tsl2X7X_device_info[] = {
> .driver_module = THIS_MODULE,
> .read_raw = &tsl2x7x_read_raw,
> .write_raw = &tsl2x7x_write_raw,
> - .read_event_value_new = &tsl2x7x_read_thresh,
> - .write_event_value_new = &tsl2x7x_write_thresh,
> - .read_event_config_new = &tsl2x7x_read_interrupt_config,
> - .write_event_config_new = &tsl2x7x_write_interrupt_config,
> + .read_event_value = &tsl2x7x_read_thresh,
> + .write_event_value = &tsl2x7x_write_thresh,
> + .read_event_config = &tsl2x7x_read_interrupt_config,
> + .write_event_config = &tsl2x7x_write_interrupt_config,
> },
> [ALSPRX2] = {
> .attrs = &tsl2X7X_device_attr_group_tbl[ALSPRX2],
> @@ -1716,10 +1716,10 @@ static const struct iio_info tsl2X7X_device_info[] = {
> .driver_module = THIS_MODULE,
> .read_raw = &tsl2x7x_read_raw,
> .write_raw = &tsl2x7x_write_raw,
> - .read_event_value_new = &tsl2x7x_read_thresh,
> - .write_event_value_new = &tsl2x7x_write_thresh,
> - .read_event_config_new = &tsl2x7x_read_interrupt_config,
> - .write_event_config_new = &tsl2x7x_write_interrupt_config,
> + .read_event_value = &tsl2x7x_read_thresh,
> + .write_event_value = &tsl2x7x_write_thresh,
> + .read_event_config = &tsl2x7x_read_interrupt_config,
> + .write_event_config = &tsl2x7x_write_interrupt_config,
> },
> };
>
> diff --git a/include/linux/iio/events.h b/include/linux/iio/events.h
> index 5dab2c4..8bbd7bc 100644
> --- a/include/linux/iio/events.h
> +++ b/include/linux/iio/events.h
> @@ -46,10 +46,6 @@ struct iio_event_data {
> ((u16)chan))
>
>
> -#define IIO_EV_DIR_MAX 4
> -#define IIO_EV_BIT(type, direction) \
> - (1 << (type*IIO_EV_DIR_MAX + direction))
> -
> /**
> * IIO_MOD_EVENT_CODE() - create event identifier for modified channels
> * @chan_type: Type of the channel. Should be one of enum iio_chan_type.
> diff --git a/include/linux/iio/iio.h b/include/linux/iio/iio.h
> index 5b125fd..75a8a20 100644
> --- a/include/linux/iio/iio.h
> +++ b/include/linux/iio/iio.h
> @@ -185,7 +185,6 @@ struct iio_event_spec {
> * by all channels of the same direction.
> * @info_mask_shared_by_all: What information is to be exported that is shared
> * by all channels.
> - * @event_mask: What events can this channel produce.
> * @event_spec: Array of events which should be registered for this
> * channel.
> * @num_event_specs: Size of the event_spec array.
> @@ -226,7 +225,6 @@ struct iio_chan_spec {
> long info_mask_shared_by_type;
> long info_mask_shared_by_dir;
> long info_mask_shared_by_all;
> - long event_mask;
> const struct iio_event_spec *event_spec;
> unsigned int num_event_specs;
> const struct iio_chan_spec_ext_info *ext_info;
> @@ -307,16 +305,8 @@ struct iio_dev;
> * returns IIO_VAL_INT_PLUS_MICRO.
> * @read_event_config: find out if the event is enabled.
> * @write_event_config: set if the event is enabled.
> - * @read_event_value: read a value associated with the event. Meaning
> - * is event dependant. event_code specifies which event.
> - * @write_event_value: write the value associated with the event.
> - * Meaning is event dependent.
> - * @read_event_config_new: find out if the event is enabled. New style interface.
> - * @write_event_config_new: set if the event is enabled. New style interface.
> - * @read_event_value_new: read a configuration value associated with the event.
> - * New style interface.
> - * @write_event_value_new: write a configuration value for the event. New style
> - * interface.
> + * @read_event_value: read a configuration value associated with the event.
> + * @write_event_value: write a configuration value for the event.
> * @validate_trigger: function to validate the trigger when the
> * current trigger gets changed.
> * @update_scan_mode: function to configure device and scan buffer when
> @@ -345,37 +335,23 @@ struct iio_info {
> long mask);
>
> int (*read_event_config)(struct iio_dev *indio_dev,
> - u64 event_code);
> -
> - int (*write_event_config)(struct iio_dev *indio_dev,
> - u64 event_code,
> - int state);
> -
> - int (*read_event_value)(struct iio_dev *indio_dev,
> - u64 event_code,
> - int *val);
> - int (*write_event_value)(struct iio_dev *indio_dev,
> - u64 event_code,
> - int val);
> -
> - int (*read_event_config_new)(struct iio_dev *indio_dev,
> const struct iio_chan_spec *chan,
> enum iio_event_type type,
> enum iio_event_direction dir);
>
> - int (*write_event_config_new)(struct iio_dev *indio_dev,
> + int (*write_event_config)(struct iio_dev *indio_dev,
> const struct iio_chan_spec *chan,
> enum iio_event_type type,
> enum iio_event_direction dir,
> int state);
>
> - int (*read_event_value_new)(struct iio_dev *indio_dev,
> + int (*read_event_value)(struct iio_dev *indio_dev,
> 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);
>
> - int (*write_event_value_new)(struct iio_dev *indio_dev,
> + int (*write_event_value)(struct iio_dev *indio_dev,
> const struct iio_chan_spec *chan,
> enum iio_event_type type,
> enum iio_event_direction dir,
>
next prev parent reply other threads:[~2013-12-08 13:21 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-07 10:45 [PATCH 1/2] iio:cm36651: Convert to new event config interface Lars-Peter Clausen
2013-12-07 10:45 ` [PATCH 2/2] iio: Remove support for the legacy " Lars-Peter Clausen
2013-12-08 13:21 ` Jonathan Cameron [this message]
2013-12-08 13:12 ` [PATCH 1/2] iio:cm36651: Convert to new " Jonathan Cameron
2013-12-09 1:18 ` Beomho Seo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52A4723F.10302@kernel.org \
--to=jic23@kernel.org \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).