summaryrefslogtreecommitdiffstats
path: root/sys/dev/usb/uhidev.c (follow)
Commit message (Collapse)AuthorAgeFilesLines
* use the UE_GET_XFERTYPE macro where applicablejasper2020-02-221-4/+4
| | | | ok mpi@
* Prevent use-after-free in uhidev_close().mpi2020-01-041-14/+14
| | | | | | | Close pipes before freeing transfers, otherwise accessing elements in pipe->queue, like in usbd_abort_pipe(), will result in a crash. Problem reported by reyk@, ok visa@
* Send uhid(4) reports via the optional interrupt out pipe. Somepatrick2019-11-131-16/+45
| | | | | | | | tiny devices don't accept reports using control transfers since they think that offering the interrupt pipe is enough. Discussed with djm@ ok mpi@
* Move HID->bus constant conversion for HID report types out of ihidevjcs2018-08-251-1/+16
| | | | | | | | | | | | | | | | | | | | | into hidmt. The HID code uses hid_feature, hid_input, and hid_output constants to refer to report types internally that then need to be converted to their bus-level counterparts before actually getting sent out (so hid_feature becomes UHID_FEATURE_REPORT for USB, I2C_HID_REPORT_TYPE_FEATURE for i2c). This conversion was hard-coded in ihidev but ihidev_[gs]et_report should assume the type passed is already an i2c-level define, not a hid one. This is how uhidev does it. Add a conversion routine callback that any hidmt callers need to set so that hidmt can convert hid constants to the bus-level versions. Also add a similar conversion function to uhidev. ok deraadt
* A pile of sizes to free(9). In test for a few days in snapshots.deraadt2017-04-081-5/+5
| | | | | Errors will result in nice clean panic messages so we know what's wrong. Reviewed by dhill visa natano jsg.
* Do not execute the callback if the device is beeing detached.mpi2016-06-131-12/+13
| | | | | Should prevent a race triggering a use-after-free reported by martijn@ on bugs@
* revert manual additions of hid.h now that usbhid.h brings it injcs2016-01-091-3/+1
|
* Move HID support files out of dev/usb into new dev/hid directoryjcs2016-01-081-2/+3
| | | | | | | | | These files aren't USB-specific and were used by the previous Bluetooth implementation, and will be used by the upcoming HID-over-i2C implementation ok deraadt previous version ok kettenis and mpi
* Match 3rd party Xbox 360 controllers, from Christian Heckendorf.mpi2015-12-081-11/+15
| | | | Tested by jeremy@
* Fix uhidev_get_report_async() from David Higgs.mpi2015-02-281-4/+7
|
* Reduce the number of intermediate buffers by pre-allocating DMA buffersmpi2015-01-221-32/+41
| | | | | | | in the report functions instead of letting the stack do it magically for us. Reviewed and tested by David Higgs, thanks!
* Correctly report memory allocation failure in uhidev_set_report_async().mpi2015-01-091-2/+2
|
* Add an asynchronous version of uhidev_get_report(), from David Higgsmpi2015-01-091-2/+78
| | | | | | | with some tweaks. Nothing use it for the moment, but upcoming upd(4) improvements need it.
* yet more mallocarray() changes.doug2014-12-131-2/+2
| | | | ok tedu@ deraadt@
* Change uhidev(4) set/get report functions in various way.mpi2014-12-111-34/+43
| | | | | | | | | | | | | | Always pass the parent uhidev(4) descriptor corresponding to the single USB device with multiple reportIDs instead of a child. Make uhidev_get_report() aware of non NUL reportID by prepending a byte to the given buffer, just like uhidev_set_report{,async}() already do. Return the number of bytes written or read upon success and -1 otherwise. This allow callers to deal with short reads without having do mess with xfer error codes madness. Reviewed and tested by David Higgs.
* Deprecate usb_*_report(). USB HID devices are always attached belowmpi2014-12-081-35/+58
| | | | | | | | | | an uhidev(4) on OpenBSD and there is not point in rerolling your own reportID handling. Simply use uhidev_*_report(). This is a first step towards better error handling required to deal with broken upd(4) firmwares. Tested by David Higgs.
* sizeof() a pointer of the right struct to appease clang.mpi2014-08-101-2/+2
|
* add a size argument to free. will be used soon, but for now default to 0.tedu2014-07-121-7/+7
| | | | after discussions with beck deraadt kettenis.
* Print an understandable error message when we fail to read the reportmpi2014-07-101-3/+3
| | | | | | descriptor. Pointed out by benno@
* Plug an xfer leak.mpi2014-04-291-3/+16
|
* Change the usbd_*_report() family of functions to take a usbd_devicempi2014-04-241-16/+32
| | | | | | | | | | | and a infaceno argument instead of an iface pointer. While here, remove some unused functions and inlined usbd_read_report_desc since it is used only once. This is part of plumbing required to convert the various USB HID devices to handle multiples report IDs. ok andre@
* Modify the uhidev_*_report() methods to always take a report ID argumentmpi2014-04-151-17/+18
| | | | | | | | instead of using the default, per-driver, one. This is a step to move away from the 1 report ID <-> 1 driver design which is no longer true since the addition of upd(4). ok andre@
* Fix uhidev_detach() when detaching a device which did not claimed for allandre2014-03-231-2/+3
| | | | | | | report IDs, on those cases some positions on sc_subdevs[] can be null. Initial analysis by guenther@. ok mpi@
* Add a temporary hack to let a subdriver claim all the reportIDs ofmpi2014-03-191-11/+25
| | | | | | | a device. This should be removed once all the drivers attaching to uhidev(4) are converted. ok andre@, sthen@
* Move the logic to deal with devices with broken descriptors into itsmpi2014-03-161-69/+69
| | | | | | | own function and don't leave the driver half-configured if a children doesn't set the interrupt function. tested by jeremy@, ok andre@
* Starting the spring cleanup.mpi2014-03-151-25/+23
|
* Remove debug leftover.mpi2014-03-151-7/+2
|
* Abort and close all the interrupt pipes when detaching HID devices sincempi2014-02-251-2/+11
| | | | | | | | drivers attaching to uhidev(4) do not always do it. Seems to fix one of the transfer aborting races triggering a lot of "ehci_idone: ..." vomit upon resume. Reported by and fix tested by Ville Valkonen, ok deraadt@
* Kill matchlvl usage throughout the usb stack.pirofti2013-11-191-7/+1
| | | | | | Allows us to move forward on cleaning. Okay mpi@
* Clean-up the activate routines from the uoak*(4) drivers.pirofti2013-11-111-1/+5
| | | | | | | | Leave the deactivate bits to the parent. Simplify by making a single dying check in uhidev(4) interrupt routine instead of having each child doing it. Okay mpi@
* Spacing.mpi2013-11-011-20/+17
|
* Do not abuse the stack of the current process to prepend a report IDmpi2013-11-011-19/+25
| | | | | | | to the report request, use malloc(9) with the appropriate size like it is done in the synchronous version. ok miod@
* Add support for Microsoft XBox 360 controller as a uhid. It doesn't usejeremy2013-10-251-4/+21
| | | | | | | | | | | | | | | | | | | | | the standard interface class and doesn't have a report descriptor, so use a manually created one. The controller shows up as multiple devices with different interface numbers, so only match the first one as a uhid. The controller shows up as both a uhid and a ugen. Rename ugraphire_rdesc.h to uhid_rdesc.h and add the manually created Xbox 360 report descriptor to the file. Add ifndef SMALL_KERNEL around the related blocks for the XBox 360 controller and Wacom Graphire, so this should actually shrink the ramdisks. Manual report descriptor taken from a 2005 FreeBSD commit. Much help from mpi@ for getting this suitable for commit to OpenBSD. OK mpi@
* Unify and document usbd_transfer(9).mpi2013-09-201-3/+9
| | | | | | | | | | | Get rid of the badly named usbd_bulk_transfer() and usbd_intr_transfer() they were two identical wrappers to submit a synchronous transfer. Now all synchronous transfers are submited using usbd_setup_xfer(9) and usbd_transfer(9) explicitly using the USBD_CATCH flag when signals must be checked. Previous version ok'd by mglocker@, with subtle inputs from miod@
* Do no try to clear the endpoint, as it if has stalled, if thempi2013-09-191-2/+2
| | | | | | | | | | | | | | USB transfer reported an I/O error. This is more likely to have no effect apart generating more errors because when this happens, the device is generally already physically detached but the *detach() task has not run yet. So simply return and wait for the device to be removed like ucom(4) does. Partially fix an issue reported by RD Thrush rt at thrush.com. ok miod@
* When deactivating child devices do not panic if their driver doesmpi2013-05-071-7/+2
| | | | | | | | | | not implement an *activate() function. This could receive more love but is good enough for the moment to start cleaning useless *activate() functions. While here remove unused sc_dying argument. Tested by and ok mglocker@
* Get rid of various 'typedef struct' definitions and use plain structuremglocker2013-04-151-4/+4
| | | | | | | | | 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@
* Remove config_activate() and DVACT_ACTIVATE. PCMCIA's the only thingmatthew2011-07-031-3/+1
| | | | | | | | 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
* garbage collect "usb events". without /dev/usb there is no way to accessjakemsr2011-01-251-7/+1
| | | | | | them from userland, and nothing in the kernel uses them. ok krw@, miod@
* remove dying flag in detach() function.yuo2010-09-241-2/+1
| | | | | | The dying flag will be set in activate()/DEACTIVATE. ok deraadt@
* In xxactivate() DVACT_DEACTIVATE, when calling a series of config_deactivate()deraadt2010-08-311-6/+8
| | | | | | for sub-devices, return the last error return value instead of |'ing the failures together and creating some value that is non-0 (bravo!) but potentially loses the specific error value...
* Handle USB_GET_REPORT, USB_GET_REPORT_DESC, USB_GET_REPORT_ID andmiod2010-08-021-1/+70
| | | | | | | USB_SET_REPORT ioctls in ukbd and ums. This allows usbhidctl to be used on these devices e.g. to dump the report descriptor of troublesome models. ok deraadt@
* suppress warning massage for polling only HID devices.yuo2009-11-231-3/+3
|
* Get rid of devact enum, substitute it with an int and coresponding defines.pirofti2009-10-131-3/+3
| | | | | | This is needed for the addition of further suspend/resume actions. Okay deraadt@, marco@.
* Use the correct buffer for non-zero report IDs in uhidev_set_report().jsg2009-07-231-2/+2
| | | | | | From Dave Huang via NetBSD rev 1.18. ok miod@ deraadt@
* Avoid a buf oflow in uhidev_set_report_async() if we have to prepend themiod2009-06-021-1/+8
| | | | | report id and the data to report is too long (this should probably use dynamic allocation for large reports).
* First pass at removing clauses 3 and 4 from NetBSD licenses.ray2008-06-261-8/+1
| | | | | | | | | Not sure what's more surprising: how long it took for NetBSD to catch up to the rest of the BSDs (including UCB), or the amount of code that NetBSD has claimed for itself without attributing to the actual authors. OK deraadt@
* remove #ifdef __macppc__ due to a misunderstanding:todd2008-06-141-13/+1
| | | | | | | | | - the wskbd/wsmouse is not phantom, but actually is pre-paired keyboard/mice talking wskbd/wsmouse to the os this permits the bluetooth device to attach as wskbd/wsmouse using bluetooth keyboards for ddb> and in general if you have paired them in an os that supports flipping the funny bluetooth device into full bluetooth mode ok drahn@
* fewer lines of dmesg output spew (matters more on newer sony laptops with... 15 lines of usb devices..deraadt2007-11-121-5/+6
|
* treat usb vendor/product names as a locator, and have usbd_print handle it,deraadt2007-10-111-6/+3
| | | | | | so that it shows up before the :. as a result, all the usb devices do not need to have name printing code anymore. all this now works and prints nicely because usbd_probe_and_attach() is serialized. ok kettenis
Copyright © 1996 – 2023 Jason A. Donenfeld. All Rights Reverse Engineered.