remove unneeded includes; ok mpi@

Remove unused device poll functions.

Also remove unneeded includes of <sys/poll.h> and <sys/select.h>.

Some addenda from jsg@.

OK miod@ mpi@

constify method tables in USB serial drivers

ok miod@

update remaining usb.org URLs

Nuke all occurrences of usbd_abort_pipe() if it gets called right
before usbd_close_pipe(), since usbd_close_pipe() already takes care
about aborting non-empty pipes.

As investigated by gerhard@ usb

Nuke all occurrences of usbd_abort_pipe() if it gets called right
before usbd_close_pipe(), since usbd_close_pipe() already takes care
about aborting non-empty pipes.

As investigated by gerhard@ usbdi.c rev. 1.57 did add usbd_abort_pipe()
to usbd_close_pipe(), but the drivers didn't get cleaned up afterwards.

ok gerhard@

show more ...

use the UE_GET_XFERTYPE macro where applicable

ok mpi@

Remove unused struct member.

ok deraadt@, mlarkin@

Delete unnecessary <sys/file.h> includes

ok millert@ krw@

Remove quirks for two devices that are known to be CDC ACM protocol 0
that are now covered by the generic class matching.

Match on class communications subclass abstract control model protocol
0 "No class specific protocol required" in addition to the existing
protocol 1 "AT Commands: V.250 etc" match.

This lets umodem

Match on class communications subclass abstract control model protocol
0 "No class specific protocol required" in addition to the existing
protocol 1 "AT Commands: V.250 etc" match.

This lets umodem(4) attach to the serial console on the overdrive 1000
which is a usb type-b socket on the back of the box not a db9 like the
overdrive 3000.

show more ...

Kill unused ioctl(2)s.

ok deraadt@

Remove some includes include-what-you-use claims don't
have any direct symbols used. Tested for indirect use by compiling
amd64/i386/sparc64 kernels.

ok tedu@ deraadt@

allow Arduino Leonardo to attach to umodem(4), from Gregor Best

tedu another bunch of DVACT_DEACTIVATE handlers that do nothing but set
the 'dying' flag.

fixed some debug messages
ok by dcoppa@

Do not match control interfaces if their associated data interface is
missing or already claimed.

This fixes a problem with the Ericsson F3507g embedded in the x200s
that has a broken configuration

Do not match control interfaces if their associated data interface is
missing or already claimed.

This fixes a problem with the Ericsson F3507g embedded in the x200s
that has a broken configuration descriptor reporting too few
interfaces.

Problem reported and fix tested by Harald Hellmuth and florian@

show more ...

remove a duplicated includ file.

Clean-up usbd_abort_pipe() usage.


This function never fails. So change it's return type to void and
adjust the copy-pasted callers that were checking the return.

"If it compiles, ok" mpi@

Sort-out activate functionality for ucom(4) and it's parents.


The child should not have to call the parent for deactivation.
The parent should handle it.
So kill the activate routine in ucom(4) and

Sort-out activate functionality for ucom(4) and it's parents.


The child should not have to call the parent for deactivation.
The parent should handle it.
So kill the activate routine in ucom(4) and adapt the parents
to no longer call it.


Also remove sc_dying in ucom(4) and use usbd_is_dying() instead.

Okay mpi@.

show more ...

Replace sc_dying in favour of usbd_is_dying() and usbd_deactivate().

Okay mpi@

Correctly abort and free the pipe when detaching the device. Found the
hard way by otto@

Rename three functions in usbdi.c to their right prefix (usb_* to usbd_*).

ok mpi@

Get rid of various 'typedef struct' definitions and use plain structure
definitions instead. We don't change usb.h for now to stay compatible
with userland.

Tested by mpi@ on macppc and myself on i

Get rid of various 'typedef struct' definitions and use plain structure
definitions instead. We don't change usb.h for now to stay compatible
with userland.

Tested by mpi@ on macppc and myself on i386.

ok mpi@

show more ...

proc.h is way too much header for usb to handle.

you probably don't need to include vnode.h or reboot.h. you most definitely
absolutely do not need to include vnode.h because it includes uvm_extern.h
and you want the idiotic TRUE FALSE defines from

you probably don't need to include vnode.h or reboot.h. you most definitely
absolutely do not need to include vnode.h because it includes uvm_extern.h
and you want the idiotic TRUE FALSE defines from uvm.

show more ...