Module Name:    src
Committed By:   mrg
Date:           Sat Oct 19 19:40:23 UTC 2013

Modified Files:
        src/sys/arch/sparc/dev: if_ie_obio.c kd.c tctrl.c ts102.c
        src/sys/arch/sparc/include: pmap.h
        src/sys/arch/sparc/sparc: cpuvar.h memecc.c timer.c

Log Message:
- remove unused but set variables.
- use __USE() where necessary.
- remove useless 'volatile' markers

kd.c:consinit() might be wrong for old proms, but i've not
changed it really.


To generate a diff of this commit:
cvs rdiff -u -r1.40 -r1.41 src/sys/arch/sparc/dev/if_ie_obio.c
cvs rdiff -u -r1.51 -r1.52 src/sys/arch/sparc/dev/kd.c
cvs rdiff -u -r1.56 -r1.57 src/sys/arch/sparc/dev/tctrl.c
cvs rdiff -u -r1.17 -r1.18 src/sys/arch/sparc/dev/ts102.c
cvs rdiff -u -r1.91 -r1.92 src/sys/arch/sparc/include/pmap.h
cvs rdiff -u -r1.90 -r1.91 src/sys/arch/sparc/sparc/cpuvar.h
cvs rdiff -u -r1.15 -r1.16 src/sys/arch/sparc/sparc/memecc.c
cvs rdiff -u -r1.30 -r1.31 src/sys/arch/sparc/sparc/timer.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/sparc/dev/if_ie_obio.c
diff -u src/sys/arch/sparc/dev/if_ie_obio.c:1.40 src/sys/arch/sparc/dev/if_ie_obio.c:1.41
--- src/sys/arch/sparc/dev/if_ie_obio.c:1.40	Fri Jul  1 18:50:41 2011
+++ src/sys/arch/sparc/dev/if_ie_obio.c	Sat Oct 19 19:40:23 2013
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_ie_obio.c,v 1.40 2011/07/01 18:50:41 dyoung Exp $	*/
+/*	$NetBSD: if_ie_obio.c,v 1.41 2013/10/19 19:40:23 mrg Exp $	*/
 
 /*-
  * Copyright (c) 1997 The NetBSD Foundation, Inc.
@@ -73,7 +73,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_ie_obio.c,v 1.40 2011/07/01 18:50:41 dyoung Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_ie_obio.c,v 1.41 2013/10/19 19:40:23 mrg Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -254,7 +254,6 @@ ie_obio_attach(device_t parent, device_t
 	int rseg;
 	int error;
 	paddr_t pa;
-	struct intrhand *ih;
 	bus_size_t memsize;
 	u_long iebase;
 	uint8_t myaddr[ETHER_ADDR_LEN];
@@ -392,7 +391,7 @@ ie_obio_attach(device_t parent, device_t
 	i82586_attach(sc, "onboard", myaddr, media, NMEDIA, media[0]);
 
 	/* Establish interrupt channel */
-	ih = bus_intr_establish(oba->oba_bustag,
-				oba->oba_pri, IPL_NET,
-				i82586_intr, sc);
+	(void)bus_intr_establish(oba->oba_bustag,
+				 oba->oba_pri, IPL_NET,
+				 i82586_intr, sc);
 }

Index: src/sys/arch/sparc/dev/kd.c
diff -u src/sys/arch/sparc/dev/kd.c:1.51 src/sys/arch/sparc/dev/kd.c:1.52
--- src/sys/arch/sparc/dev/kd.c:1.51	Sat Oct 27 17:18:11 2012
+++ src/sys/arch/sparc/dev/kd.c	Sat Oct 19 19:40:23 2013
@@ -1,4 +1,4 @@
-/*	$NetBSD: kd.c,v 1.51 2012/10/27 17:18:11 chs Exp $	*/
+/*	$NetBSD: kd.c,v 1.52 2013/10/19 19:40:23 mrg Exp $	*/
 
 /*-
  * Copyright (c) 1996 The NetBSD Foundation, Inc.
@@ -39,7 +39,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: kd.c,v 1.51 2012/10/27 17:18:11 chs Exp $");
+__KERNEL_RCSID(0, "$NetBSD: kd.c,v 1.52 2013/10/19 19:40:23 mrg Exp $");
 
 #include "opt_kgdb.h"
 #include "fb.h"
@@ -334,7 +334,6 @@ kdparam(struct tty *tp, struct termios *
 static void
 kdstart(struct tty *tp)
 {
-	struct clist *cl;
 	int s1, s2;
 
 	s1 = splsoftclock();
@@ -342,7 +341,6 @@ kdstart(struct tty *tp)
 	if (tp->t_state & (TS_BUSY|TS_TTSTOP|TS_TIMEOUT))
 		goto out;
 
-	cl = &tp->t_outq;
 	if (ttypull(tp)) {
 		tp->t_state |= TS_BUSY;
 		if ((s1 & PSR_PIL) == 0) {
@@ -619,15 +617,20 @@ prom_get_device_args(const char *prop, c
 void
 consinit(void)
 {
+#if 0
 	int inSource, outSink;
+#endif
 
 	switch (prom_version()) {
+#if 0
 	case PROM_OLDMON:
 	case PROM_OBP_V0:
 		/* The stdio handles identify the device type */
 		inSource = prom_stdin();
 		outSink  = prom_stdout();
 		break;
