Module Name:    src
Committed By:   tsutsui
Date:           Sat Jun 25 22:38:43 UTC 2022

Modified Files:
        src/sys/arch/atari/vme: if_le_vme.c

Log Message:
No need to print errors via aprint_error(9) in probe/match function.

Note aprint_error(9) triggers "WARNING: 1 error while detecting hardware"
message after device configuration, so it's a bit confusing for users.
Also check all possible variants even if bus_space_map(9) fails
(though now it shouldn't fail).


To generate a diff of this commit:
cvs rdiff -u -r1.32 -r1.33 src/sys/arch/atari/vme/if_le_vme.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/atari/vme/if_le_vme.c
diff -u src/sys/arch/atari/vme/if_le_vme.c:1.32 src/sys/arch/atari/vme/if_le_vme.c:1.33
--- src/sys/arch/atari/vme/if_le_vme.c:1.32	Sat Jun 25 22:31:09 2022
+++ src/sys/arch/atari/vme/if_le_vme.c	Sat Jun 25 22:38:43 2022
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_le_vme.c,v 1.32 2022/06/25 22:31:09 tsutsui Exp $	*/
+/*	$NetBSD: if_le_vme.c,v 1.33 2022/06/25 22:38:43 tsutsui Exp $	*/
 
 /*-
  * Copyright (c) 1998 maximum entropy.  All rights reserved.
@@ -74,7 +74,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_le_vme.c,v 1.32 2022/06/25 22:31:09 tsutsui Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_le_vme.c,v 1.33 2022/06/25 22:38:43 tsutsui Exp $");
 
 #include "opt_inet.h"
 
@@ -233,8 +233,7 @@ le_vme_match(device_t parent, cfdata_t c
 
 		if (bus_space_map(iot, le_ap->reg_addr, le_ap->reg_size, 0,
 		    &ioh)) {
-			aprint_error("leprobe: cannot map io-area\n");
-			return 0;
+			continue;
 		}
 		if (le_ap->mem_size == VMECF_MEMSIZ_DEFAULT) {
 			if (bvme410_probe(iot, ioh)) {
@@ -252,8 +251,7 @@ le_vme_match(device_t parent, cfdata_t c
 		if (bus_space_map(memt, le_ap->mem_addr, le_ap->mem_size, 0,
 		    &memh)) {
 			bus_space_unmap(iot, ioh, le_ap->reg_size);
-			aprint_error("leprobe: cannot map memory-area\n");
-			return 0;
+			continue;
 		}
 		found = probe_addresses(&iot, &memt, &ioh, &memh);
 		bus_space_unmap(iot, ioh, le_ap->reg_size);

Reply via email to