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;