Date: Tue, 5 May 2009 12:22:00 +0200 From: Hans Petter Selasky <hselasky@c2i.net> To: freebsd-current@freebsd.org Cc: Chao Shin <quakelee@geekcn.org> Subject: Re: current couldn't attach usb mouse Message-ID: <200905051222.00887.hselasky@c2i.net> In-Reply-To: <op.utf8tjbwhnq548@qld630> References: <op.utego3echnq548@qld630> <200905051040.53708.hselasky@c2i.net> <op.utf8tjbwhnq548@qld630>
next in thread | previous in thread | raw e-mail | index | archive | help
--Boundary-00=_INBAKBeYLWaJ1GH
Content-Type: text/plain;
charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline
On Tuesday 05 May 2009, Chao Shin wrote:
> =E5=9C=A8 Tue, 05 May 2009 16:40:53 +0800=EF=BC=8CHans Petter Selasky <hs=
elasky@c2i.net>
>
> =E5=86=99=E9=81=93:
> > sysctl hw.usb2.uhci.debug=3D15
>
> This is my dmesg in attchement.
Try the attached patch to uhci.c.
=2D-HPS
--Boundary-00=_INBAKBeYLWaJ1GH
Content-Type: text/x-diff;
charset="utf-8";
name="uhci.c.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
filename="uhci.c.diff"
--- /usr/8-current/src/sys/dev/usb/controller/uhci.c Tue May 5 12:15:08 2009
+++ src/sys/dev/usb/controller/uhci.c Tue May 5 12:20:53 2009
@@ -132,6 +132,7 @@
extern struct usb2_pipe_methods uhci_device_intr_methods;
extern struct usb2_pipe_methods uhci_device_isoc_methods;
+static uint8_t uhci_restart(uhci_softc_t *sc);
static void uhci_do_poll(struct usb2_bus *);
static void uhci_device_done(struct usb2_xfer *, usb2_error_t);
static void uhci_transfer_intr_enqueue(struct usb2_xfer *);
@@ -246,10 +247,51 @@
ml->buf_offset += td->len;
}
+/*
+ * Return values:
+ * 0: Success
+ * Else: Failure
+ */
+static uint8_t
+uhci_restart(uhci_softc_t *sc)
+{
+ struct usb2_page_search buf_res;
+
+ USB_BUS_LOCK_ASSERT(&sc->sc_bus, MA_OWNED);
+
+ if (UREAD2(sc, UHCI_CMD) & UHCI_CMD_RS) {
+ DPRINTFN(2, "Already started\n");
+ return (0);
+ }
+
+ DPRINTFN(2, "Restarting\n");
+
+ usb2_get_page(&sc->sc_hw.pframes_pc, 0, &buf_res);
+
+ /* Reload fresh base address */
+ UWRITE4(sc, UHCI_FLBASEADDR, buf_res.physaddr);
+
+ /*
+ * Assume 64 byte packets at frame end and start HC controller:
+ */
+ UHCICMD(sc, (UHCI_CMD_MAXP | UHCI_CMD_RS));
+
+ /* wait 10 milliseconds */
+
+ usb2_pause_mtx(&sc->sc_bus.bus_mtx, hz / 100);
+
+ /* check that controller has started */
+
+ if (!(UREAD2(sc, UHCI_STS) & UHCI_STS_HCH)) {
+ DPRINTFN(2, "Failed\n");
+ return (1);
+ }
+ return (0);
+}
+
void
uhci_reset(uhci_softc_t *sc)
{
- struct usb2_page_search buf_res;
uint16_t n;
USB_BUS_LOCK_ASSERT(&sc->sc_bus, MA_OWNED);
@@ -309,8 +351,6 @@
done_2:
/* reload the configuration */
- usb2_get_page(&sc->sc_hw.pframes_pc, 0, &buf_res);
- UWRITE4(sc, UHCI_FLBASEADDR, buf_res.physaddr);
UWRITE2(sc, UHCI_FRNUM, sc->sc_saved_frnum);
UWRITE1(sc, UHCI_SOF, sc->sc_saved_sof);
@@ -337,30 +377,11 @@
UHCI_INTR_IOCE |
UHCI_INTR_SPIE));
- /*
- * assume 64 byte packets at frame end and start HC controller
- */
-
- UHCICMD(sc, (UHCI_CMD_MAXP | UHCI_CMD_RS));
-
- uint8_t n = 10;
-
- while (n--) {
- /* wait one millisecond */
-
- usb2_pause_mtx(&sc->sc_bus.bus_mtx, hz / 1000);
-
- /* check that controller has started */
-
- if (!(UREAD2(sc, UHCI_STS) & UHCI_STS_HCH)) {
- goto done;
- }
+ if (uhci_restart(sc)) {
+ device_printf(sc->sc_bus.bdev,
+ "cannot start HC controller\n");
}
- device_printf(sc->sc_bus.bdev,
- "cannot start HC controller\n");
-
-done:
/* start root interrupt */
uhci_root_intr(sc);
}
@@ -2072,10 +2093,8 @@
qh->qh_e_next = td->td_self;
if (xfer->xroot->udev->state != USB_STATE_SUSPENDED) {
-
/* enter QHs into the controller data structures */
UHCI_APPEND_QH(qh, sc->sc_intr_p_last[xfer->qh_pos]);
-
} else {
usb2_pc_cpu_flush(qh->page_cache);
}
@@ -2391,15 +2410,7 @@
* Before we do anything, turn on SOF messages on the USB
* BUS. Some USB devices do not cope without them!
*/
- if (!(UREAD2(sc, UHCI_CMD) & UHCI_CMD_RS)) {
-
- DPRINTF("Activating SOFs!\n");
-
- UHCICMD(sc, (UHCI_CMD_MAXP | UHCI_CMD_RS));
-
- /* wait a little bit */
- usb2_pause_mtx(&sc->sc_bus.bus_mtx, hz / 100);
- }
+ uhci_restart(sc);
x = URWMASK(UREAD2(sc, port));
UWRITE2(sc, port, x | UHCI_PORTSC_PR);
@@ -3196,11 +3207,11 @@
USB_HW_POWER_INTERRUPT |
USB_HW_POWER_ISOC)) {
DPRINTF("Some USB transfer is "
- "active on %u.\n",
+ "active on unit %u.\n",
device_get_unit(sc->sc_bus.bdev));
- UHCICMD(sc, (UHCI_CMD_MAXP | UHCI_CMD_RS));
+ uhci_restart(sc);
} else {
- DPRINTF("Power save on %u.\n",
+ DPRINTF("Power save on unit %u.\n",
device_get_unit(sc->sc_bus.bdev));
UHCICMD(sc, UHCI_CMD_MAXP);
}
--Boundary-00=_INBAKBeYLWaJ1GH--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200905051222.00887.hselasky>