Remove declarations of unused local variables, an unused function
authorkrw <krw@openbsd.org>
Sat, 3 Jun 2023 21:37:53 +0000 (21:37 +0000)
committerkrw <krw@openbsd.org>
Sat, 3 Jun 2023 21:37:53 +0000 (21:37 +0000)
(get_long) and add missing {} in devsw[1] initialization.

Most from 2011 NetBSD commit by tsutui.

No functional change.

Build tested and ok kn@

sys/arch/sparc64/stand/ofwboot/boot.c
sys/arch/sparc64/stand/ofwboot/elf64_exec.c
sys/arch/sparc64/stand/ofwboot/netif_of.c
sys/arch/sparc64/stand/ofwboot/ofdev.c
sys/arch/sparc64/stand/ofwboot/softraid_sparc64.c

index 1e7869a..a047610 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: boot.c,v 1.42 2023/06/01 17:24:56 krw Exp $   */
+/*     $OpenBSD: boot.c,v 1.43 2023/06/03 21:37:53 krw Exp $   */
 /*     $NetBSD: boot.c,v 1.3 2001/05/31 08:55:19 mrg Exp $     */
 /*
  * Copyright (c) 1997, 1999 Eduardo E. Horvath.  All rights reserved.
@@ -110,7 +110,6 @@ static int
 parseargs(char *str, int *howtop)
 {
        char *cp;
-       int i;
 
        *howtop = 0;
        cp = str;
@@ -284,7 +283,7 @@ int
 loadrandom(char *path, char *buf, size_t buflen)
 {
        struct stat sb;
-       int fd, i, error = 0;
+       int fd, error = 0;
 
        fd = open(path, O_RDONLY);
        if (fd == -1)
@@ -416,7 +415,7 @@ main(void)
        int chosen;
        char bootline[512];             /* Should check size? */
        char *cp;
-       int i, fd;
+       int fd;
 #ifdef SOFTRAID
        int err;
 #endif
index 6fe116e..0b57c98 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: elf64_exec.c,v 1.15 2023/06/02 00:10:25 krw Exp $     */
+/*     $OpenBSD: elf64_exec.c,v 1.16 2023/06/03 21:37:53 krw Exp $     */
 /*     $NetBSD: elfXX_exec.c,v 1.2 2001/08/15 20:08:15 eeh Exp $       */
 
 /*
@@ -82,7 +82,6 @@ elf64_exec(int fd, Elf_Ehdr *elf, u_int64_t *entryp, void **ssymp, void **esymp)
        size_t size;
        u_int align;
        int i, first = 1;
-       int n;
        struct openbsd_bootdata *obd;
 #ifdef SOFTRAID
        struct sr_boot_volume *bv;
index 220f59e..6b2ac60 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: netif_of.c,v 1.8 2023/06/01 17:24:56 krw Exp $        */
+/*     $OpenBSD: netif_of.c,v 1.9 2023/06/03 21:37:53 krw Exp $        */
 /*     $NetBSD: netif_of.c,v 1.1 2000/08/20 14:58:39 mrg Exp $ */
 
 /*
@@ -72,8 +72,6 @@ netif_open(void *machdep_hint)
 {
        struct of_dev *op = machdep_hint;
        struct iodesc *io;
-       int fd, error;
-       char addr[32];
 
        DNPRINTF(BOOT_D_OFNET, "netif_open...");
 
index 2310867..ef9c7ff 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: ofdev.c,v 1.38 2023/06/01 17:24:56 krw Exp $  */
+/*     $OpenBSD: ofdev.c,v 1.39 2023/06/03 21:37:53 krw Exp $  */
 /*     $NetBSD: ofdev.c,v 1.1 2000/08/20 14:58:41 mrg Exp $    */
 
 /*
@@ -184,11 +184,13 @@ devclose(struct open_file *of)
 }
 
 struct devsw devsw[1] = {
-       "OpenFirmware",
-       strategy,
-       (int (*)(struct open_file *, ...))nodev,
-       devclose,
-       noioctl
+       {
+               "OpenFirmware",
+               strategy,
+               (int (*)(struct open_file *, ...))nodev,
+               devclose,
+               noioctl
+       }
 };
 int ndevs = sizeof devsw / sizeof devsw[0];
 
@@ -224,14 +226,6 @@ static struct of_dev ofdev = {
 
 char opened_name[256];
 
-static u_long
-get_long(const void *p)
-{
-       const unsigned char *cp = p;
-
-       return cp[0] | (cp[1] << 8) | (cp[2] << 16) | (cp[3] << 24);
-}
-
 /************************************************************************
  *
  * The rest of this was taken from arch/sparc64/scsi/sun_disklabel.c
index bd0b5e5..5cf58f3 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: softraid_sparc64.c,v 1.7 2023/06/01 17:24:56 krw Exp $        */
+/*     $OpenBSD: softraid_sparc64.c,v 1.8 2023/06/03 21:37:53 krw Exp $        */
 
 /*
  * Copyright (c) 2012 Joel Sing <jsing@openbsd.org>
@@ -404,7 +404,6 @@ const char *
 sr_getdisklabel(struct sr_boot_volume *bv, struct disklabel *label)
 {
        struct of_dev ofdev;
-       int err;
 #ifdef DEBUG
        int i;
 #endif