[PATCH 1/3] staging:iio: Rename IIO_EVENT_CODE_EXTRACT_NUM to IIO_EVENT_CODE_EXTRACT_CHAN

[PATCH 1/3] staging:iio: Rename IIO_EVENT_CODE_EXTRACT_NUM to IIO_EVENT_CODE_EXTRACT_CHAN

[Lars-Peter Clausen]

We name this field "chan" throughout IIO with the exception of this one macro.
Rename it to be more consistent.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
---
 drivers/staging/iio/events.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/staging/iio/events.h b/drivers/staging/iio/events.h
index bfb6340..416800e 100644
--- a/drivers/staging/iio/events.h
+++ b/drivers/staging/iio/events.h
@@ -96,7 +96,7 @@ enum iio_event_direction {
 
 /* Event code number extraction depends on which type of event we have.
  * Perhaps review this function in the future*/
-#define IIO_EVENT_CODE_EXTRACT_NUM(mask) ((__s16)(mask & 0xFFFF))
+#define IIO_EVENT_CODE_EXTRACT_CHAN(mask) ((__s16)(mask & 0xFFFF))
 
 #define IIO_EVENT_CODE_EXTRACT_MODIFIER(mask) ((mask >> 40) & 0xFF)
 
-- 
1.7.7.3

[PATCH 2/3] staging:iio: Add missing event code extract macros

[Lars-Peter Clausen]

Add macros for extracting whether the event is for a differential channel and
the second channel number from the event code. These were the only two fields
which did not have such an macro yet.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
---
 drivers/staging/iio/events.h |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/drivers/staging/iio/events.h b/drivers/staging/iio/events.h
index 416800e..c25f0e3 100644
--- a/drivers/staging/iio/events.h
+++ b/drivers/staging/iio/events.h
@@ -97,7 +97,9 @@ enum iio_event_direction {
 /* Event code number extraction depends on which type of event we have.
  * Perhaps review this function in the future*/
 #define IIO_EVENT_CODE_EXTRACT_CHAN(mask) ((__s16)(mask & 0xFFFF))
+#define IIO_EVENT_CODE_EXTRACT_CHAN2(mask) ((__s16)(((mask) >> 16) & 0xFFFF))
 
 #define IIO_EVENT_CODE_EXTRACT_MODIFIER(mask) ((mask >> 40) & 0xFF)
+#define IIO_EVENT_CODE_EXTRACT_DIFF(mask) (((mask) >> 55) & 0x1)
 
 #endif
-- 
1.7.7.3

Re: [PATCH 2/3] staging:iio: Add missing event code extract macros

[Jonathan Cameron]

Lars-Peter Clausen <lars@metafoo.de> wrote:

>Add macros for extracting whether the event is for a differential
>channel and
>the second channel number from the event code. These were the only two
>fields
>which did not have such an macro yet.
>
>Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
 Acked-by: Jonathan Cameron <jic23@kernel.org> 
>---
> drivers/staging/iio/events.h |    2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
>diff --git a/drivers/staging/iio/events.h
>b/drivers/staging/iio/events.h
>index 416800e..c25f0e3 100644
>--- a/drivers/staging/iio/events.h
>+++ b/drivers/staging/iio/events.h
>@@ -97,7 +97,9 @@ enum iio_event_direction {
>/* Event code number extraction depends on which type of event we have.
>  * Perhaps review this function in the future*/
> #define IIO_EVENT_CODE_EXTRACT_CHAN(mask) ((__s16)(mask & 0xFFFF))
>+#define IIO_EVENT_CODE_EXTRACT_CHAN2(mask) ((__s16)(((mask) >> 16) &
>0xFFFF))
> 
> #define IIO_EVENT_CODE_EXTRACT_MODIFIER(mask) ((mask >> 40) & 0xFF)
>+#define IIO_EVENT_CODE_EXTRACT_DIFF(mask) (((mask) >> 55) & 0x1)
> 
> #endif
>-- 
>1.7.7.3

-- 
Sent from my Android phone with K-9 Mail. Please excuse my brevity.

[PATCH 3/3] staging:iio: Add event monitor example application

[Lars-Peter Clausen]

Add a small evtest like application to monitor events generated by an IIO
device. The application can be used as an example on how to listen for IIO
events and also is usful for testing and debugging device drivers which
generate IIO events.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
---
 .../staging/iio/Documentation/iio_event_monitor.c  |  241 ++++++++++++++++++++
 1 files changed, 241 insertions(+), 0 deletions(-)
 create mode 100644 drivers/staging/iio/Documentation/iio_event_monitor.c

diff --git a/drivers/staging/iio/Documentation/iio_event_monitor.c b/drivers/staging/iio/Documentation/iio_event_monitor.c
new file mode 100644
index 0000000..5ca9d05
--- /dev/null
+++ b/drivers/staging/iio/Documentation/iio_event_monitor.c
@@ -0,0 +1,241 @@
+/* Industrialio buffer test code.
+ *
+ * Copyright (c) 2011-2012 Lars-Peter Clausen <lars@metafoo.de>
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 as published by
+ * the Free Software Foundation.
+ *
+ * This program is primarily intended as an example application.
+ * Reads the current buffer setup from sysfs and starts a short capture
+ * from the specified device, pretty printing the result after appropriate
+ * conversion.
+ *
+ * Usage:
+ *	iio_event_monitor <device_name>
+ *
+ */
+
+#define _GNU_SOURCE
+
+#include <unistd.h>
+#include <stdbool.h>
+#include <stdio.h>
+#include <errno.h>
+#include <string.h>
+#include <poll.h>
+#include <fcntl.h>
+#include <sys/ioctl.h>
+#include "iio_utils.h"
+#include "../events.h"
+
+static const char * const iio_chan_type_name_spec[] = {
+	[IIO_VOLTAGE] = "voltage",
+	[IIO_CURRENT] = "current",
+	[IIO_POWER] = "power",
+	[IIO_ACCEL] = "accel",
+	[IIO_ANGL_VEL] = "anglvel",
+	[IIO_MAGN] = "magn",
+	[IIO_LIGHT] = "illuminance",
+	[IIO_INTENSITY] = "intensity",
+	[IIO_PROXIMITY] = "proximity",
+	[IIO_TEMP] = "temp",
+	[IIO_INCLI] = "incli",
+	[IIO_ROT] = "rot",
+	[IIO_ANGL] = "angl",
+	[IIO_TIMESTAMP] = "timestamp",
+	[IIO_CAPACITANCE] = "capacitance",
+};
+
+static const char * const iio_ev_type_text[] = {
+	[IIO_EV_TYPE_THRESH] = "thresh",
+	[IIO_EV_TYPE_MAG] = "mag",
+	[IIO_EV_TYPE_ROC] = "roc",
+	[IIO_EV_TYPE_THRESH_ADAPTIVE] = "thresh_adaptive",
+	[IIO_EV_TYPE_MAG_ADAPTIVE] = "mag_adaptive",
+};
+
+static const char * const iio_ev_dir_text[] = {
+	[IIO_EV_DIR_EITHER] = "either",
+	[IIO_EV_DIR_RISING] = "rising",
+	[IIO_EV_DIR_FALLING] = "falling"
+};
+
+static const char * const iio_modifier_names[] = {
+	[IIO_MOD_X] = "x",
+	[IIO_MOD_Y] = "y",
+	[IIO_MOD_Z] = "z",
+	[IIO_MOD_LIGHT_BOTH] = "both",
+	[IIO_MOD_LIGHT_IR] = "ir",
+};
+
+static bool event_is_known(struct iio_event_data *event)
+{
+	enum iio_chan_type type = IIO_EVENT_CODE_EXTRACT_CHAN_TYPE(event->id);
+	enum iio_modifier mod = IIO_EVENT_CODE_EXTRACT_MODIFIER(event->id);
+	enum iio_event_type ev_type = IIO_EVENT_CODE_EXTRACT_TYPE(event->id);
+	enum iio_event_direction dir = IIO_EVENT_CODE_EXTRACT_DIR(event->id);
+
+	switch (type) {
+	case IIO_VOLTAGE:
+	case IIO_CURRENT:
+	case IIO_POWER:
+	case IIO_ACCEL:
+	case IIO_ANGL_VEL:
+	case IIO_MAGN:
+	case IIO_LIGHT:
+	case IIO_INTENSITY:
+	case IIO_PROXIMITY:
+	case IIO_TEMP:
+	case IIO_INCLI:
+	case IIO_ROT:
+	case IIO_ANGL:
+	case IIO_TIMESTAMP:
+	case IIO_CAPACITANCE:
+		break;
+	default:
+		return false;
+	}
+
+	switch (mod) {
+	case IIO_NO_MOD:
+	case IIO_MOD_X:
+	case IIO_MOD_Y:
+	case IIO_MOD_Z:
+	case IIO_MOD_LIGHT_BOTH:
+	case IIO_MOD_LIGHT_IR:
+		break;
+	default:
+		return false;
+	}
+
+	switch (ev_type) {
+	case IIO_EV_TYPE_THRESH:
+	case IIO_EV_TYPE_MAG:
+	case IIO_EV_TYPE_ROC:
+	case IIO_EV_TYPE_THRESH_ADAPTIVE:
+	case IIO_EV_TYPE_MAG_ADAPTIVE:
+		break;
+	default:
+		return false;
+	}
+
+	switch (dir) {
+	case IIO_EV_DIR_EITHER:
+	case IIO_EV_DIR_RISING:
+	case IIO_EV_DIR_FALLING:
+		break;
+	default:
+		return false;
+	}
+
+	return true;
+}
+
+static void print_event(struct iio_event_data *event)
+{
+	enum iio_chan_type type = IIO_EVENT_CODE_EXTRACT_CHAN_TYPE(event->id);
+	enum iio_modifier mod = IIO_EVENT_CODE_EXTRACT_MODIFIER(event->id);
+	enum iio_event_type ev_type = IIO_EVENT_CODE_EXTRACT_TYPE(event->id);
+	enum iio_event_direction dir = IIO_EVENT_CODE_EXTRACT_DIR(event->id);
+	int chan = IIO_EVENT_CODE_EXTRACT_CHAN(event->id);
+	int chan2 = IIO_EVENT_CODE_EXTRACT_CHAN2(event->id);
+	bool diff = IIO_EVENT_CODE_EXTRACT_DIFF(event->id);
+
+	if (!event_is_known(event)) {
+		printf("Unknown event: time: %lld, id: %llx\n",
+				event->timestamp, event->id);
+		return;
+	}
+
+	printf("Event: time: %lld, ", event->timestamp);
+
+	if (mod != IIO_NO_MOD) {
+		printf("type: %s(%s), ",
+			iio_chan_type_name_spec[type],
+			iio_modifier_names[mod]);
+	} else {
+		printf("type: %s, ",
+			iio_chan_type_name_spec[type]);
+	}
+
+	if (diff && chan >= 0 && chan2 >= 0)
+		printf("channel: %d-%d, ", chan, chan2);
+	else if (chan >= 0)
+		printf("channel: %d, ", chan);
+
+	printf("evtype: %s, direction: %s\n",
+		iio_ev_type_text[ev_type],
+		iio_ev_dir_text[dir]);
+}
+
+int main(int argc, char **argv)
+{
+	struct iio_event_data event;
+	const char *device_name;
+	char *chrdev_name;
+	int ret;
+	int dev_num;
+	int fd, event_fd;
+
+	if (argc <= 1) {
+		printf("Usage: %s <device_name>\n", argv[0]);
+		return -1;
+	}
+
+	device_name = argv[1];
+
+	dev_num = find_type_by_name(device_name, "iio:device");
+	if (dev_num >= 0) {
+		printf("Found IIO device with name %s with device number %d\n",
+			device_name, dev_num);
+		ret = asprintf(&chrdev_name, "/dev/iio:device%d", dev_num);
+		if (ret < 0) {
+			ret = -ENOMEM;
+			goto error_ret;
+		}
+	} else {
+		/* If we can't find a IIO device by name assume device_name is a
+		   IIO chrdev */
+		chrdev_name = strdup(device_name);
+	}
+
+	fd = open(chrdev_name, 0);
+	if (fd == -1) {
+		fprintf(stdout, "Failed to open %s\n", chrdev_name);
+		ret = -errno;
+		goto error_free_chrdev_name;
+	}
+
+	ret = ioctl(fd, IIO_GET_EVENT_FD_IOCTL, &event_fd);
+
+	close(fd);
+
+	if (ret == -1 || event_fd == -1) {
+		fprintf(stdout, "Failed to retrieve event fd\n");
+		ret = -errno;
+		goto error_free_chrdev_name;
+	}
+
+	while (true) {
+		ret = read(event_fd, &event, sizeof(event));
+		if (ret == -1) {
+			if (errno == EAGAIN) {
+				printf("nothing available\n");
+				continue;
+			} else {
+				perror("Failed to read event from device");
+				ret = -errno;
+				break;
+			}
+		}
+
+		print_event(&event);
+	}
+
+	close(event_fd);
+error_free_chrdev_name:
+	free(chrdev_name);
+error_ret:
+	return ret;
+}
-- 
1.7.7.3

Re: [PATCH 3/3] staging:iio: Add event monitor example application

[Lars-Peter Clausen]

On 01/03/2012 03:54 PM, Lars-Peter Clausen wrote:
> Add a small evtest like application to monitor events generated by an IIO
> device. The application can be used as an example on how to listen for IIO
> events and also is usful for testing and debugging device drivers which
> generate IIO events.
> 
> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>

Hi Jonathon,

You acked patch 1 and 2 of this series, but didn't comment on this one yet.

Thanks
- Lars

> ---
>  .../staging/iio/Documentation/iio_event_monitor.c  |  241 ++++++++++++++++++++
>  1 files changed, 241 insertions(+), 0 deletions(-)
>  create mode 100644 drivers/staging/iio/Documentation/iio_event_monitor.c
> 
> diff --git a/drivers/staging/iio/Documentation/iio_event_monitor.c b/drivers/staging/iio/Documentation/iio_event_monitor.c
> new file mode 100644
> index 0000000..5ca9d05
> --- /dev/null
> +++ b/drivers/staging/iio/Documentation/iio_event_monitor.c
> @@ -0,0 +1,241 @@
> +/* Industrialio buffer test code.
> + *
> + * Copyright (c) 2011-2012 Lars-Peter Clausen <lars@metafoo.de>
> + *
> + * This program is free software; you can redistribute it and/or modify it
> + * under the terms of the GNU General Public License version 2 as published by
> + * the Free Software Foundation.
> + *
> + * This program is primarily intended as an example application.
> + * Reads the current buffer setup from sysfs and starts a short capture
> + * from the specified device, pretty printing the result after appropriate
> + * conversion.
> + *
> + * Usage:
> + *	iio_event_monitor <device_name>
> + *
> + */
> +
> +#define _GNU_SOURCE
> +
> +#include <unistd.h>
> +#include <stdbool.h>
> +#include <stdio.h>
> +#include <errno.h>
> +#include <string.h>
> +#include <poll.h>
> +#include <fcntl.h>
> +#include <sys/ioctl.h>
> +#include "iio_utils.h"
> +#include "../events.h"
> +
> +static const char * const iio_chan_type_name_spec[] = {
> +	[IIO_VOLTAGE] = "voltage",
> +	[IIO_CURRENT] = "current",
> +	[IIO_POWER] = "power",
> +	[IIO_ACCEL] = "accel",
> +	[IIO_ANGL_VEL] = "anglvel",
> +	[IIO_MAGN] = "magn",
> +	[IIO_LIGHT] = "illuminance",
> +	[IIO_INTENSITY] = "intensity",
> +	[IIO_PROXIMITY] = "proximity",
> +	[IIO_TEMP] = "temp",
> +	[IIO_INCLI] = "incli",
> +	[IIO_ROT] = "rot",
> +	[IIO_ANGL] = "angl",
> +	[IIO_TIMESTAMP] = "timestamp",
> +	[IIO_CAPACITANCE] = "capacitance",
> +};
> +
> +static const char * const iio_ev_type_text[] = {
> +	[IIO_EV_TYPE_THRESH] = "thresh",
> +	[IIO_EV_TYPE_MAG] = "mag",
> +	[IIO_EV_TYPE_ROC] = "roc",
> +	[IIO_EV_TYPE_THRESH_ADAPTIVE] = "thresh_adaptive",
> +	[IIO_EV_TYPE_MAG_ADAPTIVE] = "mag_adaptive",
> +};
> +
> +static const char * const iio_ev_dir_text[] = {
> +	[IIO_EV_DIR_EITHER] = "either",
> +	[IIO_EV_DIR_RISING] = "rising",
> +	[IIO_EV_DIR_FALLING] = "falling"
> +};
> +
> +static const char * const iio_modifier_names[] = {
> +	[IIO_MOD_X] = "x",
> +	[IIO_MOD_Y] = "y",
> +	[IIO_MOD_Z] = "z",
> +	[IIO_MOD_LIGHT_BOTH] = "both",
> +	[IIO_MOD_LIGHT_IR] = "ir",
> +};
> +
> +static bool event_is_known(struct iio_event_data *event)
> +{
> +	enum iio_chan_type type = IIO_EVENT_CODE_EXTRACT_CHAN_TYPE(event->id);
> +	enum iio_modifier mod = IIO_EVENT_CODE_EXTRACT_MODIFIER(event->id);
> +	enum iio_event_type ev_type = IIO_EVENT_CODE_EXTRACT_TYPE(event->id);
> +	enum iio_event_direction dir = IIO_EVENT_CODE_EXTRACT_DIR(event->id);
> +
> +	switch (type) {
> +	case IIO_VOLTAGE:
> +	case IIO_CURRENT:
> +	case IIO_POWER:
> +	case IIO_ACCEL:
> +	case IIO_ANGL_VEL:
> +	case IIO_MAGN:
> +	case IIO_LIGHT:
> +	case IIO_INTENSITY:
> +	case IIO_PROXIMITY:
> +	case IIO_TEMP:
> +	case IIO_INCLI:
> +	case IIO_ROT:
> +	case IIO_ANGL:
> +	case IIO_TIMESTAMP:
> +	case IIO_CAPACITANCE:
> +		break;
> +	default:
> +		return false;
> +	}
> +
> +	switch (mod) {
> +	case IIO_NO_MOD:
> +	case IIO_MOD_X:
> +	case IIO_MOD_Y:
> +	case IIO_MOD_Z:
> +	case IIO_MOD_LIGHT_BOTH:
> +	case IIO_MOD_LIGHT_IR:
> +		break;
> +	default:
> +		return false;
> +	}
> +
> +	switch (ev_type) {
> +	case IIO_EV_TYPE_THRESH:
> +	case IIO_EV_TYPE_MAG:
> +	case IIO_EV_TYPE_ROC:
> +	case IIO_EV_TYPE_THRESH_ADAPTIVE:
> +	case IIO_EV_TYPE_MAG_ADAPTIVE:
> +		break;
> +	default:
> +		return false;
> +	}
> +
> +	switch (dir) {
> +	case IIO_EV_DIR_EITHER:
> +	case IIO_EV_DIR_RISING:
> +	case IIO_EV_DIR_FALLING:
> +		break;
> +	default:
> +		return false;
> +	}
> +
> +	return true;
> +}
> +
> +static void print_event(struct iio_event_data *event)
> +{
> +	enum iio_chan_type type = IIO_EVENT_CODE_EXTRACT_CHAN_TYPE(event->id);
> +	enum iio_modifier mod = IIO_EVENT_CODE_EXTRACT_MODIFIER(event->id);
> +	enum iio_event_type ev_type = IIO_EVENT_CODE_EXTRACT_TYPE(event->id);
> +	enum iio_event_direction dir = IIO_EVENT_CODE_EXTRACT_DIR(event->id);
> +	int chan = IIO_EVENT_CODE_EXTRACT_CHAN(event->id);
> +	int chan2 = IIO_EVENT_CODE_EXTRACT_CHAN2(event->id);
> +	bool diff = IIO_EVENT_CODE_EXTRACT_DIFF(event->id);
> +
> +	if (!event_is_known(event)) {
> +		printf("Unknown event: time: %lld, id: %llx\n",
> +				event->timestamp, event->id);
> +		return;
> +	}
> +
> +	printf("Event: time: %lld, ", event->timestamp);
> +
> +	if (mod != IIO_NO_MOD) {
> +		printf("type: %s(%s), ",
> +			iio_chan_type_name_spec[type],
> +			iio_modifier_names[mod]);
> +	} else {
> +		printf("type: %s, ",
> +			iio_chan_type_name_spec[type]);
> +	}
> +
> +	if (diff && chan >= 0 && chan2 >= 0)
> +		printf("channel: %d-%d, ", chan, chan2);
> +	else if (chan >= 0)
> +		printf("channel: %d, ", chan);
> +
> +	printf("evtype: %s, direction: %s\n",
> +		iio_ev_type_text[ev_type],
> +		iio_ev_dir_text[dir]);
> +}
> +
> +int main(int argc, char **argv)
> +{
> +	struct iio_event_data event;
> +	const char *device_name;
> +	char *chrdev_name;
> +	int ret;
> +	int dev_num;
> +	int fd, event_fd;
> +
> +	if (argc <= 1) {
> +		printf("Usage: %s <device_name>\n", argv[0]);
> +		return -1;
> +	}
> +
> +	device_name = argv[1];
> +
> +	dev_num = find_type_by_name(device_name, "iio:device");
> +	if (dev_num >= 0) {
> +		printf("Found IIO device with name %s with device number %d\n",
> +			device_name, dev_num);
> +		ret = asprintf(&chrdev_name, "/dev/iio:device%d", dev_num);
> +		if (ret < 0) {
> +			ret = -ENOMEM;
> +			goto error_ret;
> +		}
> +	} else {
> +		/* If we can't find a IIO device by name assume device_name is a
> +		   IIO chrdev */
> +		chrdev_name = strdup(device_name);
> +	}
> +
> +	fd = open(chrdev_name, 0);
> +	if (fd == -1) {
> +		fprintf(stdout, "Failed to open %s\n", chrdev_name);
> +		ret = -errno;
> +		goto error_free_chrdev_name;
> +	}
> +
> +	ret = ioctl(fd, IIO_GET_EVENT_FD_IOCTL, &event_fd);
> +
> +	close(fd);
> +
> +	if (ret == -1 || event_fd == -1) {
> +		fprintf(stdout, "Failed to retrieve event fd\n");
> +		ret = -errno;
> +		goto error_free_chrdev_name;
> +	}
> +
> +	while (true) {
> +		ret = read(event_fd, &event, sizeof(event));
> +		if (ret == -1) {
> +			if (errno == EAGAIN) {
> +				printf("nothing available\n");
> +				continue;
> +			} else {
> +				perror("Failed to read event from device");
> +				ret = -errno;
> +				break;
> +			}
> +		}
> +
> +		print_event(&event);
> +	}
> +
> +	close(event_fd);
> +error_free_chrdev_name:
> +	free(chrdev_name);
> +error_ret:
> +	return ret;
> +}

Re: [PATCH 3/3] staging:iio: Add event monitor example application

[Jonathan Cameron]

Lars-Peter Clausen <lars@metafoo.de> wrote:

>On 01/03/2012 03:54 PM, Lars-Peter Clausen wrote:
>> Add a small evtest like application to monitor events generated by an
>IIO
>> device. The application can be used as an example on how to listen
>for IIO
>> events and also is usful for testing and debugging device drivers
>which
>> generate IIO events.
>> 
>> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
>
>Hi Jonathon,
>
>You acked patch 1 and 2 of this series, but didn't comment on this one
>yet.

Looks good. I wanted to test it but never quite got to round to doing so.  Have heard from elsewhere that it works well so happy to see it go in. 

Acked-by: Jonathan Cameron <jic23@kernel.org>
>
>Thanks
>- Lars
>
>> ---
>>  .../staging/iio/Documentation/iio_event_monitor.c  |  241
>++++++++++++++++++++
>>  1 files changed, 241 insertions(+), 0 deletions(-)
>>  create mode 100644
>drivers/staging/iio/Documentation/iio_event_monitor.c
>> 
>> diff --git a/drivers/staging/iio/Documentation/iio_event_monitor.c
>b/drivers/staging/iio/Documentation/iio_event_monitor.c
>> new file mode 100644
>> index 0000000..5ca9d05
>> --- /dev/null
>> +++ b/drivers/staging/iio/Documentation/iio_event_monitor.c
>> @@ -0,0 +1,241 @@
>> +/* Industrialio buffer test code.
>> + *
>> + * Copyright (c) 2011-2012 Lars-Peter Clausen <lars@metafoo.de>
>> + *
>> + * This program is free software; you can redistribute it and/or
>modify it
>> + * under the terms of the GNU General Public License version 2 as
>published by
>> + * the Free Software Foundation.
>> + *
>> + * This program is primarily intended as an example application.
>> + * Reads the current buffer setup from sysfs and starts a short
>capture
>> + * from the specified device, pretty printing the result after
>appropriate
>> + * conversion.
>> + *
>> + * Usage:
>> + *	iio_event_monitor <device_name>
>> + *
>> + */
>> +
>> +#define _GNU_SOURCE
>> +
>> +#include <unistd.h>
>> +#include <stdbool.h>
>> +#include <stdio.h>
>> +#include <errno.h>
>> +#include <string.h>
>> +#include <poll.h>
>> +#include <fcntl.h>
>> +#include <sys/ioctl.h>
>> +#include "iio_utils.h"
>> +#include "../events.h"
>> +
>> +static const char * const iio_chan_type_name_spec[] = {
>> +	[IIO_VOLTAGE] = "voltage",
>> +	[IIO_CURRENT] = "current",
>> +	[IIO_POWER] = "power",
>> +	[IIO_ACCEL] = "accel",
>> +	[IIO_ANGL_VEL] = "anglvel",
>> +	[IIO_MAGN] = "magn",
>> +	[IIO_LIGHT] = "illuminance",
>> +	[IIO_INTENSITY] = "intensity",
>> +	[IIO_PROXIMITY] = "proximity",
>> +	[IIO_TEMP] = "temp",
>> +	[IIO_INCLI] = "incli",
>> +	[IIO_ROT] = "rot",
>> +	[IIO_ANGL] = "angl",
>> +	[IIO_TIMESTAMP] = "timestamp",
>> +	[IIO_CAPACITANCE] = "capacitance",
>> +};
>> +
>> +static const char * const iio_ev_type_text[] = {
>> +	[IIO_EV_TYPE_THRESH] = "thresh",
>> +	[IIO_EV_TYPE_MAG] = "mag",
>> +	[IIO_EV_TYPE_ROC] = "roc",
>> +	[IIO_EV_TYPE_THRESH_ADAPTIVE] = "thresh_adaptive",
>> +	[IIO_EV_TYPE_MAG_ADAPTIVE] = "mag_adaptive",
>> +};
>> +
>> +static const char * const iio_ev_dir_text[] = {
>> +	[IIO_EV_DIR_EITHER] = "either",
>> +	[IIO_EV_DIR_RISING] = "rising",
>> +	[IIO_EV_DIR_FALLING] = "falling"
>> +};
>> +
>> +static const char * const iio_modifier_names[] = {
>> +	[IIO_MOD_X] = "x",
>> +	[IIO_MOD_Y] = "y",
>> +	[IIO_MOD_Z] = "z",
>> +	[IIO_MOD_LIGHT_BOTH] = "both",
>> +	[IIO_MOD_LIGHT_IR] = "ir",
>> +};
>> +
>> +static bool event_is_known(struct iio_event_data *event)
>> +{
>> +	enum iio_chan_type type =
>IIO_EVENT_CODE_EXTRACT_CHAN_TYPE(event->id);
>> +	enum iio_modifier mod = IIO_EVENT_CODE_EXTRACT_MODIFIER(event->id);
>> +	enum iio_event_type ev_type =
>IIO_EVENT_CODE_EXTRACT_TYPE(event->id);
>> +	enum iio_event_direction dir =
>IIO_EVENT_CODE_EXTRACT_DIR(event->id);
>> +
>> +	switch (type) {
>> +	case IIO_VOLTAGE:
>> +	case IIO_CURRENT:
>> +	case IIO_POWER:
>> +	case IIO_ACCEL:
>> +	case IIO_ANGL_VEL:
>> +	case IIO_MAGN:
>> +	case IIO_LIGHT:
>> +	case IIO_INTENSITY:
>> +	case IIO_PROXIMITY:
>> +	case IIO_TEMP:
>> +	case IIO_INCLI:
>> +	case IIO_ROT:
>> +	case IIO_ANGL:
>> +	case IIO_TIMESTAMP:
>> +	case IIO_CAPACITANCE:
>> +		break;
>> +	default:
>> +		return false;
>> +	}
>> +
>> +	switch (mod) {
>> +	case IIO_NO_MOD:
>> +	case IIO_MOD_X:
>> +	case IIO_MOD_Y:
>> +	case IIO_MOD_Z:
>> +	case IIO_MOD_LIGHT_BOTH:
>> +	case IIO_MOD_LIGHT_IR:
>> +		break;
>> +	default:
>> +		return false;
>> +	}
>> +
>> +	switch (ev_type) {
>> +	case IIO_EV_TYPE_THRESH:
>> +	case IIO_EV_TYPE_MAG:
>> +	case IIO_EV_TYPE_ROC:
>> +	case IIO_EV_TYPE_THRESH_ADAPTIVE:
>> +	case IIO_EV_TYPE_MAG_ADAPTIVE:
>> +		break;
>> +	default:
>> +		return false;
>> +	}
>> +
>> +	switch (dir) {
>> +	case IIO_EV_DIR_EITHER:
>> +	case IIO_EV_DIR_RISING:
>> +	case IIO_EV_DIR_FALLING:
>> +		break;
>> +	default:
>> +		return false;
>> +	}
>> +
>> +	return true;
>> +}
>> +
>> +static void print_event(struct iio_event_data *event)
>> +{
>> +	enum iio_chan_type type =
>IIO_EVENT_CODE_EXTRACT_CHAN_TYPE(event->id);
>> +	enum iio_modifier mod = IIO_EVENT_CODE_EXTRACT_MODIFIER(event->id);
>> +	enum iio_event_type ev_type =
>IIO_EVENT_CODE_EXTRACT_TYPE(event->id);
>> +	enum iio_event_direction dir =
>IIO_EVENT_CODE_EXTRACT_DIR(event->id);
>> +	int chan = IIO_EVENT_CODE_EXTRACT_CHAN(event->id);
>> +	int chan2 = IIO_EVENT_CODE_EXTRACT_CHAN2(event->id);
>> +	bool diff = IIO_EVENT_CODE_EXTRACT_DIFF(event->id);
>> +
>> +	if (!event_is_known(event)) {
>> +		printf("Unknown event: time: %lld, id: %llx\n",
>> +				event->timestamp, event->id);
>> +		return;
>> +	}
>> +
>> +	printf("Event: time: %lld, ", event->timestamp);
>> +
>> +	if (mod != IIO_NO_MOD) {
>> +		printf("type: %s(%s), ",
>> +			iio_chan_type_name_spec[type],
>> +			iio_modifier_names[mod]);
>> +	} else {
>> +		printf("type: %s, ",
>> +			iio_chan_type_name_spec[type]);
>> +	}
>> +
>> +	if (diff && chan >= 0 && chan2 >= 0)
>> +		printf("channel: %d-%d, ", chan, chan2);
>> +	else if (chan >= 0)
>> +		printf("channel: %d, ", chan);
>> +
>> +	printf("evtype: %s, direction: %s\n",
>> +		iio_ev_type_text[ev_type],
>> +		iio_ev_dir_text[dir]);
>> +}
>> +
>> +int main(int argc, char **argv)
>> +{
>> +	struct iio_event_data event;
>> +	const char *device_name;
>> +	char *chrdev_name;
>> +	int ret;
>> +	int dev_num;
>> +	int fd, event_fd;
>> +
>> +	if (argc <= 1) {
>> +		printf("Usage: %s <device_name>\n", argv[0]);
>> +		return -1;
>> +	}
>> +
>> +	device_name = argv[1];
>> +
>> +	dev_num = find_type_by_name(device_name, "iio:device");
>> +	if (dev_num >= 0) {
>> +		printf("Found IIO device with name %s with device number %d\n",
>> +			device_name, dev_num);
>> +		ret = asprintf(&chrdev_name, "/dev/iio:device%d", dev_num);
>> +		if (ret < 0) {
>> +			ret = -ENOMEM;
>> +			goto error_ret;
>> +		}
>> +	} else {
>> +		/* If we can't find a IIO device by name assume device_name is a
>> +		   IIO chrdev */
>> +		chrdev_name = strdup(device_name);
>> +	}
>> +
>> +	fd = open(chrdev_name, 0);
>> +	if (fd == -1) {
>> +		fprintf(stdout, "Failed to open %s\n", chrdev_name);
>> +		ret = -errno;
>> +		goto error_free_chrdev_name;
>> +	}
>> +
>> +	ret = ioctl(fd, IIO_GET_EVENT_FD_IOCTL, &event_fd);
>> +
>> +	close(fd);
>> +
>> +	if (ret == -1 || event_fd == -1) {
>> +		fprintf(stdout, "Failed to retrieve event fd\n");
>> +		ret = -errno;
>> +		goto error_free_chrdev_name;
>> +	}
>> +
>> +	while (true) {
>> +		ret = read(event_fd, &event, sizeof(event));
>> +		if (ret == -1) {
>> +			if (errno == EAGAIN) {
>> +				printf("nothing available\n");
>> +				continue;
>> +			} else {
>> +				perror("Failed to read event from device");
>> +				ret = -errno;
>> +				break;
>> +			}
>> +		}
>> +
>> +		print_event(&event);
>> +	}
>> +
>> +	close(event_fd);
>> +error_free_chrdev_name:
>> +	free(chrdev_name);
>> +error_ret:
>> +	return ret;
>> +}
>
>--
>To unsubscribe from this list: send the line "unsubscribe linux-iio" in
>the body of a message to majordomo@vger.kernel.org
>More majordomo info at  http://vger.kernel.org/majordomo-info.html

-- 
Sent from my Android phone with K-9 Mail. Please excuse my brevity.

Re: [PATCH 1/3] staging:iio: Rename IIO_EVENT_CODE_EXTRACT_NUM to IIO_EVENT_CODE_EXTRACT_CHAN

[Jonathan Cameron]

Sensible. 

Lars-Peter Clausen <lars@metafoo.de> wrote:

>We name this field "chan" throughout IIO with the exception of this one
>macro.
>Rename it to be more consistent.
>
>Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
 Acked-by: Jonathan Cameron <jic23@kernel.org> 
>---
> drivers/staging/iio/events.h |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
>diff --git a/drivers/staging/iio/events.h
>b/drivers/staging/iio/events.h
>index bfb6340..416800e 100644
>--- a/drivers/staging/iio/events.h
>+++ b/drivers/staging/iio/events.h
>@@ -96,7 +96,7 @@ enum iio_event_direction {
> 
>/* Event code number extraction depends on which type of event we have.
>  * Perhaps review this function in the future*/
>-#define IIO_EVENT_CODE_EXTRACT_NUM(mask) ((__s16)(mask & 0xFFFF))
>+#define IIO_EVENT_CODE_EXTRACT_CHAN(mask) ((__s16)(mask & 0xFFFF))
> 
> #define IIO_EVENT_CODE_EXTRACT_MODIFIER(mask) ((mask >> 40) & 0xFF)
> 
>-- 
>1.7.7.3

-- 
Sent from my Android phone with K-9 Mail. Please excuse my brevity.