All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis CIOCCA <denis.ciocca@st.com>
To: Lee Jones <lee.jones@linaro.org>
Cc: Jonathan Cameron <jic23@kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"jic23@cam.ac.uk" <jic23@cam.ac.uk>,
	"arnd@arndb.de" <arnd@arndb.de>,
	"linus.walleij@linaro.org" <linus.walleij@linaro.org>,
	"linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>
Subject: Re: [PATCH 38/38] iio: magn-core: st: Provide support for the LSM303DLH
Date: Mon, 16 Sep 2013 12:18:05 +0200	[thread overview]
Message-ID: <5236DADD.3040205@st.com> (raw)
In-Reply-To: <20130916100809.GI3999@lee--X1>

SGkgTGVlLA0KDQo+IFRoZSBkcml2ZXIgbWVudGlvbnMgdGhhdCB0aGUgZmlyc3QgZ3JvdXAgb2Yg
c2Vuc29ycyBoYXZlIGEgV0FJIG9mIA0KPiAweDNjOiAjZGVmaW5lIFNUX01BR05fMV9XQUlfRVhQ
IDB4M2MgQW5kIHdoZW4gSSBwcmludCBvdXQgdGhlIFdBSSByZWFkIA0KPiBmcm9tIHRoZSBkZXZp
Y2U6IFJlcXVlc3RlZCBkZXZpY2U6IGxzbTMwM2RsaF9tYWduIC0gcmVhZCBXQUk6IDB4M2MgSSAN
Cj4gZ3Vlc3MgSSBjb3VsZCBoYXZlIGJlZW4gbGllZCB0byBhZ2FpbiBieSB0aGUgYm9hcmQncyBk
YXRhc2hlZXQgYWdhaW4gDQo+IGFuZCB0aGUgYXR0YWNoZWQgZGV2aWNlIGlzbid0IGFjdHVhbGx5
IGEgTFNNMzAzRExILCBidXQgaG93IGNhbiBJIHRlbGwgDQo+IGZvciBzdXJlPyANCk9uIExTTTMw
M0RMSCB5b3UgaGF2ZSB0byByZWFkIDB4MGEgMHgwYiBhbmQgMHgwYyByZWdpc3RlcnMsIHlvdSBo
YXZlIHRvIA0KZmluZCAweDQ4LCAweDM0IGFuZCAweDMzLiBJIHRoaW5rIGluIHRoZSBzbm93YmFs
bCB0aGVyZSBpcyB0aGUgTFNNMzAzRExIQy4NCg0KRGVuaXM=

WARNING: multiple messages have this Message-ID (diff)
From: denis.ciocca@st.com (Denis CIOCCA)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 38/38] iio: magn-core: st: Provide support for the LSM303DLH
Date: Mon, 16 Sep 2013 12:18:05 +0200	[thread overview]
Message-ID: <5236DADD.3040205@st.com> (raw)
In-Reply-To: <20130916100809.GI3999@lee--X1>

Hi Lee,

> The driver mentions that the first group of sensors have a WAI of 
> 0x3c: #define ST_MAGN_1_WAI_EXP 0x3c And when I print out the WAI read 
> from the device: Requested device: lsm303dlh_magn - read WAI: 0x3c I 
> guess I could have been lied to again by the board's datasheet again 
> and the attached device isn't actually a LSM303DLH, but how can I tell 
> for sure? 
On LSM303DLH you have to read 0x0a 0x0b and 0x0c registers, you have to 
find 0x48, 0x34 and 0x33. I think in the snowball there is the LSM303DLHC.

Denis

WARNING: multiple messages have this Message-ID (diff)
From: Denis CIOCCA <denis.ciocca@st.com>
To: Lee Jones <lee.jones@linaro.org>
Cc: Jonathan Cameron <jic23@kernel.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"jic23@cam.ac.uk" <jic23@cam.ac.uk>,
	"arnd@arndb.de" <arnd@arndb.de>,
	"linus.walleij@linaro.org" <linus.walleij@linaro.org>,
	"linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>
Subject: Re: [PATCH 38/38] iio: magn-core: st: Provide support for the LSM303DLH
Date: Mon, 16 Sep 2013 12:18:05 +0200	[thread overview]
Message-ID: <5236DADD.3040205@st.com> (raw)
In-Reply-To: <20130916100809.GI3999@lee--X1>

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 697 bytes --]

