Re: [Bluez-users] Logitech Bluetooth Mx900+keyboard support.

[Marcel Holtmann <marcel@holtmann.org>]

Hi Terence,

> I reviewed the code.  I focused on this section of code in
> switch_logitech(...)
> {
>     char devname[PATH_MAX + 1];
>     int i, fd, err = 0;
>     for (i = 0; i < 16; i++) {
> ...
> 	sprintf(devname, "%s/hiddev%d", hidpath, i);
> 	fd = open(devname, O_RDWR);
> 	if (fd < 0)
> 		continue;
> ...
> It occurred to me that open() could fail the 16 times and not report
> that to the caller.  So i init'ed err to -1.  This is fine because  if
> the open() works then err will be set based on the ioctl() or
> send_report() calls.  But if open() fails all 16 time then the caller is
> advised that something is wrong. 
> 
> Once I confirmed that in fact I was failing when I called hid2hci  I
> went to understand why.  This was I had for dev dirtree. 
>    /dev/usb/hiddev 
> not
>    /dev/usb/hid/hiddev 
> as set in hidpath. So to work on my system I needed to change the
> hidpath.   
> 
> While this works on my system.  What other systems might it break?   A
> dynamic function to pull the correct value? Or is there a naming
> authority, a header file, where this can be found?

I used the devfs path, because my Debian Sid has no default device node
for it. What is your system?

Regards

Marcel




-------------------------------------------------------
The SF.Net email is sponsored by EclipseCon 2004
Premiere Conference on Open Tools Development and Integration
See the breadth of Eclipse activity. February 3-5 in Anaheim, CA.
http://www.eclipsecon.org/osdn
_______________________________________________
Bluez-users mailing list
Bluez-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-users