iio: gyro: bmg160: fix buffer read values

itirdea · jic23 · commit b475c59b113d · 2016-04-03T11:21:33.000+01:00
When reading gyroscope axes using iio buffers, the values
returned are always 0. In the interrupt handler, the return
value of the read operation is returned to the user instead
of the value read. Return the value read to the user.

This is also fixed in commit 82d8e5da1a33 ("iio:
accel: bmg160: optimize transfers in trigger handler").

Signed-off-by: Irina Tirdea &lt;irina.tirdea@intel.com&gt;
Cc: &lt;Stable@vger.kernel.org&gt;
Signed-off-by: Jonathan Cameron &lt;jic23@kernel.org&gt;
diff --git a/drivers/iio/gyro/bmg160_core.c b/drivers/iio/gyro/bmg160_core.c
@@ -781,7 +781,7 @@ static irqreturn_t bmg160_trigger_handler(int irq, void *p)
 			mutex_unlock(&data->mutex);
 			goto err;
 		}
-		data->buffer[i++] = ret;
+		data->buffer[i++] = val;
 	}
 	mutex_unlock(&data->mutex);
 

Original file line number	Diff line number	Diff line change
`@@ -781,7 +781,7 @@ static irqreturn_t bmg160_trigger_handler(int irq, void *p)`
`781`	`781`	`mutex_unlock(&data->mutex);`
`782`	`782`	`goto err;`
`783`	`783`	`}`
`784`		`- data->buffer[i++] = ret;`
	`784`	`+ data->buffer[i++] = val;`
`785`	`785`	`}`
`786`	`786`	`mutex_unlock(&data->mutex);`
`787`	`787`