Hi Lee,

> The driver mentions that the first group of sensors have a WAI of 
> 0x3c: #define ST_MAGN_1_WAI_EXP 0x3c And when I print out the WAI read 
> from the device: Requested device: lsm303dlh_magn - read WAI: 0x3c I 
> guess I could have been lied to again by the board's datasheet again 
> and the attached device isn't actually a LSM303DLH, but how can I tell 
> for sure? 
On LSM303DLH you have to read 0x0a 0x0b and 0x0c registers, you have to 
find 0x48, 0x34 and 0x33. I think in the snowball there is the LSM303DLHC.

Denisÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

  reply	other threads:[~2013-09-16 10:18 UTC|newest]

Thread overview: 231+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-10 12:49 [PATCH v3 00/38] iio: ST clean-ups and new sensor support Lee Jones
2013-09-10 12:49 ` Lee Jones
2013-09-10 12:49 ` [PATCH 01/38] ARM: ux500: Remove PrimeCell IDs from Nomadik I2C DT nodes Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 15:23   ` Olof Johansson
2013-09-10 15:23     ` Olof Johansson
2013-09-10 15:30     ` Lee Jones
2013-09-10 15:30       ` Lee Jones
2013-09-10 16:57       ` Olof Johansson
2013-09-10 16:57         ` Olof Johansson
2013-09-11  8:06         ` Linus Walleij
2013-09-11  8:06           ` Linus Walleij
2013-09-11  8:19           ` Lee Jones
2013-09-11  8:19             ` Lee Jones
2013-09-11  9:33             ` Linus Walleij
2013-09-11  9:33               ` Linus Walleij
     [not found]               ` <CAF2Aj3iPdqM-+FQ7R9MZ-6jV1Q1me9TG1P4cEGA7fSBBYH92zA@mail.gmail.com>
2013-09-11 11:13                 ` Linus Walleij
2013-09-11 11:13                   ` Linus Walleij
2013-09-11 20:36                   ` Olof Johansson
2013-09-11 20:36                     ` Olof Johansson
2013-09-12 12:23                     ` Linus Walleij
2013-09-12 12:23                       ` Linus Walleij
2013-09-12 12:32                       ` Russell King - ARM Linux
2013-09-12 12:32                         ` Russell King - ARM Linux
2013-09-11  8:17         ` Lee Jones
2013-09-11  8:17           ` Lee Jones
2013-09-10 12:49 ` [PATCH 02/38] ARM: ux500: Enable the LPS001WP Pressure & Temperature sensor from DT Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 03/38] ARM: ux500: Enable the LSM303DLH Accelerator " Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 04/38] ARM: ux500: Enable the LSM303DLH Magnetometer " Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 05/38] ARM: ux500: Enable the L3G4200D Gyroscope " Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 06/38] ARM: ux500: CONFIG: Enable ST's IIO Pressure Sensors by default Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 07/38] ARM: ux500: CONFIG: Enable ST's IIO Accelerometer " Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 15:20   ` Olof Johansson
2013-09-10 15:20     ` Olof Johansson
2013-09-10 15:35     ` Lee Jones
2013-09-10 15:35       ` Lee Jones
2013-09-10 16:51       ` Olof Johansson
2013-09-10 16:51         ` Olof Johansson
2013-09-11  7:20         ` Lee Jones
2013-09-11  7:20           ` Lee Jones
2013-09-10 12:49 ` [PATCH 08/38] ARM: ux500: CONFIG: Enable ST's IIO Magnetometer " Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 09/38] ARM: ux500: CONFIG: Enable ST's IIO Gyroscope " Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 10/38] Documentation: dt: iio: Add binding for LPS001WP Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 12:08   ` Jonathan Cameron
2013-09-14 12:08     ` Jonathan Cameron
2013-09-14 12:08     ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 11/38] Documentation: dt: iio: Add binding for LSM303DLH Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 12:09   ` Jonathan Cameron
2013-09-14 12:09     ` Jonathan Cameron
2013-09-14 12:09     ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 12/38] Documentation: dt: iio: Add binding for L3G4200D Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 12:09   ` Jonathan Cameron
2013-09-14 12:09     ` Jonathan Cameron
2013-09-14 12:09     ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 13/38] Documentation: dt: iio: Add binding for LSM303DLH Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 12:09   ` Jonathan Cameron
2013-09-14 12:09     ` Jonathan Cameron
2013-09-14 12:09     ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 14/38] iio: accel: st: Append _accel to accelerator sensor device names Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 12:14   ` Jonathan Cameron
2013-09-14 12:14     ` Jonathan Cameron
2013-09-14 12:27     ` Jonathan Cameron
2013-09-14 12:27       ` Jonathan Cameron
2013-09-16  8:33       ` Lars-Peter Clausen
2013-09-16  8:33         ` Lars-Peter Clausen
2013-09-16  9:21         ` Denis CIOCCA
2013-09-16  9:21           ` Denis CIOCCA
2013-09-16  9:21           ` Denis CIOCCA
2013-09-16 12:03           ` Lars-Peter Clausen
2013-09-16 12:03             ` Lars-Peter Clausen
2013-09-16 19:15             ` Jonathan Cameron
2013-09-16 19:15               ` Jonathan Cameron
2013-09-16 13:31     ` Lee Jones
2013-09-16 13:31       ` Lee Jones
2013-09-16 13:43       ` Lars-Peter Clausen
2013-09-16 13:43         ` Lars-Peter Clausen
2013-09-16 19:17         ` Jonathan Cameron
2013-09-16 19:17           ` Jonathan Cameron
2013-09-17  8:05           ` Lee Jones
2013-09-17  8:05             ` Lee Jones
2013-09-10 12:49 ` [PATCH 15/38] iio: gyro: st: Append _gyro to gyroscope " Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 16/38] iio: magn: st: Append _magn to magnetometer " Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 17/38] iio: press: st: Append _press to pressure " Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 18/38] iio: sensors-core: st: Allow full-scale to be an optional feature Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 16:45   ` Jonathan Cameron
2013-09-14 16:45     ` Jonathan Cameron
2013-09-14 17:07     ` Jonathan Cameron
2013-09-14 17:07       ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 19/38] iio: sensors-core: st: Support sensors which don't have a Data Ready pin Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 16:53   ` Jonathan Cameron
2013-09-14 16:53     ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 20/38] iio: sensors-core: st: Clean-up error handling in st_sensors_init_sensor() Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 16:54   ` Jonathan Cameron
2013-09-14 16:54     ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 21/38] iio: sensors-core: st: Clean-up error handling in st_sensors_read_axis_data() Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 16:58   ` Jonathan Cameron
2013-09-14 16:58     ` Jonathan Cameron
2013-09-16  8:22     ` Lee Jones
2013-09-16  8:22       ` Lee Jones
2013-09-10 12:49 ` [PATCH 22/38] iio: sensors-core: st: Clean-up error handling in st_sensors_read_info_raw() Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 17:01   ` Jonathan Cameron
2013-09-14 17:01     ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 23/38] iio: pressure-core: st: Describe LPS331AP defines by name Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 17:09   ` Jonathan Cameron
2013-09-14 17:09     ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 24/38] iio: pressure-core: st: Expand and rename LPS331AP's channel descriptor Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 17:14   ` Jonathan Cameron
2013-09-14 17:14     ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 25/38] iio: pressure-core: st: Allow for number of channels to vary Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 17:18   ` Jonathan Cameron
2013-09-14 17:18     ` Jonathan Cameron
2013-09-16  8:17     ` Lee Jones
2013-09-16  8:17       ` Lee Jones
2013-09-16 15:32       ` Jonathan Cameron
2013-09-16 15:32         ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 26/38] iio: pressure-core: st: Clean-up probe() function Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 16:23   ` Jonathan Cameron
2013-09-10 16:23     ` Jonathan Cameron
2013-09-10 16:23     ` Jonathan Cameron
2013-09-11  7:19     ` Lee Jones
2013-09-11  7:19       ` Lee Jones
2013-09-14 17:21       ` Jonathan Cameron
2013-09-14 17:21         ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 27/38] iio: pressure-core: st: Give some indication if device probing was successful Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 16:25   ` Jonathan Cameron
2013-09-10 16:25     ` Jonathan Cameron
2013-09-10 16:25     ` Jonathan Cameron
2013-09-11  7:10     ` Lee Jones
2013-09-11  7:10       ` Lee Jones
2013-09-11  7:18       ` Lars-Peter Clausen
2013-09-11  7:18         ` Lars-Peter Clausen
2013-09-11  7:29         ` Lee Jones
2013-09-11  7:29           ` Lee Jones
2013-09-11 15:35           ` Mark Brown
2013-09-11 15:35             ` Mark Brown
2013-09-13 20:57           ` Getz, Robin
2013-09-13 20:57             ` Getz, Robin
2013-09-13 20:57             ` Getz, Robin
2013-09-16  9:47             ` Lee Jones
2013-09-16  9:47               ` Lee Jones
2013-09-11 18:30         ` Jonathan Cameron
2013-09-11 18:30           ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 28/38] iio: pressure: st: Add support for new LPS001WP pressure sensor Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 17:26   ` Jonathan Cameron
2013-09-14 17:26     ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 29/38] iio: pressure-core: st: Provide support for the Vdd power supply Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 17:28   ` Jonathan Cameron
2013-09-14 17:28     ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 30/38] iio: pressure-core: st: Provide support for the Vdd_IO " Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 31/38] iio: accel-core: st: Clean up error handling in probe() Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 17:31   ` Jonathan Cameron
2013-09-14 17:31     ` Jonathan Cameron
2013-09-10 12:49 ` [PATCH 32/38] iio: accel-core: st: Move LSM303DLH into correct group Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 17:51   ` Jonathan Cameron
2013-09-14 17:51     ` Jonathan Cameron
2013-09-16  8:05     ` Lee Jones
2013-09-16  8:05       ` Lee Jones
2013-09-16  9:19       ` Denis CIOCCA
2013-09-16  9:19         ` Denis CIOCCA
2013-09-16  9:19         ` Denis CIOCCA
2013-09-16  9:38         ` Lee Jones
2013-09-16  9:38           ` Lee Jones
2013-09-16  9:57           ` Denis CIOCCA
2013-09-16  9:57             ` Denis CIOCCA
2013-09-16  9:57             ` Denis CIOCCA
2013-09-16 10:19             ` Lee Jones
2013-09-16 10:19               ` Lee Jones
2013-09-16 10:23               ` Denis CIOCCA
2013-09-16 10:23                 ` Denis CIOCCA
2013-09-16 10:23                 ` Denis CIOCCA
2013-09-16 11:08                 ` Lee Jones
2013-09-16 11:08                   ` Lee Jones
2013-09-10 12:49 ` [PATCH 33/38] iio: accel-core: st: Give some indication if device probing was successful Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 34/38] iio: gyro-core: st: Clean up error handling in probe() Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 35/38] iio: gyro-core: st: Give some indication if device probing was successful Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 36/38] iio: magn-core: st: Clean up error handling in probe() Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 37/38] iio: magn-core: st: Give some indication if device probing was successful Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-10 12:49 ` [PATCH 38/38] iio: magn-core: st: Provide support for the LSM303DLH Lee Jones
2013-09-10 12:49   ` Lee Jones
2013-09-14 17:53   ` Jonathan Cameron
2013-09-14 17:53     ` Jonathan Cameron
2013-09-16  7:31     ` Lee Jones
2013-09-16  7:31       ` Lee Jones
2013-09-16  9:32       ` Denis CIOCCA
2013-09-16  9:32         ` Denis CIOCCA
2013-09-16  9:32         ` Denis CIOCCA
2013-09-16 10:08         ` Lee Jones
2013-09-16 10:08           ` Lee Jones
2013-09-16 10:18           ` Denis CIOCCA [this message]
2013-09-16 10:18             ` Denis CIOCCA
2013-09-16 10:18             ` Denis CIOCCA
2013-09-16 11:11             ` Lee Jones
2013-09-16 11:11               ` Lee Jones
2013-09-16 11:26             ` Lee Jones
2013-09-16 11:26               ` Lee Jones

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5236DADD.3040205@st.com \
    --to=denis.ciocca@st.com \
    --cc=arnd@arndb.de \
    --cc=jic23@cam.ac.uk \
    --cc=jic23@kernel.org \
    --cc=lee.jones@linaro.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.