Module Name:    src
Committed By:   tsutsui
Date:           Mon Apr 12 12:28:59 UTC 2010

Modified Files:
        src/sys/arch/atari/dev: dma.c

Log Message:
KNF and cosmetics.


To generate a diff of this commit:
cvs rdiff -u -r1.25 -r1.26 src/sys/arch/atari/dev/dma.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/arch/atari/dev/dma.c
diff -u src/sys/arch/atari/dev/dma.c:1.25 src/sys/arch/atari/dev/dma.c:1.26
--- src/sys/arch/atari/dev/dma.c:1.25	Sat Apr 10 18:02:05 2010
+++ src/sys/arch/atari/dev/dma.c	Mon Apr 12 12:28:59 2010
@@ -1,4 +1,4 @@
-/*	$NetBSD: dma.c,v 1.25 2010/04/10 18:02:05 tsutsui Exp $	*/
+/*	$NetBSD: dma.c,v 1.26 2010/04/12 12:28:59 tsutsui Exp $	*/
 
 /*
  * Copyright (c) 1995 Leo Weppelman.
@@ -47,7 +47,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: dma.c,v 1.25 2010/04/10 18:02:05 tsutsui Exp $");
+__KERNEL_RCSID(0, "$NetBSD: dma.c,v 1.26 2010/04/12 12:28:59 tsutsui Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -89,7 +89,7 @@
 static void
 st_dma_init(void)
 {
-	int	i;
+	int i;
 
 	TAILQ_INIT(&dma_free);
 	TAILQ_INIT(&dma_active);
@@ -102,23 +102,24 @@
 }
 
 int
-st_dmagrab(dma_farg int_func, dma_farg call_func, void *softc, int *lock_stat, int rcaller)
+st_dmagrab(dma_farg int_func, dma_farg call_func, void *softc, int *lock_stat,
+    int rcaller)
 {
-	int		sps;
-	DMA_ENTRY	*req;
+	int s;
+	DMA_ENTRY *req;
 
-	if(must_init) {
+	if (must_init) {
 		st_dma_init();
 		must_init = 0;
 	}
 	*lock_stat = DMA_LOCK_REQ;
 
-	sps = splhigh();
+	s = splhigh();
 
 	/*
 	 * Create a request...
 	 */
-	if(dma_free.tqh_first == NULL)
+	if (dma_free.tqh_first == NULL)
 		panic("st_dmagrab: Too many outstanding requests");
 	req = dma_free.tqh_first;
 	TAILQ_REMOVE(&dma_free, dma_free.tqh_first, entries);
@@ -128,50 +129,50 @@
 	req->lock_stat = lock_stat;
 	TAILQ_INSERT_TAIL(&dma_active, req, entries);
 
-	if(dma_active.tqh_first != req) {
+	if (dma_active.tqh_first != req) {
 		if (call_func == NULL) {
 			do {
 				tsleep(&dma_active, PRIBIO, "dmalck", 0);
 			} while (*req->lock_stat != DMA_LOCK_GRANT);
-			splx(sps);
-			return(1);
+			splx(s);
+			return 1;
 		}
-		splx(sps);
-		return(0);
+		splx(s);
+		return 0;
 	}
-	splx(sps);
+	splx(s);
 
 	/*
 	 * We're at the head of the queue, ergo: we got the lock.
 	 */
 	*lock_stat = DMA_LOCK_GRANT;
 
-	if(rcaller || (call_func == NULL)) {
+	if (rcaller || (call_func == NULL)) {
 		/*
 		 * Just return to caller immediately without going
 		 * through 'call_func' first.
 		 */
-		return(1);
+		return 1;
 	}
 
 	(*call_func)(softc);	/* Call followup function		*/
-	return(0);
+	return 0;
 }
 
 void
 st_dmafree(void *softc, int *lock_stat)
 {
-	int		sps;
-	DMA_ENTRY	*req;
+	int s;
+	DMA_ENTRY *req;
 	
-	sps = splhigh();
+	s = splhigh();
 
 	/*
 	 * Some validity checks first.
 	 */
-	if((req = dma_active.tqh_first) == NULL)
+	if ((req = dma_active.tqh_first) == NULL)
 		panic("st_dmafree: empty active queue");
-	if(req->softc != softc)
+	if (req->softc != softc)
 		printf("Caller of st_dmafree is not lock-owner!\n");
 
 	/*
@@ -181,37 +182,37 @@
 	TAILQ_REMOVE(&dma_active, req, entries);
 	TAILQ_INSERT_HEAD(&dma_free, req, entries);
 
-	if((req = dma_active.tqh_first) != NULL) {
+	if ((req = dma_active.tqh_first) != NULL) {
 		*req->lock_stat = DMA_LOCK_GRANT;
 
 		if (req->call_func == NULL)
 			wakeup((void *)&dma_active);
 		else {
-		    /*
-		     * Call next request through softint handler. This avoids
-		     * spl-conflicts.
-		     */
-		    add_sicallback((si_farg)req->call_func, req->softc, 0);
+			/*
+			 * Call next request through softint handler.
+			 * This avoids spl-conflicts.
+			 */
+			add_sicallback((si_farg)req->call_func, req->softc, 0);
 		}
 	}
-	splx(sps);
-	return;
+	splx(s);
 }
 
 int
 st_dmawanted(void)
 {
-	return(dma_active.tqh_first->entries.tqe_next != NULL);
+
+	return dma_active.tqh_first->entries.tqe_next != NULL;
 }
 
 int
 cdmaint(void *unused, int sr)
 	/* sr:	 sr at time of interrupt */
 {
-	dma_farg	int_func;
-	void		*softc;
+	dma_farg int_func;
+	void *softc;
 
-	if(dma_active.tqh_first != NULL) {
+	if (dma_active.tqh_first != NULL) {
 		/*
 		 * Due to the logic of the ST-DMA chip, it is not possible to
 		 * check for stray interrupts here...
@@ -229,9 +230,9 @@
  * Note: The order _is_ important!
  */
 void
-st_dmaaddr_set(void * address)
+st_dmaaddr_set(void *address)
 {
-	register u_long ad = (u_long)address;
+	u_long ad = (u_long)address;
 
 	DMA->dma_addr[AD_LOW ] = (ad     ) & 0xff;
 	DMA->dma_addr[AD_MID ] = (ad >> 8) & 0xff;
@@ -244,12 +245,12 @@
 u_long
 st_dmaaddr_get(void)
 {
-	register u_long ad = 0;
+	u_long ad = 0;
 
 	ad  = (DMA->dma_addr[AD_LOW ] & 0xff);
 	ad |= (DMA->dma_addr[AD_MID ] & 0xff) << 8;
 	ad |= (DMA->dma_addr[AD_HIGH] & 0xff) <<16;
-	return(ad);
+	return ad;
 }
 
 /*
@@ -259,6 +260,7 @@
 void
 st_dmacomm(int mode, int nblk)
 {
+
 	DMA->dma_mode = mode;
 	DMA->dma_mode = mode ^ DMA_WRBIT;
 	DMA->dma_mode = mode;

Reply via email to