[PATCH linux] Fix double accelerometer initialisation

Sascha Silbe sascha-ml-reply-to-2011-4 at silbe.org
Mon Dec 19 14:45:33 EST 2011

Excerpts from James Cameron's message of 2011-11-23 07:00:43 +0100:

> > Tested on B1 only since I don't have a C1. Are there even actual C1s
> > or just patched-up Bx? I wonder because the wiki page for C1 doesn't
> > exist yet.

> Yes, many C1 exist.  Sorry about the Wiki page.

No problem; I consider the wiki a best-effort attempt at documenting the
prototypes and am quite glad about the amount of information that _is_
available. Read: Thanks to anyone involved!

[Tested on all combinations of B1/C1 with old/new accelerometer]

Thanks for testing! My approach was based on the assumption that only
the combinations B1/old and C1/new exist, so the patch is expected to
fail with the two other combinations.

Is there any way to tell what kind of accelerometer device the board has
hooked up, other than probing the device? AFAICT I2C probing will happen
after olpc_xo_1_75_init() has run, so mmp2_add_twsi() can't give any
feedback on whether adding the device was "successful". We either need
some other way to tell the difference inside olpc_xo_1_75_init() or
modify the lis3lv02d driver to allow multiple instances.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://lists.laptop.org/pipermail/devel/attachments/20111219/20953dd9/attachment.sig>

More information about the Devel mailing list