Module Name:    src
Committed By:   njoly
Date:           Fri Dec  4 11:13:05 UTC 2009

Modified Files:
        src/sys/arch/alpha/sableio: com_sableio.c fdc_sableio.c pckbc_sableio.c
        src/sys/arch/i386/pci: gcscehci.c
        src/sys/dev/eisa: ahb.c ahc_eisa.c bha_eisa.c if_ep_eisa.c if_fea.c
            if_tlp_eisa.c uha_eisa.c
        src/sys/dev/pci/hdaudio: hdaudio_pci.c

Log Message:
Cleanup interrupt establish error messages. Do not mix
aprint_error/aprint_normal/printf calls for a single line.


To generate a diff of this commit:
cvs rdiff -u -r1.9 -r1.10 src/sys/arch/alpha/sableio/com_sableio.c \
    src/sys/arch/alpha/sableio/fdc_sableio.c
cvs rdiff -u -r1.8 -r1.9 src/sys/arch/alpha/sableio/pckbc_sableio.c
cvs rdiff -u -r1.5 -r1.6 src/sys/arch/i386/pci/gcscehci.c
cvs rdiff -u -r1.57 -r1.58 src/sys/dev/eisa/ahb.c
cvs rdiff -u -r1.37 -r1.38 src/sys/dev/eisa/ahc_eisa.c
cvs rdiff -u -r1.33 -r1.34 src/sys/dev/eisa/bha_eisa.c
cvs rdiff -u -r1.40 -r1.41 src/sys/dev/eisa/if_ep_eisa.c
cvs rdiff -u -r1.41 -r1.42 src/sys/dev/eisa/if_fea.c
cvs rdiff -u -r1.22 -r1.23 src/sys/dev/eisa/if_tlp_eisa.c
cvs rdiff -u -r1.32 -r1.33 src/sys/dev/eisa/uha_eisa.c
cvs rdiff -u -r1.2 -r1.3 src/sys/dev/pci/hdaudio/hdaudio_pci.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/alpha/sableio/com_sableio.c
diff -u src/sys/arch/alpha/sableio/com_sableio.c:1.9 src/sys/arch/alpha/sableio/com_sableio.c:1.10
--- src/sys/arch/alpha/sableio/com_sableio.c:1.9	Sat Nov 21 05:35:41 2009
+++ src/sys/arch/alpha/sableio/com_sableio.c	Fri Dec  4 11:13:04 2009
@@ -1,4 +1,4 @@
-/* $NetBSD: com_sableio.c,v 1.9 2009/11/21 05:35:41 rmind Exp $ */
+/* $NetBSD: com_sableio.c,v 1.10 2009/12/04 11:13:04 njoly Exp $ */
 
 /*-
  * Copyright (c) 1999 The NetBSD Foundation, Inc.
@@ -31,7 +31,7 @@
 
 #include <sys/cdefs.h>			/* RCS ID & Copyright macro defns */
 
