linux-toradex.git/drivers/iio/temperature, branch v4.9.14

iio: maxim_thermocouple: detect invalid storage size in read()

2016-11-13T09:08:32+00:00

As found by gcc -Wmaybe-uninitialized, having a storage_bytes value other
than 2 or 4 will result in undefined behavior:

drivers/iio/temperature/maxim_thermocouple.c: In function 'maxim_thermocouple_read':
drivers/iio/temperature/maxim_thermocouple.c:141:5: error: 'ret' may be used uninitialized in this function [-Werror=maybe-uninitialized]

This probably cannot happen, but returning -EINVAL here is appropriate
and makes gcc happy and the code more robust.

Fixes: 231147ee77f3 ("iio: maxim_thermocouple: Align 16 bit big endian value of raw reads")
Signed-off-by: Arnd Bergmann 
Signed-off-by: Jonathan Cameron 
(cherry picked from commit 32cb7d27e65df9daa7cee8f1fdf7b259f214bee2)
Signed-off-by: Greg Kroah-Hartman

iio: maxim_thermocouple: Align 16 bit big endian value of raw reads

2016-10-01T13:40:23+00:00

Driver was reporting invalid raw read values for MAX6675 on big
endian architectures. MAX6675 buffered mode is not affected, nor
is the MAX31855.

The driver was losing a 2 byte read value when it used a 32 bit
integer buffer to store a 16 bit big endian value. Use big endian
types to properly align buffers on big endian architectures.

Fixes following sparse endianness warnings:
warning: cast to restricted __be16
warning: cast to restricted __be32
Fixes checkpatch issue:
CHECK: No space is necessary after a cast

Signed-off-by: sayli karnik 
Fixes: 1f25ca11d84a ("iio: temperature: add support for Maxim
thermocouple chips")
Signed-off-by: Jonathan Cameron

iio: temperature: fix non static symbol warnings

2016-08-29T18:49:00+00:00

Fixes the following sparse warnings:

drivers/iio/temperature/maxim_thermocouple.c:35:28: warning:
 symbol 'max6675_channels' was not declared. Should it be static?
drivers/iio/temperature/maxim_thermocouple.c:52:28: warning:
 symbol 'max31855_channels' was not declared. Should it be static?
drivers/iio/temperature/maxim_thermocouple.c:98:38: warning:
 symbol 'maxim_thermocouple_chips' was not declared. Should it be static?

Signed-off-by: Wei Yongjun 
Reviewed-By: Matt Ranostay 
Signed-off-by: Jonathan Cameron

iio: temperature: add Kconfig selects for triggered buffer

2016-08-21T18:50:44+00:00

Select IIO_BUFFER and IIO_TRIGGERED_BUFFER to compile maxim_thermocouple.

Signed-off-by: Alison Schofield 
Cc: Daniel Baluta 
Signed-off-by: Jonathan Cameron

iio: temperature: add support for Maxim thermocouple chips

2016-07-10T13:44:03+00:00

Add initial driver support for MAX6675, and MAX31855 thermocouple chips.

Cc: Marek Vasut 
Cc: Matt Porter 
Signed-off-by: Matt Ranostay 
Signed-off-by: Jonathan Cameron

iio: Export I2C module alias information

2016-05-21T19:08:14+00:00

The I2C drivers have an i2c_device_id array but that information isn't
exported to the modules using the MODULE_DEVICE_TABLE() macro. So the
modules autoloading won't work if the I2C device is registered using
OF or legacy board files due missing alias information in the modules.

The issue was found using Kieran Bingham's coccinelle semantic patch:
https://lkml.org/lkml/2016/5/10/520

Signed-off-by: Javier Martinez Canillas 
Signed-off-by: Jonathan Cameron

iio: convert to common i2c_check_functionality() return value

2016-02-27T17:17:43+00:00

Previously most drivers that used a i2c_check_functionality() check
condition required various error codes on failure. This patchset
converts to a standard of -EOPNOTSUPP

Signed-off-by: Matt Ranostay 
Signed-off-by: Jonathan Cameron

Add tsys02d meas-spec driver support

2015-10-11T09:21:19+00:00

Support for TSYS02D temperature sensor

Signed-off-by: Ludovic Tancerel 
Signed-off-by: Jonathan Cameron

Add tsys01 meas-spec driver support

2015-10-04T14:02:28+00:00

Support for TSYS01 temperature sensor

Signed-off-by: Ludovic Tancerel 
Signed-off-by: Jonathan Cameron

iio: mlx96014: Error checking from positive to negative

2015-10-03T09:15:25+00:00

Dan Carpenter reported a static checker report and after his mail I
noticed that we actually return from function if positive value is
obtained from i2c read. This was remainder from when code was not in
separate function (which I changed during the review process).

Static checker reported
  drivers/iio/temperature/mlx90614.c:167
  mlx90614_iir_search()
    warn: this cast is a no-op
which meant that cast before negating is useless. Dan also proposed a
solution on nicer bit operation form.

Also changed magic number to macro in process as that was confusing.

Reported-by: Dan Carpenter 
Signed-off-by: Crt Mori 
Signed-off-by: Jonathan Cameron