Re: TEMPer and TEMPerHUM usb thermometer driver

[Paul Goyette <paul%whooppee.com@localhost>]

Thanks, plunky, for commiting the aprint's.

As for the second problem (no sensors attached), can you rebuild with 
the following patch and see what it says?

Index: uthum.c
===================================================================
RCS file: /cvsroot/src/sys/dev/usb/uthum.c,v
retrieving revision 1.2
diff -u -p -r1.2 uthum.c
--- uthum.c     24 Feb 2010 17:34:56 -0000      1.2
+++ uthum.c     24 Feb 2010 19:11:38 -0000
@@ -189,7 +189,10 @@ USB_ATTACH(uthum)
                DPRINTF(("sensor type: TEMPer\n"));
                break;
        case UTHUM_TYPE_UNKNOWN:
-               DPRINTF(("sensor type: unknown, give up to attach sensors\n"));
+               aprint_error_dev(self, "sensor type %d unknown\n",
+                                sc->sc_sensortype);
+               sc->sc_num_sensors = 0;
+               break;
        default:
                break;
        }


On Wed, 24 Feb 2010, Iain Hibbert wrote:

> On Wed, 24 Feb 2010, Andreas Gustafsson wrote:
>
> > I bought a TEMPerHUM and tried it with GENERIC kernel
> freshly built
> > from 2010.02.24.12.29.47 sources.  It didn't quite work.
> Plugging it
> > in yielded the following lines of dmesg output:
> >
> >    uhidev0 at uhub3 port 1 configuration 1 interface 0
> >    uhidev0: vendor 0x1130 PCsensor Temper, rev 1.10/1.50,
> addr 2, iclass 3/1
> >    uthum0 at uhidev0uhidev1 at uhub3 port 1 configuration
> 1 interface 1
> >    uhidev1: vendor 0x1130 PCsensor Temper, rev 1.10/1.50,
> addr 2, iclass 3/0
> >    uthum1 at uhidev1
> >
> > Note the missing newline between "uthum0 at uhidev0" and
> uhidev1 at
> > uhub3"; the output is also missing a newline at the very
> end.
>
> the attach routine should print a newline (naive and normal)
> before doing anything that will sleep, in this case before
> the uhidev_get_report_desc() call.
>
> > Running envstat resulted in:
> >
> >    envstat: no drivers registered
>
> I know nothing about that
>
> iain

-------------------------------------------------------------------------
|   Paul Goyette   | PGP DSS Key fingerprint: |  E-mail addresses:      |
| Customer Service | FA29 0E3B 35AF E8AE 6651 |  paul at whooppee.com   |
| Network Engineer | 0786 F758 55DE 53BA 7731 | pgoyette at juniper.net |
| Kernel Developer |                          | pgoyette at netbsd.org  |
-------------------------------------------------------------------------