git: c3147104fddd - main - Error is not a bool in bcm2838_pci, check for != 0

From: Andrew Turner <andrew_at_FreeBSD.org>
Date: Mon, 23 May 2022 16:40:37 UTC
The branch main has been updated by andrew:

URL: https://cgit.FreeBSD.org/src/commit/?id=c3147104fdddb5cb358726888a0a612439d10f9e

commit c3147104fdddb5cb358726888a0a612439d10f9e
Author:     Andrew Turner <andrew@FreeBSD.org>
AuthorDate: 2022-05-23 16:38:18 +0000
Commit:     Andrew Turner <andrew@FreeBSD.org>
CommitDate: 2022-05-23 16:38:18 +0000

    Error is not a bool in bcm2838_pci, check for != 0
    
    Fix for style an check if error != 0 in the bcm2838 pci driver.
    
    Sponsored by:   The FreeBSD Foundation
---
 sys/arm/broadcom/bcm2835/bcm2838_pci.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/sys/arm/broadcom/bcm2835/bcm2838_pci.c b/sys/arm/broadcom/bcm2835/bcm2838_pci.c
index 7c5fd8d787b6..dee90a4532bc 100644
--- a/sys/arm/broadcom/bcm2835/bcm2838_pci.c
+++ b/sys/arm/broadcom/bcm2835/bcm2838_pci.c
@@ -534,7 +534,7 @@ bcm_pcib_msi_attach(device_t dev)
 
 	error = bus_setup_intr(dev, sc->msi_irq_res, INTR_TYPE_BIO |
 	    INTR_MPSAFE, bcm_pcib_msi_intr, NULL, sc, &sc->msi_intr_cookie);
-	if (error) {
+	if (error != 0) {
 		device_printf(dev, "error: failed to setup MSI handler.\n");
 		return (ENXIO);
 	}
@@ -544,7 +544,7 @@ bcm_pcib_msi_attach(device_t dev)
 		sc->msi_isrcs[i].irq = i;
 		error = intr_isrc_register(&sc->msi_isrcs[i].isrc, dev, 0,
 		    "%s,%u", bcm_name, i);
-		if (error) {
+		if (error != 0) {
 			device_printf(dev,
 			"error: failed to register interrupt %d.\n", i);
 			return (ENXIO);
@@ -556,7 +556,7 @@ bcm_pcib_msi_attach(device_t dev)
 	OF_device_register_xref(xref, dev);
 
 	error = intr_msi_register(dev, xref);
-	if (error)
+	if (error != 0)
 		return (ENXIO);
 
 	mtx_init(&sc->msi_mtx, "bcm_pcib: msi_mtx", NULL, MTX_DEF);
@@ -651,15 +651,15 @@ bcm_pcib_attach(device_t dev)
 	    0, 					/* flags */
 	    NULL, NULL,				/* lockfunc, lockarg */
 	    &sc->dmat);
-	if (error)
+	if (error != 0)
 		return (error);
 
 	error = pci_host_generic_setup_fdt(dev);
-	if (error)
+	if (error != 0)
 		return (error);
 
 	error = bcm_pcib_check_ranges(dev);
-	if (error)
+	if (error != 0)
 		return (error);
 
 	mtx_init(&sc->config_mtx, "bcm_pcib: config_mtx", NULL, MTX_DEF);
@@ -743,7 +743,7 @@ bcm_pcib_attach(device_t dev)
 
 	/* Configure interrupts. */
 	error = bcm_pcib_msi_attach(dev);
-	if (error)
+	if (error != 0)
 		return (error);
 
 	/* Done. */