[i2c] i2c-tools: Testers wanted

[Jean Delvare]

Hi Ryan,

On Mon, 01 Oct 2007 09:07:40 +1300, Ryan Mallon wrote:
> Jean Delvare wrote:
> > Hi all,
> 
> > A SVN snapshot is available for download:
> > http://dl.lm-sensors.org/i2c-tools/snapshots/i2c-tools-r4909-20070930.tar.bz2
> > 
> > I would appreciate if people interested in these tools could download
> > and test the package now, and report how it works for them. Suggestions
> > for improvements are welcome, and patches implementing these even more
> > so.
> 
> I'm cross compiling the tools for the ARM. Doing make in the top level
> directory didn't work at first. I changed -I../include to -Iinclude in
> tools/Module.mk and then it built fine with no errors.

Good catch. Originally the tools directory had a standalone Makefile,
and I forgot to change the include path when integrating it into the
main build system. It's fixed in SVN now, thanks for reporting.

> I had to change the name of my i2c device name from /dev/i2c0 to
> /dev/i2c-0 as the former was not recognized by the tools. We are using
> a root filesystem created by OpenEmbedded, though I'm not sure who, or
> what was responsible for the i2c device name :-).

I don't know either. I've seen all of /dev/i2c-0, /dev/i2c/0
and /dev/i2c0 in the past, I don't know if any of these is more
official than the others. All the distributions I've been using
have /dev/i2c-0. The i2c tools currently support the former two, but
not the latter one, as you found out yourself. The reason is that it
seemed to be only rarely used. But if you think that this name is still
in use and should be supported, adding it back should be pretty
trivial, so I can do that.

> The tools seem to work correctly after I renamed the device. It
> correctly detected all the i2c devices on one of our boards.

Great, thanks a lot for testing and reporting :)

-- 
Jean Delvare