From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756082AbcH3Tt7 (ORCPT ); Tue, 30 Aug 2016 15:49:59 -0400 Received: from mga03.intel.com ([134.134.136.65]:37710 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751372AbcH3Ttz (ORCPT ); Tue, 30 Aug 2016 15:49:55 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.30,258,1470726000"; d="scan'208";a="755179704" From: "Pandruvada, Srinivas" To: "jikos@kernel.org" , "Kweh, Hock Leong" , "jic23@kernel.org" CC: "Lay, Kuan Loon" , "linux-kernel@vger.kernel.org" , "lars@metafoo.de" , "knaack.h@gmx.de" , "Ong, Boon Leong" , "linux-iio@vger.kernel.org" , "pmeerw@pmeerw.net" , "Ooi, Joyce" , "linux-input@vger.kernel.org" , "Lim, Lee Booi" Subject: Re: [PATCH] iio: fix pressure data output unit in hid-sensor-attributes Thread-Topic: [PATCH] iio: fix pressure data output unit in hid-sensor-attributes Thread-Index: AQHSAZ/ipR71SR1NHEO3Pfdqc5m/sqBiYXEA Date: Tue, 30 Aug 2016 19:49:52 +0000 Message-ID: <1472586592.5025.3.camel@intel.com> References: <1472467856-14526-1-git-send-email-hock.leong.kweh@intel.com> In-Reply-To: <1472467856-14526-1-git-send-email-hock.leong.kweh@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.7.199.80] Content-Type: text/plain; charset="utf-8" Content-ID: <18520B43B306E74ABE0EA48E64998F10@intel.com> MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id u7UJo5WD032114 On Mon, 2016-08-29 at 18:50 +0800, Kweh, Hock Leong wrote: > From: "Kweh, Hock Leong" > > According to IIO ABI definition, IIO_PRESSURE data output unit is > kilopascal: > http://lxr.free-electrons.com/source/Documentation/ABI/testing/sysfs- > bus-iio > > This patch fix output unit of HID pressure sensor IIO driver from > pascal to > kilopascal to follow IIO ABI definition. > > Signed-off-by: Kweh, Hock Leong Reviewed-by: Srinivas Pandruvada > --- >  .../iio/common/hid-sensors/hid-sensor-attributes.c |    4 ++-- >  1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c > b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c > index e81f434..dc33c1d 100644 > --- a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c > +++ b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c > @@ -56,8 +56,8 @@ static struct { >   {HID_USAGE_SENSOR_ALS, 0, 1, 0}, >   {HID_USAGE_SENSOR_ALS, HID_USAGE_SENSOR_UNITS_LUX, 1, 0}, >   > - {HID_USAGE_SENSOR_PRESSURE, 0, 100000, 0}, > - {HID_USAGE_SENSOR_PRESSURE, HID_USAGE_SENSOR_UNITS_PASCAL, > 1, 0}, > + {HID_USAGE_SENSOR_PRESSURE, 0, 100, 0}, > + {HID_USAGE_SENSOR_PRESSURE, HID_USAGE_SENSOR_UNITS_PASCAL, > 0, 1000}, >  }; >   >  static int pow_10(unsigned power)