Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 30 Apr 2019 00:54:31 +0000 (UTC)
From:      Ian Lepore <ian@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-12@freebsd.org
Subject:   svn commit: r346937 - stable/12/sys/arm/broadcom/bcm2835
Message-ID:  <201904300054.x3U0sVev048012@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ian
Date: Tue Apr 30 00:54:31 2019
New Revision: 346937
URL: https://svnweb.freebsd.org/changeset/base/346937

Log:
  MFC r346489:
  
  Move the reporting of spurious interrupts under bootverbose control, because
  occasional spurious interrupts are a normal thing on this hardware.  Also,
  change the name of the cpu-local interrupt controller driver from local_intc
  to lintc, because the name gets built into interrupt names, which have to
  fit into a 19-byte field for stats reporting (so this allows 5 more bytes
  of the actual interrupt name to be displayed).

Modified:
  stable/12/sys/arm/broadcom/bcm2835/bcm2835_intr.c
  stable/12/sys/arm/broadcom/bcm2835/bcm2836.c
Directory Properties:
  stable/12/   (props changed)

Modified: stable/12/sys/arm/broadcom/bcm2835/bcm2835_intr.c
==============================================================================
--- stable/12/sys/arm/broadcom/bcm2835/bcm2835_intr.c	Mon Apr 29 23:07:19 2019	(r346936)
+++ stable/12/sys/arm/broadcom/bcm2835/bcm2835_intr.c	Tue Apr 30 00:54:31 2019	(r346937)
@@ -230,7 +230,7 @@ bcm2835_intc_intr(void *arg)
 		}
 		arm_irq_memory_barrier(0); /* XXX */
 	}
-	if (num == 0)
+	if (num == 0 && bootverbose)
 		device_printf(sc->sc_dev, "Spurious interrupt detected\n");
 
 	return (FILTER_HANDLED);

Modified: stable/12/sys/arm/broadcom/bcm2835/bcm2836.c
==============================================================================
--- stable/12/sys/arm/broadcom/bcm2835/bcm2836.c	Mon Apr 29 23:07:19 2019	(r346936)
+++ stable/12/sys/arm/broadcom/bcm2835/bcm2836.c	Tue Apr 30 00:54:31 2019	(r346937)
@@ -421,7 +421,7 @@ bcm_lintc_intr(void *arg)
 	reg &= ~BCM_LINTC_PENDING_MASK;
 	if (reg != 0)
 		device_printf(sc->bls_dev, "Unknown interrupt(s) %x\n", reg);
-	else if (num == 0)
+	else if (num == 0 && bootverbose)
 		device_printf(sc->bls_dev, "Spurious interrupt detected\n");
 
 	return (FILTER_HANDLED);
@@ -730,12 +730,12 @@ static device_method_t bcm_lintc_methods[] = {
 };
 
 static driver_t bcm_lintc_driver = {
-	"local_intc",
+	"lintc",
 	bcm_lintc_methods,
 	sizeof(struct bcm_lintc_softc),
 };
 
 static devclass_t bcm_lintc_devclass;
 
-EARLY_DRIVER_MODULE(local_intc, simplebus, bcm_lintc_driver, bcm_lintc_devclass,
+EARLY_DRIVER_MODULE(lintc, simplebus, bcm_lintc_driver, bcm_lintc_devclass,
     0, 0, BUS_PASS_INTERRUPT);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201904300054.x3U0sVev048012>