[src/trunk]: src/sys/arch/x86/x86 Both bdt_ov->ov_dmamap_sync() and bus_dmama...

[bouyer <bouyer%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/828af3a2ce22
branches:  trunk
changeset: 770370:828af3a2ce22
user:      bouyer <bouyer%NetBSD.org@localhost>
date:      Fri Oct 14 18:28:04 2011 +0000

description:
Both bdt_ov->ov_dmamap_sync() and bus_dmamap_sync() return void,
so don't write return bdt_ov->ov_dmamap_sync(). Pointed out by njoly@

diffstat:

 sys/arch/x86/x86/bus_dma.c |  7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diffs (29 lines):

diff -r f829b8f9f189 -r 828af3a2ce22 sys/arch/x86/x86/bus_dma.c
--- a/sys/arch/x86/x86/bus_dma.c        Fri Oct 14 16:15:54 2011 +0000
+++ b/sys/arch/x86/x86/bus_dma.c        Fri Oct 14 18:28:04 2011 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: bus_dma.c,v 1.67 2011/09/28 01:45:49 dyoung Exp $      */
+/*     $NetBSD: bus_dma.c,v 1.68 2011/10/14 18:28:04 bouyer Exp $      */
 
 /*-
  * Copyright (c) 1996, 1997, 1998, 2007 The NetBSD Foundation, Inc.
@@ -31,7 +31,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: bus_dma.c,v 1.67 2011/09/28 01:45:49 dyoung Exp $");
+__KERNEL_RCSID(0, "$NetBSD: bus_dma.c,v 1.68 2011/10/14 18:28:04 bouyer Exp $");
 
 /*
  * The following is included because _bus_dma_uiomove is derived from
@@ -1317,8 +1317,9 @@
        else for (it = t; it != NULL; it = it->bdt_super) {
                if ((it->bdt_present & BUS_DMAMAP_OVERRIDE_SYNC) == 0)
                        continue;
-               return (*it->bdt_ov->ov_dmamap_sync)(it->bdt_ctx, t, p, o,
+               (*it->bdt_ov->ov_dmamap_sync)(it->bdt_ctx, t, p, o,
                    l, ops);
+               return;
        }
 
        if (ops & BUS_DMASYNC_POSTREAD)