Module Name:    src
Committed By:   rin
Date:           Mon Aug 15 11:18:12 UTC 2022

Modified Files:
        src/sys/arch/mac68k/obio: esp.c

Log Message:
Minor cosmetic changes. NFC.

- Use switch instead of if-else-if chain.
- Dedup codes a bit.


To generate a diff of this commit:
cvs rdiff -u -r1.60 -r1.61 src/sys/arch/mac68k/obio/esp.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/mac68k/obio/esp.c
diff -u src/sys/arch/mac68k/obio/esp.c:1.60 src/sys/arch/mac68k/obio/esp.c:1.61
--- src/sys/arch/mac68k/obio/esp.c:1.60	Mon Feb 22 05:23:36 2021
+++ src/sys/arch/mac68k/obio/esp.c	Mon Aug 15 11:18:12 2022
@@ -1,4 +1,4 @@
-/*	$NetBSD: esp.c,v 1.60 2021/02/22 05:23:36 rin Exp $	*/
+/*	$NetBSD: esp.c,v 1.61 2022/08/15 11:18:12 rin Exp $	*/
 
 /*
  * Copyright (c) 1997 Jason R. Thorpe.
@@ -81,7 +81,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: esp.c,v 1.60 2021/02/22 05:23:36 rin Exp $");
+__KERNEL_RCSID(0, "$NetBSD: esp.c,v 1.61 2022/08/15 11:18:12 rin Exp $");
 
 #include <sys/types.h>
 #include <sys/param.h>
@@ -202,12 +202,15 @@ espattach(device_t parent, device_t self
 	 * pseudo-DMA timing.  The default value is 0x1d1.
 	 */
 	if (oa->oa_addr == 0) {
-		if (reg_offset == 0x10000) {
+		switch (reg_offset) {
+		case 0x10000:
 			quick = 1;
 			esp_have_dreq = esp_iosb_have_dreq;
-		} else if (reg_offset == 0x18000)
+			break;
+		case 0x18000:
 			avdma = 1;
-		else {
+			break;
+		default:
 			addr = 0xf9800024;
 			goto dafb_dreq;
 		}
@@ -262,11 +265,6 @@ dafb_dreq:	bst = oa->oa_tag;
 			sc->sc_freq = 16500000;
 		} else
 			sc->sc_freq = 25000000;
-
-		if (quick)
-			aprint_normal(" (quick)");
-		else if (avdma)
-			aprint_normal(" (avdma)");
 	} else {
 		esp1 = esc;
 
@@ -274,11 +272,13 @@ dafb_dreq:	bst = oa->oa_tag;
 		via2_register_irq(VIA2_SCSIIRQ, esp_dualbus_intr, NULL);
 		irq_mask = 0;
 		sc->sc_freq = 25000000;
-
-		if (quick)
-			printf(" (quick)");
 	}
 
+	if (quick)
+		aprint_normal(" (quick)");
+	else if (avdma)
+		aprint_normal(" (avdma)");
+
 	aprint_normal(": address %p", esc->sc_reg);
 
 	sc->sc_id = 7;

Reply via email to