[RFC PATCH 1/9] iio: imu: inv_mpu6050: Fix multiline comments style

2016-02-18 Thread Daniel Baluta
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 --- drivers/iio/imu/inv_mpu6050/inv_mpu_core.c | 37

[RFC PATCH 0/9] iio: Fix ABBA deadlock in inv-mpu6050

2016-02-18 Thread Daniel Baluta
and mpu lock Daniel Baluta (7): iio: imu: inv_mpu6050: Fix multiline comments style iio: imu: inv_mpu6050: Fix Yoda conditions iio: imu: inv_mpu6050: Fix newlines to make code easier to read iio: imu: inv_mpu6050: Remove unnecessary parentheses iio: imu: inv_mpu6050: Delete space before

[RFC PATCH 0/9] iio: Fix ABBA deadlock in inv-mpu6050

2016-02-18 Thread Daniel Baluta
and mpu lock Daniel Baluta (7): iio: imu: inv_mpu6050: Fix multiline comments style iio: imu: inv_mpu6050: Fix Yoda conditions iio: imu: inv_mpu6050: Fix newlines to make code easier to read iio: imu: inv_mpu6050: Remove unnecessary parentheses iio: imu: inv_mpu6050: Delete space before

Re: [PATCH] iio: pressure: ms5611: select IIO_BUFFER

2016-02-16 Thread Daniel Baluta
on] > > This adds the second select. > > Signed-off-by: Arnd Bergmann <a...@arndb.de> > Fixes: 713bbb4efb9d ("iio: pressure: ms5611: Add triggered buffer support") Acked-by: Daniel Baluta <daniel.bal...@intel.com> > --- > drivers/iio/pressure/Kconfig | 1

Re: [PATCH] iio: pressure: ms5611: select IIO_BUFFER

2016-02-16 Thread Daniel Baluta
his adds the second select. > > Signed-off-by: Arnd Bergmann > Fixes: 713bbb4efb9d ("iio: pressure: ms5611: Add triggered buffer support") Acked-by: Daniel Baluta > --- > drivers/iio/pressure/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/iio/pr

Re: [PATCH] iio: health/afe4403: mark suspend/resume functions __maybe_unused

2016-02-15 Thread Daniel Baluta
On Mon, Feb 15, 2016 at 11:02 AM, Arnd Bergmann wrote: > The newly added afe4403 driver implements suspend/resume using the > SIMPLE_DEV_PM_OPS() macro, which leaves out references to the actual > functions when CONFIG_PM is disabled, causing a harmless warning: > >

Re: [PATCH] iio: health/afe4403: mark suspend/resume functions __maybe_unused

2016-02-15 Thread Daniel Baluta
On Mon, Feb 15, 2016 at 11:02 AM, Arnd Bergmann wrote: > The newly added afe4403 driver implements suspend/resume using the > SIMPLE_DEV_PM_OPS() macro, which leaves out references to the actual > functions when CONFIG_PM is disabled, causing a harmless warning: > > health/afe4403.c:509:12:

[PATCH v5] iio: adc: Add TI ADS1015 ADC driver support

2016-02-11 Thread Daniel Baluta
The driver has sysfs readings with runtime PM support for power saving. It also offers buffer support that can be used together with IIO software triggers. Datasheet can be found here: http://www.ti.com.cn/cn/lit/ds/symlink/ads1015.pdf Signed-off-by: Daniel Baluta --- Changes since v4

[PATCH v5] iio: adc: Add TI ADS1015 ADC driver support

2016-02-11 Thread Daniel Baluta
The driver has sysfs readings with runtime PM support for power saving. It also offers buffer support that can be used together with IIO software triggers. Datasheet can be found here: http://www.ti.com.cn/cn/lit/ds/symlink/ads1015.pdf Signed-off-by: Daniel Baluta <daniel.

Re: [PATCH v4] iio: adc: Add TI ADS1015 ADC driver support