-__KERNEL_RCSID(0, "$NetBSD: com_sableio.c,v 1.9 2009/11/21 05:35:41 rmind Exp $");
+__KERNEL_RCSID(0, "$NetBSD: com_sableio.c,v 1.10 2009/12/04 11:13:04 njoly Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -111,8 +111,8 @@
 	if (ssc->sc_ih == NULL) {
 		aprint_error_dev(self, "unable to establish interrupt");
 		if (intrstr != NULL)
-			aprint_normal(" at %s", intrstr);
-		aprint_normal("\n");
+			aprint_error(" at %s", intrstr);
+		aprint_error("\n");
 		return;
 	}
 	aprint_normal_dev(self, "interrupting at %s\n", intrstr);
Index: src/sys/arch/alpha/sableio/fdc_sableio.c
diff -u src/sys/arch/alpha/sableio/fdc_sableio.c:1.9 src/sys/arch/alpha/sableio/fdc_sableio.c:1.10
--- src/sys/arch/alpha/sableio/fdc_sableio.c:1.9	Mon Apr 28 20:23:12 2008
+++ src/sys/arch/alpha/sableio/fdc_sableio.c	Fri Dec  4 11:13:04 2009
@@ -1,4 +1,4 @@
-/* $NetBSD: fdc_sableio.c,v 1.9 2008/04/28 20:23:12 martin Exp $ */
+/* $NetBSD: fdc_sableio.c,v 1.10 2009/12/04 11:13:04 njoly Exp $ */
 
 /*-
  * Copyright (c) 1999, 2000 The NetBSD Foundation, Inc.
@@ -31,7 +31,7 @@
 
 #include <sys/cdefs.h>			/* RCS ID & Copyright macro defns */
 
-__KERNEL_RCSID(0, "$NetBSD: fdc_sableio.c,v 1.9 2008/04/28 20:23:12 martin Exp $");
+__KERNEL_RCSID(0, "$NetBSD: fdc_sableio.c,v 1.10 2009/12/04 11:13:04 njoly Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -118,8 +118,8 @@
 	if (fdc->sc_ih == NULL) {
 		aprint_error_dev(self, "unable to establish interrupt");
 		if (intrstr != NULL)
-			aprint_normal(" at %s", intrstr);
-		aprint_normal("\n");
+			aprint_error(" at %s", intrstr);
+		aprint_error("\n");
 		return;
 	}
 	aprint_normal_dev(self, "interrupting at %s\n", intrstr);

Index: src/sys/arch/alpha/sableio/pckbc_sableio.c
diff -u src/sys/arch/alpha/sableio/pckbc_sableio.c:1.8 src/sys/arch/alpha/sableio/pckbc_sableio.c:1.9
--- src/sys/arch/alpha/sableio/pckbc_sableio.c:1.8	Mon Apr 28 20:23:12 2008
+++ src/sys/arch/alpha/sableio/pckbc_sableio.c	Fri Dec  4 11:13:04 2009
@@ -1,4 +1,4 @@
-/* $NetBSD: pckbc_sableio.c,v 1.8 2008/04/28 20:23:12 martin Exp $ */
+/* $NetBSD: pckbc_sableio.c,v 1.9 2009/12/04 11:13:04 njoly Exp $ */
 
 /*-
  * Copyright (c) 1999 The NetBSD Foundation, Inc.
@@ -31,7 +31,7 @@
 
 #include <sys/cdefs.h>			/* RCS ID & Copyright macro defns */
 
-__KERNEL_RCSID(0, "$NetBSD: pckbc_sableio.c,v 1.8 2008/04/28 20:23:12 martin Exp $");
+__KERNEL_RCSID(0, "$NetBSD: pckbc_sableio.c,v 1.9 2009/12/04 11:13:04 njoly Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -144,8 +144,8 @@
 		    "unable to establish interrupt for %s slot",
 		    pckbc_slot_names[slot]);
 		if (intrstr != NULL)
-			aprint_normal(" at %s", intrstr);
-		aprint_normal("\n");
+			aprint_error(" at %s", intrstr);
+		aprint_error("\n");
 		return;
 	}
 	aprint_normal_dev(sc->sc_dv, "%s slot interrupting at %s\n",

Index: src/sys/arch/i386/pci/gcscehci.c
diff -u src/sys/arch/i386/pci/gcscehci.c:1.5 src/sys/arch/i386/pci/gcscehci.c:1.6
--- src/sys/arch/i386/pci/gcscehci.c:1.5	Mon May  4 12:15:51 2009
+++ src/sys/arch/i386/pci/gcscehci.c	Fri Dec  4 11:13:04 2009
@@ -1,4 +1,4 @@
-/* $NetBSD: gcscehci.c,v 1.5 2009/05/04 12:15:51 cegger Exp $ */
+/* $NetBSD: gcscehci.c,v 1.6 2009/12/04 11:13:04 njoly Exp $ */
 
 /*
  * Copyright (c) 2001, 2002, 2007 The NetBSD Foundation, Inc.
@@ -31,7 +31,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: gcscehci.c,v 1.5 2009/05/04 12:15:51 cegger Exp $");
+__KERNEL_RCSID(0, "$NetBSD: gcscehci.c,v 1.6 2009/12/04 11:13:04 njoly Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -141,8 +141,8 @@
 	if (sc->sc_ih == NULL) {
 		aprint_error("%s: couldn't establish interrupt", devname);
 		if (intrstr != NULL)
-			aprint_normal(" at %s", intrstr);
-		aprint_normal("\n");
+			aprint_error(" at %s", intrstr);
+		aprint_error("\n");
 		return;
 	}
 	aprint_normal("%s: interrupting at %s\n", devname, intrstr);

Index: src/sys/dev/eisa/ahb.c
diff -u src/sys/dev/eisa/ahb.c:1.57 src/sys/dev/eisa/ahb.c:1.58
--- src/sys/dev/eisa/ahb.c:1.57	Mon Nov 23 02:13:45 2009
+++ src/sys/dev/eisa/ahb.c	Fri Dec  4 11:13:04 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: ahb.c,v 1.57 2009/11/23 02:13:45 rmind Exp $	*/
+/*	$NetBSD: ahb.c,v 1.58 2009/12/04 11:13:04 njoly Exp $	*/
 
 /*-
  * Copyright (c) 1997, 1998 The NetBSD Foundation, Inc.
@@ -46,7 +46,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ahb.c,v 1.57 2009/11/23 02:13:45 rmind Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ahb.c,v 1.58 2009/12/04 11:13:04 njoly Exp $");
 
 #include "opt_ddb.h"
 
@@ -257,12 +257,12 @@
 	if (sc->sc_ih == NULL) {
 		aprint_error_dev(&sc->sc_dev, "couldn't establish interrupt");
 		if (intrstr != NULL)
-			printf(" at %s", intrstr);
-		printf("\n");
+			aprint_error(" at %s", intrstr);
+		aprint_error("\n");
 		return;
 	}
 	if (intrstr != NULL)
-		printf("%s: interrupting at %s\n", device_xname(&sc->sc_dev),
+		aprint_normal_dev(&sc->sc_dev, "interrupting at %s\n",
 		    intrstr);
 
 	/*

Index: src/sys/dev/eisa/ahc_eisa.c
diff -u src/sys/dev/eisa/ahc_eisa.c:1.37 src/sys/dev/eisa/ahc_eisa.c:1.38
--- src/sys/dev/eisa/ahc_eisa.c:1.37	Tue May  5 09:51:23 2009
+++ src/sys/dev/eisa/ahc_eisa.c	Fri Dec  4 11:13:04 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: ahc_eisa.c,v 1.37 2009/05/05 09:51:23 cegger Exp $	*/
+/*	$NetBSD: ahc_eisa.c,v 1.38 2009/12/04 11:13:04 njoly Exp $	*/
 
 /*
  * Product specific probe and attach routines for:
@@ -32,7 +32,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ahc_eisa.c,v 1.37 2009/05/05 09:51:23 cegger Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ahc_eisa.c,v 1.38 2009/12/04 11:13:04 njoly Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -172,12 +172,12 @@
 		aprint_error_dev(ahc->sc_dev, "couldn't establish %s interrupt",
 		    intrtypestr);
 		if (intrstr != NULL)
-			printf(" at %s", intrstr);
-		printf("\n");
+			aprint_error(" at %s", intrstr);
+		aprint_error("\n");
 		goto free_io;
 	}
 	if (intrstr != NULL)
-		printf("%s: %s interrupting at %s\n", device_xname(ahc->sc_dev),
+		aprint_normal_dev(ahc->sc_dev, "%s interrupting at %s\n",
 		       intrtypestr, intrstr);
 
 	/*

Index: src/sys/dev/eisa/bha_eisa.c
diff -u src/sys/dev/eisa/bha_eisa.c:1.33 src/sys/dev/eisa/bha_eisa.c:1.34
--- src/sys/dev/eisa/bha_eisa.c:1.33	Tue May 12 14:21:32 2009
+++ src/sys/dev/eisa/bha_eisa.c	Fri Dec  4 11:13:04 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: bha_eisa.c,v 1.33 2009/05/12 14:21:32 cegger Exp $	*/
+/*	$NetBSD: bha_eisa.c,v 1.34 2009/12/04 11:13:04 njoly Exp $	*/
 
 /*-
  * Copyright (c) 1998 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: bha_eisa.c,v 1.33 2009/05/12 14:21:32 cegger Exp $");
+__KERNEL_RCSID(0, "$NetBSD: bha_eisa.c,v 1.34 2009/12/04 11:13:04 njoly Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -184,11 +184,11 @@
 	if (sc->sc_ih == NULL) {
 		aprint_error_dev(&sc->sc_dev, "couldn't establish interrupt");
 		if (intrstr != NULL)
-			printf(" at %s", intrstr);
-		printf("\n");
+			aprint_error(" at %s", intrstr);
+		aprint_error("\n");
 		return;
 	}
-	printf("%s: interrupting at %s\n", device_xname(&sc->sc_dev), intrstr);
+	aprint_normal_dev(&sc->sc_dev, "interrupting at %s\n", intrstr);
 
 	bha_attach(sc);
 }

Index: src/sys/dev/eisa/if_ep_eisa.c
diff -u src/sys/dev/eisa/if_ep_eisa.c:1.40 src/sys/dev/eisa/if_ep_eisa.c:1.41
--- src/sys/dev/eisa/if_ep_eisa.c:1.40	Sat Jan 31 13:54:10 2009
+++ src/sys/dev/eisa/if_ep_eisa.c	Fri Dec  4 11:13:04 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_ep_eisa.c,v 1.40 2009/01/31 13:54:10 martin Exp $	*/
+/*	$NetBSD: if_ep_eisa.c,v 1.41 2009/12/04 11:13:04 njoly Exp $	*/
 
 /*-
  * Copyright (c) 1998 The NetBSD Foundation, Inc.
@@ -64,7 +64,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_ep_eisa.c,v 1.40 2009/01/31 13:54:10 martin Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_ep_eisa.c,v 1.41 2009/12/04 11:13:04 njoly Exp $");
 
 #include "opt_inet.h"
 #include "bpfilter.h"
@@ -266,13 +266,12 @@
 	if (sc->sc_ih == NULL) {
 		aprint_error_dev(sc->sc_dev, "couldn't establish interrupt");
 		if (intrstr != NULL)
-			printf(" at %s", intrstr);
-		printf("\n");
+			aprint_error(" at %s", intrstr);
+		aprint_error("\n");
 		return;
 	}
 	if (intrstr != NULL)
-		printf("%s: interrupting at %s\n", device_xname(sc->sc_dev),
-		    intrstr);
+		aprint_normal_dev(sc->sc_dev, "interrupting at %s\n", intrstr);
 
 	epconfig(sc, eep->eep_chipset, NULL);
 }

Index: src/sys/dev/eisa/if_fea.c
diff -u src/sys/dev/eisa/if_fea.c:1.41 src/sys/dev/eisa/if_fea.c:1.42
--- src/sys/dev/eisa/if_fea.c:1.41	Tue May 12 14:21:32 2009
+++ src/sys/dev/eisa/if_fea.c	Fri Dec  4 11:13:04 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_fea.c,v 1.41 2009/05/12 14:21:32 cegger Exp $	*/
+/*	$NetBSD: if_fea.c,v 1.42 2009/12/04 11:13:04 njoly Exp $	*/
 
 /*-
  * Copyright (c) 1995, 1996 Matt Thomas <m...@3am-software.com>
@@ -33,7 +33,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_fea.c,v 1.41 2009/05/12 14:21:32 cegger Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_fea.c,v 1.42 2009/12/04 11:13:04 njoly Exp $");
 
 #include "opt_inet.h"
 
@@ -517,8 +517,8 @@
     if (sc->sc_ih == NULL) {
 	aprint_error_dev(&sc->sc_dev, "couldn't establish interrupt");
 	if (intrstr != NULL)
-	    printf(" at %s", intrstr);
-	printf("\n");
+	    aprint_error(" at %s", intrstr);
+	aprint_error("\n");
 	return;
     }
     sc->sc_ats = shutdownhook_establish((void (*)(void *)) pdq_hwreset, sc->sc_pdq);

Index: src/sys/dev/eisa/if_tlp_eisa.c
diff -u src/sys/dev/eisa/if_tlp_eisa.c:1.22 src/sys/dev/eisa/if_tlp_eisa.c:1.23
--- src/sys/dev/eisa/if_tlp_eisa.c:1.22	Fri Apr 17 10:20:32 2009
+++ src/sys/dev/eisa/if_tlp_eisa.c	Fri Dec  4 11:13:04 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_tlp_eisa.c,v 1.22 2009/04/17 10:20:32 cegger Exp $	*/
+/*	$NetBSD: if_tlp_eisa.c,v 1.23 2009/12/04 11:13:04 njoly Exp $	*/
 
 /*-
  * Copyright (c) 1999, 2000 The NetBSD Foundation, Inc.
@@ -36,7 +36,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_tlp_eisa.c,v 1.22 2009/04/17 10:20:32 cegger Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_tlp_eisa.c,v 1.23 2009/12/04 11:13:04 njoly Exp $");
 
 #include "opt_inet.h"
 #include "bpfilter.h"
@@ -275,13 +275,12 @@
 	if (esc->sc_ih == NULL) {
 		aprint_error_dev(self, "unable to establish interrupt");
 		if (intrstr != NULL)
-			printf(" at %s", intrstr);
-		printf("\n");
+			aprint_error(" at %s", intrstr);
+		aprint_error("\n");
 		return;
 	}
 	if (intrstr != NULL)
-		printf("%s: interrupting at %s\n", device_xname(self),
-		    intrstr);
+		aprint_normal_dev(self, "interrupting at %s\n", intrstr);
 
 	/*
 	 * Finish off the attach.

Index: src/sys/dev/eisa/uha_eisa.c
diff -u src/sys/dev/eisa/uha_eisa.c:1.32 src/sys/dev/eisa/uha_eisa.c:1.33
--- src/sys/dev/eisa/uha_eisa.c:1.32	Mon Nov 23 02:13:45 2009
+++ src/sys/dev/eisa/uha_eisa.c	Fri Dec  4 11:13:04 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: uha_eisa.c,v 1.32 2009/11/23 02:13:45 rmind Exp $	*/
+/*	$NetBSD: uha_eisa.c,v 1.33 2009/12/04 11:13:04 njoly Exp $	*/
 
 /*-
  * Copyright (c) 1998 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: uha_eisa.c,v 1.32 2009/11/23 02:13:45 rmind Exp $");
+__KERNEL_RCSID(0, "$NetBSD: uha_eisa.c,v 1.33 2009/12/04 11:13:04 njoly Exp $");
 
 #include "opt_ddb.h"
 
@@ -147,11 +147,11 @@
 	if (sc->sc_ih == NULL) {
 		aprint_error_dev(&sc->sc_dev, "couldn't establish interrupt");
 		if (intrstr != NULL)
-			printf(" at %s", intrstr);
-		printf("\n");
+			aprint_error(" at %s", intrstr);
+		aprint_error("\n");
 		return;
 	}
-	printf("%s: interrupting at %s\n", device_xname(&sc->sc_dev), intrstr);
+	aprint_normal_dev(&sc->sc_dev, "interrupting at %s\n", intrstr);
 
 	/* Save function pointers for later use. */
 	sc->start_mbox = u24_start_mbox;

Index: src/sys/dev/pci/hdaudio/hdaudio_pci.c
diff -u src/sys/dev/pci/hdaudio/hdaudio_pci.c:1.2 src/sys/dev/pci/hdaudio/hdaudio_pci.c:1.3
--- src/sys/dev/pci/hdaudio/hdaudio_pci.c:1.2	Sun Sep  6 17:33:53 2009
+++ src/sys/dev/pci/hdaudio/hdaudio_pci.c	Fri Dec  4 11:13:05 2009
@@ -1,4 +1,4 @@
-/* $NetBSD: hdaudio_pci.c,v 1.2 2009/09/06 17:33:53 sborrill Exp $ */
+/* $NetBSD: hdaudio_pci.c,v 1.3 2009/12/04 11:13:05 njoly Exp $ */
 
 /*
  * Copyright (c) 2009 Precedence Technologies Ltd <supp...@precedence.co.uk>
@@ -34,7 +34,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: hdaudio_pci.c,v 1.2 2009/09/06 17:33:53 sborrill Exp $");
+__KERNEL_RCSID(0, "$NetBSD: hdaudio_pci.c,v 1.3 2009/12/04 11:13:05 njoly Exp $");
 
 #include <sys/types.h>
 #include <sys/param.h>
@@ -144,8 +144,8 @@
 	if (sc->sc_ih == NULL) {
 		aprint_error_dev(self, "couldn't establish interrupt");
 		if (intrstr)
-			aprint_normal(" at %s", intrstr);
-		aprint_normal("\n");
+			aprint_error(" at %s", intrstr);
+		aprint_error("\n");
 		return;
 	}
 	aprint_normal_dev(self, "interrupting at %s\n", intrstr);

Reply via email to