aboutsummaryrefslogtreecommitdiff
path: root/sys/dev/usb/usb_busdma.c
Commit message (Collapse)AuthorAgeFilesLines
* sys: Remove $FreeBSD$: one-line .c comment patternWarner Losh2023-08-161-1/+0
| | | | Remove /^/[*/]\s*\$FreeBSD\$.*\n/
* spdx: The BSD-2-Clause-FreeBSD identifier is obsolete, drop -FreeBSDWarner Losh2023-05-121-1/+1
| | | | | | | | | The SPDX folks have obsoleted the BSD-2-Clause-FreeBSD identifier. Catch up to that fact and revert to their recommended match of BSD-2-Clause. Discussed with: pfg MFC After: 3 days Sponsored by: Netflix
* net80211(4): Bring back an accidently reverted grammar correctionGordon Bergling2022-09-071-1/+1
| | | | | | | - s/that that/that/ Reported by: se, danfe MFC after: 3 days
* Revert "usb: Remove a double words in a source code comments"Gordon Bergling2022-09-051-1/+1
| | | | | | | | This reverts commit 8ca67bf1273a5168f8a3787183159c477632e582. The original comment was correct; changing it loses a key part. Reported by: jrtc27
* usb: Remove a double words in a source code commentsGordon Bergling2022-09-041-1/+1
| | | | | | - s/that that/that/ MFC after: 3 days
* Fix busdma resource leak on usb device detach.Ian Lepore2021-09-281-3/+13
| | | | | | | | | | | | | | | | When a usb device is detached, usb_pc_dmamap_destroy() called bus_dmamap_destroy() while the map was still loaded. That's harmless on x86 architectures, but on all other platforms it causes bus_dmamap_destroy() to return EBUSY and leak away any memory resources (including bounce buffers) associated with the mapping, as well as any allocated map structure itself. This change introduces a new is_loaded flag to the usb_page_cache struct to track whether a map is loaded or not. If the map is loaded, bus_dmamap_unload() is called before bus_dmamap_destroy() to avoid leaking away resources. MFC after: 7 days Differential Revision: https://reviews.freebsd.org/D32208
* Fix a common typo in source code commentsGordon Bergling2021-08-141-1/+1
| | | | | | - s/aligment/alignment/ MFC after: 5 days
* usb: clean up empty lines in .c and .h filesMateusz Guzik2020-09-011-18/+0
| | | | Notes: svn path=/head/; revision=365084
* sys/dev: further adoption of SPDX licensing ID tags.Pedro F. Giffuni2017-11-271-0/+2
| | | | | | | | | | | | | | | Mainly focus on files that use BSD 2-Clause license, however the tool I was using misidentified many licenses so this was mostly a manual - error prone - task. The Software Package Data Exchange (SPDX) group provides a specification to make it easier for automated tools to detect and summarize well known opensource licenses. We are gradually adopting the specification, noting that the tags are considered only advisory and do not, in any way, superceed or replace the license texts. Notes: svn path=/head/; revision=326255
* Remove register keyword from sys/ and ANSIfy prototypesEd Maste2017-05-171-1/+1
| | | | | | | | | | | | | | | A long long time ago the register keyword told the compiler to store the corresponding variable in a CPU register, but it is not relevant for any compiler used in the FreeBSD world today. ANSIfy related prototypes while here. Reviewed by: cem, jhb Sponsored by: The FreeBSD Foundation Differential Revision: https://reviews.freebsd.org/D10193 Notes: svn path=/head/; revision=318389
* Improve USB polling mode by not locking any mutexes, asserting anyHans Petter Selasky2016-09-141-7/+7
| | | | | | | | | | | | | | | | | | | | | mutexes or using any callouts when active. Trying to lock a mutex when KDB is active or the scheduler is stopped can result in infinite wait loops. The same goes for calling callout related functions which in turn lock mutexes. If the USB controller at which a USB keyboard is connected is idle when KDB is entered, polling the USB keyboard via USB will always succeed. Else polling may fail depending on which state the USB subsystem and USB interrupt handler is in. This is unavoidable unless KDB can wait for USB interrupt threads to complete before stalling the CPU(s). Tested by: Bruce Evans <bde@freebsd.org> MFC after: 4 weeks Notes: svn path=/head/; revision=305806
* sys: use our roundup2/rounddown2() macros when param.h is available.Pedro F. Giffuni2016-04-211-2/+2
| | | | | | | | | | | | | rounddown2 tends to produce longer lines than the original code and when the code has a high indentation level it was not really advantageous to do the replacement. This tries to strike a balance between readability using the macros and flexibility of having the expressions, so not everything is converted. Notes: svn path=/head/; revision=298433
* Verify that all segments in the loaded segment list are back to backHans Petter Selasky2016-03-291-11/+16
| | | | | | | with regard to the page offset, and not only the two first ones. Notes: svn path=/head/; revision=297381
* Add helper function to check if a USB page cache buffer is properlyHans Petter Selasky2015-11-071-0/+29
| | | | | | | | | aligned to reduce the use of bounce buffers in PIO mode. MFC after: 1 week Notes: svn path=/head/; revision=290489
* Relax the BUS_DMA_KEEP_PG_OFFSET requirement to allow optimisingHans Petter Selasky2015-11-031-2/+6
| | | | | | | | | | allocation of DMA bounce buffers. Discussed with: ian @ MFC after: 3 weeks Notes: svn path=/head/; revision=290326
* Fix a regression issue. Contiguous single segment allocations aboveHans Petter Selasky2014-02-051-1/+1
| | | | | | | | | | PAGE_SIZE bytes should only use one USB page structure. Fixes a problem with some external drivers. MFC after: 2 days Notes: svn path=/head/; revision=261505
* Fix external compiler warning about write-only assigned variable.Hans Petter Selasky2013-12-051-3/+1
| | | | Notes: svn path=/head/; revision=258961
* Modify the FreeBSD USB kernel code so that it can be compiled directlyHans Petter Selasky2013-01-301-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | into the FreeBSD boot loader, typically for non-USB aware BIOSes, EFI systems or embedded platforms. This is also useful for out of the system compilation of the FreeBSD USB stack for various purposes. The USB kernel files can now optionally include a global header file which should include all needed definitions required to compile the FreeBSD USB stack. When the global USB header file is included, no other USB header files will be included by default. Add new file containing the USB stack configuration for the FreeBSD loader build. Replace some __FBSDID()'s by /* $FreeBSD$ */ comments. Now all USB files follow the same style. Use cases: - console in loader via USB - loading kernel via USB Discussed with: Hiroki Sato, hrs @ EuroBSDCon Notes: svn path=/head/; revision=246122
* Fix regression issue after r244500 and r244503:Hans Petter Selasky2013-01-271-2/+3
| | | | | | | | | | | | | | If a BUSDMA load operation results in a single segment which is greater than the PAGE_SIZE, the USB computed physical addresses will not be correct. Make sure that the first segment is unfolded like the sub-sequent segments are into USB_PAGE_SIZE big ranges. Found by: Alexander Nedotsukov MFC after: 1 week Notes: svn path=/head/; revision=245995
* Regression issue:Hans Petter Selasky2012-12-211-5/+13
| | | | | | | | | | | | | | | | | | | | | | | Use a boundary of zero, hence a PAGE_SIZE boundary is implied by all memory allocations. Background: Busdma has problems to allocate more than PAGE_SIZE bytes when the boundary is PAGE_SIZE bytes too. Initially it was thought that a boundary of PAGE_SIZE bytes will only affect loading of DMA memory, so that segments get split correctly, but it also affects allocation of DMA'able memory. Solution: USB can detect big segments and split them as required by the USB code. MFC after: 1 week Reported by: gonzo Notes: svn path=/head/; revision=244535
* Fix compiler warnings, mostly signed issues,Hans Petter Selasky2012-04-021-2/+2
| | | | | | | | | when USB modules are compiled with WARNS=9. MFC after: 1 weeks Notes: svn path=/head/; revision=233774
* Style change.Hans Petter Selasky2011-11-121-7/+7
| | | | | | | | | | | | | | | - Make it easier to port the USB code to other platforms by only using one set of memory functions for clearing and copying memory. None of the memory copies are overlapping. This means using bcopy() is not required. - Fix a compile warning when USB_HAVE_BUSDMA=0 - Add missing semicolon in avr32dci. - Update some comments. MFC after: 1 week Notes: svn path=/head/; revision=227461
* Remove unneeded includes of <sys/linker_set.h>. Other headers that useJohn Baldwin2011-01-111-1/+0
| | | | | | | | | it internally contain nested includes. Reviewed by: bde Notes: svn path=/head/; revision=217265
* Use saner nsegments and maxsegsz parameters when creating certain DMA tags;Marius Strobl2010-09-141-2/+2
| | | | | | | | tags for 1-byte allocations cannot possibly be split across 2 segments and maxsegsz must not exceed maxsize. Notes: svn path=/head/; revision=212621
* Remove overuse of exclamation marks in kernel printfs, there mere fact aAndrew Thompson2009-11-261-3/+3
| | | | | | | | | message has been printed is enough to get someones attention. Also remove the line number for DPRINTF/DPRINTFN, it already prints the funtion name and a unique message. Notes: svn path=/head/; revision=199816
* Add a reminder comment to optimize bus_dmamap_sync calls.Alfred Perlstein2009-08-241-0/+6
| | | | | | | Submitted by: hps Notes: svn path=/head/; revision=196496
* Fix USB cache sync operations for platforms with non-coherent DMA.Rafal Jaworowski2009-08-191-4/+3
| | | | | | | | | | | | | | | | | | | | - usb_pc_cpu_invalidate() is called between [consecutive] reads from a device, so a sequence of BUS_DMASYNC_POSTREAD and _PREREAD should be used. Note we cannot use or'ed shorthand ( _POSTREAD | _PREREAD) for BUS_DMASYNC flags, as the low level bus dma sync operation is implementation dependent and we cannot assume the required order of operations to be guaranteed. - usb_pc_cpu_flush() is called before writing to a device, so BUS_DMASYNC_PREWRITE should be used. Submitted by: Grzegorz Bernacki Reviewed by: HPS, arm@, usb@ ML Tested by: HPS, Mike Tancsa Approved by: re (kib) Obtained from: Semihalf Notes: svn path=/head/; revision=196380
* MFp4:Marcel Moolenaar2009-07-121-1/+2
| | | | | | | | | | | | | | | | | USB CORE: busdma improvement For single segment allocations the boundary field of the BUSDMA tag should be zero. Currently all single segment allocations are less than or equal to 4096 bytes, so the limit does not kick in. If any single segment USB allocations would be greater than 4K, then it would be a problem. Approved by: re (kensmith) Obtained from: HPS Notes: svn path=/head/; revision=195639
* - Make struct usb_xfer opaque so that drivers can not access the internalsAndrew Thompson2009-06-231-3/+23
| | | | | | | - Reduce the number of headers needed for a usb driver, the common case is just usb.h and usbdi.h Notes: svn path=/head/; revision=194677
* s/usb2_/usb_|usbd_/ on all function names for the USB stack.Andrew Thompson2009-06-151-97/+97
| | | | Notes: svn path=/head/; revision=194228
* Remove usb2_cv_* and just use the kernel condvar implementation, it was neededAndrew Thompson2009-06-151-5/+5
| | | | | | | | earlier since condition variables didnt work with Giant but this was fixed 10 months ago. Notes: svn path=/head/; revision=194227
* Revert the size_t part of the last commit for the moment, this blows up theAndrew Thompson2009-05-301-8/+8
| | | | | | | USB_ADD_BYTES macro. Notes: svn path=/head/; revision=193074
* s/usb2_/usb_/ on all typedefs for the USB stack.Andrew Thompson2009-05-291-27/+27
| | | | Notes: svn path=/head/; revision=193045
* s/usb2_/usb_/ on all C structs for the USB stack.Andrew Thompson2009-05-281-53/+53
| | | | Notes: svn path=/head/; revision=192984
* Unifdef __NetBSD__ here, the usb stack as a whole doesnt support NetBSD and itAndrew Thompson2009-05-261-396/+1
| | | | | | | obsfucates the code. Notes: svn path=/head/; revision=192857
* - rename usb2_mode to usb_mode [1]Andrew Thompson2009-05-211-1/+1
| | | | | | | | | - change variable types to use the enum Submitted by: Hans Petter Selasky [1] Notes: svn path=/head/; revision=192499
* MFp4 //depot/projects/usb @159431,159437,159438Andrew Thompson2009-03-201-35/+36
| | | | | | | | | | - start using the new USB typedefs in the USB core - Remove usage of USB_ADD_BYTES() Submitted by: Hans Petter Selasky Notes: svn path=/head/; revision=190181
* MFp4 //depot/projects/usb@159392Andrew Thompson2009-03-201-20/+32
| | | | | | | | | Add ifdefs for making parts of usb conditional. Submitted by: Hans Petter Selasky Notes: svn path=/head/; revision=190180
* MFp4 //depot/projects/usb @159430Andrew Thompson2009-03-201-1/+0
| | | | | | | | | | | | | - Move tunable defines into usb_core.h and dependancy towards usb_defs.h - Leave hardcoded defines in "usb_defs.h". - Allow overriding all tunable defines. - Add more customisable typedefs. - Correct maximum device number. Submitted by: Hans Petter Selasky Notes: svn path=/head/; revision=190174
* Move the new USB stack into its new home.Andrew Thompson2009-02-231-0/+1426
Notes: svn path=/head/; revision=188942
generated by cgit v1.2.3 (git 2.25.1) at 2025-08-03 01:16:43 +0000