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
This commit is contained in:
parent
b25d7f7477
commit
c3147104fd
@ -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. */
|
||||
|
Loading…
Reference in New Issue
Block a user