<html><head></head><body bgcolor="#FFFFFF"><div>Thanks for looking at this. I thought I was seeing this error message continue to scroll by...</div><div><br></div><div>Cheers,</div><div>was</div><div><br><br><div><br></div></div><div><br>On Dec 2, 2011, at 11:59 AM, Saadia Husain Baloch <<a href="mailto:saadia@laptop.org">saadia@laptop.org</a>> wrote:<br><br></div><div></div><blockquote type="cite"><div>Sasha and James,<br>I thought this was pushed to arm-3.0-wip, but it doesn't show up there. After trying it out, I found that the board revision was not current, and got that fixed by jnettlet.<br>Any clues as to whether the patch was pushed?<br>
-Saadia<br><br><br><div class="gmail_quote">On Wed, Nov 23, 2011 at 1:00 AM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Tue, Nov 22, 2011 at 01:13:55PM +0100, Sascha Silbe wrote:<br>
> The lis3lv02d driver doesn't support multiple instances of itself, so<br>
> we need to make sure we instantiate only the one that's actually<br>
> present. This also avoids logging an error message about not being<br>
> able to initialise the 8-bit sensor on C1 and above.<br>
><br>
> Signed-off-by: Sascha Silbe <<a href="mailto:silbe@activitycentral.com">silbe@activitycentral.com</a>><br>
> ---<br>
> Tested on B1 only since I don't have a C1. Are there even actual C1s<br>
> or just patched-up Bx? I wonder because the wiki page for C1 doesn't<br>
> exist yet.<br>
<br>
</div>Yes, many C1 exist. Sorry about the Wiki page.<br>
<br>
Tested your patch with 5611fd36a7b30edac640e1bd8ab2948ca91d09d5 by<br>
Andres, since this is needed for board detect to work, and also with<br>
q4c05jc [1] which changes /proc/device-tree/openprom/architecture to<br>
OLPC.<br>
<br>
On an XO-1.75 C1 with new accelerometer, dmesg contains:<br>
<br>
lis3lv02d: 16 bits sensor found<br>
input: ST LIS3LV02DL Accelerometer as /devices/platform/lis3lv02d/input/input0<br>
<br>
On an XO-1.75 B1 with old accelerometer, dmesg contains:<br>
<br>
lis3lv02d: 8 bits sensor found<br>
input: ST LIS3LV02DL Accelerometer as /devices/platform/lis3lv02d/input/input0<br>
<br>
On an XO-1.75 B1 with new accelerometer, dmesg contains:<br>
<br>
lis3lv02d: unknown sensor type 0x87<br>
lis3lv02d_i2c: probe of 5-001d failed with error -22<br>
<br>
On an XO-1.75 B4 with new accelerometer, dmesg contains:<br>
<br>
lis3lv02d: unknown sensor type 0x87<br>
lis3lv02d_i2c: probe of 5-001d failed with error -22<br>
<br>
So effectively your patch breaks XO-1.75 B1 and B4 with new<br>
accelerometer chip. ;-}<br>
<br>
Saadia is looking into this, she said.<br>
<br>
--<br>
<br>
[1] <a href="http://dev.laptop.org/%7Equozl/q4c05jc.rom" target="_blank">http://dev.laptop.org/~quozl/q4c05jc.rom</a><br>
<font color="#888888"><br>
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/</a><br>
</font><div><div></div><div class="h5">_______________________________________________<br>
Devel mailing list<br>
<a href="mailto:Devel@lists.laptop.org">Devel@lists.laptop.org</a><br>
<a href="http://lists.laptop.org/listinfo/devel" target="_blank">http://lists.laptop.org/listinfo/devel</a><br>
</div></div></blockquote></div><br>
</div></blockquote><blockquote type="cite"><div><span>_______________________________________________</span><br><span>Devel mailing list</span><br><span><a href="mailto:Devel@lists.laptop.org">Devel@lists.laptop.org</a></span><br><span><a href="http://lists.laptop.org/listinfo/devel">http://lists.laptop.org/listinfo/devel</a></span><br></div></blockquote></body></html>