diff for duplicates of <5236CD17.5010901@st.com> diff --git a/a/1.txt b/N1/1.txt index 14a1e8d..78bc1ec 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,41 +1,59 @@ -SGkgTGVlLA0KPj4gT24gMDkvMTAvMTMgMTM6NDksIExlZSBKb25lcyB3cm90ZToNCj4+PiBUaGUg -TFNNMzAzRExIJ3MgV0FJIChXaG9BbUkpIGlzIDB4MzMsIG1lYW5pbmcgaXQgc2hvdWxkIGJlIGVu -YWJsZWQgYnkNCj4+PiBBY2NlbCBTZW5zb3IgZ3JvdXAgb25lLiBGb3IgdGhlIGRldmljZSB0byBw -cm9iZSB3aXRob3V0IGVycm9yLCB3ZSdsbA0KPj4+IG5lZWQgdG8gZW5zdXJlIGl0J3MgcmVnaXN0 -ZXJlZCB3aXRoIHRoZSBjb3JyZWN0IFdBSS4NCj4+Pg0KPj4+IFNpZ25lZC1vZmYtYnk6IExlZSBK -b25lcyA8bGVlLmpvbmVzQGxpbmFyby5vcmc+DQo+PiBZb3UgY2xlYXJseSBoYXZlIGEgYmV0dGVy -IGRhdGFzaGVldCB0aGFuIEkgaGF2ZSBhcyBmb3IgdGhhdCBwYXJ0IGl0IGRvZXNuJ3QgZXZlbiBj -bGFpbSB0bw0KPj4gaGF2ZSB0aGUgcmVsZXZhbnQgcmVnaXN0ZXIgdG8gcmVhZCBhIHdobyBhbSBJ -IGZyb20uDQo+Pg0KPj4gTm93IHRoYXQgZGF0YXNoZWV0IGRvZXMgbGlzdCBvZHIgdmFsdWVzIGFz -IDUwLCAxMDAsIDQwMCAxMDAwIHdoaWNoIHdvdWxkIHB1dCBpdCB3aGVyZSBpdCBvcmlnaW5hbGx5 -DQo+PiB3YXMgaW4gdGhlc2UgdGFibGVzLg0KPj4NCj4+IGh0dHA6Ly93d3cuc3QuY29tL3dlYi9l -bi9yZXNvdXJjZS90ZWNobmljYWwvZG9jdW1lbnQvZGF0YXNoZWV0L0NEMDAyNjAyODgucGRmDQo+ -Pg0KPj4gSSBoYXZlbid0IGNoZWNrZWQgb3RoZXIgZWxlbWVudHMuLi4NCj4+DQo+PiBJJ20gY29u -ZnVzZWQgYnV0IHN1c3BlY3Qgd2UgbWF5IG5lZWQgYW5vdGhlciB0eXBlIGVudHJ5IHRvIGRlYWwg -d2l0aCB0aGlzIG9uZS4NCj4gSG1tbS4uLiBvbiBpbml0aWFsIHRob3VnaHQsIEkgZG9uJ3Qga25v -dyBob3cgd2UgY2FuIGhhbmRsZSB0aGlzIG9uZS4gSQ0KPiBhc3N1bWVkIHRoZSBXQUlzIHdlcmUg -YSBvbmUtc3RvcC1zaG9wIGZvciBkZXZpY2UgaWRlbnRpZmljYXRpb24uIEkgYWxzbw0KPiB0aG91 -Z2h0IHRoYXQgdGhleSB3b3VsZCBvbmx5IG1hdGNoIGlmIHRoZSBmdW5jdGlvbmFsaXR5IGFuZCBy -ZWdpc3Rlcg0KPiBtYXBzIHdvdWxkIGJlIHRoZSBzYW1lLg0KPg0KPiBJJ2xsIHRvdWNoIGJhc2Ug -d2l0aCBEZW5pcyBvbiB0aGlzLCBwZXJoYXBzIGhlIGNhbiBzaGVkIHNvbWUgbGlnaHQuDQo+DQpJ -J3ZlIGp1c3QgY2hlY2tlZCwgdGhlIFdBSSBpcyAweDMyLiBJdCB3YXMgY29ycmVjdC4uLkFyZSB5 -b3Ugc3VyZSB0aGF0IA0KeW91IHVzZSB0aGUgTFNNMzAzRExIPw0KDQpEZW5pcw0KDQoNCg0KPj4+ -IC0tLQ0KPj4+ICAgZHJpdmVycy9paW8vYWNjZWwvc3RfYWNjZWxfY29yZS5jIHwgNCArKy0tDQo+ -Pj4gICAxIGZpbGUgY2hhbmdlZCwgMiBpbnNlcnRpb25zKCspLCAyIGRlbGV0aW9ucygtKQ0KPj4+ -DQo+Pj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvaWlvL2FjY2VsL3N0X2FjY2VsX2NvcmUuYyBiL2Ry -aXZlcnMvaWlvL2FjY2VsL3N0X2FjY2VsX2NvcmUuYw0KPj4+IGluZGV4IGVhNjIyOTEuLjAzYTJi -NmIgMTAwNjQ0DQo+Pj4gLS0tIGEvZHJpdmVycy9paW8vYWNjZWwvc3RfYWNjZWxfY29yZS5jDQo+ -Pj4gKysrIGIvZHJpdmVycy9paW8vYWNjZWwvc3RfYWNjZWxfY29yZS5jDQo+Pj4gQEAgLTE3MCw2 -ICsxNzAsNyBAQCBzdGF0aWMgY29uc3Qgc3RydWN0IHN0X3NlbnNvcnMgc3RfYWNjZWxfc2Vuc29y -c1tdID0gew0KPj4+ICAgCQkJWzJdID0gTFNNMzMwRF9BQ0NFTF9ERVZfTkFNRSwNCj4+PiAgIAkJ -CVszXSA9IExTTTMzMERMX0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICAgCQkJWzRdID0gTFNNMzMwRExD -X0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICsJCQlbNV0gPSBMU00zMDNETEhfQUNDRUxfREVWX05BTUUs -DQo+Pj4gICAJCX0sDQo+Pj4gICAJCS5jaCA9IChzdHJ1Y3QgaWlvX2NoYW5fc3BlYyAqKXN0X2Fj -Y2VsXzEyYml0X2NoYW5uZWxzLA0KPj4+ICAgCQkub2RyID0gew0KPj4+IEBAIC0yMzgsOCArMjM5 -LDcgQEAgc3RhdGljIGNvbnN0IHN0cnVjdCBzdF9zZW5zb3JzIHN0X2FjY2VsX3NlbnNvcnNbXSA9 -IHsNCj4+PiAgIAkJLnNlbnNvcnNfc3VwcG9ydGVkID0gew0KPj4+ICAgCQkJWzBdID0gTElTMzMx -RExIX0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICAgCQkJWzFdID0gTFNNMzAzRExfQUNDRUxfREVWX05B -TUUsDQo+Pj4gLQkJCVsyXSA9IExTTTMwM0RMSF9BQ0NFTF9ERVZfTkFNRSwNCj4+PiAtCQkJWzNd -ID0gTFNNMzAzRExNX0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICsJCQlbMl0gPSBMU00zMDNETE1fQUND -RUxfREVWX05BTUUsDQo+Pj4gICAJCX0sDQo+Pj4gICAJCS5jaCA9IChzdHJ1Y3QgaWlvX2NoYW5f -c3BlYyAqKXN0X2FjY2VsXzEyYml0X2NoYW5uZWxzLA0KPj4+ICAgCQkub2RyID0gew0KPj4+DQo= +Hi Lee, +>> On 09/10/13 13:49, Lee Jones wrote: +>>> The LSM303DLH's WAI (WhoAmI) is 0x33, meaning it should be enabled by +>>> Accel Sensor group one. For the device to probe without error, we'll +>>> need to ensure it's registered with the correct WAI. +>>> +>>> Signed-off-by: Lee Jones <lee.jones@linaro.org> +>> You clearly have a better datasheet than I have as for that part it doesn't even claim to +>> have the relevant register to read a who am I from. +>> +>> Now that datasheet does list odr values as 50, 100, 400 1000 which would put it where it originally +>> was in these tables. +>> +>> http://www.st.com/web/en/resource/technical/document/datasheet/CD00260288.pdf +>> +>> I haven't checked other elements... +>> +>> I'm confused but suspect we may need another type entry to deal with this one. +> Hmmm... on initial thought, I don't know how we can handle this one. I +> assumed the WAIs were a one-stop-shop for device identification. I also +> thought that they would only match if the functionality and register +> maps would be the same. +> +> I'll touch base with Denis on this, perhaps he can shed some light. +> +I've just checked, the WAI is 0x32. It was correct...Are you sure that +you use the LSM303DLH? + +Denis + + + +>>> --- +>>> drivers/iio/accel/st_accel_core.c | 4 ++-- +>>> 1 file changed, 2 insertions(+), 2 deletions(-) +>>> +>>> diff --git a/drivers/iio/accel/st_accel_core.c b/drivers/iio/accel/st_accel_core.c +>>> index ea62291..03a2b6b 100644 +>>> --- a/drivers/iio/accel/st_accel_core.c +>>> +++ b/drivers/iio/accel/st_accel_core.c +>>> @@ -170,6 +170,7 @@ static const struct st_sensors st_accel_sensors[] = { +>>> [2] = LSM330D_ACCEL_DEV_NAME, +>>> [3] = LSM330DL_ACCEL_DEV_NAME, +>>> [4] = LSM330DLC_ACCEL_DEV_NAME, +>>> + [5] = LSM303DLH_ACCEL_DEV_NAME, +>>> }, +>>> .ch = (struct iio_chan_spec *)st_accel_12bit_channels, +>>> .odr = { +>>> @@ -238,8 +239,7 @@ static const struct st_sensors st_accel_sensors[] = { +>>> .sensors_supported = { +>>> [0] = LIS331DLH_ACCEL_DEV_NAME, +>>> [1] = LSM303DL_ACCEL_DEV_NAME, +>>> - [2] = LSM303DLH_ACCEL_DEV_NAME, +>>> - [3] = LSM303DLM_ACCEL_DEV_NAME, +>>> + [2] = LSM303DLM_ACCEL_DEV_NAME, +>>> }, +>>> .ch = (struct iio_chan_spec *)st_accel_12bit_channels, +>>> .odr = { +>>> diff --git a/a/content_digest b/N1/content_digest index c870cf5..2f6332b 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,58 +2,70 @@ "ref\01378817379-8238-33-git-send-email-lee.jones@linaro.org\0" "ref\05234A20F.7040008@kernel.org\0" "ref\020130916080546.GC3999@lee--X1\0" - "From\0Denis CIOCCA <denis.ciocca@st.com>\0" - "Subject\0Re: [PATCH 32/38] iio: accel-core: st: Move LSM303DLH into correct group\0" + "From\0denis.ciocca@st.com (Denis CIOCCA)\0" + "Subject\0[PATCH 32/38] iio: accel-core: st: Move LSM303DLH into correct group\0" "Date\0Mon, 16 Sep 2013 11:19:19 +0200\0" - "To\0Lee Jones <lee.jones@linaro.org>\0" - "Cc\0Jonathan 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> - arnd@arndb.de <arnd@arndb.de> - linus.walleij@linaro.org <linus.walleij@linaro.org> - " linux-iio@vger.kernel.org <linux-iio@vger.kernel.org>\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" - "SGkgTGVlLA0KPj4gT24gMDkvMTAvMTMgMTM6NDksIExlZSBKb25lcyB3cm90ZToNCj4+PiBUaGUg\n" - "TFNNMzAzRExIJ3MgV0FJIChXaG9BbUkpIGlzIDB4MzMsIG1lYW5pbmcgaXQgc2hvdWxkIGJlIGVu\n" - "YWJsZWQgYnkNCj4+PiBBY2NlbCBTZW5zb3IgZ3JvdXAgb25lLiBGb3IgdGhlIGRldmljZSB0byBw\n" - "cm9iZSB3aXRob3V0IGVycm9yLCB3ZSdsbA0KPj4+IG5lZWQgdG8gZW5zdXJlIGl0J3MgcmVnaXN0\n" - "ZXJlZCB3aXRoIHRoZSBjb3JyZWN0IFdBSS4NCj4+Pg0KPj4+IFNpZ25lZC1vZmYtYnk6IExlZSBK\n" - "b25lcyA8bGVlLmpvbmVzQGxpbmFyby5vcmc+DQo+PiBZb3UgY2xlYXJseSBoYXZlIGEgYmV0dGVy\n" - "IGRhdGFzaGVldCB0aGFuIEkgaGF2ZSBhcyBmb3IgdGhhdCBwYXJ0IGl0IGRvZXNuJ3QgZXZlbiBj\n" - "bGFpbSB0bw0KPj4gaGF2ZSB0aGUgcmVsZXZhbnQgcmVnaXN0ZXIgdG8gcmVhZCBhIHdobyBhbSBJ\n" - "IGZyb20uDQo+Pg0KPj4gTm93IHRoYXQgZGF0YXNoZWV0IGRvZXMgbGlzdCBvZHIgdmFsdWVzIGFz\n" - "IDUwLCAxMDAsIDQwMCAxMDAwIHdoaWNoIHdvdWxkIHB1dCBpdCB3aGVyZSBpdCBvcmlnaW5hbGx5\n" - "DQo+PiB3YXMgaW4gdGhlc2UgdGFibGVzLg0KPj4NCj4+IGh0dHA6Ly93d3cuc3QuY29tL3dlYi9l\n" - "bi9yZXNvdXJjZS90ZWNobmljYWwvZG9jdW1lbnQvZGF0YXNoZWV0L0NEMDAyNjAyODgucGRmDQo+\n" - "Pg0KPj4gSSBoYXZlbid0IGNoZWNrZWQgb3RoZXIgZWxlbWVudHMuLi4NCj4+DQo+PiBJJ20gY29u\n" - "ZnVzZWQgYnV0IHN1c3BlY3Qgd2UgbWF5IG5lZWQgYW5vdGhlciB0eXBlIGVudHJ5IHRvIGRlYWwg\n" - "d2l0aCB0aGlzIG9uZS4NCj4gSG1tbS4uLiBvbiBpbml0aWFsIHRob3VnaHQsIEkgZG9uJ3Qga25v\n" - "dyBob3cgd2UgY2FuIGhhbmRsZSB0aGlzIG9uZS4gSQ0KPiBhc3N1bWVkIHRoZSBXQUlzIHdlcmUg\n" - "YSBvbmUtc3RvcC1zaG9wIGZvciBkZXZpY2UgaWRlbnRpZmljYXRpb24uIEkgYWxzbw0KPiB0aG91\n" - "Z2h0IHRoYXQgdGhleSB3b3VsZCBvbmx5IG1hdGNoIGlmIHRoZSBmdW5jdGlvbmFsaXR5IGFuZCBy\n" - "ZWdpc3Rlcg0KPiBtYXBzIHdvdWxkIGJlIHRoZSBzYW1lLg0KPg0KPiBJJ2xsIHRvdWNoIGJhc2Ug\n" - "d2l0aCBEZW5pcyBvbiB0aGlzLCBwZXJoYXBzIGhlIGNhbiBzaGVkIHNvbWUgbGlnaHQuDQo+DQpJ\n" - "J3ZlIGp1c3QgY2hlY2tlZCwgdGhlIFdBSSBpcyAweDMyLiBJdCB3YXMgY29ycmVjdC4uLkFyZSB5\n" - "b3Ugc3VyZSB0aGF0IA0KeW91IHVzZSB0aGUgTFNNMzAzRExIPw0KDQpEZW5pcw0KDQoNCg0KPj4+\n" - "IC0tLQ0KPj4+ICAgZHJpdmVycy9paW8vYWNjZWwvc3RfYWNjZWxfY29yZS5jIHwgNCArKy0tDQo+\n" - "Pj4gICAxIGZpbGUgY2hhbmdlZCwgMiBpbnNlcnRpb25zKCspLCAyIGRlbGV0aW9ucygtKQ0KPj4+\n" - "DQo+Pj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvaWlvL2FjY2VsL3N0X2FjY2VsX2NvcmUuYyBiL2Ry\n" - "aXZlcnMvaWlvL2FjY2VsL3N0X2FjY2VsX2NvcmUuYw0KPj4+IGluZGV4IGVhNjIyOTEuLjAzYTJi\n" - "NmIgMTAwNjQ0DQo+Pj4gLS0tIGEvZHJpdmVycy9paW8vYWNjZWwvc3RfYWNjZWxfY29yZS5jDQo+\n" - "Pj4gKysrIGIvZHJpdmVycy9paW8vYWNjZWwvc3RfYWNjZWxfY29yZS5jDQo+Pj4gQEAgLTE3MCw2\n" - "ICsxNzAsNyBAQCBzdGF0aWMgY29uc3Qgc3RydWN0IHN0X3NlbnNvcnMgc3RfYWNjZWxfc2Vuc29y\n" - "c1tdID0gew0KPj4+ICAgCQkJWzJdID0gTFNNMzMwRF9BQ0NFTF9ERVZfTkFNRSwNCj4+PiAgIAkJ\n" - "CVszXSA9IExTTTMzMERMX0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICAgCQkJWzRdID0gTFNNMzMwRExD\n" - "X0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICsJCQlbNV0gPSBMU00zMDNETEhfQUNDRUxfREVWX05BTUUs\n" - "DQo+Pj4gICAJCX0sDQo+Pj4gICAJCS5jaCA9IChzdHJ1Y3QgaWlvX2NoYW5fc3BlYyAqKXN0X2Fj\n" - "Y2VsXzEyYml0X2NoYW5uZWxzLA0KPj4+ICAgCQkub2RyID0gew0KPj4+IEBAIC0yMzgsOCArMjM5\n" - "LDcgQEAgc3RhdGljIGNvbnN0IHN0cnVjdCBzdF9zZW5zb3JzIHN0X2FjY2VsX3NlbnNvcnNbXSA9\n" - "IHsNCj4+PiAgIAkJLnNlbnNvcnNfc3VwcG9ydGVkID0gew0KPj4+ICAgCQkJWzBdID0gTElTMzMx\n" - "RExIX0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICAgCQkJWzFdID0gTFNNMzAzRExfQUNDRUxfREVWX05B\n" - "TUUsDQo+Pj4gLQkJCVsyXSA9IExTTTMwM0RMSF9BQ0NFTF9ERVZfTkFNRSwNCj4+PiAtCQkJWzNd\n" - "ID0gTFNNMzAzRExNX0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICsJCQlbMl0gPSBMU00zMDNETE1fQUND\n" - "RUxfREVWX05BTUUsDQo+Pj4gICAJCX0sDQo+Pj4gICAJCS5jaCA9IChzdHJ1Y3QgaWlvX2NoYW5f\n" - c3BlYyAqKXN0X2FjY2VsXzEyYml0X2NoYW5uZWxzLA0KPj4+ICAgCQkub2RyID0gew0KPj4+DQo= + "Hi Lee,\n" + ">> On 09/10/13 13:49, Lee Jones wrote:\n" + ">>> The LSM303DLH's WAI (WhoAmI) is 0x33, meaning it should be enabled by\n" + ">>> Accel Sensor group one. For the device to probe without error, we'll\n" + ">>> need to ensure it's registered with the correct WAI.\n" + ">>>\n" + ">>> Signed-off-by: Lee Jones <lee.jones@linaro.org>\n" + ">> You clearly have a better datasheet than I have as for that part it doesn't even claim to\n" + ">> have the relevant register to read a who am I from.\n" + ">>\n" + ">> Now that datasheet does list odr values as 50, 100, 400 1000 which would put it where it originally\n" + ">> was in these tables.\n" + ">>\n" + ">> http://www.st.com/web/en/resource/technical/document/datasheet/CD00260288.pdf\n" + ">>\n" + ">> I haven't checked other elements...\n" + ">>\n" + ">> I'm confused but suspect we may need another type entry to deal with this one.\n" + "> Hmmm... on initial thought, I don't know how we can handle this one. I\n" + "> assumed the WAIs were a one-stop-shop for device identification. I also\n" + "> thought that they would only match if the functionality and register\n" + "> maps would be the same.\n" + ">\n" + "> I'll touch base with Denis on this, perhaps he can shed some light.\n" + ">\n" + "I've just checked, the WAI is 0x32. It was correct...Are you sure that \n" + "you use the LSM303DLH?\n" + "\n" + "Denis\n" + "\n" + "\n" + "\n" + ">>> ---\n" + ">>> drivers/iio/accel/st_accel_core.c | 4 ++--\n" + ">>> 1 file changed, 2 insertions(+), 2 deletions(-)\n" + ">>>\n" + ">>> diff --git a/drivers/iio/accel/st_accel_core.c b/drivers/iio/accel/st_accel_core.c\n" + ">>> index ea62291..03a2b6b 100644\n" + ">>> --- a/drivers/iio/accel/st_accel_core.c\n" + ">>> +++ b/drivers/iio/accel/st_accel_core.c\n" + ">>> @@ -170,6 +170,7 @@ static const struct st_sensors st_accel_sensors[] = {\n" + ">>> \t\t\t[2] = LSM330D_ACCEL_DEV_NAME,\n" + ">>> \t\t\t[3] = LSM330DL_ACCEL_DEV_NAME,\n" + ">>> \t\t\t[4] = LSM330DLC_ACCEL_DEV_NAME,\n" + ">>> +\t\t\t[5] = LSM303DLH_ACCEL_DEV_NAME,\n" + ">>> \t\t},\n" + ">>> \t\t.ch = (struct iio_chan_spec *)st_accel_12bit_channels,\n" + ">>> \t\t.odr = {\n" + ">>> @@ -238,8 +239,7 @@ static const struct st_sensors st_accel_sensors[] = {\n" + ">>> \t\t.sensors_supported = {\n" + ">>> \t\t\t[0] = LIS331DLH_ACCEL_DEV_NAME,\n" + ">>> \t\t\t[1] = LSM303DL_ACCEL_DEV_NAME,\n" + ">>> -\t\t\t[2] = LSM303DLH_ACCEL_DEV_NAME,\n" + ">>> -\t\t\t[3] = LSM303DLM_ACCEL_DEV_NAME,\n" + ">>> +\t\t\t[2] = LSM303DLM_ACCEL_DEV_NAME,\n" + ">>> \t\t},\n" + ">>> \t\t.ch = (struct iio_chan_spec *)st_accel_12bit_channels,\n" + ">>> \t\t.odr = {\n" + >>> -d4e1cf399e4cd88f6cdd3f6f6933925a019510991be42dd2f8a87b7b21754a3c +3b68502fbf8349bd81092f9bdd3ddd75f486a88a227e36b2437fe8b17f69d416
diff --git a/a/1.txt b/N2/1.txt index 14a1e8d..51a3ed5 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,41 +1,60 @@ -SGkgTGVlLA0KPj4gT24gMDkvMTAvMTMgMTM6NDksIExlZSBKb25lcyB3cm90ZToNCj4+PiBUaGUg -TFNNMzAzRExIJ3MgV0FJIChXaG9BbUkpIGlzIDB4MzMsIG1lYW5pbmcgaXQgc2hvdWxkIGJlIGVu -YWJsZWQgYnkNCj4+PiBBY2NlbCBTZW5zb3IgZ3JvdXAgb25lLiBGb3IgdGhlIGRldmljZSB0byBw -cm9iZSB3aXRob3V0IGVycm9yLCB3ZSdsbA0KPj4+IG5lZWQgdG8gZW5zdXJlIGl0J3MgcmVnaXN0 -ZXJlZCB3aXRoIHRoZSBjb3JyZWN0IFdBSS4NCj4+Pg0KPj4+IFNpZ25lZC1vZmYtYnk6IExlZSBK -b25lcyA8bGVlLmpvbmVzQGxpbmFyby5vcmc+DQo+PiBZb3UgY2xlYXJseSBoYXZlIGEgYmV0dGVy -IGRhdGFzaGVldCB0aGFuIEkgaGF2ZSBhcyBmb3IgdGhhdCBwYXJ0IGl0IGRvZXNuJ3QgZXZlbiBj -bGFpbSB0bw0KPj4gaGF2ZSB0aGUgcmVsZXZhbnQgcmVnaXN0ZXIgdG8gcmVhZCBhIHdobyBhbSBJ -IGZyb20uDQo+Pg0KPj4gTm93IHRoYXQgZGF0YXNoZWV0IGRvZXMgbGlzdCBvZHIgdmFsdWVzIGFz -IDUwLCAxMDAsIDQwMCAxMDAwIHdoaWNoIHdvdWxkIHB1dCBpdCB3aGVyZSBpdCBvcmlnaW5hbGx5 -DQo+PiB3YXMgaW4gdGhlc2UgdGFibGVzLg0KPj4NCj4+IGh0dHA6Ly93d3cuc3QuY29tL3dlYi9l -bi9yZXNvdXJjZS90ZWNobmljYWwvZG9jdW1lbnQvZGF0YXNoZWV0L0NEMDAyNjAyODgucGRmDQo+ -Pg0KPj4gSSBoYXZlbid0IGNoZWNrZWQgb3RoZXIgZWxlbWVudHMuLi4NCj4+DQo+PiBJJ20gY29u -ZnVzZWQgYnV0IHN1c3BlY3Qgd2UgbWF5IG5lZWQgYW5vdGhlciB0eXBlIGVudHJ5IHRvIGRlYWwg -d2l0aCB0aGlzIG9uZS4NCj4gSG1tbS4uLiBvbiBpbml0aWFsIHRob3VnaHQsIEkgZG9uJ3Qga25v -dyBob3cgd2UgY2FuIGhhbmRsZSB0aGlzIG9uZS4gSQ0KPiBhc3N1bWVkIHRoZSBXQUlzIHdlcmUg -YSBvbmUtc3RvcC1zaG9wIGZvciBkZXZpY2UgaWRlbnRpZmljYXRpb24uIEkgYWxzbw0KPiB0aG91 -Z2h0IHRoYXQgdGhleSB3b3VsZCBvbmx5IG1hdGNoIGlmIHRoZSBmdW5jdGlvbmFsaXR5IGFuZCBy -ZWdpc3Rlcg0KPiBtYXBzIHdvdWxkIGJlIHRoZSBzYW1lLg0KPg0KPiBJJ2xsIHRvdWNoIGJhc2Ug -d2l0aCBEZW5pcyBvbiB0aGlzLCBwZXJoYXBzIGhlIGNhbiBzaGVkIHNvbWUgbGlnaHQuDQo+DQpJ -J3ZlIGp1c3QgY2hlY2tlZCwgdGhlIFdBSSBpcyAweDMyLiBJdCB3YXMgY29ycmVjdC4uLkFyZSB5 -b3Ugc3VyZSB0aGF0IA0KeW91IHVzZSB0aGUgTFNNMzAzRExIPw0KDQpEZW5pcw0KDQoNCg0KPj4+ -IC0tLQ0KPj4+ICAgZHJpdmVycy9paW8vYWNjZWwvc3RfYWNjZWxfY29yZS5jIHwgNCArKy0tDQo+ -Pj4gICAxIGZpbGUgY2hhbmdlZCwgMiBpbnNlcnRpb25zKCspLCAyIGRlbGV0aW9ucygtKQ0KPj4+ -DQo+Pj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvaWlvL2FjY2VsL3N0X2FjY2VsX2NvcmUuYyBiL2Ry -aXZlcnMvaWlvL2FjY2VsL3N0X2FjY2VsX2NvcmUuYw0KPj4+IGluZGV4IGVhNjIyOTEuLjAzYTJi -NmIgMTAwNjQ0DQo+Pj4gLS0tIGEvZHJpdmVycy9paW8vYWNjZWwvc3RfYWNjZWxfY29yZS5jDQo+ -Pj4gKysrIGIvZHJpdmVycy9paW8vYWNjZWwvc3RfYWNjZWxfY29yZS5jDQo+Pj4gQEAgLTE3MCw2 -ICsxNzAsNyBAQCBzdGF0aWMgY29uc3Qgc3RydWN0IHN0X3NlbnNvcnMgc3RfYWNjZWxfc2Vuc29y -c1tdID0gew0KPj4+ICAgCQkJWzJdID0gTFNNMzMwRF9BQ0NFTF9ERVZfTkFNRSwNCj4+PiAgIAkJ -CVszXSA9IExTTTMzMERMX0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICAgCQkJWzRdID0gTFNNMzMwRExD -X0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICsJCQlbNV0gPSBMU00zMDNETEhfQUNDRUxfREVWX05BTUUs -DQo+Pj4gICAJCX0sDQo+Pj4gICAJCS5jaCA9IChzdHJ1Y3QgaWlvX2NoYW5fc3BlYyAqKXN0X2Fj -Y2VsXzEyYml0X2NoYW5uZWxzLA0KPj4+ICAgCQkub2RyID0gew0KPj4+IEBAIC0yMzgsOCArMjM5 -LDcgQEAgc3RhdGljIGNvbnN0IHN0cnVjdCBzdF9zZW5zb3JzIHN0X2FjY2VsX3NlbnNvcnNbXSA9 -IHsNCj4+PiAgIAkJLnNlbnNvcnNfc3VwcG9ydGVkID0gew0KPj4+ICAgCQkJWzBdID0gTElTMzMx -RExIX0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICAgCQkJWzFdID0gTFNNMzAzRExfQUNDRUxfREVWX05B -TUUsDQo+Pj4gLQkJCVsyXSA9IExTTTMwM0RMSF9BQ0NFTF9ERVZfTkFNRSwNCj4+PiAtCQkJWzNd -ID0gTFNNMzAzRExNX0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICsJCQlbMl0gPSBMU00zMDNETE1fQUND -RUxfREVWX05BTUUsDQo+Pj4gICAJCX0sDQo+Pj4gICAJCS5jaCA9IChzdHJ1Y3QgaWlvX2NoYW5f -c3BlYyAqKXN0X2FjY2VsXzEyYml0X2NoYW5uZWxzLA0KPj4+ICAgCQkub2RyID0gew0KPj4+DQo= +Hi Lee, +>> On 09/10/13 13:49, Lee Jones wrote: +>>> The LSM303DLH's WAI (WhoAmI) is 0x33, meaning it should be enabled by +>>> Accel Sensor group one. For the device to probe without error, we'll +>>> need to ensure it's registered with the correct WAI. +>>> +>>> Signed-off-by: Lee Jones <lee.jones@linaro.org> +>> You clearly have a better datasheet than I have as for that part it doesn't even claim to +>> have the relevant register to read a who am I from. +>> +>> Now that datasheet does list odr values as 50, 100, 400 1000 which would put it where it originally +>> was in these tables. +>> +>> http://www.st.com/web/en/resource/technical/document/datasheet/CD00260288.pdf +>> +>> I haven't checked other elements... +>> +>> I'm confused but suspect we may need another type entry to deal with this one. +> Hmmm... on initial thought, I don't know how we can handle this one. I +> assumed the WAIs were a one-stop-shop for device identification. I also +> thought that they would only match if the functionality and register +> maps would be the same. +> +> I'll touch base with Denis on this, perhaps he can shed some light. +> +I've just checked, the WAI is 0x32. It was correct...Are you sure that +you use the LSM303DLH? + +Denis + + + +>>> --- +>>> drivers/iio/accel/st_accel_core.c | 4 ++-- +>>> 1 file changed, 2 insertions(+), 2 deletions(-) +>>> +>>> diff --git a/drivers/iio/accel/st_accel_core.c b/drivers/iio/accel/st_accel_core.c +>>> index ea62291..03a2b6b 100644 +>>> --- a/drivers/iio/accel/st_accel_core.c +>>> +++ b/drivers/iio/accel/st_accel_core.c +>>> @@ -170,6 +170,7 @@ static const struct st_sensors st_accel_sensors[] = { +>>> [2] = LSM330D_ACCEL_DEV_NAME, +>>> [3] = LSM330DL_ACCEL_DEV_NAME, +>>> [4] = LSM330DLC_ACCEL_DEV_NAME, +>>> + [5] = LSM303DLH_ACCEL_DEV_NAME, +>>> }, +>>> .ch = (struct iio_chan_spec *)st_accel_12bit_channels, +>>> .odr = { +>>> @@ -238,8 +239,7 @@ static const struct st_sensors st_accel_sensors[] = { +>>> .sensors_supported = { +>>> [0] = LIS331DLH_ACCEL_DEV_NAME, +>>> [1] = LSM303DL_ACCEL_DEV_NAME, +>>> - [2] = LSM303DLH_ACCEL_DEV_NAME, +>>> - [3] = LSM303DLM_ACCEL_DEV_NAME, +>>> + [2] = LSM303DLM_ACCEL_DEV_NAME, +>>> }, +>>> .ch = (struct iio_chan_spec *)st_accel_12bit_channels, +>>> .odr = { +>>> +ÿôèº{.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¥ diff --git a/a/content_digest b/N2/content_digest index c870cf5..d919e2a 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -14,46 +14,65 @@ " linux-iio@vger.kernel.org <linux-iio@vger.kernel.org>\0" "\00:1\0" "b\0" - "SGkgTGVlLA0KPj4gT24gMDkvMTAvMTMgMTM6NDksIExlZSBKb25lcyB3cm90ZToNCj4+PiBUaGUg\n" - "TFNNMzAzRExIJ3MgV0FJIChXaG9BbUkpIGlzIDB4MzMsIG1lYW5pbmcgaXQgc2hvdWxkIGJlIGVu\n" - "YWJsZWQgYnkNCj4+PiBBY2NlbCBTZW5zb3IgZ3JvdXAgb25lLiBGb3IgdGhlIGRldmljZSB0byBw\n" - "cm9iZSB3aXRob3V0IGVycm9yLCB3ZSdsbA0KPj4+IG5lZWQgdG8gZW5zdXJlIGl0J3MgcmVnaXN0\n" - "ZXJlZCB3aXRoIHRoZSBjb3JyZWN0IFdBSS4NCj4+Pg0KPj4+IFNpZ25lZC1vZmYtYnk6IExlZSBK\n" - "b25lcyA8bGVlLmpvbmVzQGxpbmFyby5vcmc+DQo+PiBZb3UgY2xlYXJseSBoYXZlIGEgYmV0dGVy\n" - "IGRhdGFzaGVldCB0aGFuIEkgaGF2ZSBhcyBmb3IgdGhhdCBwYXJ0IGl0IGRvZXNuJ3QgZXZlbiBj\n" - "bGFpbSB0bw0KPj4gaGF2ZSB0aGUgcmVsZXZhbnQgcmVnaXN0ZXIgdG8gcmVhZCBhIHdobyBhbSBJ\n" - "IGZyb20uDQo+Pg0KPj4gTm93IHRoYXQgZGF0YXNoZWV0IGRvZXMgbGlzdCBvZHIgdmFsdWVzIGFz\n" - "IDUwLCAxMDAsIDQwMCAxMDAwIHdoaWNoIHdvdWxkIHB1dCBpdCB3aGVyZSBpdCBvcmlnaW5hbGx5\n" - "DQo+PiB3YXMgaW4gdGhlc2UgdGFibGVzLg0KPj4NCj4+IGh0dHA6Ly93d3cuc3QuY29tL3dlYi9l\n" - "bi9yZXNvdXJjZS90ZWNobmljYWwvZG9jdW1lbnQvZGF0YXNoZWV0L0NEMDAyNjAyODgucGRmDQo+\n" - "Pg0KPj4gSSBoYXZlbid0IGNoZWNrZWQgb3RoZXIgZWxlbWVudHMuLi4NCj4+DQo+PiBJJ20gY29u\n" - "ZnVzZWQgYnV0IHN1c3BlY3Qgd2UgbWF5IG5lZWQgYW5vdGhlciB0eXBlIGVudHJ5IHRvIGRlYWwg\n" - "d2l0aCB0aGlzIG9uZS4NCj4gSG1tbS4uLiBvbiBpbml0aWFsIHRob3VnaHQsIEkgZG9uJ3Qga25v\n" - "dyBob3cgd2UgY2FuIGhhbmRsZSB0aGlzIG9uZS4gSQ0KPiBhc3N1bWVkIHRoZSBXQUlzIHdlcmUg\n" - "YSBvbmUtc3RvcC1zaG9wIGZvciBkZXZpY2UgaWRlbnRpZmljYXRpb24uIEkgYWxzbw0KPiB0aG91\n" - "Z2h0IHRoYXQgdGhleSB3b3VsZCBvbmx5IG1hdGNoIGlmIHRoZSBmdW5jdGlvbmFsaXR5IGFuZCBy\n" - "ZWdpc3Rlcg0KPiBtYXBzIHdvdWxkIGJlIHRoZSBzYW1lLg0KPg0KPiBJJ2xsIHRvdWNoIGJhc2Ug\n" - "d2l0aCBEZW5pcyBvbiB0aGlzLCBwZXJoYXBzIGhlIGNhbiBzaGVkIHNvbWUgbGlnaHQuDQo+DQpJ\n" - "J3ZlIGp1c3QgY2hlY2tlZCwgdGhlIFdBSSBpcyAweDMyLiBJdCB3YXMgY29ycmVjdC4uLkFyZSB5\n" - "b3Ugc3VyZSB0aGF0IA0KeW91IHVzZSB0aGUgTFNNMzAzRExIPw0KDQpEZW5pcw0KDQoNCg0KPj4+\n" - "IC0tLQ0KPj4+ICAgZHJpdmVycy9paW8vYWNjZWwvc3RfYWNjZWxfY29yZS5jIHwgNCArKy0tDQo+\n" - "Pj4gICAxIGZpbGUgY2hhbmdlZCwgMiBpbnNlcnRpb25zKCspLCAyIGRlbGV0aW9ucygtKQ0KPj4+\n" - "DQo+Pj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvaWlvL2FjY2VsL3N0X2FjY2VsX2NvcmUuYyBiL2Ry\n" - "aXZlcnMvaWlvL2FjY2VsL3N0X2FjY2VsX2NvcmUuYw0KPj4+IGluZGV4IGVhNjIyOTEuLjAzYTJi\n" - "NmIgMTAwNjQ0DQo+Pj4gLS0tIGEvZHJpdmVycy9paW8vYWNjZWwvc3RfYWNjZWxfY29yZS5jDQo+\n" - "Pj4gKysrIGIvZHJpdmVycy9paW8vYWNjZWwvc3RfYWNjZWxfY29yZS5jDQo+Pj4gQEAgLTE3MCw2\n" - "ICsxNzAsNyBAQCBzdGF0aWMgY29uc3Qgc3RydWN0IHN0X3NlbnNvcnMgc3RfYWNjZWxfc2Vuc29y\n" - "c1tdID0gew0KPj4+ICAgCQkJWzJdID0gTFNNMzMwRF9BQ0NFTF9ERVZfTkFNRSwNCj4+PiAgIAkJ\n" - "CVszXSA9IExTTTMzMERMX0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICAgCQkJWzRdID0gTFNNMzMwRExD\n" - "X0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICsJCQlbNV0gPSBMU00zMDNETEhfQUNDRUxfREVWX05BTUUs\n" - "DQo+Pj4gICAJCX0sDQo+Pj4gICAJCS5jaCA9IChzdHJ1Y3QgaWlvX2NoYW5fc3BlYyAqKXN0X2Fj\n" - "Y2VsXzEyYml0X2NoYW5uZWxzLA0KPj4+ICAgCQkub2RyID0gew0KPj4+IEBAIC0yMzgsOCArMjM5\n" - "LDcgQEAgc3RhdGljIGNvbnN0IHN0cnVjdCBzdF9zZW5zb3JzIHN0X2FjY2VsX3NlbnNvcnNbXSA9\n" - "IHsNCj4+PiAgIAkJLnNlbnNvcnNfc3VwcG9ydGVkID0gew0KPj4+ICAgCQkJWzBdID0gTElTMzMx\n" - "RExIX0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICAgCQkJWzFdID0gTFNNMzAzRExfQUNDRUxfREVWX05B\n" - "TUUsDQo+Pj4gLQkJCVsyXSA9IExTTTMwM0RMSF9BQ0NFTF9ERVZfTkFNRSwNCj4+PiAtCQkJWzNd\n" - "ID0gTFNNMzAzRExNX0FDQ0VMX0RFVl9OQU1FLA0KPj4+ICsJCQlbMl0gPSBMU00zMDNETE1fQUND\n" - "RUxfREVWX05BTUUsDQo+Pj4gICAJCX0sDQo+Pj4gICAJCS5jaCA9IChzdHJ1Y3QgaWlvX2NoYW5f\n" - c3BlYyAqKXN0X2FjY2VsXzEyYml0X2NoYW5uZWxzLA0KPj4+ICAgCQkub2RyID0gew0KPj4+DQo= + "Hi Lee,\n" + ">> On 09/10/13 13:49, Lee Jones wrote:\n" + ">>> The LSM303DLH's WAI (WhoAmI) is 0x33, meaning it should be enabled by\n" + ">>> Accel Sensor group one. For the device to probe without error, we'll\n" + ">>> need to ensure it's registered with the correct WAI.\n" + ">>>\n" + ">>> Signed-off-by: Lee Jones <lee.jones@linaro.org>\n" + ">> You clearly have a better datasheet than I have as for that part it doesn't even claim to\n" + ">> have the relevant register to read a who am I from.\n" + ">>\n" + ">> Now that datasheet does list odr values as 50, 100, 400 1000 which would put it where it originally\n" + ">> was in these tables.\n" + ">>\n" + ">> http://www.st.com/web/en/resource/technical/document/datasheet/CD00260288.pdf\n" + ">>\n" + ">> I haven't checked other elements...\n" + ">>\n" + ">> I'm confused but suspect we may need another type entry to deal with this one.\n" + "> Hmmm... on initial thought, I don't know how we can handle this one. I\n" + "> assumed the WAIs were a one-stop-shop for device identification. I also\n" + "> thought that they would only match if the functionality and register\n" + "> maps would be the same.\n" + ">\n" + "> I'll touch base with Denis on this, perhaps he can shed some light.\n" + ">\n" + "I've just checked, the WAI is 0x32. It was correct...Are you sure that \n" + "you use the LSM303DLH?\n" + "\n" + "Denis\n" + "\n" + "\n" + "\n" + ">>> ---\n" + ">>> drivers/iio/accel/st_accel_core.c | 4 ++--\n" + ">>> 1 file changed, 2 insertions(+), 2 deletions(-)\n" + ">>>\n" + ">>> diff --git a/drivers/iio/accel/st_accel_core.c b/drivers/iio/accel/st_accel_core.c\n" + ">>> index ea62291..03a2b6b 100644\n" + ">>> --- a/drivers/iio/accel/st_accel_core.c\n" + ">>> +++ b/drivers/iio/accel/st_accel_core.c\n" + ">>> @@ -170,6 +170,7 @@ static const struct st_sensors st_accel_sensors[] = {\n" + ">>> \t\t\t[2] = LSM330D_ACCEL_DEV_NAME,\n" + ">>> \t\t\t[3] = LSM330DL_ACCEL_DEV_NAME,\n" + ">>> \t\t\t[4] = LSM330DLC_ACCEL_DEV_NAME,\n" + ">>> +\t\t\t[5] = LSM303DLH_ACCEL_DEV_NAME,\n" + ">>> \t\t},\n" + ">>> \t\t.ch = (struct iio_chan_spec *)st_accel_12bit_channels,\n" + ">>> \t\t.odr = {\n" + ">>> @@ -238,8 +239,7 @@ static const struct st_sensors st_accel_sensors[] = {\n" + ">>> \t\t.sensors_supported = {\n" + ">>> \t\t\t[0] = LIS331DLH_ACCEL_DEV_NAME,\n" + ">>> \t\t\t[1] = LSM303DL_ACCEL_DEV_NAME,\n" + ">>> -\t\t\t[2] = LSM303DLH_ACCEL_DEV_NAME,\n" + ">>> -\t\t\t[3] = LSM303DLM_ACCEL_DEV_NAME,\n" + ">>> +\t\t\t[2] = LSM303DLM_ACCEL_DEV_NAME,\n" + ">>> \t\t},\n" + ">>> \t\t.ch = (struct iio_chan_spec *)st_accel_12bit_channels,\n" + ">>> \t\t.odr = {\n" + ">>>\n" + "\303\277\303\264\303\250\302\272{.n\303\207+\302\211\302\267\302\237\302\256\302\211\302\255\302\206+%\302\212\303\213\303\277\302\261\303\251\303\235\302\266\027\302\245\302\212w\303\277\302\272{.n\303\207+\302\211\302\267\302\245\302\212{\302\261\303\276G\302\253\302\235\303\251\303\277\302\212{ay\302\272\035\303\212\302\207\303\232\302\231\303\253,j\a\302\255\302\242f\302\243\302\242\302\267h\302\232\302\217\303\257\302\201\303\252\303\277\302\221\303\252\303\247z_\303\250\302\256\003(\302\255\303\251\302\232\302\216\302\212\303\235\302\242j\"\302\235\303\272\032\302\266\033m\302\247\303\277\303\277\302\276\a\302\253\303\276G\302\253\302\235\303\251\303\277\302\242\302\270?\302\231\302\250\303\250\302\255\303\232&\302\243\303\270\302\247~\302\217\303\241\302\266iO\302\225\303\246\302\254z\302\267\302\232v\303\230^\024\004\032\302\266\033m\302\247\303\277\303\277\303\203\f\303\277\302\266\303\254\303\277\302\242\302\270?\302\226I\302\245" -d4e1cf399e4cd88f6cdd3f6f6933925a019510991be42dd2f8a87b7b21754a3c +3626abda52f1c191c3a16292cec9ff0562a6d6bcb4634e386803d82916ff4843
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.