Age | Commit message (Collapse) | Author |
|
|
|
Note that pseudo-drivers not using if_input() are not affected by this
conversion.
ok mikeb@, kettenis@, claudio@, dlg@
|
|
have any direct symbols used. Tested for indirect use by compiling
amd64/i386/sparc64 kernels.
ok tedu@ deraadt@
|
|
Tested and ok armani@
|
|
claims to support both RNDIS and CDC Ethernet. However, RNDIS doesn't seem
to work, at least not with our driver. So blacklist it here such that cdce(4)
takes over.
ok miod@, armani@ (both a while back)
|
|
|
|
command in urndis_ctrl_set(). Seems many RNDIS hardware doesn't care about
the overwritten fields, but the SunMicro Virtual Eth Device found in newer
Oracle SPARC hardware does.
ok armani@
|
|
|
|
after discussions with beck deraadt kettenis.
|
|
set the 'dying' flag of a device. Such handlers are useless now that
usbd_detach() already set this flag.
Even if the purpose of this flag is questionnable on OpenBSD because
DVACT_DEACTIVATE acts as a pre-detach event, this is just a first small
step towards the simplification of the autoconf(9) device states.
This cleaning is now possible thanks to the work of pirofti@ to convert
all the USB drivers to properly use usbd_is_dying().
Discussed many times with deraadt@
|
|
When detaching my nokia c2-01 dohooks in if_detach would panic.
This was due to a failure to setup a pipe in urndis_ctrl_init()
which results in a TIMEOUT on receive with side-effects later on
during detach due to assumptions regarding the existence of interface
hooks during dohooks on if_detach tear-down.
That resulted on dereferencing a NULL function pointer which triggered
a panic.
Discussed with mpi@, tested and okay giovanni@.
|
|
with the other drivers. No functional change.
|
|
driver is not yet attached, it is here to prevent the driver to free
unitialized structures in case something wrong happened during the
the initialization...
Found the hard way and ok giovanni@
|
|
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@
|
|
Okay mpi@
|
|
by the same explore task and cannot happen in parallel.
|
|
while there, fix a few %d into %u
|
|
don't have to. Just remove these include lines.
Compiled on amd64 i386 sparc64; OK henning@ mikeb@
|
|
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@
|
|
|
|
- print what type of device is.
tested by sthen@. similar diff tested by jasper@ and gonzalo@.
ok mpi@ and sthen@
if you have one of these devices please give it a try and mail the details.
|
|
ok guenther millert kettenis
|
|
- in urndis_attach() use the first interface descriptor that's different
from the control interface as the data interface.
from armani's colleague with some tweaks by me.
kettenis@ sthen@ ok
|
|
committed over it.
|
|
ok dlg@
|
|
that's ever used it, and it's long since been changed to use
DVACT_{QUIESCE,SUSPEND,RESUME} instead.
ok deraadt@, dlg@; miod@ also agreed with this idea when I brought it
up a few weeks ago
|
|
It may be possible to use better heuristics to detect compatible devices
(Linux driver appears to hand over from CDC to urndis) which would avoid
listing IDs, but this gets tethering working on this phone for now.
|
|
them from userland, and nothing in the kernel uses them.
ok krw@, miod@
|
|
interfaces as being claimed in the usbd_device's copy of the interface
descriptors
* allow ugen(4) to be attached if there are unused interfaces in a
configuration that has had drivers attached
* make ugen(4) aware that it may be sharing a device with (an)other
driver(s), and if so:
* do not let ugen(4) change the configuration
* do not let ugen(4) access the already claimed interfaces
discussed with deraadt and miod
|
|
|
|
The dying flag will be set in activate()/DEACTIVATE.
ok deraadt@
|
|
function is not.
This change #if 0's urndis_watchdog() and some functions that are only
called from here.
At some point we probably want to use the watchdog functionality but the
current code is completely untested so disable it entirely rather than
enabling it this close to release.
|
|
|
|
no point in copying before having done the final sanity check (we copied
before the last one).
ok armani
|
|
get rid of it.
ok armani
|
|
|
|
"iff you commit over the Hero" mk@
|
|
ok armani
|
|
This fix a crash on detach.
|
|
product ID. The former code was a bit promiscuous and would attempt to
attach urndis(4) to e.g. E71 (as noticed by jmc) and then fail.
ok fabien armani deraadt
|
|
Found when looking at ckuethe problem.
with and ok fabien@, "definitivly" mk@, "that diff can go in" kettenis
|
|
multipacket tx.
ok fabien "if you commit over urndis" armani
|
|
ok armani, mk.
|
|
Now option to debug is URNDIS_DEBUG.
ok mk, armani.
|
|
usb ethernet drivers.
ok armani
|
|
that shows how many packets the device will accept in one transfer.
ok armani fabien
|
|
other drivers do otherwise we panic trying to remove an unexistent
ifp during detach.
looks good to fabien@, with comments from mk@, ok mk
|
|
|
|
ok mk@
|
|
|