2016-02-10 Thread Daniel Baluta
>> > > > > > >> +static int ads1015_read_raw(struct iio_dev *indio_dev, >> > > > > > >> + struct iio_chan_spec const *chan, >> > > > > > >> +int >> > *val, >> > > > > > >> + int *val2, long mask) { >> > > > > > >> + int ret, idx; >> > > > > >

Re: [PATCH v4] iio: adc: Add TI ADS1015 ADC driver support

2016-02-10 Thread Daniel Baluta
>> > > > > > >> +static int ads1015_read_raw(struct iio_dev *indio_dev, >> > > > > > >> + struct iio_chan_spec const *chan, >> > > > > > >> +int >> > *val, >> > > > > > >> + int *val2, long mask) { >> > > > > > >> + int ret, idx; >> > > > > >

Re: [PATCH v4] iio: adc: Add TI ADS1015 ADC driver support

2016-02-08 Thread Daniel Baluta
On Mon, Feb 8, 2016 at 4:44 PM, Daniel Baluta wrote: > On Mon, Feb 8, 2016 at 12:25 PM, Wolfram Sang wrote: >> On Fri, Feb 05, 2016 at 06:32:45PM -0600, Michael Welling wrote: >>> On Fri, Feb 05, 2016 at 09:32:34PM +0200, Daniel Baluta wrote: >>> > >> +static

[PATCH] iio: Fix documentation for iio_dev mlock

2016-02-08 Thread Daniel Baluta
mlock *must* be used by core and drivers to protect access to devices state changes. Signed-off-by: Daniel Baluta --- At a first glance there are few places where access to iio_dev state changes are done without mlock protection. Will fix them with follow up patches. include/linux/iio/iio.h

Re: [PATCH v4] iio: adc: Add TI ADS1015 ADC driver support

2016-02-08 Thread Daniel Baluta
On Mon, Feb 8, 2016 at 12:25 PM, Wolfram Sang wrote: > On Fri, Feb 05, 2016 at 06:32:45PM -0600, Michael Welling wrote: >> On Fri, Feb 05, 2016 at 09:32:34PM +0200, Daniel Baluta wrote: >> > >> +static int ads1015_read_raw(struct iio_dev *indio_dev, >> > &g

Re: [PATCH v4] iio: adc: Add TI ADS1015 ADC driver support

2016-02-08 Thread Daniel Baluta
On Mon, Feb 8, 2016 at 12:25 PM, Wolfram Sang <w...@the-dreams.de> wrote: > On Fri, Feb 05, 2016 at 06:32:45PM -0600, Michael Welling wrote: >> On Fri, Feb 05, 2016 at 09:32:34PM +0200, Daniel Baluta wrote: >> > >> +static int ads1015_read

Re: [PATCH v4] iio: adc: Add TI ADS1015 ADC driver support

2016-02-08 Thread Daniel Baluta
On Mon, Feb 8, 2016 at 4:44 PM, Daniel Baluta <daniel.bal...@intel.com> wrote: > On Mon, Feb 8, 2016 at 12:25 PM, Wolfram Sang <w...@the-dreams.de> wrote: >> On Fri, Feb 05, 2016 at 06:32:45PM -0600, Michael Welling wrote: >>> On Fri, Feb 05, 2016 at 09:32:3

[PATCH] iio: Fix documentation for iio_dev mlock

2016-02-08 Thread Daniel Baluta
mlock *must* be used by core and drivers to protect access to devices state changes. Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- At a first glance there are few places where access to iio_dev state changes are done without mlock protection. Will fix them with follow up p

Re: [PATCH v4] iio: adc: Add TI ADS1015 ADC driver support

2016-02-05 Thread Daniel Baluta
On Fri, Feb 5, 2016 at 11:02 PM, Lucas De Marchi wrote: > Hi Daniel, > > On Fri, Feb 5, 2016 at 11:17 AM, Daniel Baluta > wrote: >> + >> +static const struct i2c_device_id ads1015_id[] = { >> + {"ads1015", 0}, >> + {} >&

Re: [PATCH v4] iio: adc: Add TI ADS1015 ADC driver support

2016-02-05 Thread Daniel Baluta
On Fri, Feb 5, 2016 at 7:25 PM, Michael Welling wrote: > On Fri, Feb 05, 2016 at 03:17:18PM +0200, Daniel Baluta wrote: >> The driver has sysfs readings with runtime PM support for power saving. >> It also offers buffer support that can be used together with IIO softw

[PATCH v4] iio: adc: Add TI ADS1015 ADC driver support

2016-02-05 Thread Daniel Baluta
The driver has sysfs readings with runtime PM support for power saving. It also offers buffer support that can be used together with IIO software triggers. Datasheet can be found here: http://www.ti.com.cn/cn/lit/ds/symlink/ads1015.pdf Signed-off-by: Daniel Baluta --- Changes since v3

Re: [RFC PATCH 1/2] iio: hmc5843 Add channel attribute for bias configuration

2016-02-05 Thread Daniel Baluta
On Thu, Feb 4, 2016 at 7:45 PM, Lars-Peter Clausen wrote: > On 02/04/2016 03:50 PM, Cristina Moraru wrote: >> Replace non standard meas_conf attribute with the standard IIO >> calibbias attribute. >> >> API for setting bias measurement configuration: >> >> 0 - Normal measurement configuration

Re: [PATCH v4] iio: adc: Add TI ADS1015 ADC driver support

2016-02-05 Thread Daniel Baluta
On Fri, Feb 5, 2016 at 11:02 PM, Lucas De Marchi <lucas.de.mar...@gmail.com> wrote: > Hi Daniel, > > On Fri, Feb 5, 2016 at 11:17 AM, Daniel Baluta <daniel.bal...@intel.com> > wrote: >> + >> +static const struct i2c_device_id ads10

Re: [PATCH v4] iio: adc: Add TI ADS1015 ADC driver support

2016-02-05 Thread Daniel Baluta
On Fri, Feb 5, 2016 at 7:25 PM, Michael Welling <mwell...@ieee.org> wrote: > On Fri, Feb 05, 2016 at 03:17:18PM +0200, Daniel Baluta wrote: >> The driver has sysfs readings with runtime PM support for power saving. >> It also offers buffer support that can be used toget

Re: [RFC PATCH 1/2] iio: hmc5843 Add channel attribute for bias configuration

2016-02-05 Thread Daniel Baluta
On Thu, Feb 4, 2016 at 7:45 PM, Lars-Peter Clausen wrote: > On 02/04/2016 03:50 PM, Cristina Moraru wrote: >> Replace non standard meas_conf attribute with the standard IIO >> calibbias attribute. >> >> API for setting bias measurement configuration: >> >> 0 - Normal measurement

[PATCH v4] iio: adc: Add TI ADS1015 ADC driver support

2016-02-05 Thread Daniel Baluta
The driver has sysfs readings with runtime PM support for power saving. It also offers buffer support that can be used together with IIO software triggers. Datasheet can be found here: http://www.ti.com.cn/cn/lit/ds/symlink/ads1015.pdf Signed-off-by: Daniel Baluta <daniel.

Re: [PATCH v3] iio: adc: Add TI ADS1015 ADC driver support

2016-02-04 Thread Daniel Baluta
> Would it be more consistent to handle the mutex outside of the switch above > similar > to how it is handled in ads1015_write_raw? > > Also the ads1015_set_power_state(data, false) is called either way so why not > just > use one call? > I don't have a strong preference for that. I think

Re: [PATCH v3] iio: adc: Add TI ADS1015 ADC driver support

2016-02-04 Thread Daniel Baluta
> Would it be more consistent to handle the mutex outside of the switch above > similar > to how it is handled in ads1015_write_raw? > > Also the ads1015_set_power_state(data, false) is called either way so why not > just > use one call? > I don't have a strong preference for that. I think

[PATCH v3] iio: adc: Add TI ADS1015 ADC driver support

2016-02-03 Thread Daniel Baluta
The driver has sysfs readings with runtime PM support for power saving. It also offers buffer support that can be used together with IIO software triggers. Datasheet can be found here: http://www.ti.com.cn/cn/lit/ds/symlink/ads1015.pdf Signed-off-by: Daniel Baluta --- Changes since v2

[PATCH v3 0/2] ms5611: Add triggered buffer support

2016-02-03 Thread Daniel Baluta
* correctly handle data copy into buffer when not all channels are enabled * fixed timestamp channel index (2 instead of 3) * added some new lines for readability Daniel Baluta (2): iio: pressure: ms5611: Add IIO_CHAN_INFO_SCALE to mask iio: pressure: ms5611: Add triggered buff

[PATCH v3 2/2] iio: pressure: ms5611: Add triggered buffer support

2016-02-03 Thread Daniel Baluta
This will be used together with an external trigger (e.g hrtimer based software trigger). Signed-off-by: Daniel Baluta --- drivers/iio/pressure/Kconfig | 1 + drivers/iio/pressure/ms5611.h | 1 + drivers/iio/pressure/ms5611_core.c | 74

[PATCH v3 1/2] iio: pressure: ms5611: Add IIO_CHAN_INFO_SCALE to mask

2016-02-03 Thread Daniel Baluta
This allows data exported via buffer interface to be converted to standard units in userspace. Signed-off-by: Daniel Baluta --- drivers/iio/pressure/ms5611_core.c | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/drivers/iio/pressure/ms5611_core.c b

[PATCH v2] iio: adc: Add TI ADS1015 ADC driver support

2016-02-03 Thread Daniel Baluta
The driver has sysfs readings with runtime PM support for power saving. It also offers buffer support that can be used together with IIO software triggers. Datasheet can be found here: http://www.ti.com.cn/cn/lit/ds/symlink/ads1015.pdf Signed-off-by: Daniel Baluta --- Changes since v1

[PATCH v2] iio: adc: Add TI ADS1015 ADC driver support

2016-02-03 Thread Daniel Baluta
The driver has sysfs readings with runtime PM support for power saving. It also offers buffer support that can be used together with IIO software triggers. Datasheet can be found here: http://www.ti.com.cn/cn/lit/ds/symlink/ads1015.pdf Signed-off-by: Daniel Baluta <daniel.

[PATCH v3 2/2] iio: pressure: ms5611: Add triggered buffer support

2016-02-03 Thread Daniel Baluta
This will be used together with an external trigger (e.g hrtimer based software trigger). Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- drivers/iio/pressure/Kconfig | 1 + drivers/iio/pressure/ms5611.h | 1 + drivers/iio/pressure/ms5611_core.

[PATCH v3 1/2] iio: pressure: ms5611: Add IIO_CHAN_INFO_SCALE to mask

2016-02-03 Thread Daniel Baluta
This allows data exported via buffer interface to be converted to standard units in userspace. Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- drivers/iio/pressure/ms5611_core.c | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/drive

[PATCH v3 0/2] ms5611: Add triggered buffer support

2016-02-03 Thread Daniel Baluta
* correctly handle data copy into buffer when not all channels are enabled * fixed timestamp channel index (2 instead of 3) * added some new lines for readability Daniel Baluta (2): iio: pressure: ms5611: Add IIO_CHAN_INFO_SCALE to mask iio: pressure: ms5611: Add triggered buff

[PATCH v3] iio: adc: Add TI ADS1015 ADC driver support

2016-02-03 Thread Daniel Baluta
The driver has sysfs readings with runtime PM support for power saving. It also offers buffer support that can be used together with IIO software triggers. Datasheet can be found here: http://www.ti.com.cn/cn/lit/ds/symlink/ads1015.pdf Signed-off-by: Daniel Baluta <daniel.

Re: [PATCH] iio: adc: Add TI ADS1015 ADC driver support

2016-01-27 Thread Daniel Baluta
On Sat, Jan 16, 2016 at 2:07 PM, Jonathan Cameron wrote: > On 14/01/16 16:25, Daniel Baluta wrote: >> The driver has sysfs readings with runtime PM support for power saving. >> It also offers buffer support that can be used together with IIO software >> triggers. >> >

Re: [PATCH] iio: adc: Add TI ADS1015 ADC driver support

2016-01-27 Thread Daniel Baluta
On Wed, Jan 27, 2016 at 12:28 AM, Lucas De Marchi wrote: > Hi, > > On Thu, Jan 14, 2016 at 2:25 PM, Daniel Baluta > wrote: >> The driver has sysfs readings with runtime PM support for power saving. >> It also offers buffer support that can be used together with

Re: [PATCH 2/2] iio: si7005: add support for Hoperf th02

2016-01-27 Thread Daniel Baluta
On Wed, Jan 27, 2016 at 12:56 AM, Matt Ranostay wrote: > On Tue, Jan 26, 2016 at 12:21 PM, Cristina Moraru > wrote: >> This patch adds support for Hoperf th02 humidity and >> temperature sensor as it uses same register definitions >> as si7005 >> >> th02 Datasheet: >>

Re: [PATCH] iio: adc: Add TI ADS1015 ADC driver support

2016-01-27 Thread Daniel Baluta
On Sat, Jan 16, 2016 at 2:07 PM, Jonathan Cameron <ji...@kernel.org> wrote: > On 14/01/16 16:25, Daniel Baluta wrote: >> The driver has sysfs readings with runtime PM support for power saving. >> It also offers buffer support that can be used together with IIO software >>

Re: [PATCH] iio: adc: Add TI ADS1015 ADC driver support

2016-01-27 Thread Daniel Baluta
On Wed, Jan 27, 2016 at 12:28 AM, Lucas De Marchi <lucas.de.mar...@gmail.com> wrote: > Hi, > > On Thu, Jan 14, 2016 at 2:25 PM, Daniel Baluta <daniel.bal...@intel.com> > wrote: >> The driver has sysfs readings with runtime PM support for power saving. >> It

Re: [PATCH 2/2] iio: si7005: add support for Hoperf th02

2016-01-27 Thread Daniel Baluta
On Wed, Jan 27, 2016 at 12:56 AM, Matt Ranostay wrote: > On Tue, Jan 26, 2016 at 12:21 PM, Cristina Moraru > wrote: >> This patch adds support for Hoperf th02 humidity and >> temperature sensor as it uses same register definitions >> as si7005 >>

[PATCH v2 0/2] ms5611: Add triggered buffer support

2016-01-26 Thread Daniel Baluta
ressed comments from Peter * explain why the need for allocating a buffer of 4 s32 elements * correctly handle data copy into buffer when not all channels are enabled * fixed timestamp channel index (2 instead of 3) * added some new lines for readability Dani

[PATCH v2 1/2] iio: pressure: ms5611: Add IIO_CHAN_INFO_SCALE to mask

2016-01-26 Thread Daniel Baluta
This allows data exported via buffer interface to be converted to standard units in userspace. Signed-off-by: Daniel Baluta --- drivers/iio/pressure/ms5611_core.c | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/drivers/iio/pressure/ms5611_core.c b

[PATCH v2 2/2] iio: pressure: ms5611: Add triggered buffer support

2016-01-26 Thread Daniel Baluta
This will be used together with an external trigger (e.g hrtimer based software trigger). Signed-off-by: Daniel Baluta --- drivers/iio/pressure/Kconfig | 1 + drivers/iio/pressure/ms5611.h | 1 + drivers/iio/pressure/ms5611_core.c | 77

[PATCH 1/2] iio: pressure: ms5611: Add IIO_CHAN_INFO_SCALE to mask

2016-01-26 Thread Daniel Baluta
This allows data exported via buffer interface to be converted to standard units in userspace. Signed-off-by: Daniel Baluta --- drivers/iio/pressure/ms5611_core.c | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/drivers/iio/pressure/ms5611_core.c b

[PATCH 2/2] iio: pressure: ms5611: Add triggered buffer support

2016-01-26 Thread Daniel Baluta
This will be used together with an external trigger (e.g hrtimer based software trigger). Signed-off-by: Daniel Baluta --- drivers/iio/pressure/Kconfig | 1 + drivers/iio/pressure/ms5611.h | 1 + drivers/iio/pressure/ms5611_core.c | 71

[PATCH 0/2] ms5611: Add triggered buffer support

2016-01-26 Thread Daniel Baluta
First patch reverts 7cb46c2a066 ("iio: pressure: ms5611: remove IIO_CHAN_INFO_SCALE from mask") because we need the scale to correctly transform "raw" data from buffer into correct measurement units. Second patch adds the actual buffer support. Daniel Baluta (2): iio: p

[PATCH 0/2] ms5611: Add triggered buffer support

2016-01-26 Thread Daniel Baluta
First patch reverts 7cb46c2a066 ("iio: pressure: ms5611: remove IIO_CHAN_INFO_SCALE from mask") because we need the scale to correctly transform "raw" data from buffer into correct measurement units. Second patch adds the actual buffer support. Daniel Baluta (2): iio: p

[PATCH 2/2] iio: pressure: ms5611: Add triggered buffer support

2016-01-26 Thread Daniel Baluta
This will be used together with an external trigger (e.g hrtimer based software trigger). Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- drivers/iio/pressure/Kconfig | 1 + drivers/iio/pressure/ms5611.h | 1 + drivers/iio/pressure/ms5611_core.

[PATCH 1/2] iio: pressure: ms5611: Add IIO_CHAN_INFO_SCALE to mask

2016-01-26 Thread Daniel Baluta
This allows data exported via buffer interface to be converted to standard units in userspace. Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- drivers/iio/pressure/ms5611_core.c | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/drive

[PATCH v2 2/2] iio: pressure: ms5611: Add triggered buffer support

2016-01-26 Thread Daniel Baluta
This will be used together with an external trigger (e.g hrtimer based software trigger). Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- drivers/iio/pressure/Kconfig | 1 + drivers/iio/pressure/ms5611.h | 1 + drivers/iio/pressure/ms5611_core.

[PATCH v2 1/2] iio: pressure: ms5611: Add IIO_CHAN_INFO_SCALE to mask

2016-01-26 Thread Daniel Baluta
This allows data exported via buffer interface to be converted to standard units in userspace. Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- drivers/iio/pressure/ms5611_core.c | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/drive

[PATCH v2 0/2] ms5611: Add triggered buffer support

2016-01-26 Thread Daniel Baluta
ressed comments from Peter * explain why the need for allocating a buffer of 4 s32 elements * correctly handle data copy into buffer when not all channels are enabled * fixed timestamp channel index (2 instead of 3) * added some new lines for readability Dani

Re: Issues with Lenovo Yoga 900 IIO devices (accelerometer, etc.)

2015-12-16 Thread Daniel Baluta
On Tue, Dec 15, 2015 at 9:19 PM, Nish Aravamudan wrote: > So, I apologize in advance for this relatively vague report, but I'm fairly > sure > the Yoga 900 has an accelerometer amongst other sensors (ambient light?) > exported over IIO. > > But, these sensors seem to not be updating at all with

Re: Issues with Lenovo Yoga 900 IIO devices (accelerometer, etc.)

2015-12-16 Thread Daniel Baluta
On Tue, Dec 15, 2015 at 9:19 PM, Nish Aravamudan wrote: > So, I apologize in advance for this relatively vague report, but I'm fairly > sure > the Yoga 900 has an accelerometer amongst other sensors (ambient light?) > exported over IIO. > > But, these sensors seem to

[PATCH] tools/lguest: Don't bork the terminal in case of wrong args

2015-12-08 Thread Daniel Baluta
Running lguest without arguments or with a wrong argument name borks the terminal, because the cleanup handler is set up too late in the initialization process. Signed-off-by: Daniel Baluta --- tools/lguest/lguest.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tools

[PATCH] tools/lguest: Don't bork the terminal in case of wrong args

2015-12-08 Thread Daniel Baluta
Running lguest without arguments or with a wrong argument name borks the terminal, because the cleanup handler is set up too late in the initialization process. Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- tools/lguest/lguest.c | 6 +++--- 1 file changed, 3 insertions

Re: [PATCH] Doc: Docbook/iio: Fix typo in iio.tmpl

2015-11-20 Thread Daniel Baluta
On 11/20/2015 05:31 PM, Masanari Iida wrote: This patch fix a spelling typo in iio.tmpl. Signed-off-by: Masanari Iida Acked-by: Daniel Baluta --- Documentation/DocBook/iio.tmpl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/DocBook/iio.tmpl b

Re: [PATCH] Doc: Docbook/iio: Fix typo in iio.tmpl

2015-11-20 Thread Daniel Baluta
On 11/20/2015 05:31 PM, Masanari Iida wrote: This patch fix a spelling typo in iio.tmpl. Signed-off-by: Masanari Iida <standby2...@gmail.com> Acked-by: Daniel Baluta <daniel.bal...@intel.com> --- Documentation/DocBook/iio.tmpl | 2 +- 1 file changed, 1 insertion(+), 1 delet

Re: [RFC 5/9] iio: Documentation: Add IIO configfs documentation

2015-11-18 Thread Daniel Baluta
On Wed, Nov 18, 2015 at 6:06 PM, Marc Titinger wrote: > On 18/11/2015 16:38, Crt Mori wrote: >> >> This has my acked-by, yet it is missing a commit message? Can we get that >> and the removed From: as I think this is a new patch and not resend of >> Daniel's? >> Also it would need your

Re: [RFC 5/9] iio: Documentation: Add IIO configfs documentation

2015-11-18 Thread Daniel Baluta
On Wed, Nov 18, 2015 at 6:06 PM, Marc Titinger wrote: > On 18/11/2015 16:38, Crt Mori wrote: >> >> This has my acked-by, yet it is missing a commit message? Can we get that >> and the removed From: as I think this is a new patch and not resend of >> Daniel's? >> Also it

Re: [RFC 1/4] iio: ina2xx: add direct IO support for TI INA2xx Power Monitors

2015-11-11 Thread Daniel Baluta
> Signed-off-by: Marc Titinger > --- > drivers/iio/adc/Kconfig | 9 + > drivers/iio/adc/Makefile | 1 + > drivers/iio/adc/ina2xx-iio.c | 404 > +++ One more thing. In IIO we do not prefer generic names for files. Lets name this after the

Re: [RFC 2/4] iio: ina2xx: add SAMP_FREQ attribute.

2015-11-11 Thread Daniel Baluta
On Tue, Nov 10, 2015 at 6:07 PM, Marc Titinger wrote: > Signed-off-by: Marc Titinger I'm in favor of small incremental patches, anyhow since this is the initial submission I can't see why this patch isn't squashed in the previous one? > --- > drivers/iio/adc/ina2xx-iio.c | 51 >

Re: [RFC 1/4] iio: ina2xx: add direct IO support for TI INA2xx Power Monitors

2015-11-11 Thread Daniel Baluta
On Tue, Nov 10, 2015 at 6:07 PM, Marc Titinger wrote: > Basic support or direct IO raw read, with averaging attribute. > Values are RAW, INT_PLUS_MICRO (Volt/Ampere/Watt). > > IIO context has 1 devices: Is this from libiio right? Perhaps you should specify this: "libiio context has 1 devices" >

Re: [RFC 1/4] iio: ina2xx: add direct IO support for TI INA2xx Power Monitors

2015-11-11 Thread Daniel Baluta
On Tue, Nov 10, 2015 at 6:07 PM, Marc Titinger wrote: > Basic support or direct IO raw read, with averaging attribute. > Values are RAW, INT_PLUS_MICRO (Volt/Ampere/Watt). > > IIO context has 1 devices: Is this from libiio right? Perhaps you should specify this: "libiio

Re: [RFC 2/4] iio: ina2xx: add SAMP_FREQ attribute.

2015-11-11 Thread Daniel Baluta
On Tue, Nov 10, 2015 at 6:07 PM, Marc Titinger wrote: > Signed-off-by: Marc Titinger I'm in favor of small incremental patches, anyhow since this is the initial submission I can't see why this patch isn't squashed in the previous one? > --- >

Re: [RFC 1/4] iio: ina2xx: add direct IO support for TI INA2xx Power Monitors

2015-11-11 Thread Daniel Baluta
> Signed-off-by: Marc Titinger > --- > drivers/iio/adc/Kconfig | 9 + > drivers/iio/adc/Makefile | 1 + > drivers/iio/adc/ina2xx-iio.c | 404 > +++ One more thing. In IIO we do not prefer generic names for files.

Re: [PATCH v10 1/5] configfs: Allow dynamic group creation

2015-11-06 Thread Daniel Baluta
On Fri, Nov 6, 2015 at 9:50 AM, Christoph Hellwig wrote: > Is this going into 4.4 through the iio tree? If not is there any chance > to get it in through some other tree? While we're not past the merge > window is trivial new functionality that doesn't change code, and I'd like to > move

Re: [PATCH v10 1/5] configfs: Allow dynamic group creation

2015-11-06 Thread Daniel Baluta
On Fri, Nov 6, 2015 at 9:50 AM, Christoph Hellwig wrote: > Is this going into 4.4 through the iio tree? If not is there any chance > to get it in through some other tree? While we're not past the merge > window is trivial new functionality that doesn't change code, and I'd like to

[PATCH v10 3/5] iio: core: Introduce IIO software triggers

2015-10-23 Thread Daniel Baluta
The first supported trigger type is "hrtimer" found under /config/iio/triggers/hrtimer. Signed-off-by: Daniel Baluta --- drivers/iio/Kconfig | 8 ++ drivers/iio/Makefile | 1 + drivers/iio/industrialio-sw-trigger.c | 183 +

[PATCH v10 1/5] configfs: Allow dynamic group creation

2015-10-23 Thread Daniel Baluta
-by: Lars-Peter Clausen Signed-off-by: Daniel Baluta --- fs/configfs/dir.c| 110 +++ include/linux/configfs.h | 10 + 2 files changed, 120 insertions(+) diff --git a/fs/configfs/dir.c b/fs/configfs/dir.c index c81ce7f..a7a1b21 100644 --- a/fs

[PATCH v10 4/5] iio: trigger: Introduce IIO hrtimer based trigger

2015-10-23 Thread Daniel Baluta
is based on an older version from Marten and Lars-Peter. Signed-off-by: Marten Svanfeldt Signed-off-by: Lars-Peter Clausen Signed-off-by: Daniel Baluta --- drivers/iio/trigger/Kconfig| 10 ++ drivers/iio/trigger/Makefile | 2 + drivers/iio/trigger/iio-trig-hrtimer.c | 193

[PATCH v10 2/5] iio: core: Introduce IIO configfs support

2015-10-23 Thread Daniel Baluta
-by: Daniel Baluta --- drivers/iio/Kconfig | 8 ++ drivers/iio/Makefile| 1 + drivers/iio/industrialio-configfs.c | 50 + 3 files changed, 59 insertions(+) create mode 100644 drivers/iio/industrialio-configfs.c diff --git

[PATCH v10 0/5] Add initial configfs support for IIO

2015-10-23 Thread Daniel Baluta
r work" chapter in iio_configfs.txt - added configfs-iio file in Documentation/ABI/testing Daniel Baluta (5): configfs: Allow dynamic group creation iio: core: Introduce IIO configfs support iio: core: Introduce IIO software triggers iio: trigger: Introduce IIO hrtimer

[PATCH v10 5/5] iio: Documentation: Add IIO configfs documentation

2015-10-23 Thread Daniel Baluta
Signed-off-by: Daniel Baluta --- Documentation/ABI/testing/configfs-iio | 21 Documentation/iio/iio_configfs.txt | 93 ++ 2 files changed, 114 insertions(+) create mode 100644 Documentation/ABI/testing/configfs-iio create mode 100644 Documentation

Re: [PATCH v9 1/5] configfs: Allow dynamic group creation

2015-10-23 Thread Daniel Baluta
Christoph, one question. See below. On Fri, Oct 23, 2015 at 3:48 PM, Christoph Hellwig wrote: > Hi Daniel, > > all these functions look great, but can you also move your new > iio_sw_trigger_type_configfs_register/unregister functions to the > core code as those are the wrappers that everyone

Re: [PATCH v9 1/5] configfs: Allow dynamic group creation

2015-10-23 Thread Daniel Baluta
On Fri, Oct 23, 2015 at 3:48 PM, Christoph Hellwig wrote: > Hi Daniel, > > all these functions look great, but can you also move your new > iio_sw_trigger_type_configfs_register/unregister functions to the > core code as those are the wrappers that everyone would have to write, > e.g. something

[PATCH v9 1/5] configfs: Allow dynamic group creation

2015-10-23 Thread Daniel Baluta
-by: Lars-Peter Clausen Signed-off-by: Daniel Baluta --- fs/configfs/dir.c| 102 +++ include/linux/configfs.h | 8 2 files changed, 110 insertions(+) diff --git a/fs/configfs/dir.c b/fs/configfs/dir.c index c81ce7f..bf76795 100644 --- a/fs

[PATCH v9 4/5] iio: trigger: Introduce IIO hrtimer based trigger

2015-10-23 Thread Daniel Baluta
is based on an older version from Marten and Lars-Peter. Signed-off-by: Marten Svanfeldt Signed-off-by: Lars-Peter Clausen Signed-off-by: Daniel Baluta --- drivers/iio/trigger/Kconfig| 10 ++ drivers/iio/trigger/Makefile | 2 + drivers/iio/trigger/iio-trig-hrtimer.c | 193

[PATCH v9 0/5] Add initial configfs support for IIO

2015-10-23 Thread Daniel Baluta
patch 4/4 - removed "Further work" chapter in iio_configfs.txt - added configfs-iio file in Documentation/ABI/testing Daniel Baluta (5): configfs: Allow dynamic group creation iio: core: Introduce IIO configfs support iio: core: Introduce IIO sof

[PATCH v9 2/5] iio: core: Introduce IIO configfs support

2015-10-23 Thread Daniel Baluta
-by: Daniel Baluta --- drivers/iio/Kconfig | 8 ++ drivers/iio/Makefile| 1 + drivers/iio/industrialio-configfs.c | 50 + 3 files changed, 59 insertions(+) create mode 100644 drivers/iio/industrialio-configfs.c diff --git

[PATCH v9 5/5] iio: Documentation: Add IIO configfs documentation

2015-10-23 Thread Daniel Baluta
Signed-off-by: Daniel Baluta --- Documentation/ABI/testing/configfs-iio | 21 Documentation/iio/iio_configfs.txt | 93 ++ 2 files changed, 114 insertions(+) create mode 100644 Documentation/ABI/testing/configfs-iio create mode 100644 Documentation

[PATCH v9 3/5] iio: core: Introduce IIO software triggers

2015-10-23 Thread Daniel Baluta
The first support trigger type is "hrtimer" found under /config/iio/triggers/hrtimer. Signed-off-by: Daniel Baluta --- drivers/iio/Kconfig | 8 ++ drivers/iio/Makefile | 1 + drivers/iio/industrialio-sw-trigger.c | 206 +

Re: [PATCH v9 1/5] configfs: Allow dynamic group creation

2015-10-23 Thread Daniel Baluta
On Fri, Oct 23, 2015 at 3:48 PM, Christoph Hellwig wrote: > Hi Daniel, > > all these functions look great, but can you also move your new > iio_sw_trigger_type_configfs_register/unregister functions to the > core code as those are the wrappers that everyone would have to write, >

Re: [PATCH v9 1/5] configfs: Allow dynamic group creation

2015-10-23 Thread Daniel Baluta
Christoph, one question. See below. On Fri, Oct 23, 2015 at 3:48 PM, Christoph Hellwig wrote: > Hi Daniel, > > all these functions look great, but can you also move your new > iio_sw_trigger_type_configfs_register/unregister functions to the > core code as those are the wrappers

[PATCH v9 3/5] iio: core: Introduce IIO software triggers

2015-10-23 Thread Daniel Baluta
The first support trigger type is "hrtimer" found under /config/iio/triggers/hrtimer. Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- drivers/iio/Kconfig | 8 ++ drivers/iio/Makefile | 1 + drivers/iio/indus

[PATCH v9 1/5] configfs: Allow dynamic group creation

2015-10-23 Thread Daniel Baluta
-by: Lars-Peter Clausen <l...@metafoo.de> Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- fs/configfs/dir.c| 102 +++ include/linux/configfs.h | 8 2 files changed, 110 insertions(+) diff --git a/fs/configfs/dir.c b/fs/co

[PATCH v9 4/5] iio: trigger: Introduce IIO hrtimer based trigger

2015-10-23 Thread Daniel Baluta
is based on an older version from Marten and Lars-Peter. Signed-off-by: Marten Svanfeldt <mar...@intuitiveaerial.com> Signed-off-by: Lars-Peter Clausen <l...@metafoo.de> Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- drivers/iio/trigger/Kconfig| 10 ++ d

[PATCH v9 0/5] Add initial configfs support for IIO

2015-10-23 Thread Daniel Baluta
patch 4/4 - removed "Further work" chapter in iio_configfs.txt - added configfs-iio file in Documentation/ABI/testing Daniel Baluta (5): configfs: Allow dynamic group creation iio: core: Introduce IIO configfs support iio: core: Introduce IIO sof

[PATCH v9 2/5] iio: core: Introduce IIO configfs support

2015-10-23 Thread Daniel Baluta
-by: Daniel Baluta <daniel.bal...@intel.com> --- drivers/iio/Kconfig | 8 ++ drivers/iio/Makefile| 1 + drivers/iio/industrialio-configfs.c | 50 + 3 files changed, 59 insertions(+) create mode 100644 drivers/iio/indust

[PATCH v9 5/5] iio: Documentation: Add IIO configfs documentation

2015-10-23 Thread Daniel Baluta
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- Documentation/ABI/testing/configfs-iio | 21 Documentation/iio/iio_configfs.txt | 93 ++ 2 files changed, 114 insertions(+) create mode 100644 Documentation/ABI/testing/configfs-iio

[PATCH v10 1/5] configfs: Allow dynamic group creation

2015-10-23 Thread Daniel Baluta
-by: Lars-Peter Clausen <l...@metafoo.de> Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- fs/configfs/dir.c| 110 +++ include/linux/configfs.h | 10 + 2 files changed, 120 insertions(+) diff --git a/fs/configfs/dir.c b/fs/co

[PATCH v10 3/5] iio: core: Introduce IIO software triggers

2015-10-23 Thread Daniel Baluta
The first supported trigger type is "hrtimer" found under /config/iio/triggers/hrtimer. Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- drivers/iio/Kconfig | 8 ++ drivers/iio/Makefile | 1 + drivers/iio/indus

[PATCH v10 5/5] iio: Documentation: Add IIO configfs documentation

2015-10-23 Thread Daniel Baluta
Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- Documentation/ABI/testing/configfs-iio | 21 Documentation/iio/iio_configfs.txt | 93 ++ 2 files changed, 114 insertions(+) create mode 100644 Documentation/ABI/testing/configfs-iio

[PATCH v10 2/5] iio: core: Introduce IIO configfs support

2015-10-23 Thread Daniel Baluta
-by: Daniel Baluta <daniel.bal...@intel.com> --- drivers/iio/Kconfig | 8 ++ drivers/iio/Makefile| 1 + drivers/iio/industrialio-configfs.c | 50 + 3 files changed, 59 insertions(+) create mode 100644 drivers/iio/indust

[PATCH v10 4/5] iio: trigger: Introduce IIO hrtimer based trigger

2015-10-23 Thread Daniel Baluta
is based on an older version from Marten and Lars-Peter. Signed-off-by: Marten Svanfeldt <mar...@intuitiveaerial.com> Signed-off-by: Lars-Peter Clausen <l...@metafoo.de> Signed-off-by: Daniel Baluta <daniel.bal...@intel.com> --- drivers/iio/trigger/Kconfig| 10 ++ d

[PATCH v10 0/5] Add initial configfs support for IIO

2015-10-23 Thread Daniel Baluta
r work" chapter in iio_configfs.txt - added configfs-iio file in Documentation/ABI/testing Daniel Baluta (5): configfs: Allow dynamic group creation iio: core: Introduce IIO configfs support iio: core: Introduce IIO software triggers iio: trigger: Introduce IIO hrtimer

Re: [PATCH v8 1/5] configfs: Allow dynamic group creation

2015-10-22 Thread Daniel Baluta
On Thu, Oct 22, 2015 at 11:40 AM, Christoph Hellwig wrote: > On Tue, Oct 20, 2015 at 01:53:42PM +0300, Daniel Baluta wrote: >> We don't want to hardcode default groups at subsystem >> creation time. We export: >> * configfs_register_group >> * configfs_u

<    3   4   5   6   7   8   9   10   11   12   >