From ffab6de9c0e87fa8019d6a090d847b9017d2e015 Mon Sep 17 00:00:00 2001 From: Clément Bénier Date: Fri, 22 Jun 2018 15:49:47 +0200 Subject: Remove useless code and declare iio_infos in C code MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Remove treat_iio_infos function that is no used at all in source code. - it's really not a good idea to define static variable in header file (causing redefinition each time header file is included), so move static definition of iio_infos in C code. Change-Id: I66c6c5593c9d10581b49ce5f2a9b2f4952584259 Signed-off-by: Sebastien Douheret Signed-off-by: Clément Bénier --- README.md | 10 +++++----- binding/config_iiodevices.h | 9 +-------- binding/iiodevices-binding.c | 9 ++++++++- 3 files changed, 14 insertions(+), 14 deletions(-) diff --git a/README.md b/README.md index e712116..85f97e5 100644 --- a/README.md +++ b/README.md @@ -16,16 +16,16 @@ subscription. | Name | Description | JSON Parameters | |:-------------------|:--------------------------------------------|:--------------------------------------------------------------- | -| subscribe | subscribe to 9 axis events | *Request:* {"event": "accel", "args":"xy", "frequency": "10" }| -| unsubscribe | unsubscribe to accelero events | *Request:* {"event": "accel" } | +| subscribe | subscribe to 9 axis events | *Request:* {"event": "acceleration", "args":"xy", "frequency": "10" }| +| unsubscribe | unsubscribe to accelero events | *Request:* {"event": "acceleration" } | ## Events For now, there are 3 different events matching with the different available sensors. -* "accel": is for acceleration data -* "magn": is for gyroscope data -* "anglvel": is for electronic compass data +* "acceleration": is for acceleration data +* "gyroscope": is for gyroscope data +* "compass": is for electronic compass data ## Frequency diff --git a/binding/config_iiodevices.h b/binding/config_iiodevices.h index 9e99045..85189bc 100644 --- a/binding/config_iiodevices.h +++ b/binding/config_iiodevices.h @@ -9,20 +9,13 @@ struct iio_info { const char *dev_name; const char *id; -}; - -#define IIO_INFOS_SIZE 3 -static struct iio_info iio_infos[IIO_INFOS_SIZE] = { - { "16-001d", "accel"}, - { "16-001d", "magn"}, - { "16-006b", "anglvel"} + const char *middlename; }; enum iio_elements { X = 1, Y = 2, Z = 4 }; void set_channel_name(char *name, enum iio_elements iioelts); enum iio_elements treat_iio_elts(const char *iioelts_string); -enum iio_infos treat_iio_infos(const char *infos); int get_iio_nb(enum iio_elements iioelts); #endif //_CONFIG_IIODEVICES_ diff --git a/binding/iiodevices-binding.c b/binding/iiodevices-binding.c index 8e18fd0..affd4b9 100644 --- a/binding/iiodevices-binding.c +++ b/binding/iiodevices-binding.c @@ -58,6 +58,13 @@ static struct client_sub * clients = NULL; /*save last registered client*/ static struct client_sub * last_client = NULL; +/* Static definition of supported iiodevices */ +static struct iio_info iio_infos[] = { + { "16-001d", "acceleration", "accel"}, + { "16-001d", "gyroscope", "magn"}, + { "16-006b", "compass", "anglvel"} +}; + /*get event by afb_event*/ static struct event *event_get_event(const struct afb_event *event) { @@ -397,7 +404,7 @@ static struct channels* set_channel( chn->iioelts = i; /*set channel name with iio_elements*/ - strncpy(chn->name, client->infos->id, PATH_MAX); + strncpy(chn->name, client->infos->middlename, PATH_MAX); set_channel_name(chn->name, i); if(!(chn->chn = iio_device_find_channel(client->dev, chn->name, false))) { -- cgit 1.2.3-korg