Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756103Ab1CWJey (ORCPT ); Wed, 23 Mar 2011 05:34:54 -0400 Received: from nwd2mail11.analog.com ([137.71.25.57]:20352 "EHLO nwd2mail11.analog.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755693Ab1CWJew (ORCPT ); Wed, 23 Mar 2011 05:34:52 -0400 X-IronPort-AV: E=Sophos;i="4.63,230,1299474000"; d="scan'208";a="30681999" From: "Hennerich, Michael" To: Jonathan Cameron CC: "Cai, Cliff" , "linux-iio@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Drivers , "device-drivers-devel@blackfin.uclinux.org" , Cliff Cai Date: Wed, 23 Mar 2011 09:34:46 +0000 Subject: RE: [PATCH RESEND v2]IIO driver for Analog Devices Digital Output Gyroscope ADXRS450 Thread-Topic: [PATCH RESEND v2]IIO driver for Analog Devices Digital Output Gyroscope ADXRS450 Thread-Index: AcvofbOSP5AkRZWxTq2IFJ3e+OoPFQAvw3Ag Message-ID: <544AC56F16B56944AEC3BD4E3D5917713754315FCA@LIMKCMBX1.ad.analog.com> References: <1300526846-27183-1-git-send-email-cliff.cai@analog.com> <4D878D8B.6070902@cam.ac.uk> <544AC56F16B56944AEC3BD4E3D5917713754315B05@LIMKCMBX1.ad.analog.com> <4D887D1E.90503@cam.ac.uk> In-Reply-To: <4D887D1E.90503@cam.ac.uk> Accept-Language: de-DE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: de-DE, en-US Content-Type: text/plain; charset="utf-8" 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 p2N9Z3dw011418 Content-Length: 4263 Lines: 112 Jonathan Cameron wrote on 2011-03-22: > On 03/22/11 08:16, Hennerich, Michael wrote: >> Cai, Cliff wrote on 2011-03-22: >>> >>> >>>> -----Original Message----- >>>> From: Jonathan Cameron [mailto:jic23@cam.ac.uk] >>>> Sent: 2011年3月22日 1:40 >>>> To: Cai, Cliff >>>> Cc: linux-iio@vger.kernel.org; linux-kernel@vger.kernel.org; >>>> Drivers; device-drivers-devel@blackfin.uclinux.org; Cliff Cai >>>> Subject: Re: [PATCH RESEND v2]IIO driver for Analog Devices >>>> Digital Output Gyroscope ADXRS450 >>>> >>>> On 03/19/11 09:27, cliff.cai@analog.com wrote: >>>>> From: Cliff Cai >>>>> >>>>> Change v2:v1: >>>>> >>>>> Make modification according to Michael Hennerich's comments, >>>>> correct the spi transfer way,use existing sysfs interfaces. >>>> Hi Cliff, >>>> >>>> As you are proposing a couple of new interfaces we need to have >>>> documentation for them. They are quad and dynamic_null_correction. >>>> We need to first establish whether they are of general utility and >>>> hence should be in the main abi doc. The quadrature one isn't >>>> something I've seen before. Is it common in gyros? >>> >>> I'm not sure about this. >>> Michael,do you have any ideas? >> >> The ADXRS450 is a quite new part and features a new sensor design - So >> I don't think the quadrature error is to date very common. It might >> become in future... >> >> From the datasheet: >> >> "The quad sensor design rejects linear and angular acceleration, >> including external g-forces and vibration. This is achieved by >> mechanically coupling the four sensing structures such that external >> g-forces appear as common-mode signals that can be removed by the >> fully differential architecture implemented in the ADXRS450." >> >> "The quad memory registers contain a value corresponding to the amount >> of quadrature error present in the device at a given time. Quadrature >> can be likened to a measurement of the error of the motion of the >> resonator structure, and can be caused by stresses and aging effects. >> The quadrature data is filtered to 80 Hz and can be read frequently to >> detect sudden shifts in the level of quadrature. The data is presented >> as a 16-bit, twos complement number." >> > Cool. So what is a good general name for this? I guess from this > description if it were in a multi axis device you would have this > measure for each axis? Yes > So perhaps gyro_z_quadrature_correction_raw? Sounds good to me. > This thing also looks > rather similar to a dynamically changing calibbias. Perhaps we need > another term for a general dynamic linear (I think this is linear) > correction inside a device? Probably - but don't know for sure. Bottom line is that the error is removed from the result. In case it can't be removed, the part flags it with an error bit. gyro_z_quadrature_correction_raw is just an measure on how much error was is present at a given time. > Perhaps go with a gyro specific term for now and wait to see if we get > many more parts with this feature... ok >>> >>>> Dynamic null correction looks like it should be gyro_z_calibbias to >>>> me but I could be wrong. The doc says " The user can make small >>>> adjustments to the rateout of the device by asserting these bits. >>>> This 10-bit register allows the user to adjust the static rateout of >>>> the device by up to ±6.4°/sec. " >>>> >>>> which makes me think it is an internally applied offset on the >>>> output signal and hence calibbias in our abi. >>> >>> Thanks >>> >>>> Other than that, various minor nitpicks inline. >>>> >>>> Jonathan >> >> Greetings, >> Michael >> >> -- >> Analog Devices GmbH Wilhelm-Wagenfeld-Str. 6 80807 Muenchen >> Sitz der Gesellschaft: Muenchen; Registergericht: Muenchen HRB >> 40368; Geschaeftsfuehrer:Dr.Carsten Suckrow, Thomas Wessel, William A. >> Martin, Margaret Seif >> Greetings, Michael -- Analog Devices GmbH Wilhelm-Wagenfeld-Str. 6 80807 Muenchen Sitz der Gesellschaft: Muenchen; Registergericht: Muenchen HRB 40368; Geschaeftsfuehrer:Dr.Carsten Suckrow, Thomas Wessel, William A. Martin, Margaret Seif ????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?