+	// XXXMRG  should these just set prom_stdin_node / prom_stdout_node?
+#endif
 
 	case PROM_OBP_V2:
 	case PROM_OBP_V3:

Index: src/sys/arch/sparc/dev/tctrl.c
diff -u src/sys/arch/sparc/dev/tctrl.c:1.56 src/sys/arch/sparc/dev/tctrl.c:1.57
--- src/sys/arch/sparc/dev/tctrl.c:1.56	Sat Oct 27 17:18:11 2012
+++ src/sys/arch/sparc/dev/tctrl.c	Sat Oct 19 19:40:23 2013
@@ -1,4 +1,4 @@
-/*	$NetBSD: tctrl.c,v 1.56 2012/10/27 17:18:11 chs Exp $	*/
+/*	$NetBSD: tctrl.c,v 1.57 2013/10/19 19:40:23 mrg Exp $	*/
 
 /*-
  * Copyright (c) 1998, 2005, 2006 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: tctrl.c,v 1.56 2012/10/27 17:18:11 chs Exp $");
+__KERNEL_RCSID(0, "$NetBSD: tctrl.c,v 1.57 2013/10/19 19:40:23 mrg Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -747,7 +747,7 @@ static void
 tctrl_read_event_status(struct tctrl_softc *sc)
 {
 	struct tctrl_req req;
-	int s, lid;
+	int s;
 	uint32_t v;
 
 	req.cmdbuf[0] = TS102_OP_RD_EVENT_STATUS;
@@ -800,7 +800,6 @@ tctrl_read_event_status(struct tctrl_sof
 		    (sc->sc_ext_status & TS102_EXT_STATUS_LID_DOWN)
 		    ? "closed" : "opened");
 #endif
-		lid = (sc->sc_ext_status & TS102_EXT_STATUS_LID_DOWN) == 0;
 	}
 	if (v & TS102_EVENT_STATUS_EXTERNAL_VGA_STATUS_CHANGE) {
 		int vga;

Index: src/sys/arch/sparc/dev/ts102.c
diff -u src/sys/arch/sparc/dev/ts102.c:1.17 src/sys/arch/sparc/dev/ts102.c:1.18
--- src/sys/arch/sparc/dev/ts102.c:1.17	Sat Oct 27 17:18:11 2012
+++ src/sys/arch/sparc/dev/ts102.c	Sat Oct 19 19:40:23 2013
@@ -1,5 +1,5 @@
 /*	$OpenBSD: ts102.c,v 1.14 2005/01/27 17:03:23 millert Exp $	*/
-/*	$NetBSD: ts102.c,v 1.17 2012/10/27 17:18:11 chs Exp $ */
+/*	$NetBSD: ts102.c,v 1.18 2013/10/19 19:40:23 mrg Exp $ */
 /*
  * Copyright (c) 2003, 2004, Miodrag Vallat.
  * Copyright (c) 2005, Michael Lorenz.
@@ -866,6 +866,8 @@ tslot_slot_intr(struct tslot_data *td, i
 #ifdef TSLOT_DEBUG
 	printf("%s: interrupt on socket %d ir %x sts %x\n",
 	    device_xname(sc->sc_dev), td->td_slot, intreg, status);
+#else	
+	__USE(status);
 #endif
 
 	sockstat = td->td_status;

Index: src/sys/arch/sparc/include/pmap.h
diff -u src/sys/arch/sparc/include/pmap.h:1.91 src/sys/arch/sparc/include/pmap.h:1.92
--- src/sys/arch/sparc/include/pmap.h:1.91	Mon Jan  7 16:59:18 2013
+++ src/sys/arch/sparc/include/pmap.h	Sat Oct 19 19:40:23 2013
@@ -1,4 +1,4 @@
-/*	$NetBSD: pmap.h,v 1.91 2013/01/07 16:59:18 chs Exp $ */
+/*	$NetBSD: pmap.h,v 1.92 2013/10/19 19:40:23 mrg Exp $ */
 
 /*
  * Copyright (c) 1996
@@ -260,7 +260,7 @@ vaddr_t		pmap_map(vaddr_t, paddr_t, padd
 #define		pmap_phys_address(x) (x)
 void		pmap_reference(pmap_t);
 void		pmap_remove(pmap_t, vaddr_t, vaddr_t);
-#define		pmap_update(pmap)		/* nothing (yet) */
+#define		pmap_update(pmap)		__USE(pmap)
 void		pmap_virtual_space(vaddr_t *, vaddr_t *);
 #ifdef PMAP_GROWKERNEL
 vaddr_t		pmap_growkernel(vaddr_t);

