Module Name: src Committed By: christos Date: Sat Mar 22 01:52:44 UTC 2014
Modified Files: src/sys/arch/amiga/amiga: machdep.c src/sys/arch/amiga/dev: aucc.c drsc.c ite.c src/sys/arch/amiga/pci: em4k.c p5pb.c pci_machdep.c src/sys/arch/amiga/stand/bootblock/boot: printf.s Log Message: kill sprintf fix unused variables To generate a diff of this commit: cvs rdiff -u -r1.241 -r1.242 src/sys/arch/amiga/amiga/machdep.c cvs rdiff -u -r1.42 -r1.43 src/sys/arch/amiga/dev/aucc.c cvs rdiff -u -r1.32 -r1.33 src/sys/arch/amiga/dev/drsc.c cvs rdiff -u -r1.96 -r1.97 src/sys/arch/amiga/dev/ite.c cvs rdiff -u -r1.2 -r1.3 src/sys/arch/amiga/pci/em4k.c cvs rdiff -u -r1.12 -r1.13 src/sys/arch/amiga/pci/p5pb.c cvs rdiff -u -r1.1 -r1.2 src/sys/arch/amiga/pci/pci_machdep.c cvs rdiff -u -r1.8 -r1.9 src/sys/arch/amiga/stand/bootblock/boot/printf.s 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/amiga/amiga/machdep.c diff -u src/sys/arch/amiga/amiga/machdep.c:1.241 src/sys/arch/amiga/amiga/machdep.c:1.242 --- src/sys/arch/amiga/amiga/machdep.c:1.241 Mon Jan 28 11:36:10 2013 +++ src/sys/arch/amiga/amiga/machdep.c Fri Mar 21 21:52:44 2014 @@ -1,4 +1,4 @@ -/* $NetBSD: machdep.c,v 1.241 2013/01/28 16:36:10 rkujawa Exp $ */ +/* $NetBSD: machdep.c,v 1.242 2014/03/22 01:52:44 christos Exp $ */ /* * Copyright (c) 1988 University of Utah. @@ -50,7 +50,7 @@ #include "empm.h" #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: machdep.c,v 1.241 2013/01/28 16:36:10 rkujawa Exp $"); +__KERNEL_RCSID(0, "$NetBSD: machdep.c,v 1.242 2014/03/22 01:52:44 christos Exp $"); #include <sys/param.h> #include <sys/systm.h> @@ -308,7 +308,7 @@ identifycpu(void) char machbuf[16]; if (is_draco()) { - sprintf(machbuf, "DraCo rev.%d", is_draco()); + snprintf(machbuf, sizeof(machbuf), "DraCo rev.%d", is_draco()); mach = machbuf; } else #endif @@ -327,7 +327,7 @@ identifycpu(void) #ifdef M68060 if (machineid & AMIGA_68060) { __asm(".word 0x4e7a,0x0808; movl %%d0,%0" : "=d"(pcr) : : "d0"); - sprintf(cpubuf, "68%s060 rev.%d", + snprintf(cpubuf, sizeof(cpubuf), "68%s060 rev.%d", pcr & 0x10000 ? "LC/EC" : "", (pcr>>8)&0xff); cpu_type = cpubuf; mmu = "/MMU"; @@ -367,7 +367,8 @@ identifycpu(void) fputype = FPU_NONE; } } - sprintf(cpu_model, "%s (%s CPU%s%s)", mach, cpu_type, mmu, fpu); + snprintf(cpu_model, sizeof(cpu_model), "%s (%s CPU%s%s)", mach, + cpu_type, mmu, fpu); printf("%s\n", cpu_model); } @@ -820,39 +821,35 @@ int *nofault; int badaddr(register void *addr) { - register int i; + int i; label_t faultbuf; -#ifdef lint - i = *addr; if (i) return(0); -#endif nofault = (int *) &faultbuf; if (setjmp((label_t *)nofault)) { - nofault = (int *) 0; - return(1); + nofault = NULL; + return 1; } i = *(volatile short *)addr; - nofault = (int *) 0; - return(0); + __USE(i); + nofault = NULL; + return 0; } int badbaddr(register void *addr) { - register int i; + int i; label_t faultbuf; -#ifdef lint - i = *addr; if (i) return(0); -#endif nofault = (int *) &faultbuf; if (setjmp((label_t *)nofault)) { - nofault = (int *) 0; - return(1); + nofault = NULL; + return 1; } i = *(volatile char *)addr; - nofault = (int *) 0; - return(0); + __USE(i); + nofault = NULL; + return 0; } struct isr *isr_ports; Index: src/sys/arch/amiga/dev/aucc.c diff -u src/sys/arch/amiga/dev/aucc.c:1.42 src/sys/arch/amiga/dev/aucc.c:1.43 --- src/sys/arch/amiga/dev/aucc.c:1.42 Sat Oct 27 13:17:27 2012 +++ src/sys/arch/amiga/dev/aucc.c Fri Mar 21 21:52:44 2014 @@ -1,4 +1,4 @@ -/* $NetBSD: aucc.c,v 1.42 2012/10/27 17:17:27 chs Exp $ */ +/* $NetBSD: aucc.c,v 1.43 2014/03/22 01:52:44 christos Exp $ */ /* * Copyright (c) 1999 Bernardo Innocenti @@ -53,7 +53,7 @@ #if NAUCC > 0 #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: aucc.c,v 1.42 2012/10/27 17:17:27 chs Exp $"); +__KERNEL_RCSID(0, "$NetBSD: aucc.c,v 1.43 2014/03/22 01:52:44 christos Exp $"); #include <sys/param.h> #include <sys/systm.h> @@ -927,9 +927,11 @@ aucc_query_devinfo(void *addr, register dip->type = AUDIO_MIXER_SET; dip->mixer_class = AUCC_OUTPUT_CLASS; dip->prev = dip->next = AUDIO_MIXER_LAST; - strcpy(dip->label.name, AudioNspeaker); +#define setname(a) strlcpy(dip->label.name, (a), sizeof(dip->label.name)) + setname(AudioNspeaker); for (i = 0; i < 16; i++) { - sprintf(dip->un.s.member[i].label.name, + snprintf(dip->un.s.member[i].label.name, + sizeof(dip->un.s.member[i].label.name), "channelmask%d", i); dip->un.s.member[i].mask = i; } @@ -940,7 +942,7 @@ aucc_query_devinfo(void *addr, register dip->type = AUDIO_MIXER_VALUE; dip->mixer_class = AUCC_OUTPUT_CLASS; dip->prev = dip->next = AUDIO_MIXER_LAST; - strcpy(dip->label.name, AudioNmaster); + setname(AudioNmaster); dip->un.v.num_channels = 4; strcpy(dip->un.v.units.name, AudioNvolume); break; @@ -949,7 +951,7 @@ aucc_query_devinfo(void *addr, register dip->type = AUDIO_MIXER_CLASS; dip->mixer_class = AUCC_OUTPUT_CLASS; dip->next = dip->prev = AUDIO_MIXER_LAST; - strcpy(dip->label.name, AudioCoutputs); + setname(AudioCoutputs); break; default: Index: src/sys/arch/amiga/dev/drsc.c diff -u src/sys/arch/amiga/dev/drsc.c:1.32 src/sys/arch/amiga/dev/drsc.c:1.33 --- src/sys/arch/amiga/dev/drsc.c:1.32 Sat Oct 27 13:17:28 2012 +++ src/sys/arch/amiga/dev/drsc.c Fri Mar 21 21:52:44 2014 @@ -1,4 +1,4 @@ -/* $NetBSD: drsc.c,v 1.32 2012/10/27 17:17:28 chs Exp $ */ +/* $NetBSD: drsc.c,v 1.33 2014/03/22 01:52:44 christos Exp $ */ /* * Copyright (c) 1996 Ignatios Souvatzis @@ -59,7 +59,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: drsc.c,v 1.32 2012/10/27 17:17:28 chs Exp $"); +__KERNEL_RCSID(0, "$NetBSD: drsc.c,v 1.33 2014/03/22 01:52:44 christos Exp $"); #include <sys/param.h> #include <sys/systm.h> @@ -115,15 +115,12 @@ void drscattach(device_t parent, device_t self, void *aux) { struct siop_softc *sc = device_private(self); - struct zbus_args *zap; siop_regmap_p rp; struct scsipi_adapter *adapt = &sc->sc_adapter; struct scsipi_channel *chan = &sc->sc_channel; printf("\n"); - zap = aux; - sc->sc_dev = self; sc->sc_siopp = rp = (siop_regmap_p)(DRCCADDR+PAGE_SIZE*DRSCSIPG); Index: src/sys/arch/amiga/dev/ite.c diff -u src/sys/arch/amiga/dev/ite.c:1.96 src/sys/arch/amiga/dev/ite.c:1.97 --- src/sys/arch/amiga/dev/ite.c:1.96 Sun Mar 16 01:20:22 2014 +++ src/sys/arch/amiga/dev/ite.c Fri Mar 21 21:52:44 2014 @@ -1,4 +1,4 @@ -/* $NetBSD: ite.c,v 1.96 2014/03/16 05:20:22 dholland Exp $ */ +/* $NetBSD: ite.c,v 1.97 2014/03/22 01:52:44 christos Exp $ */ /* * Copyright (c) 1988 University of Utah. @@ -46,7 +46,7 @@ #include "opt_ddb.h" #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: ite.c,v 1.96 2014/03/16 05:20:22 dholland Exp $"); +__KERNEL_RCSID(0, "$NetBSD: ite.c,v 1.97 2014/03/22 01:52:44 christos Exp $"); #include <sys/param.h> #include <sys/kernel.h> @@ -1642,7 +1642,8 @@ iteputchar(register int c, struct ite_so break; case 6: /* cursor position report */ - sprintf(ip->argbuf, "\033[%d;%dR", + snprintf(ip->argbuf, sizeof(ip->argbuf), + "\033[%d;%dR", ip->cury + 1, ip->curx + 1); ite_sendstr(ip->argbuf); break; Index: src/sys/arch/amiga/pci/em4k.c diff -u src/sys/arch/amiga/pci/em4k.c:1.2 src/sys/arch/amiga/pci/em4k.c:1.3 --- src/sys/arch/amiga/pci/em4k.c:1.2 Tue Jan 29 16:02:50 2013 +++ src/sys/arch/amiga/pci/em4k.c Fri Mar 21 21:52:44 2014 @@ -1,4 +1,4 @@ -/* $NetBSD: em4k.c,v 1.2 2013/01/29 21:02:50 rkujawa Exp $ */ +/* $NetBSD: em4k.c,v 1.3 2014/03/22 01:52:44 christos Exp $ */ /*- * Copyright (c) 2013 The NetBSD Foundation, Inc. @@ -314,9 +314,6 @@ em4k_pci_conf_read(pci_chipset_tag_t pc, { uint32_t data; uint32_t bus, dev, func; - struct em4k_softc *sc; - - sc = pc->cookie; pci_decompose_tag(pc, tag, &bus, &dev, &func); @@ -335,10 +332,7 @@ void em4k_pci_conf_write(pci_chipset_tag_t pc, pcitag_t tag, int reg, pcireg_t val) { uint32_t bus, dev, func; - struct em4k_softc *sc; - sc = pc->cookie; - pci_decompose_tag(pc, tag, &bus, &dev, &func); bus_space_write_4(pc->pci_conf_datat, pc->pci_conf_datah, Index: src/sys/arch/amiga/pci/p5pb.c diff -u src/sys/arch/amiga/pci/p5pb.c:1.12 src/sys/arch/amiga/pci/p5pb.c:1.13 --- src/sys/arch/amiga/pci/p5pb.c:1.12 Sat Oct 27 13:17:34 2012 +++ src/sys/arch/amiga/pci/p5pb.c Fri Mar 21 21:52:44 2014 @@ -1,4 +1,4 @@ -/* $NetBSD: p5pb.c,v 1.12 2012/10/27 17:17:34 chs Exp $ */ +/* $NetBSD: p5pb.c,v 1.13 2014/03/22 01:52:44 christos Exp $ */ /*- * Copyright (c) 2011, 2012 The NetBSD Foundation, Inc. @@ -321,6 +321,7 @@ p5pb_find_resources(struct p5pb_softc *s void p5pb_set_props(struct p5pb_softc *sc) { +#if NGENFB > 0 prop_dictionary_t dict; device_t dev; @@ -328,7 +329,6 @@ p5pb_set_props(struct p5pb_softc *sc) dict = device_properties(dev); /* genfb needs additional properties, like virtual, physical address */ -#if (NGENFB > 0) /* XXX: currently genfb is supported only on CVPPC/BVPPC */ prop_dictionary_set_uint64(dict, "virtual_address", sc->pci_mem_area.base); Index: src/sys/arch/amiga/pci/pci_machdep.c diff -u src/sys/arch/amiga/pci/pci_machdep.c:1.1 src/sys/arch/amiga/pci/pci_machdep.c:1.2 --- src/sys/arch/amiga/pci/pci_machdep.c:1.1 Mon Sep 19 15:15:29 2011 +++ src/sys/arch/amiga/pci/pci_machdep.c Fri Mar 21 21:52:44 2014 @@ -1,4 +1,4 @@ -/* $NetBSD: pci_machdep.c,v 1.1 2011/09/19 19:15:29 rkujawa Exp $ */ +/* $NetBSD: pci_machdep.c,v 1.2 2014/03/22 01:52:44 christos Exp $ */ /*- * Copyright (c) 2011 The NetBSD Foundation, Inc. @@ -101,7 +101,7 @@ amiga_pci_intr_string(pci_chipset_tag_t { static char str[10]; - sprintf(str, "INT%d", (int) ih); + snprintf(str, sizeof(str), "INT%d", (int) ih); return str; } Index: src/sys/arch/amiga/stand/bootblock/boot/printf.s diff -u src/sys/arch/amiga/stand/bootblock/boot/printf.s:1.8 src/sys/arch/amiga/stand/bootblock/boot/printf.s:1.9 --- src/sys/arch/amiga/stand/bootblock/boot/printf.s:1.8 Mon Apr 28 16:23:13 2008 +++ src/sys/arch/amiga/stand/bootblock/boot/printf.s Fri Mar 21 21:52:44 2014 @@ -1,4 +1,4 @@ -/* $NetBSD: printf.s,v 1.8 2008/04/28 20:23:13 martin Exp $ */ +/* $NetBSD: printf.s,v 1.9 2014/03/22 01:52:44 christos Exp $ */ /*- * Copyright (c) 1996,2006 The NetBSD Foundation, Inc. @@ -56,6 +56,7 @@ Lstorech: movb %d0, %a3@+ rts +#ifdef notdef ENTRY_NOPROFILE(sprintf) movml #0x0032,%sp@- movl %sp@(16),%a3 @@ -66,6 +67,7 @@ ENTRY_NOPROFILE(sprintf) jsr %a6@(-0x20a) movml %sp@+, #0x4c00 rts +#endif /* * XXX cheating - at least for now.