Date: Fri, 13 Dec 2019 13:41:21 -0800 From: Ravi Pokala <rpokala@freebsd.org> To: Warner Losh <imp@FreeBSD.org>, <src-committers@freebsd.org>, <svn-src-all@freebsd.org>, <svn-src-head@freebsd.org> Subject: Re: svn commit: r355735 - in head/sys: arm/broadcom/bcm2835 arm/mv dev/ow Message-ID: <DDD8601D-C642-4BD3-8B80-83DF3D1DCAF9@panasas.com> In-Reply-To: <201912132139.xBDLdK99042422@repo.freebsd.org> References: <201912132139.xBDLdK99042422@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Thanks!
-Ravi (rpokala@)
=EF=BB=BF-----Original Message-----
From: <owner-src-committers@freebsd.org> on behalf of Warner Losh <imp@Free=
BSD.org>
Date: 2019-12-13, Friday at 13:39
To: <src-committers@freebsd.org>, <svn-src-all@freebsd.org>, <svn-src-head@=
freebsd.org>
Subject: svn commit: r355735 - in head/sys: arm/broadcom/bcm2835 arm/mv dev=
/ow
Author: imp
Date: Fri Dec 13 21:39:20 2019
New Revision: 355735
URL: https://svnweb.freebsd.org/changeset/base/355735
=20
Log:
Be consistent about checking return value from bus_delayed_attach_chi=
ldren.
=20
Most places checked, but a couple last minute changes didn't. Make th=
em all use
the return value.
=20
Noticed by: rpokala@
=20
Modified:
head/sys/arm/broadcom/bcm2835/bcm2835_bsc.c
head/sys/arm/mv/a37x0_spi.c
head/sys/dev/ow/owc_gpiobus.c
=20
Modified: head/sys/arm/broadcom/bcm2835/bcm2835_bsc.c
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D
--- head/sys/arm/broadcom/bcm2835/bcm2835_bsc.c Fri Dec 13 21:39:10 201=
9 (r355734)
+++ head/sys/arm/broadcom/bcm2835/bcm2835_bsc.c Fri Dec 13 21:39:20 201=
9 (r355735)
@@ -347,9 +347,7 @@ bcm_bsc_attach(device_t dev)
}
=20
/* Probe and attach the iicbus when interrupts are available. */
- bus_delayed_attach_children(dev);
-
- return (0);
+ return (bus_delayed_attach_children(dev));
}
=20
static int
=20
Modified: head/sys/arm/mv/a37x0_spi.c
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D
--- head/sys/arm/mv/a37x0_spi.c Fri Dec 13 21:39:10 2019 (r355734)
+++ head/sys/arm/mv/a37x0_spi.c Fri Dec 13 21:39:20 2019 (r355735)
@@ -220,9 +220,7 @@ a37x0_spi_attach(device_t dev)
device_add_child(dev, "spibus", -1);
=20
/* Probe and attach the spibus when interrupts are available. */
- bus_delayed_attach_children(dev);
-
- return (0);
+ return (bus_delayed_attach_children(dev));
}
=20
static int
=20
Modified: head/sys/dev/ow/owc_gpiobus.c
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D
--- head/sys/dev/ow/owc_gpiobus.c Fri Dec 13 21:39:10 2019 (r355734)
+++ head/sys/dev/ow/owc_gpiobus.c Fri Dec 13 21:39:20 2019 (r355735)
@@ -131,9 +131,7 @@ owc_gpiobus_attach(device_t dev)
free(kids, M_TEMP);
if (nkid =3D=3D 0)
device_add_child(dev, "ow", -1);
- bus_delayed_attach_children(dev);
-
- return (0);
+ return (bus_delayed_attach_children(dev));
}
=20
static int
=20
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?DDD8601D-C642-4BD3-8B80-83DF3D1DCAF9>