Commit 9fbfb4b3 authored by Srinivas Pandruvada's avatar Srinivas Pandruvada Committed by Jonathan Cameron

IIO: core: Introduce read_raw_multi

This callback is introduced to overcome some limitations of existing
read_raw callback. The functionality of both existing read_raw and
read_raw_multi is similar, both are used to request values from the
device. The current read_raw callback allows only two return values.
The new read_raw_multi allows returning multiple values. Instead of
passing just address of val and val2, it passes length and pointer
to values. Depending on the type and length of passed buffer, iio
client drivers can return multiple values.
Signed-off-by: default avatarSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Signed-off-by: default avatarJonathan Cameron <jic23@kernel.org>
parent 3046365b
...@@ -35,7 +35,7 @@ int __iio_add_chan_devattr(const char *postfix, ...@@ -35,7 +35,7 @@ int __iio_add_chan_devattr(const char *postfix,
struct list_head *attr_list); struct list_head *attr_list);
void iio_free_chan_devattr_list(struct list_head *attr_list); void iio_free_chan_devattr_list(struct list_head *attr_list);
ssize_t iio_format_value(char *buf, unsigned int type, int val, int val2); ssize_t iio_format_value(char *buf, unsigned int type, int size, int *vals);
/* Event interface flags */ /* Event interface flags */
#define IIO_BUSY_BIT_POS 1 #define IIO_BUSY_BIT_POS 1
......
...@@ -373,41 +373,53 @@ EXPORT_SYMBOL_GPL(iio_enum_write); ...@@ -373,41 +373,53 @@ EXPORT_SYMBOL_GPL(iio_enum_write);
* @buf: The buffer to which the formated value gets written * @buf: The buffer to which the formated value gets written
* @type: One of the IIO_VAL_... constants. This decides how the val and val2 * @type: One of the IIO_VAL_... constants. This decides how the val and val2
* parameters are formatted. * parameters are formatted.
* @val: First part of the value, exact meaning depends on the type parameter. * @vals: pointer to the values, exact meaning depends on the type parameter.
* @val2: Second part of the value, exact meaning depends on the type parameter.
*/ */
ssize_t iio_format_value(char *buf, unsigned int type, int val, int val2) ssize_t iio_format_value(char *buf, unsigned int type, int size, int *vals)
{ {
unsigned long long tmp; unsigned long long tmp;
bool scale_db = false; bool scale_db = false;
switch (type) { switch (type) {
case IIO_VAL_INT: case IIO_VAL_INT:
return sprintf(buf, "%d\n", val); return sprintf(buf, "%d\n", vals[0]);
case IIO_VAL_INT_PLUS_MICRO_DB: case IIO_VAL_INT_PLUS_MICRO_DB:
scale_db = true; scale_db = true;
case IIO_VAL_INT_PLUS_MICRO: case IIO_VAL_INT_PLUS_MICRO:
if (val2 < 0) if (vals[1] < 0)
return sprintf(buf, "-%ld.%06u%s\n", abs(val), -val2, return sprintf(buf, "-%ld.%06u%s\n", abs(vals[0]),
-vals[1],
scale_db ? " dB" : ""); scale_db ? " dB" : "");
else else
return sprintf(buf, "%d.%06u%s\n", val, val2, return sprintf(buf, "%d.%06u%s\n", vals[0], vals[1],
scale_db ? " dB" : ""); scale_db ? " dB" : "");
case IIO_VAL_INT_PLUS_NANO: case IIO_VAL_INT_PLUS_NANO:
if (val2 < 0) if (vals[1] < 0)
return sprintf(buf, "-%ld.%09u\n", abs(val), -val2); return sprintf(buf, "-%ld.%09u\n", abs(vals[0]),
-vals[1]);
else else
return sprintf(buf, "%d.%09u\n", val, val2); return sprintf(buf, "%d.%09u\n", vals[0], vals[1]);
case IIO_VAL_FRACTIONAL: case IIO_VAL_FRACTIONAL:
tmp = div_s64((s64)val * 1000000000LL, val2); tmp = div_s64((s64)vals[0] * 1000000000LL, vals[1]);
val2 = do_div(tmp, 1000000000LL); vals[1] = do_div(tmp, 1000000000LL);
val = tmp; vals[0] = tmp;
return sprintf(buf, "%d.%09u\n", val, val2); return sprintf(buf, "%d.%09u\n", vals[0], vals[1]);
case IIO_VAL_FRACTIONAL_LOG2: case IIO_VAL_FRACTIONAL_LOG2:
tmp = (s64)val * 1000000000LL >> val2; tmp = (s64)vals[0] * 1000000000LL >> vals[1];
val2 = do_div(tmp, 1000000000LL); vals[1] = do_div(tmp, 1000000000LL);
val = tmp; vals[0] = tmp;
return sprintf(buf, "%d.%09u\n", val, val2); return sprintf(buf, "%d.%09u\n", vals[0], vals[1]);
case IIO_VAL_INT_MULTIPLE:
{
int i;
int len = 0;
for (i = 0; i < size; ++i)
len += snprintf(&buf[len], PAGE_SIZE - len, "%d ",
vals[i]);
len += snprintf(&buf[len], PAGE_SIZE - len, "\n");
return len;
}
default: default:
return 0; return 0;
} }
...@@ -419,14 +431,23 @@ static ssize_t iio_read_channel_info(struct device *dev, ...@@ -419,14 +431,23 @@ static ssize_t iio_read_channel_info(struct device *dev,
{ {
struct iio_dev *indio_dev = dev_to_iio_dev(dev); struct iio_dev *indio_dev = dev_to_iio_dev(dev);
struct iio_dev_attr *this_attr = to_iio_dev_attr(attr); struct iio_dev_attr *this_attr = to_iio_dev_attr(attr);
int val, val2; int vals[INDIO_MAX_RAW_ELEMENTS];
int ret = indio_dev->info->read_raw(indio_dev, this_attr->c, int ret;
&val, &val2, this_attr->address); int val_len = 2;
if (indio_dev->info->read_raw_multi)
ret = indio_dev->info->read_raw_multi(indio_dev, this_attr->c,
INDIO_MAX_RAW_ELEMENTS,
vals, &val_len,
this_attr->address);
else
ret = indio_dev->info->read_raw(indio_dev, this_attr->c,
&vals[0], &vals[1], this_attr->address);
if (ret < 0) if (ret < 0)
return ret; return ret;
return iio_format_value(buf, ret, val, val2); return iio_format_value(buf, ret, val_len, vals);
} }
/** /**
......
...@@ -270,7 +270,7 @@ static ssize_t iio_ev_value_show(struct device *dev, ...@@ -270,7 +270,7 @@ static ssize_t iio_ev_value_show(struct device *dev,
{ {
struct iio_dev *indio_dev = dev_to_iio_dev(dev); struct iio_dev *indio_dev = dev_to_iio_dev(dev);
struct iio_dev_attr *this_attr = to_iio_dev_attr(attr); struct iio_dev_attr *this_attr = to_iio_dev_attr(attr);
int val, val2; int val, val2, val_arr[2];
int ret; int ret;
ret = indio_dev->info->read_event_value(indio_dev, ret = indio_dev->info->read_event_value(indio_dev,
...@@ -279,7 +279,9 @@ static ssize_t iio_ev_value_show(struct device *dev, ...@@ -279,7 +279,9 @@ static ssize_t iio_ev_value_show(struct device *dev,
&val, &val2); &val, &val2);
if (ret < 0) if (ret < 0)
return ret; return ret;
return iio_format_value(buf, ret, val, val2); val_arr[0] = val;
val_arr[1] = val2;
return iio_format_value(buf, ret, 2, val_arr);
} }
static ssize_t iio_ev_value_store(struct device *dev, static ssize_t iio_ev_value_store(struct device *dev,
......
...@@ -417,12 +417,24 @@ static int iio_channel_read(struct iio_channel *chan, int *val, int *val2, ...@@ -417,12 +417,24 @@ static int iio_channel_read(struct iio_channel *chan, int *val, int *val2,
enum iio_chan_info_enum info) enum iio_chan_info_enum info)
{ {
int unused; int unused;
int vals[INDIO_MAX_RAW_ELEMENTS];
int ret;
int val_len = 2;
if (val2 == NULL) if (val2 == NULL)
val2 = &unused; val2 = &unused;
return chan->indio_dev->info->read_raw(chan->indio_dev, chan->channel, if (chan->indio_dev->info->read_raw_multi) {
val, val2, info); ret = chan->indio_dev->info->read_raw_multi(chan->indio_dev,
chan->channel, INDIO_MAX_RAW_ELEMENTS,
vals, &val_len, info);
*val = vals[0];
*val2 = vals[1];
} else
ret = chan->indio_dev->info->read_raw(chan->indio_dev,
chan->channel, val, val2, info);
return ret;
} }
int iio_read_channel_raw(struct iio_channel *chan, int *val) int iio_read_channel_raw(struct iio_channel *chan, int *val)
......
...@@ -288,6 +288,8 @@ static inline s64 iio_get_time_ns(void) ...@@ -288,6 +288,8 @@ static inline s64 iio_get_time_ns(void)
#define INDIO_ALL_BUFFER_MODES \ #define INDIO_ALL_BUFFER_MODES \
(INDIO_BUFFER_TRIGGERED | INDIO_BUFFER_HARDWARE) (INDIO_BUFFER_TRIGGERED | INDIO_BUFFER_HARDWARE)
#define INDIO_MAX_RAW_ELEMENTS 4
struct iio_trigger; /* forward declaration */ struct iio_trigger; /* forward declaration */
struct iio_dev; struct iio_dev;
...@@ -302,6 +304,14 @@ struct iio_dev; ...@@ -302,6 +304,14 @@ struct iio_dev;
* the channel in question. Return value will specify the * the channel in question. Return value will specify the
* type of value returned by the device. val and val2 will * type of value returned by the device. val and val2 will
* contain the elements making up the returned value. * contain the elements making up the returned value.
* @read_raw_multi: function to return values from the device.
* mask specifies which value. Note 0 means a reading of
* the channel in question. Return value will specify the
* type of value returned by the device. vals pointer
* contain the elements making up the returned value.
* max_len specifies maximum number of elements
* vals pointer can contain. val_len is used to return
* length of valid elements in vals.
* @write_raw: function to write a value to the device. * @write_raw: function to write a value to the device.
* Parameters are the same as for read_raw. * Parameters are the same as for read_raw.
* @write_raw_get_fmt: callback function to query the expected * @write_raw_get_fmt: callback function to query the expected
...@@ -328,6 +338,13 @@ struct iio_info { ...@@ -328,6 +338,13 @@ struct iio_info {
int *val2, int *val2,
long mask); long mask);
int (*read_raw_multi)(struct iio_dev *indio_dev,
struct iio_chan_spec const *chan,
int max_len,
int *vals,
int *val_len,
long mask);
int (*write_raw)(struct iio_dev *indio_dev, int (*write_raw)(struct iio_dev *indio_dev,
struct iio_chan_spec const *chan, struct iio_chan_spec const *chan,
int val, int val,
......
...@@ -79,6 +79,7 @@ enum iio_event_direction { ...@@ -79,6 +79,7 @@ enum iio_event_direction {
#define IIO_VAL_INT_PLUS_MICRO 2 #define IIO_VAL_INT_PLUS_MICRO 2
#define IIO_VAL_INT_PLUS_NANO 3 #define IIO_VAL_INT_PLUS_NANO 3
#define IIO_VAL_INT_PLUS_MICRO_DB 4 #define IIO_VAL_INT_PLUS_MICRO_DB 4
#define IIO_VAL_INT_MULTIPLE 5
#define IIO_VAL_FRACTIONAL 10 #define IIO_VAL_FRACTIONAL 10
#define IIO_VAL_FRACTIONAL_LOG2 11 #define IIO_VAL_FRACTIONAL_LOG2 11
......
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