Index: src/sys/arch/sparc/sparc/cpuvar.h
diff -u src/sys/arch/sparc/sparc/cpuvar.h:1.90 src/sys/arch/sparc/sparc/cpuvar.h:1.91
--- src/sys/arch/sparc/sparc/cpuvar.h:1.90	Mon Aug 15 02:19:44 2011
+++ src/sys/arch/sparc/sparc/cpuvar.h	Sat Oct 19 19:40:23 2013
@@ -1,4 +1,4 @@
-/*	$NetBSD: cpuvar.h,v 1.90 2011/08/15 02:19:44 mrg Exp $ */
+/*	$NetBSD: cpuvar.h,v 1.91 2013/10/19 19:40:23 mrg Exp $ */
 
 /*
  *  Copyright (c) 1996 The NetBSD Foundation, Inc.
@@ -171,9 +171,9 @@ struct cpu_info {
 	 * the pending register to avoid a hardware bug.
 	 */
 #define raise_ipi(cpi,lvl)	do {			\
-	volatile int x;					\
+	int x;						\
 	(cpi)->intreg_4m->pi_set = PINTR_SINTRLEV(lvl);	\
-	x = (cpi)->intreg_4m->pi_pend;			\
+	x = (cpi)->intreg_4m->pi_pend; __USE(x);	\
 } while (0)
 
 	int		sun4_mmu3l;	/* [4]: 3-level MMU present */

Index: src/sys/arch/sparc/sparc/memecc.c
diff -u src/sys/arch/sparc/sparc/memecc.c:1.15 src/sys/arch/sparc/sparc/memecc.c:1.16
--- src/sys/arch/sparc/sparc/memecc.c:1.15	Wed Oct 10 16:51:51 2012
+++ src/sys/arch/sparc/sparc/memecc.c	Sat Oct 19 19:40:23 2013
@@ -1,4 +1,4 @@
-/*	$NetBSD: memecc.c,v 1.15 2012/10/10 16:51:51 tsutsui Exp $	*/
+/*	$NetBSD: memecc.c,v 1.16 2013/10/19 19:40:23 mrg Exp $	*/
 
 /*-
  * Copyright (c) 1998 The NetBSD Foundation, Inc.
@@ -34,7 +34,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: memecc.c,v 1.15 2012/10/10 16:51:51 tsutsui Exp $");
+__KERNEL_RCSID(0, "$NetBSD: memecc.c,v 1.16 2013/10/19 19:40:23 mrg Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -77,7 +77,6 @@ memecc_attach(device_t parent, device_t 
 {
 	struct memecc_softc *sc = device_private(self);
 	struct mainbus_attach_args *ma = aux;
-	int node;
 	uint32_t reg;
 
 	if (memerr_handler) {
@@ -86,7 +85,6 @@ memecc_attach(device_t parent, device_t 
 	}
 
 	sc->sc_bt = ma->ma_bustag;
-	node = ma->ma_node;
 
 	/*
 	 * Map registers

Index: src/sys/arch/sparc/sparc/timer.c
diff -u src/sys/arch/sparc/sparc/timer.c:1.30 src/sys/arch/sparc/sparc/timer.c:1.31
--- src/sys/arch/sparc/sparc/timer.c:1.30	Sun Jul 29 00:04:05 2012
+++ src/sys/arch/sparc/sparc/timer.c	Sat Oct 19 19:40:23 2013
@@ -1,4 +1,4 @@
-/*	$NetBSD: timer.c,v 1.30 2012/07/29 00:04:05 matt Exp $ */
+/*	$NetBSD: timer.c,v 1.31 2013/10/19 19:40:23 mrg Exp $ */
 
 /*
  * Copyright (c) 1992, 1993
@@ -60,7 +60,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: timer.c,v 1.30 2012/07/29 00:04:05 matt Exp $");
+__KERNEL_RCSID(0, "$NetBSD: timer.c,v 1.31 2013/10/19 19:40:23 mrg Exp $");
 
 #include <sys/param.h>
 #include <sys/kernel.h>
@@ -163,11 +163,12 @@ timerattach(volatile int *cntreg, volati
 	 * Note: sun4m clocks tick with 500ns periods.
 	 */
 	for (timerblurb = 1; ; timerblurb++) {
-		volatile int discard;
+		int discard;
 		u_int t1;
 
 		/* Reset counter register by writing some large limit value */
 		discard = *limreg;
+		__USE(discard);
 		*limreg = tmr_ustolim(TMR_MASK-1);
 
 		t0 = *cntreg;

Reply via email to