svn commit: r285293 - head/sys/sys

Zbigniew Bodek zbb at FreeBSD.org
Wed Jul 8 22:09:48 UTC 2015


Author: zbb
Date: Wed Jul  8 22:09:47 2015
New Revision: 285293
URL: https://svnweb.freebsd.org/changeset/base/285293

Log:
  Style cleanups after r285270
  
  There should be no semicolons in added macro definitions.
  Define empty macro as "do {} while (0)".
  
  Pointed out by: jmg

Modified:
  head/sys/sys/bus_dma.h

Modified: head/sys/sys/bus_dma.h
==============================================================================
--- head/sys/sys/bus_dma.h	Wed Jul  8 21:32:57 2015	(r285292)
+++ head/sys/sys/bus_dma.h	Wed Jul  8 22:09:47 2015	(r285293)
@@ -289,18 +289,18 @@ void bus_dmamem_free(bus_dma_tag_t dmat,
  */
 void _bus_dmamap_sync(bus_dma_tag_t, bus_dmamap_t, bus_dmasync_op_t);
 #if defined(__arm__)
-	#define __BUS_DMAMAP_SYNC_DEFAULT		mb();
+	#define __BUS_DMAMAP_SYNC_DEFAULT		mb()
 #elif defined(__aarch64__)
-	#define	__BUS_DMAMAP_SYNC_DEFAULT		dmb(sy);
+	#define	__BUS_DMAMAP_SYNC_DEFAULT		dmb(sy)
 #else
-	#define	__BUS_DMAMAP_SYNC_DEFAULT		{}
+	#define	__BUS_DMAMAP_SYNC_DEFAULT		do {} while (0)
 #endif
 #define bus_dmamap_sync(dmat, dmamap, op) 			\
 	do {							\
 		if ((dmamap) != NULL)				\
 			_bus_dmamap_sync(dmat, dmamap, op);	\
 		else						\
-			__BUS_DMAMAP_SYNC_DEFAULT		\
+			__BUS_DMAMAP_SYNC_DEFAULT;		\
 	} while (0)
 
 /*


More information about the svn-src-all mailing list