weird usb problem

[Maksim Yevmenkin]

[...]

> > > From a quick glimpse, you need to add something like
> > >
> > > "ehci_pcd(sc, sc->sc_intrxfer);" after "ehci_pcd_able(sc, 1);".
> > >
> > > In the middle you have to add something like this:
> > >
> > >         /* acknowledge any PCD interrupt */
> > >         EOWRITE4(sc, EHCI_USBSTS, EHCI_STS_PCD);
> > >
> > > Just look at the code in my USB driver, and if it fixes the problem, just
> > > backport it.
> >
> > well, i tried a naive thing like this
> >
> > beetle% diff -u10 ehci.c.orig ehci.c
> > --- ehci.c.orig Sat May 28 21:42:27 2005
> > +++ ehci.c      Tue Aug  9 09:53:13 2005
> > @@ -632,20 +632,21 @@
> >                 sc->sc_eintrs &= ~EHCI_STS_PCD;
> >         EOWRITE4(sc, EHCI_USBINTR, sc->sc_eintrs);
> >  }
> >
> >  void
> >  ehci_pcd_enable(void *v_sc)
> >  {
> >         ehci_softc_t *sc = v_sc;
> >
> >         ehci_pcd_able(sc, 1);
> 
> You need to clear the PCD interrupt bit here, because it is possibly still
> set, and enabling bits in the interrupt mask does not generate another
> interrupt!
> 
>          /* acknowledge any PCD interrupt */
>          EOWRITE4(sc, EHCI_USBSTS, EHCI_STS_PCD);
> 
> > +       ehci_pcd(sc, sc->sc_intrxfer);
> >  }

oops, my bad. i did not realize i have to use  EHCI_USBSTS and not
EHCI_USBINTR (as in ehci_pcd_able()), so the diff  looks like

beetle% diff -u10 ehci.c.orig ehci.c
--- ehci.c.orig Sat May 28 21:42:27 2005
+++ ehci.c      Tue Aug  9 11:47:17 2005
@@ -632,20 +632,25 @@
                sc->sc_eintrs &= ~EHCI_STS_PCD;
        EOWRITE4(sc, EHCI_USBINTR, sc->sc_eintrs);
 }
 
 void
 ehci_pcd_enable(void *v_sc)
 {
        ehci_softc_t *sc = v_sc;
 
        ehci_pcd_able(sc, 1);
+
+       /* acknowledge any PCD interrupt */
+       EOWRITE4(sc, EHCI_USBSTS, EHCI_STS_PCD);
+
+       ehci_pcd(sc, sc->sc_intrxfer);
 }
 
 void
  ehci_pcd(ehci_softc_t *sc, usbd_xfer_handle xfer)
 {
        usbd_pipe_handle pipe;
        u_char *p;
        int i, m;
 
        if (xfer == NULL) {

==

does it look right to you?

> > other usb slots are connected to the uhub5 ports 2 to 4 and the usb
> > 2.0 device is _not_ working when plugged into these slots.
> >
> > usb 1.0 devices work in every usb slot.
> 
> If you have got your source tree in another directory, which makes installing
> to "/usr/src" difficult, you can set the variable "S" in the Makefile to that
> director and it will install there.
> 
> Do like this:
> 
> sysctl hw.usb.ehci.debug=15
> 
> Then plug your device.
> 
> sysctl hw.usb.ehci.debug=0
> 
> If you see a pcd change message your device is detected. But just watch and
> you will see that it stops interrupting.
> 
> The problem is the same with OHCI.

ok, i have tried this and  *nothing* happened when i plug the usb 2.0
device. no debug output whatsoever.

so, i started looking at my usb 2.0 device, and guess what. the
frigging usb connector on my usb 2.0 jump drive device is a about 2-3
millimeters short than on my other usb devices!!! also there is a
little bit of extra plastic molding around usb socket, so there was no
electrical connection!!!

i just got an usb extension cable with _proper_sized_ usb plug and
tried it, and it just worked. i will make a clean build without any
ehci(4) changes to see if the problem still there.

thanks,
max