xref: /freebsd/sys/dev/acpica/acpi_pci.c (revision a7a3177f270261a8051f75d428247a889d2610b3)
1098ca2bdSWarner Losh /*-
228bf1fadSJohn Baldwin  * Copyright (c) 1997, Stefan Esser <se@freebsd.org>
328bf1fadSJohn Baldwin  * Copyright (c) 2000, Michael Smith <msmith@freebsd.org>
428bf1fadSJohn Baldwin  * Copyright (c) 2000, BSDi
528bf1fadSJohn Baldwin  * All rights reserved.
628bf1fadSJohn Baldwin  *
728bf1fadSJohn Baldwin  * Redistribution and use in source and binary forms, with or without
828bf1fadSJohn Baldwin  * modification, are permitted provided that the following conditions
928bf1fadSJohn Baldwin  * are met:
1028bf1fadSJohn Baldwin  * 1. Redistributions of source code must retain the above copyright
1128bf1fadSJohn Baldwin  *    notice unmodified, this list of conditions, and the following
1228bf1fadSJohn Baldwin  *    disclaimer.
1328bf1fadSJohn Baldwin  * 2. Redistributions in binary form must reproduce the above copyright
1428bf1fadSJohn Baldwin  *    notice, this list of conditions and the following disclaimer in the
1528bf1fadSJohn Baldwin  *    documentation and/or other materials provided with the distribution.
1628bf1fadSJohn Baldwin  *
1728bf1fadSJohn Baldwin  * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR
1828bf1fadSJohn Baldwin  * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
1928bf1fadSJohn Baldwin  * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
2028bf1fadSJohn Baldwin  * IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT,
2128bf1fadSJohn Baldwin  * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
2228bf1fadSJohn Baldwin  * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
2328bf1fadSJohn Baldwin  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
2428bf1fadSJohn Baldwin  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
2528bf1fadSJohn Baldwin  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
2628bf1fadSJohn Baldwin  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
2728bf1fadSJohn Baldwin  */
2828bf1fadSJohn Baldwin 
29aad970f1SDavid E. O'Brien #include <sys/cdefs.h>
30aad970f1SDavid E. O'Brien __FBSDID("$FreeBSD$");
31aad970f1SDavid E. O'Brien 
3228bf1fadSJohn Baldwin #include <sys/param.h>
3328bf1fadSJohn Baldwin #include <sys/systm.h>
3428bf1fadSJohn Baldwin #include <sys/bus.h>
3528bf1fadSJohn Baldwin #include <sys/kernel.h>
3628bf1fadSJohn Baldwin #include <sys/malloc.h>
3728bf1fadSJohn Baldwin #include <sys/module.h>
3828bf1fadSJohn Baldwin 
39129d3046SJung-uk Kim #include <contrib/dev/acpica/include/acpi.h>
40129d3046SJung-uk Kim #include <contrib/dev/acpica/include/accommon.h>
41129d3046SJung-uk Kim 
4228bf1fadSJohn Baldwin #include <dev/acpica/acpivar.h>
4328bf1fadSJohn Baldwin 
4428bf1fadSJohn Baldwin #include <sys/pciio.h>
4528bf1fadSJohn Baldwin #include <dev/pci/pcireg.h>
4628bf1fadSJohn Baldwin #include <dev/pci/pcivar.h>
4728bf1fadSJohn Baldwin #include <dev/pci/pci_private.h>
4828bf1fadSJohn Baldwin 
4928bf1fadSJohn Baldwin #include "pcib_if.h"
5028bf1fadSJohn Baldwin #include "pci_if.h"
5128bf1fadSJohn Baldwin 
525acd0218SNate Lawson /* Hooks for the ACPI CA debugging infrastructure. */
5328bf1fadSJohn Baldwin #define _COMPONENT	ACPI_BUS
5428bf1fadSJohn Baldwin ACPI_MODULE_NAME("PCI")
5528bf1fadSJohn Baldwin 
5628bf1fadSJohn Baldwin struct acpi_pci_devinfo {
5728bf1fadSJohn Baldwin 	struct pci_devinfo	ap_dinfo;
5828bf1fadSJohn Baldwin 	ACPI_HANDLE		ap_handle;
59d4b9ff91SNate Lawson 	int			ap_flags;
6028bf1fadSJohn Baldwin };
6128bf1fadSJohn Baldwin 
62f4e6d08dSNate Lawson ACPI_SERIAL_DECL(pci_powerstate, "ACPI PCI power methods");
63f4e6d08dSNate Lawson 
64a811035eSNate Lawson /* Be sure that ACPI and PCI power states are equivalent. */
65a811035eSNate Lawson CTASSERT(ACPI_STATE_D0 == PCI_POWERSTATE_D0);
66a811035eSNate Lawson CTASSERT(ACPI_STATE_D1 == PCI_POWERSTATE_D1);
67a811035eSNate Lawson CTASSERT(ACPI_STATE_D2 == PCI_POWERSTATE_D2);
68a811035eSNate Lawson CTASSERT(ACPI_STATE_D3 == PCI_POWERSTATE_D3);
69a811035eSNate Lawson 
7028bf1fadSJohn Baldwin static int	acpi_pci_attach(device_t dev);
71cd284d7aSTakanori Watanabe static int	acpi_pci_child_location_str_method(device_t cbdev,
72cd284d7aSTakanori Watanabe 		    device_t child, char *buf, size_t buflen);
7381bb99d2SJohn Baldwin static int	acpi_pci_probe(device_t dev);
7481bb99d2SJohn Baldwin static int	acpi_pci_read_ivar(device_t dev, device_t child, int which,
7581bb99d2SJohn Baldwin 		    uintptr_t *result);
76d4b9ff91SNate Lawson static int	acpi_pci_write_ivar(device_t dev, device_t child, int which,
77d4b9ff91SNate Lawson 		    uintptr_t value);
7828bf1fadSJohn Baldwin static ACPI_STATUS acpi_pci_save_handle(ACPI_HANDLE handle, UINT32 level,
7928bf1fadSJohn Baldwin 		    void *context, void **status);
8081bb99d2SJohn Baldwin static int	acpi_pci_set_powerstate_method(device_t dev, device_t child,
8181bb99d2SJohn Baldwin 		    int state);
8281bb99d2SJohn Baldwin static void	acpi_pci_update_device(ACPI_HANDLE handle, device_t pci_child);
8328bf1fadSJohn Baldwin 
8428bf1fadSJohn Baldwin static device_method_t acpi_pci_methods[] = {
8528bf1fadSJohn Baldwin 	/* Device interface */
8628bf1fadSJohn Baldwin 	DEVMETHOD(device_probe,		acpi_pci_probe),
8728bf1fadSJohn Baldwin 	DEVMETHOD(device_attach,	acpi_pci_attach),
8828bf1fadSJohn Baldwin 
8928bf1fadSJohn Baldwin 	/* Bus interface */
9028bf1fadSJohn Baldwin 	DEVMETHOD(bus_read_ivar,	acpi_pci_read_ivar),
91d4b9ff91SNate Lawson 	DEVMETHOD(bus_write_ivar,	acpi_pci_write_ivar),
92cd284d7aSTakanori Watanabe 	DEVMETHOD(bus_child_location_str, acpi_pci_child_location_str_method),
9328bf1fadSJohn Baldwin 
9428bf1fadSJohn Baldwin 	/* PCI interface */
955233e9ffSJohn Baldwin 	DEVMETHOD(pci_set_powerstate,	acpi_pci_set_powerstate_method),
9628bf1fadSJohn Baldwin 
9728bf1fadSJohn Baldwin 	{ 0, 0 }
9828bf1fadSJohn Baldwin };
9928bf1fadSJohn Baldwin 
10014450110SJohn Baldwin static devclass_t pci_devclass;
10128bf1fadSJohn Baldwin 
10214450110SJohn Baldwin DEFINE_CLASS_1(pci, acpi_pci_driver, acpi_pci_methods, 0, pci_driver);
10328bf1fadSJohn Baldwin DRIVER_MODULE(acpi_pci, pcib, acpi_pci_driver, pci_devclass, 0, 0);
10464278df5SNate Lawson MODULE_DEPEND(acpi_pci, acpi, 1, 1, 1);
10528bf1fadSJohn Baldwin MODULE_DEPEND(acpi_pci, pci, 1, 1, 1);
10664278df5SNate Lawson MODULE_VERSION(acpi_pci, 1);
10728bf1fadSJohn Baldwin 
10828bf1fadSJohn Baldwin static int
10928bf1fadSJohn Baldwin acpi_pci_read_ivar(device_t dev, device_t child, int which, uintptr_t *result)
11028bf1fadSJohn Baldwin {
11128bf1fadSJohn Baldwin     struct acpi_pci_devinfo *dinfo;
11228bf1fadSJohn Baldwin 
113d4b9ff91SNate Lawson     dinfo = device_get_ivars(child);
11428bf1fadSJohn Baldwin     switch (which) {
11528bf1fadSJohn Baldwin     case ACPI_IVAR_HANDLE:
11628bf1fadSJohn Baldwin 	*result = (uintptr_t)dinfo->ap_handle;
11728bf1fadSJohn Baldwin 	return (0);
118d4b9ff91SNate Lawson     case ACPI_IVAR_FLAGS:
119d4b9ff91SNate Lawson 	*result = (uintptr_t)dinfo->ap_flags;
120d4b9ff91SNate Lawson 	return (0);
12128bf1fadSJohn Baldwin     }
12228bf1fadSJohn Baldwin     return (pci_read_ivar(dev, child, which, result));
12328bf1fadSJohn Baldwin }
12428bf1fadSJohn Baldwin 
125cd284d7aSTakanori Watanabe static int
126d4b9ff91SNate Lawson acpi_pci_write_ivar(device_t dev, device_t child, int which, uintptr_t value)
127d4b9ff91SNate Lawson {
128d4b9ff91SNate Lawson     struct acpi_pci_devinfo *dinfo;
129d4b9ff91SNate Lawson 
130d4b9ff91SNate Lawson     dinfo = device_get_ivars(child);
131d4b9ff91SNate Lawson     switch (which) {
132d4b9ff91SNate Lawson     case ACPI_IVAR_HANDLE:
133d4b9ff91SNate Lawson 	dinfo->ap_handle = (ACPI_HANDLE)value;
134d4b9ff91SNate Lawson 	return (0);
135d4b9ff91SNate Lawson     case ACPI_IVAR_FLAGS:
136d4b9ff91SNate Lawson 	dinfo->ap_flags = (int)value;
137d4b9ff91SNate Lawson 	return (0);
138d4b9ff91SNate Lawson     }
139d4b9ff91SNate Lawson     return (pci_write_ivar(dev, child, which, value));
140d4b9ff91SNate Lawson }
141d4b9ff91SNate Lawson 
142d4b9ff91SNate Lawson static int
143cd284d7aSTakanori Watanabe acpi_pci_child_location_str_method(device_t cbdev, device_t child, char *buf,
144cd284d7aSTakanori Watanabe     size_t buflen)
145cd284d7aSTakanori Watanabe {
146cd284d7aSTakanori Watanabe     struct acpi_pci_devinfo *dinfo = device_get_ivars(child);
14763600cc3SNate Lawson 
148cd284d7aSTakanori Watanabe     pci_child_location_str_method(cbdev, child, buf, buflen);
149cd284d7aSTakanori Watanabe 
150cd284d7aSTakanori Watanabe     if (dinfo->ap_handle) {
1518c4c207dSTakanori Watanabe 	strlcat(buf, " handle=", buflen);
152cd284d7aSTakanori Watanabe 	strlcat(buf, acpi_name(dinfo->ap_handle), buflen);
153cd284d7aSTakanori Watanabe     }
154cd284d7aSTakanori Watanabe     return (0);
155cd284d7aSTakanori Watanabe }
156cd284d7aSTakanori Watanabe 
15728bf1fadSJohn Baldwin /*
15828bf1fadSJohn Baldwin  * PCI power manangement
15928bf1fadSJohn Baldwin  */
16028bf1fadSJohn Baldwin static int
16128bf1fadSJohn Baldwin acpi_pci_set_powerstate_method(device_t dev, device_t child, int state)
16228bf1fadSJohn Baldwin {
163916dc0e2SNate Lawson 	ACPI_HANDLE h;
1645233e9ffSJohn Baldwin 	ACPI_STATUS status;
165a811035eSNate Lawson 	int old_state, error;
1665233e9ffSJohn Baldwin 
167f4e6d08dSNate Lawson 	error = 0;
168a811035eSNate Lawson 	if (state < ACPI_STATE_D0 || state > ACPI_STATE_D3)
1695233e9ffSJohn Baldwin 		return (EINVAL);
17028bf1fadSJohn Baldwin 
1715233e9ffSJohn Baldwin 	/*
1725233e9ffSJohn Baldwin 	 * We set the state using PCI Power Management outside of setting
1735233e9ffSJohn Baldwin 	 * the ACPI state.  This means that when powering down a device, we
1745233e9ffSJohn Baldwin 	 * first shut it down using PCI, and then using ACPI, which lets ACPI
1755233e9ffSJohn Baldwin 	 * try to power down any Power Resources that are now no longer used.
1765233e9ffSJohn Baldwin 	 * When powering up a device, we let ACPI set the state first so that
1775233e9ffSJohn Baldwin 	 * it can enable any needed Power Resources before changing the PCI
1785233e9ffSJohn Baldwin 	 * power state.
1795233e9ffSJohn Baldwin 	 */
180f4e6d08dSNate Lawson 	ACPI_SERIAL_BEGIN(pci_powerstate);
1815233e9ffSJohn Baldwin 	old_state = pci_get_powerstate(child);
1825233e9ffSJohn Baldwin 	if (old_state < state) {
1835233e9ffSJohn Baldwin 		error = pci_set_powerstate_method(dev, child, state);
1845233e9ffSJohn Baldwin 		if (error)
185f4e6d08dSNate Lawson 			goto out;
18628bf1fadSJohn Baldwin 	}
1871f36889fSNate Lawson 	h = acpi_get_handle(child);
188a811035eSNate Lawson 	status = acpi_pwr_switch_consumer(h, state);
189*a7a3177fSJung-uk Kim 	if (ACPI_SUCCESS(status)) {
190*a7a3177fSJung-uk Kim 		if (bootverbose)
191*a7a3177fSJung-uk Kim 			device_printf(dev, "set ACPI power state D%d on %s\n",
192*a7a3177fSJung-uk Kim 			    state, acpi_name(h));
193*a7a3177fSJung-uk Kim 	} else if (status != AE_NOT_FOUND)
1945233e9ffSJohn Baldwin 		device_printf(dev,
195*a7a3177fSJung-uk Kim 		    "failed to set ACPI power state D%d on %s: %s\n",
196a811035eSNate Lawson 		    state, acpi_name(h), AcpiFormatException(status));
197edc0cb7dSJung-uk Kim 	if (old_state > state && pci_do_power_resume)
198f4e6d08dSNate Lawson 		error = pci_set_powerstate_method(dev, child, state);
199f4e6d08dSNate Lawson 
200f4e6d08dSNate Lawson out:
201f4e6d08dSNate Lawson 	ACPI_SERIAL_END(pci_powerstate);
202f4e6d08dSNate Lawson 	return (error);
2035233e9ffSJohn Baldwin }
20428bf1fadSJohn Baldwin 
20581bb99d2SJohn Baldwin static void
20681bb99d2SJohn Baldwin acpi_pci_update_device(ACPI_HANDLE handle, device_t pci_child)
20781bb99d2SJohn Baldwin {
20881bb99d2SJohn Baldwin 	ACPI_STATUS status;
20981bb99d2SJohn Baldwin 	device_t child;
21081bb99d2SJohn Baldwin 
21181bb99d2SJohn Baldwin 	/*
21281bb99d2SJohn Baldwin 	 * Lookup and remove the unused device that acpi0 creates when it walks
21381bb99d2SJohn Baldwin 	 * the namespace creating devices.
21481bb99d2SJohn Baldwin 	 */
21581bb99d2SJohn Baldwin 	child = acpi_get_device(handle);
21681bb99d2SJohn Baldwin 	if (child != NULL) {
2173052da8bSJohn Baldwin 		if (device_is_alive(child)) {
2183052da8bSJohn Baldwin 			/*
2193052da8bSJohn Baldwin 			 * The TabletPC TC1000 has a second PCI-ISA bridge
2203052da8bSJohn Baldwin 			 * that has a _HID for an acpi_sysresource device.
2213052da8bSJohn Baldwin 			 * In that case, leave ACPI-CA's device data pointing
2223052da8bSJohn Baldwin 			 * at the ACPI-enumerated device.
2233052da8bSJohn Baldwin 			 */
2243052da8bSJohn Baldwin 			device_printf(child,
2253052da8bSJohn Baldwin 			    "Conflicts with PCI device %d:%d:%d\n",
2263052da8bSJohn Baldwin 			    pci_get_bus(pci_child), pci_get_slot(pci_child),
2273052da8bSJohn Baldwin 			    pci_get_function(pci_child));
2283052da8bSJohn Baldwin 			return;
2293052da8bSJohn Baldwin 		}
23081bb99d2SJohn Baldwin 		KASSERT(device_get_parent(child) ==
23181bb99d2SJohn Baldwin 		    devclass_get_device(devclass_find("acpi"), 0),
23281bb99d2SJohn Baldwin 		    ("%s: child (%s)'s parent is not acpi0", __func__,
23381bb99d2SJohn Baldwin 		    acpi_name(handle)));
23481bb99d2SJohn Baldwin 		device_delete_child(device_get_parent(child), child);
23581bb99d2SJohn Baldwin 	}
23681bb99d2SJohn Baldwin 
23781bb99d2SJohn Baldwin 	/*
23881bb99d2SJohn Baldwin 	 * Update ACPI-CA to use the PCI enumerated device_t for this handle.
23981bb99d2SJohn Baldwin 	 */
24081bb99d2SJohn Baldwin 	status = AcpiDetachData(handle, acpi_fake_objhandler);
24181bb99d2SJohn Baldwin 	if (ACPI_FAILURE(status))
24281bb99d2SJohn Baldwin 		printf("WARNING: Unable to detach object data from %s - %s\n",
24381bb99d2SJohn Baldwin 		    acpi_name(handle), AcpiFormatException(status));
24430d577a0SNate Lawson 	status = AcpiAttachData(handle, acpi_fake_objhandler, pci_child);
24581bb99d2SJohn Baldwin 	if (ACPI_FAILURE(status))
24681bb99d2SJohn Baldwin 		printf("WARNING: Unable to attach object data to %s - %s\n",
24781bb99d2SJohn Baldwin 		    acpi_name(handle), AcpiFormatException(status));
24881bb99d2SJohn Baldwin }
24981bb99d2SJohn Baldwin 
25028bf1fadSJohn Baldwin static ACPI_STATUS
25128bf1fadSJohn Baldwin acpi_pci_save_handle(ACPI_HANDLE handle, UINT32 level, void *context,
25228bf1fadSJohn Baldwin     void **status)
25328bf1fadSJohn Baldwin {
25428bf1fadSJohn Baldwin 	struct acpi_pci_devinfo *dinfo;
25528bf1fadSJohn Baldwin 	device_t *devlist;
25628bf1fadSJohn Baldwin 	int devcount, i, func, slot;
25728bf1fadSJohn Baldwin 	UINT32 address;
25828bf1fadSJohn Baldwin 
25928bf1fadSJohn Baldwin 	ACPI_FUNCTION_TRACE((char *)(uintptr_t)__func__);
26028bf1fadSJohn Baldwin 
261c310653eSNate Lawson 	if (ACPI_FAILURE(acpi_GetInteger(handle, "_ADR", &address)))
26228bf1fadSJohn Baldwin 		return_ACPI_STATUS (AE_OK);
263e0a93586SJohn Baldwin 	slot = ACPI_ADR_PCI_SLOT(address);
264e0a93586SJohn Baldwin 	func = ACPI_ADR_PCI_FUNC(address);
26528bf1fadSJohn Baldwin 	if (device_get_children((device_t)context, &devlist, &devcount) != 0)
26628bf1fadSJohn Baldwin 		return_ACPI_STATUS (AE_OK);
26728bf1fadSJohn Baldwin 	for (i = 0; i < devcount; i++) {
26828bf1fadSJohn Baldwin 		dinfo = device_get_ivars(devlist[i]);
26928bf1fadSJohn Baldwin 		if (dinfo->ap_dinfo.cfg.func == func &&
27028bf1fadSJohn Baldwin 		    dinfo->ap_dinfo.cfg.slot == slot) {
27128bf1fadSJohn Baldwin 			dinfo->ap_handle = handle;
27281bb99d2SJohn Baldwin 			acpi_pci_update_device(handle, devlist[i]);
27330d577a0SNate Lawson 			break;
27428bf1fadSJohn Baldwin 		}
27528bf1fadSJohn Baldwin 	}
27628bf1fadSJohn Baldwin 	free(devlist, M_TEMP);
27728bf1fadSJohn Baldwin 	return_ACPI_STATUS (AE_OK);
27828bf1fadSJohn Baldwin }
27928bf1fadSJohn Baldwin 
28028bf1fadSJohn Baldwin static int
28128bf1fadSJohn Baldwin acpi_pci_probe(device_t dev)
28228bf1fadSJohn Baldwin {
28328bf1fadSJohn Baldwin 
28428bf1fadSJohn Baldwin 	if (pcib_get_bus(dev) < 0)
28528bf1fadSJohn Baldwin 		return (ENXIO);
28628bf1fadSJohn Baldwin 	if (acpi_get_handle(dev) == NULL)
28728bf1fadSJohn Baldwin 		return (ENXIO);
2885acd0218SNate Lawson 	device_set_desc(dev, "ACPI PCI bus");
28928bf1fadSJohn Baldwin 	return (0);
29028bf1fadSJohn Baldwin }
29128bf1fadSJohn Baldwin 
29228bf1fadSJohn Baldwin static int
29328bf1fadSJohn Baldwin acpi_pci_attach(device_t dev)
29428bf1fadSJohn Baldwin {
29555aaf894SMarius Strobl 	int busno, domain;
29628bf1fadSJohn Baldwin 
29728bf1fadSJohn Baldwin 	/*
29828bf1fadSJohn Baldwin 	 * Since there can be multiple independantly numbered PCI
29970bc2d3fSJohn Baldwin 	 * busses on systems with multiple PCI domains, we can't use
30070bc2d3fSJohn Baldwin 	 * the unit number to decide which bus we are probing. We ask
30155aaf894SMarius Strobl 	 * the parent pcib what our domain and bus numbers are.
30228bf1fadSJohn Baldwin 	 */
30355aaf894SMarius Strobl 	domain = pcib_get_domain(dev);
30428bf1fadSJohn Baldwin 	busno = pcib_get_bus(dev);
30528bf1fadSJohn Baldwin 	if (bootverbose)
30655aaf894SMarius Strobl 		device_printf(dev, "domain=%d, physical bus=%d\n",
30755aaf894SMarius Strobl 		    domain, busno);
30828bf1fadSJohn Baldwin 
30928bf1fadSJohn Baldwin 	/*
31028bf1fadSJohn Baldwin 	 * First, PCI devices are added as in the normal PCI bus driver.
31128bf1fadSJohn Baldwin 	 * Afterwards, the ACPI namespace under the bridge driver is
31228bf1fadSJohn Baldwin 	 * walked to save ACPI handles to all the devices that appear in
31328bf1fadSJohn Baldwin 	 * the ACPI namespace as immediate descendants of the bridge.
31428bf1fadSJohn Baldwin 	 *
31528bf1fadSJohn Baldwin 	 * XXX: Sometimes PCI devices show up in the ACPI namespace that
31628bf1fadSJohn Baldwin 	 * pci_add_children() doesn't find.  We currently just ignore
31728bf1fadSJohn Baldwin 	 * these devices.
31828bf1fadSJohn Baldwin 	 */
31955aaf894SMarius Strobl 	pci_add_children(dev, domain, busno, sizeof(struct acpi_pci_devinfo));
3205acd0218SNate Lawson 	AcpiWalkNamespace(ACPI_TYPE_DEVICE, acpi_get_handle(dev), 1,
3212272d050SJung-uk Kim 	    acpi_pci_save_handle, NULL, dev, NULL);
32228bf1fadSJohn Baldwin 
32328bf1fadSJohn Baldwin 	return (bus_generic_attach(dev));
32428bf1fadSJohn Baldwin }
325