From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk ([178.18.118.26]:45526 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751122AbaLZNb2 (ORCPT ); Fri, 26 Dec 2014 08:31:28 -0500 Message-ID: <549D632E.4000004@kernel.org> Date: Fri, 26 Dec 2014 13:31:26 +0000 From: Jonathan Cameron MIME-Version: 1.0 To: Irina Tirdea , linux-iio@vger.kernel.org CC: linux-kernel@vger.kernel.org, Vlad Dogaru , Daniel Baluta , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald Subject: Re: [PATCH 4/8] iio: core: Introduce IO_CHAN_INFO_CALIBWEIGHT References: <1419029879-12024-1-git-send-email-irina.tirdea@intel.com> <1419029879-12024-5-git-send-email-irina.tirdea@intel.com> In-Reply-To: <1419029879-12024-5-git-send-email-irina.tirdea@intel.com> Content-Type: text/plain; charset=windows-1252 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 19/12/14 22:57, Irina Tirdea wrote: > Some devices need the weight of the user to compute other > parameters. One of this devices is Freescale's MMA9553L > (http://www.freescale.com/files/sensors/doc/ref_manual/MMA9553LSWRM.pdf) > that needs the weight of the user to compute the number of calories burnt. > > Signed-off-by: Irina Tirdea > --- > Documentation/ABI/testing/sysfs-bus-iio | 7 +++++++ > drivers/iio/industrialio-core.c | 1 + > include/linux/iio/iio.h | 1 + > 3 files changed, 9 insertions(+) > > diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio > index 07acef7..e480175 100644 > --- a/Documentation/ABI/testing/sysfs-bus-iio > +++ b/Documentation/ABI/testing/sysfs-bus-iio > @@ -351,6 +351,13 @@ Description: > to compute the stride length, distance, speed and activity > type. > > +What: /sys/bus/iio/devices/iio:deviceX/in_steps_calibweight > +KernelVersion: 3.19 > +Contact: linux-iio@vger.kernel.org > +Description: > + Weight of the user (in kg). It is needed by some pedometers > + to compute the calories burnt by the user. How about grams? Nice to keep to SI units going forward (I appreciate we have broken that for what seemed like good reasons at the time) in one or two places, but it makes it much harder to define consistent interfaces in the long run. J > + > What: /sys/bus/iio/devices/iio:deviceX/in_accel_scale_available > What: /sys/.../iio:deviceX/in_voltageX_scale_available > What: /sys/.../iio:deviceX/in_voltage-voltage_scale_available > diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c > index 5e50aca..4c435c8 100644 > --- a/drivers/iio/industrialio-core.c > +++ b/drivers/iio/industrialio-core.c > @@ -124,6 +124,7 @@ static const char * const iio_chan_info_postfix[] = { > [IIO_CHAN_INFO_INT_TIME] = "integration_time", > [IIO_CHAN_INFO_ENABLE] = "en", > [IIO_CHAN_INFO_CALIBHEIGHT] = "calibheight", > + [IIO_CHAN_INFO_CALIBWEIGHT] = "calibweight", > }; > > /** > diff --git a/include/linux/iio/iio.h b/include/linux/iio/iio.h > index 878d861..752a929 100644 > --- a/include/linux/iio/iio.h > +++ b/include/linux/iio/iio.h > @@ -40,6 +40,7 @@ enum iio_chan_info_enum { > IIO_CHAN_INFO_INT_TIME, > IIO_CHAN_INFO_ENABLE, > IIO_CHAN_INFO_CALIBHEIGHT, > + IIO_CHAN_INFO_CALIBWEIGHT, > }; > > enum iio_shared_by { >