Re: [PATCH 10/10] iio: imu: kmx61: Use correct base when reading data

2015-01-01 Thread Hartmut Knaack 
Daniel Baluta schrieb am 23.12.2014 um 14:22:
> We have two IIO devices and we need to adjust the base
> when reading data.
> 
> Signed-off-by: Daniel Baluta 
Reviewed-by: Hartmut Knaack 
> ---
>  drivers/iio/imu/kmx61.c | 8 +++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/iio/imu/kmx61.c b/drivers/iio/imu/kmx61.c
> index 98369eb..6178cea 100644
> --- a/drivers/iio/imu/kmx61.c
> +++ b/drivers/iio/imu/kmx61.c
> @@ -1210,12 +1210,18 @@ static irqreturn_t kmx61_trigger_handler(int irq, 
> void *p)
>   struct iio_dev *indio_dev = pf->indio_dev;
>   struct kmx61_data *data = kmx61_get_data(indio_dev);
>   int bit, ret, i = 0;
> + u8 base;
>   s16 buffer[8];
>  
> + if (indio_dev == data->acc_indio_dev)
> + base = KMX61_ACC_XOUT_L;
> + else
> + base = KMX61_MAG_XOUT_L;
> +
>   mutex_lock(>lock);
>   for_each_set_bit(bit, indio_dev->buffer->scan_mask,
>indio_dev->masklength) {
> - ret = kmx61_read_measurement(data, KMX61_ACC_XOUT_L, bit);
> + ret = kmx61_read_measurement(data, base, bit);
>   if (ret < 0) {
>   mutex_unlock(>lock);
>   goto err;
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 10/10] iio: imu: kmx61: Use correct base when reading data

2015-01-01 Thread Hartmut Knaack 
Daniel Baluta schrieb am 23.12.2014 um 14:22:
 We have two IIO devices and we need to adjust the base
 when reading data.
 
 Signed-off-by: Daniel Baluta daniel.bal...@intel.com
Reviewed-by: Hartmut Knaack knaac...@gmx.de
 ---
  drivers/iio/imu/kmx61.c | 8 +++-
  1 file changed, 7 insertions(+), 1 deletion(-)
 
 diff --git a/drivers/iio/imu/kmx61.c b/drivers/iio/imu/kmx61.c
 index 98369eb..6178cea 100644
 --- a/drivers/iio/imu/kmx61.c
 +++ b/drivers/iio/imu/kmx61.c
 @@ -1210,12 +1210,18 @@ static irqreturn_t kmx61_trigger_handler(int irq, 
 void *p)
   struct iio_dev *indio_dev = pf-indio_dev;
   struct kmx61_data *data = kmx61_get_data(indio_dev);
   int bit, ret, i = 0;
 + u8 base;
   s16 buffer[8];
  
 + if (indio_dev == data-acc_indio_dev)
 + base = KMX61_ACC_XOUT_L;
 + else
 + base = KMX61_MAG_XOUT_L;
 +
   mutex_lock(data-lock);
   for_each_set_bit(bit, indio_dev-buffer-scan_mask,
indio_dev-masklength) {
 - ret = kmx61_read_measurement(data, KMX61_ACC_XOUT_L, bit);
 + ret = kmx61_read_measurement(data, base, bit);
   if (ret  0) {
   mutex_unlock(data-lock);
   goto err;
 

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 10/10] iio: imu: kmx61: Use correct base when reading data

2014-12-23 Thread Daniel Baluta 
We have two IIO devices and we need to adjust the base
when reading data.

Signed-off-by: Daniel Baluta 
---
 drivers/iio/imu/kmx61.c | 8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/drivers/iio/imu/kmx61.c b/drivers/iio/imu/kmx61.c
index 98369eb..6178cea 100644
--- a/drivers/iio/imu/kmx61.c
+++ b/drivers/iio/imu/kmx61.c
@@ -1210,12 +1210,18 @@ static irqreturn_t kmx61_trigger_handler(int irq, void 
*p)
struct iio_dev *indio_dev = pf->indio_dev;
struct kmx61_data *data = kmx61_get_data(indio_dev);
int bit, ret, i = 0;
+   u8 base;
s16 buffer[8];
 
+   if (indio_dev == data->acc_indio_dev)
+   base = KMX61_ACC_XOUT_L;
+   else
+   base = KMX61_MAG_XOUT_L;
+
mutex_lock(>lock);
for_each_set_bit(bit, indio_dev->buffer->scan_mask,
 indio_dev->masklength) {
-   ret = kmx61_read_measurement(data, KMX61_ACC_XOUT_L, bit);
+   ret = kmx61_read_measurement(data, base, bit);
if (ret < 0) {
mutex_unlock(>lock);
goto err;
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 10/10] iio: imu: kmx61: Use correct base when reading data

2014-12-23 Thread Daniel Baluta 
We have two IIO devices and we need to adjust the base
when reading data.

Signed-off-by: Daniel Baluta daniel.bal...@intel.com
---
 drivers/iio/imu/kmx61.c | 8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/drivers/iio/imu/kmx61.c b/drivers/iio/imu/kmx61.c
index 98369eb..6178cea 100644
--- a/drivers/iio/imu/kmx61.c
+++ b/drivers/iio/imu/kmx61.c
@@ -1210,12 +1210,18 @@ static irqreturn_t kmx61_trigger_handler(int irq, void 
*p)
struct iio_dev *indio_dev = pf-indio_dev;
struct kmx61_data *data = kmx61_get_data(indio_dev);
int bit, ret, i = 0;
+   u8 base;
s16 buffer[8];
 
+   if (indio_dev == data-acc_indio_dev)
+   base = KMX61_ACC_XOUT_L;
+   else
+   base = KMX61_MAG_XOUT_L;
+
mutex_lock(data-lock);
for_each_set_bit(bit, indio_dev-buffer-scan_mask,
 indio_dev-masklength) {
-   ret = kmx61_read_measurement(data, KMX61_ACC_XOUT_L, bit);
+   ret = kmx61_read_measurement(data, base, bit);
if (ret  0) {
mutex_unlock(data-lock);
goto err;
-- 
1.9.1

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/