linux-stable.git/drivers/staging/iio, branch linux-4.1.y

staging: iio: cdc: fix improper return value

2017-12-08T23:01:10+00:00

[ Upstream commit 91ca1a8c584f55857b1f6ab20a1d3a1ce7a559bb ]

At the end of function ad7150_write_event_config(), directly returns 0.
As a result, the errors will be ignored by the callers. It may be better
to return variable "ret".

Signed-off-by: Pan Bian 
Signed-off-by: Jonathan Cameron 
Signed-off-by: Sasha Levin

iio: trigger: free trigger resource correctly

2017-12-07T02:19:58+00:00

[ Upstream commit 10e840dfb0b7fc345082dd9e5fff3c1c02e7690e ]

These stand-alone trigger drivers were using iio_trigger_put()
where they should have been using iio_trigger_free().  The
iio_trigger_put() adds a module_put which is bad since they
never did a module_get.

In the sysfs driver, module_get/put's are used as triggers are
added & removed. This extra module_put() occurs on an error path
in the probe routine (probably rare).

In the bfin-timer & interrupt trigger drivers, the module resources
are not explicitly managed, so it's doing a put on something that
was never get'd.  It occurs on the probe error path and on the
remove path (not so rare).

Tested with the sysfs trigger driver.
The bfin & interrupt drivers were build tested & inspected only.

Signed-off-by: Alison Schofield 
Signed-off-by: Jonathan Cameron 
Signed-off-by: Sasha Levin

staging: iio: ad7192: Fix - use the dedicated reset function avoiding dma from stack.

2017-11-06T04:54:22+00:00

[ Upstream commit f790923f146140a261ad211e5baf75d169f16fb2 ]

Depends on: 691c4b95d1 ("iio: ad_sigma_delta: Implement a dedicated reset function")

SPI host drivers can use DMA to transfer data, so the buffer should be properly allocated.
Keeping it on the stack could cause an undefined behavior.

The dedicated reset function solves this issue.

Signed-off-by: Stefan Popa 
Acked-by: Lars-Peter Clausen 
Acked-by: Michael Hennerich 
Cc: 
Signed-off-by: Jonathan Cameron 
Signed-off-by: Sasha Levin

staging:iio:resolver:ad2s1210 fix negative IIO_ANGL_VEL read

2017-09-10T20:36:10+00:00

[ Upstream commit 105967ad68d2eb1a041bc041f9cf96af2a653b65 ]

gcc-7 points out an older regression:

drivers/staging/iio/resolver/ad2s1210.c: In function 'ad2s1210_read_raw':
drivers/staging/iio/resolver/ad2s1210.c:515:42: error: '<<' in boolean context, did you mean '<' ? [-Werror=int-in-bool-context]

The original code had 'unsigned short' here, but incorrectly got
converted to 'bool'. This reverts the regression and uses a normal
type instead.

Fixes: 29148543c521 ("staging:iio:resolver:ad2s1210 minimal chan spec conversion.")
Cc: stable@vger.kernel.org
Signed-off-by: Arnd Bergmann 
Signed-off-by: Jonathan Cameron 
Signed-off-by: Sasha Levin

staging: iio: accel: fix error check

2016-08-20T03:08:43+00:00

[ Upstream commit ef3149eb3ddb7f9125e11c90f8330e371b55cffd ]

sca3000_read_ctrl_reg() returns a negative number on failure, check for
this instead of zero.

Signed-off-by: Luis de Bethencourt 
Cc: 
Signed-off-by: Jonathan Cameron 
Signed-off-by: Sasha Levin

iio: accel: sca3000: memory corruption in sca3000_read_first_n_hw_rb()

2015-11-09T22:33:35+00:00

commit eda7d0f38aaf50dbb2a2de15e8db386c4f6f65fc upstream.

"num_read" is in byte units but we are write u16s so we end up write
twice as much as intended.

Signed-off-by: Dan Carpenter 
Signed-off-by: Jonathan Cameron 
Signed-off-by: Greg Kroah-Hartman

iio: mxs-lradc: Fix temperature offset

2015-11-09T22:33:35+00:00

commit b94e22805a2224061bb263a82b72e09544a5fbb3 upstream.

0° Kelvin is actually −273.15°C, not -272.15°C. Fix the temperature offset.
Also improve the comment explaining the calculation.

Reported-by: Janusz Użycki 
Signed-off-by: Alexandre Belloni 
Acked-by: Stefan Wahren 
Acked-by: Marek Vasut 
Cc: 
Signed-off-by: Jonathan Cameron 
Signed-off-by: Greg Kroah-Hartman

Merge 4.0-rc7 into staging-next

2015-04-07T09:03:02+00:00

We want those fixes (iio primarily) into the -next branch to help with
merge and testing issues.

Signed-off-by: Greg Kroah-Hartman

Staging: iio: accel: Remove explicit NULL comparison

2015-04-01T15:23:21+00:00

This patch removes explicit NULL comparison and writes it in its
equivalent shorter form. Done with coccinelle.

@replace_rule@
expression e;
@@

-e == NULL
+ !e

Signed-off-by: Cristina Opriceana 
Signed-off-by: Greg Kroah-Hartman

Staging: iio: adc: Remove explicit NULL comparison

2015-04-01T15:23:21+00:00

This patch removes explicit NULL comparison and writes it in its
shorter form. Detected with coccinelle.

Signed-off-by: Cristina Opriceana 
Signed-off-by: Greg Kroah-Hartman