Remove definitions and usage of usb_callout and related macros. These macros
were used as a layer of confusion^Wabstraction around the timeout(9) API.

No binary change.

ok jsg.

Remove the "Static" declaration of many functions. It was defined to be empty
and it was not consistently used. It was confusing as it suggested these
functions were static, which they were not.

d

Remove the "Static" declaration of many functions. It was defined to be empty
and it was not consistently used. It was confusing as it suggested these
functions were static, which they were not.

discussed with dlg and jsg, ok jsg.

show more ...

Remove le{16,32}toh macros

Remove ifdef __otherbsd__ mess; no binary change.

Correct URL to UHCI spec.
From dunceor@gmail.com

Reapplication of rev 1.49.

Save SOF (frame timing adjustment) before the very first host controller
reset, rather than at suspend/resume, otherwise any BIOS inserted value
is lost immediately.

ok d

Reapplication of rev 1.49.

Save SOF (frame timing adjustment) before the very first host controller
reset, rather than at suspend/resume, otherwise any BIOS inserted value
is lost immediately.

ok dlg@

show more ...

Backout pascoe@'s last USB change because it page faults at attachment
time.

Chris; Get your shiz fixed and tested for the next time. We have
better todo then wasting our time by backing out untest

Backout pascoe@'s last USB change because it page faults at attachment
time.

Chris; Get your shiz fixed and tested for the next time. We have
better todo then wasting our time by backing out untested stuff.

OK deraadt, OK ckuethe

show more ...

Save SOF (frame timing adjustment) before the very first host controller
reset, rather than at suspend/resume, otherwise any BIOS inserted value
is lost immediately.

ok dlg@

In a drivers activate() entry point, if on DVACT_DEACTIVATE it does
something, then it MUST return zero on DVACT_ACTIVATE, not EOPNOTSUPP;
this very popular bug has been cut and pasted a lot of times

In a drivers activate() entry point, if on DVACT_DEACTIVATE it does
something, then it MUST return zero on DVACT_ACTIVATE, not EOPNOTSUPP;
this very popular bug has been cut and pasted a lot of times...
ok deraadt@ mickey@

show more ...

Save and restore the data toggle value when a pipe to an endpoint
is closed and then reopened. This may be necessary now that
we no longer clear endpoint stalls every time a pipe is opened.
Previous

Save and restore the data toggle value when a pipe to an endpoint
is closed and then reopened. This may be necessary now that
we no longer clear endpoint stalls every time a pipe is opened.
Previously we could assume an initial toggle value of zero because
a clear-stall operation resets the device's toggle state.

Derived from work in FreeBSD.

This is most likely to affect devices like printers, which open
pipes for short periods of time and close them again.

ok dlg@

show more ...

Fix a comment, remove a stray variable assignment.

Only do the software part of an abort if we are dying.

Attach routines can fail before calling *hook_establish(), and they
often rely on the detach routine for cleanup. So be consistant and
careful by checking for a NULL hook before calling *hook_disesta

Attach routines can fail before calling *hook_establish(), and they
often rely on the detach routine for cleanup. So be consistant and
careful by checking for a NULL hook before calling *hook_disestablish
in detach routines.

ok mickey@ brad@ dlg@

show more ...

implement outgoing interrupt pipes, lets my creative voip blaster work

from freebsd, ok dlg@

Allow strings descriptor 0 to be fetched. It's the default language index.

From augustss NetBSD

ok dlg@

Move contents of sys/select.h to sys/selinfo.h in preparation for a
userland-visible sys/select.h. Consistent with what Net and Free do.
OK deraadt@, tested with full ports build by naddy@.

More typos (especially indicies -> indices), checked with jmc@

Make sure we don't enable host controller interrupts until all
initialization is finished.

From augustss NetBSD

ok dlg@ pascoe@

fix for isochronous pipes:
xfer's busy_free status can be XFER_ONQU when uhci_device_isoc_done()
is called. this happens when the xfer is queued in the driver supplied
callback function that is calle

fix for isochronous pipes:
xfer's busy_free status can be XFER_ONQU when uhci_device_isoc_done()
is called. this happens when the xfer is queued in the driver supplied
callback function that is called before uhci_device_isoc_done().

ok dlg@

show more ...

Handle all types of interrupts when operating in polled mode.

from Danovitsch@vitsch.net

from netbsd, uhci.c revision 1.175, 1.179, 1.181
log message for 1.175:
Tweak a debugging printf().

log message for 1.179:
Make one message dependent on ohcidebug, so it doesn't interfere with polle

from netbsd, uhci.c revision 1.175, 1.179, 1.181
log message for 1.175:
Tweak a debugging printf().

log message for 1.179:
Make one message dependent on ohcidebug, so it doesn't interfere with polled
operation; e.g. when entering a root device or in DDB.

log message for 1.181:
typo in comment

show more ...

from netbsd, uhci.c revision 1.180

hcpriv is not actually used here. Remove references to it.

from netbsd, uhci.c revision 1.178

Interrupt descriptors might become invalid while being processed in
uhci_check_intr - so remember their next pointer before calling it.

from netbsd, uhci.c revision 1.176

Ignore a CRCTO error on a SETUP transaction in combination with STALLED or NAK.
This fixes problems with the GL641.

from netbsd via loki@animata.net
uhub.c revision 1.65, ohci.c revision 1.146, uhci.c revision 1.177
Use the correct wValue to get hub desriptors.
Also, make wValue checks of root hub codes less stric

from netbsd via loki@animata.net
uhub.c revision 1.65, ohci.c revision 1.146, uhci.c revision 1.177
Use the correct wValue to get hub desriptors.
Also, make wValue checks of root hub codes less strict.

show more ...