Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
drivers/iio/imu/bmi160/bmi160_core.c | 27 +++
1 file changed, 27 insertions(+)
diff --git a/drivers/iio/imu/bmi160/bmi160_core.c
b/drivers/iio/imu/bmi160/bmi160_core.c
index b8a290e..97928d5 100644
--- a/d
mask and val parameters of regmap_update_bits were reveresed.
Fixes: 77c4ad2d6a9 ("iio: imu: Add initial support for Bosch BMI160")
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
drivers/iio/imu/bmi160/bmi160_core.c | 4 ++--
1 file changed, 2 insertions(+), 2 de
Signed-off-by: Daniel Baluta
---
drivers/iio/imu/bmi160/bmi160_core.c | 27 +++
1 file changed, 27 insertions(+)
diff --git a/drivers/iio/imu/bmi160/bmi160_core.c
b/drivers/iio/imu/bmi160/bmi160_core.c
index b8a290e..97928d5 100644
--- a/drivers/iio/imu/bmi160
mask and val parameters of regmap_update_bits were reveresed.
Fixes: 77c4ad2d6a9 ("iio: imu: Add initial support for Bosch BMI160")
Signed-off-by: Daniel Baluta
---
drivers/iio/imu/bmi160/bmi160_core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drive
First two patches are bugfixes related to sampling frequency and the last
one is adding support for exporting available sampling frequency and scale.
Daniel Baluta (3):
iio: bmi160: Fix output data rate for accel
iio: bmi160: Fix ODR setting
imu: bmi160: Add avail frequency and scale
On Tue, Apr 26, 2016 at 4:14 PM, Yong Li wrote:
> I am thinking if there is any application is using this incorrect
> name, the application should be fix too
The rule is: "Don't break the userspace ABI". So, if we got this wrong
from the beginning we are stuck with this name.
On Tue, Apr 26, 2016 at 4:14 PM, Yong Li wrote:
> I am thinking if there is any application is using this incorrect
> name, the application should be fix too
The rule is: "Don't break the userspace ABI". So, if we got this wrong
from the beginning we are stuck with this name.
The only thing
BMM150 is register compatible with magnetometer part of
BMC156.
Datasheet is at:
http://www.mouser.com/ds/2/783/BST-BMM150-DS001-01-786480.pdf
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
Lucas let me know if it works for you. I don't have yet the BMM150 evaluation
board. It
BMM150 is register compatible with magnetometer part of
BMC156.
Datasheet is at:
http://www.mouser.com/ds/2/783/BST-BMM150-DS001-01-786480.pdf
Signed-off-by: Daniel Baluta
---
Lucas let me know if it works for you. I don't have yet the BMM150 evaluation
board. It should be available in few
This is similar with support for creating triggers via configfs.
Devices will be hosted under:
* /config/iio/devices
We allow users to register "device types" under:
* /config/iio/devices//
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
dri
patch 1)
* make iio_simple_dummy depend on SW_DEVICE config option in order to
avoid
compilation errors reported by kbuild robot.
Daniel Baluta (3):
iio: Add support for creating IIO devices via configfs
iio: dummy: Convert IIO dummy to configfs
Documentation: iio: Add II
We register a new device type named "dummy", this will create a
configfs entry under:
* /config/iio/devices/dummy.
Creating dummy devices is now as simple as:
$ mkdir /config/iio/devices/dummy/my_dummy_device
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
d
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
Documentation/ABI/testing/configfs-iio | 13 +
1 file changed, 13 insertions(+)
diff --git a/Documentation/ABI/testing/configfs-iio
b/Documentation/ABI/testing/configfs-iio
index 2483756..aebda53 100644
--- a/Documen
This is similar with support for creating triggers via configfs.
Devices will be hosted under:
* /config/iio/devices
We allow users to register "device types" under:
* /config/iio/devices//
Signed-off-by: Daniel Baluta
---
drivers/iio/Kconfig | 8 +
patch 1)
* make iio_simple_dummy depend on SW_DEVICE config option in order to
avoid
compilation errors reported by kbuild robot.
Daniel Baluta (3):
iio: Add support for creating IIO devices via configfs
iio: dummy: Convert IIO dummy to configfs
Documentation: iio: Add II
We register a new device type named "dummy", this will create a
configfs entry under:
* /config/iio/devices/dummy.
Creating dummy devices is now as simple as:
$ mkdir /config/iio/devices/dummy/my_dummy_device
Signed-off-by: Daniel Baluta
---
drivers/iio/dummy/Kconfig
Signed-off-by: Daniel Baluta
---
Documentation/ABI/testing/configfs-iio | 13 +
1 file changed, 13 insertions(+)
diff --git a/Documentation/ABI/testing/configfs-iio
b/Documentation/ABI/testing/configfs-iio
index 2483756..aebda53 100644
--- a/Documentation/ABI/testing/configfs-iio
On Sun, Apr 24, 2016 at 2:28 PM, Jonathan Cameron <ji...@kernel.org> wrote:
> On 20/04/16 16:45, Daniel Baluta wrote:
>> For testing purposes is nice to have a quick way of creating IIO devices.
>> This patch series introduces support for creating IIO devices via configs
&
On Sun, Apr 24, 2016 at 2:28 PM, Jonathan Cameron wrote:
> On 20/04/16 16:45, Daniel Baluta wrote:
>> For testing purposes is nice to have a quick way of creating IIO devices.
>> This patch series introduces support for creating IIO devices via configs
>> (patch 1), allow
This is similar with support for creating triggers via configfs.
Devices will be hosted under:
* /config/iio/devices
We allow users to register "device types" under:
* /config/iio/devices//
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
dri
This is similar with support for creating triggers via configfs.
Devices will be hosted under:
* /config/iio/devices
We allow users to register "device types" under:
* /config/iio/devices//
Signed-off-by: Daniel Baluta
---
drivers/iio/Kconfig | 9 +
On Wed, Apr 20, 2016 at 6:44 PM, Lars-Peter Clausen <l...@metafoo.de> wrote:
> On 04/20/2016 05:45 PM, Daniel Baluta wrote:
>
>> +
>> +What:/config/iio/triggers/dummy
>
> s/triggers/devices
:D, will fix in v2. Obviously, one problem of this RFC is t
On Wed, Apr 20, 2016 at 6:44 PM, Lars-Peter Clausen wrote:
> On 04/20/2016 05:45 PM, Daniel Baluta wrote:
>
>> +
>> +What:/config/iio/triggers/dummy
>
> s/triggers/devices
:D, will fix in v2. Obviously, one problem of this RFC is that we
have a lot o
RFC in order to see if the interface is acceptable. We also
need a way to create IIO devices with configurable number of channels.
Patch 3 introduces configfs entries documentation for easier review.
Daniel Baluta (3):
iio: Add support for creating IIO devices via configfs
iio: dummy: Convert I
RFC in order to see if the interface is acceptable. We also
need a way to create IIO devices with configurable number of channels.
Patch 3 introduces configfs entries documentation for easier review.
Daniel Baluta (3):
iio: Add support for creating IIO devices via configfs
iio: dummy: Convert I
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
Documentation/ABI/testing/configfs-iio | 13 +
1 file changed, 13 insertions(+)
diff --git a/Documentation/ABI/testing/configfs-iio
b/Documentation/ABI/testing/configfs-iio
index 2483756..84b6bd1 100644
--- a/Documen
Signed-off-by: Daniel Baluta
---
Documentation/ABI/testing/configfs-iio | 13 +
1 file changed, 13 insertions(+)
diff --git a/Documentation/ABI/testing/configfs-iio
b/Documentation/ABI/testing/configfs-iio
index 2483756..84b6bd1 100644
--- a/Documentation/ABI/testing/configfs-iio
We register a new device type named "dummy", this will create a
configfs entry under:
* /config/iio/devices/dummy.
Creating dummy devices is now as simple as:
$ mkdir /config/iio/devices/dummy/my_dummy_device
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
d
We register a new device type named "dummy", this will create a
configfs entry under:
* /config/iio/devices/dummy.
Creating dummy devices is now as simple as:
$ mkdir /config/iio/devices/dummy/my_dummy_device
Signed-off-by: Daniel Baluta
---
drivers/iio/dummy/iio_simple_du
ppens to fix the deadlock described in
>> http://patchwork.ozlabs.org/patch/584776/ authored by
>> Adriana Reus <adriana.r...@intel.com> and submitted by
>> Daniel Baluta <daniel.bal...@intel.com>
>>
>> --8<--
>> iio: imu: inv_mpu6050: F
http://patchwork.ozlabs.org/patch/584776/ authored by
>> Adriana Reus and submitted by
>> Daniel Baluta
>>
>> --8<--
>> iio: imu: inv_mpu6050: Fix deadlock between i2c adapter lock and mpu lock
>>
>> This deadlock occurs if the accel/gyr
interface using an external trigger (e.g.
hrtimer). Data is retrieved from IMU via I2C or SPI interface.
Datasheet is at:
http://www.mouser.com/ds/2/783/BST-BMI160-DS000-07-786474.pdf
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
Changes since v3:
* make sample an
interface using an external trigger (e.g.
hrtimer). Data is retrieved from IMU via I2C or SPI interface.
Datasheet is at:
http://www.mouser.com/ds/2/783/BST-BMI160-DS000-07-786474.pdf
Signed-off-by: Daniel Baluta
---
Changes since v3:
* make sample an __le16 instead of int.
Changes
On Sun, Apr 10, 2016 at 5:35 PM, Jonathan Cameron <ji...@kernel.org> wrote:
> On 06/04/16 15:58, Daniel Baluta wrote:
>> BMI160 is an Inertial Measurement Unit (IMU) which provides acceleration
>> and angular rate measurement. It also offers a secondary I2C interf
On Sun, Apr 10, 2016 at 5:35 PM, Jonathan Cameron wrote:
> On 06/04/16 15:58, Daniel Baluta wrote:
>> BMI160 is an Inertial Measurement Unit (IMU) which provides acceleration
>> and angular rate measurement. It also offers a secondary I2C interface
>> for connecting a magneto
and uses core
Daniel Baluta (2):
iio: magn: Split bmc150 driver in common/i2c parts
iio: magn: bmc150: Introduce SPI support
drivers/iio/magnetometer/Kconfig | 31 --
drivers/iio/magnetometer/Makefile | 3 +
drivers/iio/magnetometer/bmc150_magn.c | 155
and uses core
Daniel Baluta (2):
iio: magn: Split bmc150 driver in common/i2c parts
iio: magn: bmc150: Introduce SPI support
drivers/iio/magnetometer/Kconfig | 31 --
drivers/iio/magnetometer/Makefile | 3 +
drivers/iio/magnetometer/bmc150_magn.c | 155
This is useful for easily adding SPI support in later patches.
Now bmc150_magn exports core functions to be used by I2C/SPI drivers
instances. For the moment only I2C driver is supported.
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
drivers/iio/magnetometer/Kconfig
This is useful for easily adding SPI support in later patches.
Now bmc150_magn exports core functions to be used by I2C/SPI drivers
instances. For the moment only I2C driver is supported.
Signed-off-by: Daniel Baluta
---
drivers/iio/magnetometer/Kconfig | 17 ++--
drivers/iio
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
drivers/iio/magnetometer/Kconfig | 16 +++
drivers/iio/magnetometer/Makefile | 1 +
drivers/iio/magnetometer/bmc150_magn_spi.c | 68 ++
3 files changed, 85 insertions(+)
creat
Signed-off-by: Daniel Baluta
---
drivers/iio/magnetometer/Kconfig | 16 +++
drivers/iio/magnetometer/Makefile | 1 +
drivers/iio/magnetometer/bmc150_magn_spi.c | 68 ++
3 files changed, 85 insertions(+)
create mode 100644 drivers/iio
interface using an external trigger (e.g.
hrtimer). Data is retrieved from IMU via I2C or SPI interface.
Datasheet is at:
http://www.mouser.com/ds/2/783/BST-BMI160-DS000-07-786474.pdf
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
Changes since v2:
* include
interface using an external trigger (e.g.
hrtimer). Data is retrieved from IMU via I2C or SPI interface.
Datasheet is at:
http://www.mouser.com/ds/2/783/BST-BMI160-DS000-07-786474.pdf
Signed-off-by: Daniel Baluta
---
Changes since v2:
* include to fix compile errors reported
On Wed, Apr 6, 2016 at 1:31 PM, Laxman Dewangan wrote:
> Some of kernel driver uses the IIO framework to get the sensor
> value via ADC or IIO HW driver. The client driver get iio channel
> by iio_channel_get() and release it by calling iio_channel_release().
>
> Add
On Wed, Apr 6, 2016 at 1:31 PM, Laxman Dewangan wrote:
> Some of kernel driver uses the IIO framework to get the sensor
> value via ADC or IIO HW driver. The client driver get iio channel
> by iio_channel_get() and release it by calling iio_channel_release().
>
> Add resource managed version
interface using an external trigger (e.g.
hrtimer). Data is retrieved from IMU via I2C or SPI interface.
Datasheet is at:
http://www.mouser.com/ds/2/783/BST-BMI160-DS000-07-786474.pdf
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
Changes since v1:
* addressed co
interface using an external trigger (e.g.
hrtimer). Data is retrieved from IMU via I2C or SPI interface.
Datasheet is at:
http://www.mouser.com/ds/2/783/BST-BMI160-DS000-07-786474.pdf
Signed-off-by: Daniel Baluta
---
Changes since v1:
* addressed comments from Jonathan and Peter
On Sun, Apr 3, 2016 at 11:53 AM, Jonathan Cameron <ji...@kernel.org> wrote:
> On 01/04/16 13:31, Daniel Baluta wrote:
>> BMI160 is an Inertial Measurement Unit (IMU) which provides acceleration
>> and angular rate measurement. It also offers a secondary I2C interf
On Sun, Apr 3, 2016 at 11:53 AM, Jonathan Cameron wrote:
> On 01/04/16 13:31, Daniel Baluta wrote:
>> BMI160 is an Inertial Measurement Unit (IMU) which provides acceleration
>> and angular rate measurement. It also offers a secondary I2C interface
>> for connecting a magneto
On Fri, Apr 1, 2016 at 5:28 PM, Peter Meerwald-Stadler
wrote:
>
>> BMI160 is an Inertial Measurement Unit (IMU) which provides acceleration
>> and angular rate measurement. It also offers a secondary I2C interface
>> for connecting a magnetometer sensor (usually BMM160).
>>
>>
On Fri, Apr 1, 2016 at 5:28 PM, Peter Meerwald-Stadler
wrote:
>
>> BMI160 is an Inertial Measurement Unit (IMU) which provides acceleration
>> and angular rate measurement. It also offers a secondary I2C interface
>> for connecting a magnetometer sensor (usually BMM160).
>>
>> Current driver
interface using an external trigger (e.g.
hrtimer). Data is retrieved from IMU via I2C or SPI interface.
Datasheet is at:
http://www.mouser.com/ds/2/783/BST-BMI160-DS000-07-786474.pdf
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
drivers/iio/imu/Kconfig
interface using an external trigger (e.g.
hrtimer). Data is retrieved from IMU via I2C or SPI interface.
Datasheet is at:
http://www.mouser.com/ds/2/783/BST-BMI160-DS000-07-786474.pdf
Signed-off-by: Daniel Baluta
---
drivers/iio/imu/Kconfig | 2 +
drivers/iio/imu/Makefile
On Fri, Mar 25, 2016 at 12:20 PM, Peter Rosin <p...@axentia.se> wrote:
> Hi again,
>
> Cristina Moraru wrote:
>> Add implementation for Maxim MAX5487, MAX5488, MAX5489
>> digital potentiometers.
>>
>> Signed-off-by: Cristina Moraru <cristina.morar...@g
On Fri, Mar 25, 2016 at 12:20 PM, Peter Rosin wrote:
> Hi again,
>
> Cristina Moraru wrote:
>> Add implementation for Maxim MAX5487, MAX5488, MAX5489
>> digital potentiometers.
>>
>> Signed-off-by: Cristina Moraru
>> CC: Daniel Baluta
>
> Some m
> .wipers and .max_pos need not be in max5487_cfg, they are common.
> .wipers isn't even used. Which means that if you like, you can
> use the ohms reading as the driver_data directly instead of going
> via the MAX548x enumeration, see below. Or is there some reason
> not doing so?
You make a
> .wipers and .max_pos need not be in max5487_cfg, they are common.
> .wipers isn't even used. Which means that if you like, you can
> use the ohms reading as the driver_data directly instead of going
> via the MAX548x enumeration, see below. Or is there some reason
> not doing so?
You make a
ru <cristina.morar...@gmail.com>
> CC: Daniel Baluta <daniel.bal...@intel.com>
Tested-by: Daniel Baluta <daniel.bal...@intel.com>
On Thu, Mar 24, 2016 at 1:21 PM, Cristina Moraru
wrote:
> Add implementation for Maxim MAX5487, MAX5488, MAX5489
> digital potentiometers.
Datasheet: http://datasheets.maximintegrated.com/en/ds/MAX5487-MAX5489.pdf
>
> Signed-off-by: Cristina Moraru
> CC: Daniel Baluta
Tested-by: Daniel Baluta
: Matt Gumbel <matthew.k.gum...@intel.com>
Signed-off-by: Mohamed Abbas <mohamed.ab...@intel.com>
Signed-off-by: Constantin Musca <constantin.mu...@intel.com>
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
Changes since v1:
* updated Makefile after changing s
Abbas
Signed-off-by: Constantin Musca
Signed-off-by: Daniel Baluta
---
Changes since v1:
* updated Makefile after changing source name from efibc.c -> efi-bc.c
to comply with naming rules in drivers/firmware/efi/
drivers/firmware/efi/Kconfig | 11 ++
drivers/firmware/
: Matt Gumbel <matthew.k.gum...@intel.com>
Signed-off-by: Mohamed Abbas <mohamed.ab...@intel.com>
Signed-off-by: Constantin Musca <constantin.mu...@intel.com>
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
drivers/firmware/efi/Kconfig | 11 ++
drivers/firmware
Abbas
Signed-off-by: Constantin Musca
Signed-off-by: Daniel Baluta
---
drivers/firmware/efi/Kconfig | 11 ++
drivers/firmware/efi/Makefile | 1 +
drivers/firmware/efi/efi-bc.c | 251 ++
3 files changed, 263 insertions(+)
create mode 100644 drivers
On Wed, Mar 16, 2016 at 6:25 PM, Slawomir Stepien wrote:
> On Mar 16, 2016 13:30, Peter Meerwald-Stadler wrote:
>> On Wed, 16 Mar 2016, Slawomir Stepien wrote:
>>
>> > The following functionalities are supported:
>> > - write, read from volatile and non volatile memory
>> > -
On Wed, Mar 16, 2016 at 6:25 PM, Slawomir Stepien wrote:
> On Mar 16, 2016 13:30, Peter Meerwald-Stadler wrote:
>> On Wed, 16 Mar 2016, Slawomir Stepien wrote:
>>
>> > The following functionalities are supported:
>> > - write, read from volatile and non volatile memory
>> > - increase and
On Sun, Mar 13, 2016 at 9:21 PM, Cosmin-Gabriel Samoila
wrote:
> Fixed errors and warnings reported by checkpatch.pl.
>
Use indicative mood in your commit message.
So, s/Fixed/Fix.
http://chris.beams.io/posts/git-commit/
> Signed-off-by: Cosmin-Gabriel Samoila
On Sun, Mar 13, 2016 at 9:21 PM, Cosmin-Gabriel Samoila
wrote:
> Fixed errors and warnings reported by checkpatch.pl.
>
Use indicative mood in your commit message.
So, s/Fixed/Fix.
http://chris.beams.io/posts/git-commit/
> Signed-off-by: Cosmin-Gabriel Samoila
> ---
>
On Fri, Mar 4, 2016 at 1:09 AM, Peter Rosin <p...@lysator.liu.se> wrote:
> From: Peter Rosin <p...@axentia.se>
>
> Hi Daniel,
>
> Daniel Baluta wrote:
>>On Mon, Feb 22, 2016 at 1:17 AM, Wolfram Sang wrote:
>>> On Thu, Feb 18, 2016 at 05:53:05P
On Fri, Mar 4, 2016 at 1:09 AM, Peter Rosin wrote:
> From: Peter Rosin
>
> Hi Daniel,
>
> Daniel Baluta wrote:
>>On Mon, Feb 22, 2016 at 1:17 AM, Wolfram Sang wrote:
>>> On Thu, Feb 18, 2016 at 05:53:05PM +0200, Daniel Baluta wrote:
>>>> Sending this as
On Tue, Mar 1, 2016 at 10:50 PM, Wolfram Sang <w...@the-dreams.de> wrote:
> On Thu, Feb 18, 2016 at 05:53:14PM +0200, Daniel Baluta wrote:
>> From: Adriana Reus <adriana.r...@intel.com>
>>
>> This deadlock occurs if the accel/gyro and the sensor on the auxiliary
&
On Tue, Mar 1, 2016 at 10:50 PM, Wolfram Sang wrote:
> On Thu, Feb 18, 2016 at 05:53:14PM +0200, Daniel Baluta wrote:
>> From: Adriana Reus
>>
>> This deadlock occurs if the accel/gyro and the sensor on the auxiliary
>> I2C (in my setup it's an ak8975) a
On Tue, Mar 1, 2016 at 10:30 PM, Wolfram Sang <w...@the-dreams.de> wrote:
> On Thu, Feb 18, 2016 at 05:53:13PM +0200, Daniel Baluta wrote:
>> From: Adriana Reus <adriana.r...@intel.com>
>>
>> Add a check in i2c_mux_master_xfer before calling the select callback.
On Tue, Mar 1, 2016 at 10:30 PM, Wolfram Sang wrote:
> On Thu, Feb 18, 2016 at 05:53:13PM +0200, Daniel Baluta wrote:
>> From: Adriana Reus
>>
>> Add a check in i2c_mux_master_xfer before calling the select callback.
>> This is necessary so that NULL callbac
On Tue, Mar 1, 2016 at 4:42 AM, Michael Welling <mwell...@ieee.org> wrote:
> On Mon, Feb 29, 2016 at 10:09:10PM -0300, Lucas De Marchi wrote:
>> On Mon, Feb 29, 2016 at 9:50 PM, Michael Welling <mwell...@ieee.org> wrote:
>> > On Fri, Feb 05, 2016 at 03:17:1
On Tue, Mar 1, 2016 at 4:42 AM, Michael Welling wrote:
> On Mon, Feb 29, 2016 at 10:09:10PM -0300, Lucas De Marchi wrote:
>> On Mon, Feb 29, 2016 at 9:50 PM, Michael Welling wrote:
>> > On Fri, Feb 05, 2016 at 03:17:18PM +0200, Daniel Baluta wrote:
>> >>
On Tue, Mar 1, 2016 at 10:59 AM, Martin Kepplinger wrote:
> Would it be ok, if adding in_accelX_power_mode to a driver, to extend it
> so that in_accel_power_mode_available offers:
>
> low_noise low_power low_power_low_noise normal
>
> if there's a default "normal" mode, plus
On Tue, Mar 1, 2016 at 10:59 AM, Martin Kepplinger wrote:
> Would it be ok, if adding in_accelX_power_mode to a driver, to extend it
> so that in_accel_power_mode_available offers:
>
> low_noise low_power low_power_low_noise normal
>
> if there's a default "normal" mode, plus options to increase
On Mon, Feb 22, 2016 at 1:17 AM, Wolfram Sang <w...@the-dreams.de> wrote:
> On Thu, Feb 18, 2016 at 05:53:05PM +0200, Daniel Baluta wrote:
>> Sending this as an RFC because I don't know if style fixes are appropriate
>> for this driver and also not sure if deadlock fix
On Mon, Feb 22, 2016 at 1:17 AM, Wolfram Sang wrote:
> On Thu, Feb 18, 2016 at 05:53:05PM +0200, Daniel Baluta wrote:
>> Sending this as an RFC because I don't know if style fixes are appropriate
>> for this driver and also not sure if deadlock fix is the best solution.
>>
On Mon, Feb 22, 2016 at 1:17 AM, Wolfram Sang <w...@the-dreams.de> wrote:
> On Thu, Feb 18, 2016 at 05:53:05PM +0200, Daniel Baluta wrote:
>> Sending this as an RFC because I don't know if style fixes are appropriate
>> for this driver and also not sure if deadlock fix
On Mon, Feb 22, 2016 at 1:17 AM, Wolfram Sang wrote:
> On Thu, Feb 18, 2016 at 05:53:05PM +0200, Daniel Baluta wrote:
>> Sending this as an RFC because I don't know if style fixes are appropriate
>> for this driver and also not sure if deadlock fix is the best solution.
>>
entries to the 'Industrial I/O support ->
>> Magnetometer sensors' menu.
>>
>> Signed-off-by: Cristina Moraru <cristina.morar...@gmail.com>
>> Cc: Daniel Baluta <daniel.bal...@intel.com>
> For some reason I couldn't immediately identify this patch wouldn't
ndustrial I/O support ->
>> Magnetometer sensors' menu.
>>
>> Signed-off-by: Cristina Moraru
>> Cc: Daniel Baluta
> For some reason I couldn't immediately identify this patch wouldn't
> apply to my tree. I hand applied it and all seemed fine, but please do
> sani
This fixes the following checkpatch warning:
* WARNING: Comparisons should place the constant
on the right side of the test
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
drivers/iio/imu/inv_mpu6050/inv_mpu_core.c | 6 +++---
drivers/iio/imu/inv_m
This fixes the following checkpatch warning:
* WARNING: Comparisons should place the constant
on the right side of the test
Signed-off-by: Daniel Baluta
---
drivers/iio/imu/inv_mpu6050/inv_mpu_core.c | 6 +++---
drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c | 2 +-
2 files
This fixes the following checkpatch.pl warnings:
* WARNING: Missing a blank line after declarations
* CHECK: Blank lines aren't necessary before a close brace '}'
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.c | 2 +-
d
This fixes the following checkpatch.pl warnings:
* WARNING: Missing a blank line after declarations
* CHECK: Blank lines aren't necessary before a close brace '}'
Signed-off-by: Daniel Baluta
---
drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.c | 2 +-
drivers/iio/imu/inv_mpu6050
This fixes the following checkpatch.pl warning:
WARNING: suspect code indent for conditional statements (8, 24)
+ if (kfifo_len(>timestamps) >
[...]
+ goto flush_fifo;
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
drivers/iio/imu
This fixes the following checkpatch.pl warning:
WARNING: suspect code indent for conditional statements (8, 24)
+ if (kfifo_len(>timestamps) >
[...]
+ goto flush_fifo;
Signed-off-by: Daniel Baluta
---
drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c | 4 ++--
This fixes the following checkpatch.pl warning:
ERROR: space prohibited before that ',' (ctx:WxE)
.shift = 0 ,
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
drivers/iio/imu/inv_mpu6050/inv_mpu_core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
Fixes the following checkpatch warning:
CHECK: Unnecessary parentheses around cpm->package.elements[i]
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/driver
y sensor i2c transaction so if only the
compass is used this would be more power efficient.
Signed-off-by: Adriana Reus <adriana.r...@intel.com>
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c | 88 ++-
1 file chang
From: Adriana Reus <adriana.r...@intel.com>
Add a check in i2c_mux_master_xfer before calling the select callback.
This is necessary so that NULL callbacks can be safely registered.
Signed-off-by: Adriana Reus <adriana.r...@intel.com>
Signed-off-by: Daniel Baluta <daniel.b
This fixes the following checkpatch.pl warning:
ERROR: space prohibited before that ',' (ctx:WxE)
.shift = 0 ,
Signed-off-by: Daniel Baluta
---
drivers/iio/imu/inv_mpu6050/inv_mpu_core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/iio/imu
Fixes the following checkpatch warning:
CHECK: Unnecessary parentheses around cpm->package.elements[i]
Signed-off-by: Daniel Baluta
---
drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_acp
ompass is used this would be more power efficient.
Signed-off-by: Adriana Reus
Signed-off-by: Daniel Baluta
---
drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c | 88 ++-
1 file changed, 15 insertions(+), 73 deletions(-)
diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_i2
From: Adriana Reus
Add a check in i2c_mux_master_xfer before calling the select callback.
This is necessary so that NULL callbacks can be safely registered.
Signed-off-by: Adriana Reus
Signed-off-by: Daniel Baluta
---
drivers/i2c/i2c-mux.c | 10 ++
1 file changed, 6 insertions(+), 4
This makes code consistent around inv_mpu6050 driver and
fixes the following checkpatch.pl warning:
CHECK: Alignment should match open parenthesis
Note that there were few cases were it was not possible to
fix this due to making the line too long, but we can live with that.
Signed-off-by: Daniel
The preffered style for long (multi-line) comments is:
/*
* this is a multiline
* comment
*/
This also fixes checkpatch.pl warning:
WARNING: Block comments use * on subsequent lines
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com>
---
drivers/iio/imu/inv_mpu6050/inv_mpu_core.
This makes code consistent around inv_mpu6050 driver and
fixes the following checkpatch.pl warning:
CHECK: Alignment should match open parenthesis
Note that there were few cases were it was not possible to
fix this due to making the line too long, but we can live with that.
Signed-off-by: Daniel
601 - 700 of 1511 matches
Mail list logo