kern/54658: [patch] Several spelling and other minor corrections in sys/i386/include/bus_dma.h

Roderick van Domburg r.s.a.vandomburg at student.utwente.nl
Sun Jul 20 08:30:20 PDT 2003


>Number:         54658
>Category:       kern
>Synopsis:       [patch] Several spelling and other minor corrections in sys/i386/include/bus_dma.h
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Sun Jul 20 08:30:18 PDT 2003
>Closed-Date:
>Last-Modified:
>Originator:     Roderick van Domburg
>Release:        FreeBSD 5.1-CURRENT sparc64
>Organization:
University of Twente
>Environment:
System: FreeBSD stud187236.mobiel.utwente.nl 5.1-CURRENT FreeBSD 5.1-CURRENT #0: Tue Jul 1 12:10:05 CEST 2003 roderick at stud187236.mobiel.utwente.nl:/usr/obj/usr/src/sys/E250 sparc64


	
>Description:
Whilst browsing the kernel source I found several spelling mistakes.
in sys/i386/include/bus_dma.h.

Since I had time on my hands, I corrected them along a couple of
other minor documentation corrections (mostly whitespacing). I am
unsure if this should be fixed upstream?
>How-To-Repeat:
N/A
>Fix:
--- /usr/src/sys/i386/include/bus_dma.h.old     Sun Jul 20 17:05:53 2003
+++ /usr/src/sys/i386/include/bus_dma.h Sun Jul 20 17:09:33 2003
@@ -103,7 +103,7 @@
  *     A machine-dependent opaque type describing the characteristics
  *     of how to perform DMA mappings.  This structure encapsultes
  *     information concerning address and alignment restrictions, number
- *     of S/G  segments, amount of data per S/G segment, etc.
+ *     of S/G segments, amount of data per S/G segment, etc.
  */
 typedef struct bus_dma_tag     *bus_dma_tag_t;

@@ -132,7 +132,7 @@
 typedef int bus_dma_filter_t(void *, bus_addr_t);

 /*
- * A function that performs driver-specific syncronization on behalf of
+ * A function that performs driver-specific synchronization on behalf of
  * busdma.
  */
 typedef enum {
@@ -146,7 +146,7 @@
  * Allocate a device specific dma_tag encapsulating the constraints of
  * the parent tag in addition to other restrictions specified:
  *
- *     alignment:      alignment for segments.
+ *     alignment:      Alignment for segments.
  *     boundary:       Boundary that segments cannot cross.
  *     lowaddr:        Low restricted address that cannot appear in a mapping.
  *     highaddr:       High restricted address that cannot appear in a mapping.
@@ -179,21 +179,21 @@
 int bus_dmamap_create(bus_dma_tag_t dmat, int flags, bus_dmamap_t *mapp);

 /*
- * Destroy  a handle for mapping from kva/uva/physical
+ * Destroy a handle for mapping from kva/uva/physical
  * address space into bus device space.
  */
 int bus_dmamap_destroy(bus_dma_tag_t dmat, bus_dmamap_t map);

 /*
  * Allocate a piece of memory that can be efficiently mapped into
- * bus device space based on the constraints lited in the dma tag.
+ * bus device space based on the constraints listed in the dma tag.
  * A dmamap to for use with dmamap_load is also allocated.
  */
 int bus_dmamem_alloc(bus_dma_tag_t dmat, void** vaddr, int flags,
                     bus_dmamap_t *mapp);

 /*
- * Free a piece of memory and it's allociated dmamap, that was allocated
+ * Free a piece of memory and its allocated dmamap, that was allocated
  * via bus_dmamem_alloc.
  */
 void bus_dmamem_free(bus_dma_tag_t dmat, void *vaddr, bus_dmamap_t map);
@@ -213,7 +213,7 @@

 /*
  * Like bus_dmamap_callback but includes map size in bytes.  This is
- * defined as a separate interface to maintain compatiiblity for users
+ * defined as a separate interface to maintain compatibility for users
  * of bus_dmamap_callback_t--at some point these interfaces should be merged.
  */
 typedef void bus_dmamap_callback2_t(void *, bus_dma_segment_t *, int, bus_size_t, int);
@@ -235,7 +235,7 @@
                        int flags);

 /*
- * Perform a syncronization operation on the given map.
+ * Perform a synchronization operation on the given map.
  */
 void _bus_dmamap_sync(bus_dma_tag_t, bus_dmamap_t, bus_dmasync_op_t);
 #define bus_dmamap_sync(dmat, dmamap, op)              \
>Release-Note:
>Audit-Trail:
>Unformatted:


More information about the freebsd-bugs mailing list