- 12 Aug, 2015 5 commits
-
-
Cristina Opriceana authored
Negative return values are not supported by iio_event_poll since its return type is unsigned int. Fixes: f18e7a06 ("iio: Return -ENODEV for file operations if the device has been unregistered") Signed-off-by:
Cristina Opriceana <cristina.opriceana@gmail.com> Cc: <Stable@vger.kernel.org> Signed-off-by:
Jonathan Cameron <jic23@kernel.org>
-
Cristina Opriceana authored
Change return value to 0 if no device is bound since unsigned int cannot support negative error codes. Fixes: f18e7a06 ("iio: Return -ENODEV for file operations if the device has been unregistered") Signed-off-by:
-
Linus Walleij authored
commit acf50b35 "tools:iio:lsiio: add error handling" introduced error handling of errors returned from read_sysfs_string(), but with a simple if (retval), missing the fact that these functions return a positive value if the read was successful. As a result lsiio regresses and does not show any devices on my filesystem. Fix this by checking for only negative error codes. Cc: Hartmut Knaack <knaack.h@gmx.de> Signed-off-by:
Linus Walleij <linus.walleij@linaro.org> Acked-by:
Hartmut Knaack <knaack.h@gmx.de> Signed-off-by:
-
Markus Pargmann authored
This patch adds selects for IIO_BUFFER and IIO_TRIGGERED_BUFFER. Without IIO_BUFFER, the driver does not compile. Signed-off-by:
Markus Pargmann <mpa@pengutronix.de> Reviewed-by:
Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> Cc: <Stable@vger.kernel.org> Signed-off-by:
-
Hartmut Knaack authored
Active channel number is stored in BERLIN2_SM_CTRL as value, instead of a bit map. The masks for channel interrupts and data ready are a 16 bits wide bit map each, instead of just 4 bits. Also correct the data mask for the temperature sensor, which was Reported-by:
Antoine Tenart <antoine.tenart@free-electrons.com> Signed-off-by:
-
- 08 Aug, 2015 31 commits
-
-
Joo Aun Saw authored
Remove unnecessary double pointer from channel sorting function. Signed-off-by:
Joo Aun Saw <jasaw@dius.com.au> Signed-off-by:
-
Joo Aun Saw authored
On error, caller's ci_array is freed and set to NULL to avoid potential double free if some other user of this code is not sufficiently careful. Counter is reset to zero for consistency. Signed-off-by:
-
Vladimir Barinov authored
Fix typos in ABI documentation Signed-off-by:
Vladimir Barinov <vladimir.barinov@cogentembedded.com> Signed-off-by:
-
Hartmut Knaack authored
Move the entry in Kconfig to its alphabetically correct position. Signed-off-by:
-
Hartmut Knaack authored
Some coding style cleanups, mainly indicated by checkpatch.pl, which includes indentation changes, drop spaces after casts and befor tabs. Also insert empty lines after logical blocks and before unconditional returns. Signed-off-by:
-
Hartmut Knaack authored
Rework register definitions to be sorted by register and bit number, with bit definitions cascaded under the appropriate register, use GENMASK for consecutive bitmasks and realign properly. Signed-off-by:
-
Hartmut Knaack authored
Check values to be written to the device for valid lower and upper bounds. Signed-off-by:
-
Hartmut Knaack authored
Pass up the error code provided by functions. Signed-off-by:
-
Hartmut Knaack authored
One inclusion of linux/iio/trigger_consumer.h is sufficient. Fixes: ae6d9ce0 ("iio: mma8452: Add support for interrupt driven triggers.") Signed-off-by:
-
Hartmut Knaack authored
To iterate through the available frequencies of mma8452_hp_filter_cutoff[], the array size of a row of that table needs to be provided to _get_int_plus_micros_index(). Fixes: 1e79841a ("iio: mma8452: Add highpass filter configuration.") Signed-off-by:
-
Bastien Nocera authored
Instead of using the I2C or ACPI ID to determine which variant of the chipset to use, determine that from the chip ID. Under Windows, the same driver is used for those variants and, despite incorrect ACPI data, it is able to load and operate the accelerometer. Fixes the accelerometer failing with: bmc150_accel i2c-BMA250E:00: Invalid chip f8 on the WinBook TW100 Signed-off-by:
Bastien Nocera <hadess@hadess.net> Signed-off-by:
-
Bastien Nocera authored
Move bmc150_accel_chip_init() so that we can use bmc150_accel_chip_info_tbl[] in it. Signed-off-by:
-
Lars Svensson authored
Fixing indentation errors in drivers/staging/iio/iio_simple_dummy_events.c. Signed-off-by:
Lars Svensson <lars1.svensson@sonymobile.com> Acked-by:
Daniel Baluta <daniel.baluta@intel.com> Signed-off-by:
-
Lars-Peter Clausen authored
The ADIS16137 is register map compatible to the ADIS16136, but has a different scale factor for the gyroscope output. Signed-off-by:
Lars-Peter Clausen <lars@metafoo.de> Signed-off-by:
-
Lars-Peter Clausen authored
The ADIS16445 is similar to the ADIS16448, but without the magnetometer and pressure channels as well as different scale factors for the gyroscope and accelerometer outputs. Signed-off-by:
-
Lars-Peter Clausen authored
The ADIS16367 is mostly register compatible to the ADIS16360. The only difference is the scale factor for the gyroscope output. Signed-off-by:
-
Lars-Peter Clausen authored
The ADIS16266 is mostly register compatible to the ADIS16260. The difference is a different gyroscope scale factor as well not having the relative angular displacement channel. Signed-off-by:
-
Lars-Peter Clausen authored
The ADIS16305 is fully register map compatible to the ADIS16300. Signed-off-by:
-
Lars-Peter Clausen authored
The ADIS16300 has the product ID and serial number registers, they are just not documented. Set the appropriate flags so the driver makes use of them. Signed-off-by:
-
Lars-Peter Clausen authored
The different devices support by the adis16480 driver have slightly different scales for the gyroscope and accelerometer channels. Signed-off-by:
-
Lars-Peter Clausen authored
Add inverse unit conversion macro to convert from standard IIO units to units that might be used by some devices. Those are useful in combination with scale factors that are specified as IIO_VAL_FRACTIONAL. Typically the denominator for those specifications will contain the maximum raw value the sensor will generate and the numerator the value it maps to in a specific unit. Sometimes datasheets specify those in different units than the standard IIO units (e.g. degree/s instead of rad/s) and so we need to do a unit conversion. From a mathematical point of view it does not make a difference whether we apply the unit conversion to the numerator or the inverse unit conversion to the denominator since (x / y) / z = x / (y * z). But as the denominator is typically a larger value and we are rounding both the numerator and denominator to integer values using the later method gives us a better precision (E.g. the relative error is smaller if we round 8000.3 to 8000 rather than rounding 8.3 to 8). This is where in inverse unit conversion macros will be used. Marked for stable as used by some upcoming fixes. Signed-off-by:
-
Lars-Peter Clausen authored
Use the correct scale for the adis16448 gyroscope output. Signed-off-by:
-
Cristina Opriceana authored
Fix kernel docs warnings by adding the missing description for each of the existing function parameters. Signed-off-by:
-
Cristina Opriceana authored
This patch adds an entry in ABI Documentation for the name attribute issued when a trigger is created. Signed-off-by:
-
Daniel Baluta authored
This is intended to help developers faster find their way inside the Industrial I/O core and reduce time spent on IIO drivers development. Signed-off-by:
Crt Mori <cmo@melexis.com> Reviewed-by:
-
Pengyu Ma authored
When compile iio related driver the following warning shown: include/linux/iio/trigger.h:35:34: warning: 'struct iio_trigger' declared inside parameter list int (*set_trigger_state)(struct iio_trigger *trig, bool state); include/linux/iio/trigger.h:38:18: warning: 'struct iio_dev' declared inside parameter list struct iio_dev *indio_dev); 'struct iio_dev' and 'struct iio_trigger' was used before declaration, forward declaration for these structs to fix warning. Signed-off-by:Pengyu Ma <pengyu.ma@windriver.com> Acked-by:
-
Hartmut Knaack authored
Drop the local buffer in stk8312_trigger_handler() and use data->buffer instead for bulk reads. Signed-off-by:
Tiberiu Breana <tiberiu.a.breana@intel.com> Signed-off-by:
-
Hartmut Knaack authored
Adjust some indentation issues to make checkpatch.pl happy in strict mode. Signed-off-by:
-
Hartmut Knaack authored
Adapt some variable types to reduce unnecessary casting. Signed-off-by:
-
Hartmut Knaack authored
Make use of BIT to describe register bits, GENMASK for consecutive bitmasks, rename and sort existing definitions, replace magic value with an expressive definition, drop an unused definition. Signed-off-by:
-
Hartmut Knaack authored
Improve error handling in the following ways: - set return value on error condition to an appropriate error code - return error code immediately in case of an error (slightly changes code structure) - pass up real error code - add missing error handling - return 0 when error have been caught already - put device back in active mode after error occurs Signed-off-by:
-
- 02 Aug, 2015 4 commits
-
-
Hartmut Knaack authored
Revision 1.2 of the datasheet recommends on page 22 to only write non-zero values read from OTP register 0x70 into AFECTRL register. Signed-off-by:
-
Hartmut Knaack authored
Add the still missing dependencies for triggered buffer support. Fixes: 95c12bba ("iio: accel: Add buffer mode for Sensortek STK8312") Signed-off-by:
-
Hartmut Knaack authored
Drop the otherwise unused definition of the channel-array size and use it directly in _probe - makes it a bit more obvious. Signed-off-by:
-
Hartmut Knaack authored
Use augmented assignment to subtract the offset for negative temperature values. Specify the amount of private data to be allocated through devm_iio_device_alloc() with sizeof(*priv), as it is shorter and common practice in IIO. Signed-off-by:
-
