xref: /freebsd/sys/dev/acpica/acpi_pci.c (revision 30d577a014c96b5db1e22912ac996db31f6a4c31)
128bf1fadSJohn Baldwin /*
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 
3928bf1fadSJohn Baldwin #include "acpi.h"
4028bf1fadSJohn Baldwin #include <dev/acpica/acpivar.h>
4128bf1fadSJohn Baldwin 
4228bf1fadSJohn Baldwin #include <sys/pciio.h>
4328bf1fadSJohn Baldwin #include <dev/pci/pcireg.h>
4428bf1fadSJohn Baldwin #include <dev/pci/pcivar.h>
4528bf1fadSJohn Baldwin #include <dev/pci/pci_private.h>
4628bf1fadSJohn Baldwin 
4728bf1fadSJohn Baldwin #include "pcib_if.h"
4828bf1fadSJohn Baldwin #include "pci_if.h"
4928bf1fadSJohn Baldwin 
505acd0218SNate Lawson /* Hooks for the ACPI CA debugging infrastructure. */
5128bf1fadSJohn Baldwin #define _COMPONENT	ACPI_BUS
5228bf1fadSJohn Baldwin ACPI_MODULE_NAME("PCI")
5328bf1fadSJohn Baldwin 
5428bf1fadSJohn Baldwin struct acpi_pci_devinfo {
5528bf1fadSJohn Baldwin 	struct pci_devinfo	ap_dinfo;
5628bf1fadSJohn Baldwin 	ACPI_HANDLE		ap_handle;
5728bf1fadSJohn Baldwin };
5828bf1fadSJohn Baldwin 
5928bf1fadSJohn Baldwin static int	acpi_pci_attach(device_t dev);
60cd284d7aSTakanori Watanabe static int	acpi_pci_child_location_str_method(device_t cbdev,
61cd284d7aSTakanori Watanabe 		    device_t child, char *buf, size_t buflen);
6281bb99d2SJohn Baldwin static int	acpi_pci_probe(device_t dev);
6381bb99d2SJohn Baldwin static int	acpi_pci_read_ivar(device_t dev, device_t child, int which,
6481bb99d2SJohn Baldwin 		    uintptr_t *result);
6528bf1fadSJohn Baldwin static ACPI_STATUS acpi_pci_save_handle(ACPI_HANDLE handle, UINT32 level,
6628bf1fadSJohn Baldwin 		    void *context, void **status);
6781bb99d2SJohn Baldwin static int	acpi_pci_set_powerstate_method(device_t dev, device_t child,
6881bb99d2SJohn Baldwin 		    int state);
6981bb99d2SJohn Baldwin static void	acpi_pci_update_device(ACPI_HANDLE handle, device_t pci_child);
7028bf1fadSJohn Baldwin 
7128bf1fadSJohn Baldwin static device_method_t acpi_pci_methods[] = {
7228bf1fadSJohn Baldwin 	/* Device interface */
7328bf1fadSJohn Baldwin 	DEVMETHOD(device_probe,		acpi_pci_probe),
7428bf1fadSJohn Baldwin 	DEVMETHOD(device_attach,	acpi_pci_attach),
7528bf1fadSJohn Baldwin 	DEVMETHOD(device_shutdown,	bus_generic_shutdown),
76cd8b53edSWarner Losh 	DEVMETHOD(device_suspend,	pci_suspend),
77526b5e65SMitsuru IWASAKI 	DEVMETHOD(device_resume,	pci_resume),
7828bf1fadSJohn Baldwin 
7928bf1fadSJohn Baldwin 	/* Bus interface */
8028bf1fadSJohn Baldwin 	DEVMETHOD(bus_print_child,	pci_print_child),
8128bf1fadSJohn Baldwin 	DEVMETHOD(bus_probe_nomatch,	pci_probe_nomatch),
8228bf1fadSJohn Baldwin 	DEVMETHOD(bus_read_ivar,	acpi_pci_read_ivar),
8328bf1fadSJohn Baldwin 	DEVMETHOD(bus_write_ivar,	pci_write_ivar),
84cd8b53edSWarner Losh 	DEVMETHOD(bus_driver_added,	pci_driver_added),
8528bf1fadSJohn Baldwin 	DEVMETHOD(bus_setup_intr,	bus_generic_setup_intr),
8628bf1fadSJohn Baldwin 	DEVMETHOD(bus_teardown_intr,	bus_generic_teardown_intr),
8728bf1fadSJohn Baldwin 
8828bf1fadSJohn Baldwin 	DEVMETHOD(bus_get_resource_list,pci_get_resource_list),
8928bf1fadSJohn Baldwin 	DEVMETHOD(bus_set_resource,	bus_generic_rl_set_resource),
9028bf1fadSJohn Baldwin 	DEVMETHOD(bus_get_resource,	bus_generic_rl_get_resource),
9128bf1fadSJohn Baldwin 	DEVMETHOD(bus_delete_resource,	pci_delete_resource),
9228bf1fadSJohn Baldwin 	DEVMETHOD(bus_alloc_resource,	pci_alloc_resource),
9328bf1fadSJohn Baldwin 	DEVMETHOD(bus_release_resource,	bus_generic_rl_release_resource),
9428bf1fadSJohn Baldwin 	DEVMETHOD(bus_activate_resource, bus_generic_activate_resource),
9528bf1fadSJohn Baldwin 	DEVMETHOD(bus_deactivate_resource, bus_generic_deactivate_resource),
965794c593SWarner Losh 	DEVMETHOD(bus_child_pnpinfo_str, pci_child_pnpinfo_str_method),
97cd284d7aSTakanori Watanabe 	DEVMETHOD(bus_child_location_str, acpi_pci_child_location_str_method),
9828bf1fadSJohn Baldwin 
9928bf1fadSJohn Baldwin 	/* PCI interface */
10028bf1fadSJohn Baldwin 	DEVMETHOD(pci_read_config,	pci_read_config_method),
10128bf1fadSJohn Baldwin 	DEVMETHOD(pci_write_config,	pci_write_config_method),
10228bf1fadSJohn Baldwin 	DEVMETHOD(pci_enable_busmaster,	pci_enable_busmaster_method),
10328bf1fadSJohn Baldwin 	DEVMETHOD(pci_disable_busmaster, pci_disable_busmaster_method),
10428bf1fadSJohn Baldwin 	DEVMETHOD(pci_enable_io,	pci_enable_io_method),
10528bf1fadSJohn Baldwin 	DEVMETHOD(pci_disable_io,	pci_disable_io_method),
10628bf1fadSJohn Baldwin 	DEVMETHOD(pci_get_powerstate,	pci_get_powerstate_method),
1075233e9ffSJohn Baldwin 	DEVMETHOD(pci_set_powerstate,	acpi_pci_set_powerstate_method),
1083920999dSThomas Moestl 	DEVMETHOD(pci_assign_interrupt, pci_assign_interrupt_method),
10928bf1fadSJohn Baldwin 
11028bf1fadSJohn Baldwin 	{ 0, 0 }
11128bf1fadSJohn Baldwin };
11228bf1fadSJohn Baldwin 
11328bf1fadSJohn Baldwin static driver_t acpi_pci_driver = {
11428bf1fadSJohn Baldwin 	"pci",
11528bf1fadSJohn Baldwin 	acpi_pci_methods,
11628bf1fadSJohn Baldwin 	0,			/* no softc */
11728bf1fadSJohn Baldwin };
11828bf1fadSJohn Baldwin 
11928bf1fadSJohn Baldwin DRIVER_MODULE(acpi_pci, pcib, acpi_pci_driver, pci_devclass, 0, 0);
12064278df5SNate Lawson MODULE_DEPEND(acpi_pci, acpi, 1, 1, 1);
12128bf1fadSJohn Baldwin MODULE_DEPEND(acpi_pci, pci, 1, 1, 1);
12264278df5SNate Lawson MODULE_VERSION(acpi_pci, 1);
12328bf1fadSJohn Baldwin 
12428bf1fadSJohn Baldwin static int
12528bf1fadSJohn Baldwin acpi_pci_read_ivar(device_t dev, device_t child, int which, uintptr_t *result)
12628bf1fadSJohn Baldwin {
12728bf1fadSJohn Baldwin     struct acpi_pci_devinfo *dinfo;
12828bf1fadSJohn Baldwin 
12928bf1fadSJohn Baldwin     switch (which) {
13028bf1fadSJohn Baldwin     case ACPI_IVAR_HANDLE:
13128bf1fadSJohn Baldwin 	dinfo = device_get_ivars(child);
13228bf1fadSJohn Baldwin 	*result = (uintptr_t)dinfo->ap_handle;
13328bf1fadSJohn Baldwin 	return (0);
13428bf1fadSJohn Baldwin     }
13528bf1fadSJohn Baldwin     return (pci_read_ivar(dev, child, which, result));
13628bf1fadSJohn Baldwin }
13728bf1fadSJohn Baldwin 
138cd284d7aSTakanori Watanabe static int
139cd284d7aSTakanori Watanabe acpi_pci_child_location_str_method(device_t cbdev, device_t child, char *buf,
140cd284d7aSTakanori Watanabe     size_t buflen)
141cd284d7aSTakanori Watanabe {
142cd284d7aSTakanori Watanabe     struct acpi_pci_devinfo *dinfo = device_get_ivars(child);
14363600cc3SNate Lawson 
144cd284d7aSTakanori Watanabe     pci_child_location_str_method(cbdev, child, buf, buflen);
145cd284d7aSTakanori Watanabe 
146cd284d7aSTakanori Watanabe     if (dinfo->ap_handle) {
147cd284d7aSTakanori Watanabe 	strlcat(buf, " path=", buflen);
148cd284d7aSTakanori Watanabe 	strlcat(buf, acpi_name(dinfo->ap_handle), buflen);
149cd284d7aSTakanori Watanabe     }
150cd284d7aSTakanori Watanabe     return (0);
151cd284d7aSTakanori Watanabe }
152cd284d7aSTakanori Watanabe 
15328bf1fadSJohn Baldwin /*
15428bf1fadSJohn Baldwin  * PCI power manangement
15528bf1fadSJohn Baldwin  */
15628bf1fadSJohn Baldwin static int
15728bf1fadSJohn Baldwin acpi_pci_set_powerstate_method(device_t dev, device_t child, int state)
15828bf1fadSJohn Baldwin {
159916dc0e2SNate Lawson 	ACPI_HANDLE h;
1605233e9ffSJohn Baldwin 	ACPI_STATUS status;
1615233e9ffSJohn Baldwin 	int acpi_state, old_state, error;
1625233e9ffSJohn Baldwin 
1635233e9ffSJohn Baldwin 	switch (state) {
1645233e9ffSJohn Baldwin 	case PCI_POWERSTATE_D0:
1655233e9ffSJohn Baldwin 		acpi_state = ACPI_STATE_D0;
1665233e9ffSJohn Baldwin 		break;
1675233e9ffSJohn Baldwin 	case PCI_POWERSTATE_D1:
1685233e9ffSJohn Baldwin 		acpi_state = ACPI_STATE_D1;
1695233e9ffSJohn Baldwin 		break;
1705233e9ffSJohn Baldwin 	case PCI_POWERSTATE_D2:
1715233e9ffSJohn Baldwin 		acpi_state = ACPI_STATE_D2;
1725233e9ffSJohn Baldwin 		break;
1735233e9ffSJohn Baldwin 	case PCI_POWERSTATE_D3:
1745233e9ffSJohn Baldwin 		acpi_state = ACPI_STATE_D3;
1755233e9ffSJohn Baldwin 		break;
1765233e9ffSJohn Baldwin 	default:
1775233e9ffSJohn Baldwin 		return (EINVAL);
17828bf1fadSJohn Baldwin 	}
17928bf1fadSJohn Baldwin 
1805233e9ffSJohn Baldwin 	/*
1815233e9ffSJohn Baldwin 	 * We set the state using PCI Power Management outside of setting
1825233e9ffSJohn Baldwin 	 * the ACPI state.  This means that when powering down a device, we
1835233e9ffSJohn Baldwin 	 * first shut it down using PCI, and then using ACPI, which lets ACPI
1845233e9ffSJohn Baldwin 	 * try to power down any Power Resources that are now no longer used.
1855233e9ffSJohn Baldwin 	 * When powering up a device, we let ACPI set the state first so that
1865233e9ffSJohn Baldwin 	 * it can enable any needed Power Resources before changing the PCI
1875233e9ffSJohn Baldwin 	 * power state.
1885233e9ffSJohn Baldwin 	 */
1895233e9ffSJohn Baldwin 	old_state = pci_get_powerstate(child);
1905233e9ffSJohn Baldwin 	if (old_state < state) {
1915233e9ffSJohn Baldwin 		error = pci_set_powerstate_method(dev, child, state);
1925233e9ffSJohn Baldwin 		if (error)
1935233e9ffSJohn Baldwin 			return (error);
19428bf1fadSJohn Baldwin 	}
1951f36889fSNate Lawson 	h = acpi_get_handle(child);
196916dc0e2SNate Lawson 	status = acpi_pwr_switch_consumer(h, acpi_state);
19745e556ccSNate Lawson 	if (ACPI_FAILURE(status) && status != AE_NOT_FOUND)
1985233e9ffSJohn Baldwin 		device_printf(dev,
1995233e9ffSJohn Baldwin 		    "Failed to set ACPI power state D%d on %s: %s\n",
20045e556ccSNate Lawson 		    acpi_state, acpi_name(h), AcpiFormatException(status));
2011c168bb7SWarner Losh 	if (old_state > state)
2025233e9ffSJohn Baldwin 		return (pci_set_powerstate_method(dev, child, state));
2035233e9ffSJohn Baldwin 	else
2045233e9ffSJohn Baldwin 		return (0);
2055233e9ffSJohn Baldwin }
20628bf1fadSJohn Baldwin 
20781bb99d2SJohn Baldwin static void
20881bb99d2SJohn Baldwin acpi_pci_update_device(ACPI_HANDLE handle, device_t pci_child)
20981bb99d2SJohn Baldwin {
21081bb99d2SJohn Baldwin 	ACPI_STATUS status;
21181bb99d2SJohn Baldwin 	device_t child;
21281bb99d2SJohn Baldwin 
21381bb99d2SJohn Baldwin 	/*
21481bb99d2SJohn Baldwin 	 * Lookup and remove the unused device that acpi0 creates when it walks
21581bb99d2SJohn Baldwin 	 * the namespace creating devices.
21681bb99d2SJohn Baldwin 	 */
21781bb99d2SJohn Baldwin 	child = acpi_get_device(handle);
21881bb99d2SJohn Baldwin 	if (child != NULL) {
21981bb99d2SJohn Baldwin 		KASSERT(!device_is_alive(child), ("%s: deleting alive child %s",
22081bb99d2SJohn Baldwin 		    __func__, device_get_nameunit(child)));
22181bb99d2SJohn Baldwin 		KASSERT(device_get_parent(child) ==
22281bb99d2SJohn Baldwin 		    devclass_get_device(devclass_find("acpi"), 0),
22381bb99d2SJohn Baldwin 		    ("%s: child (%s)'s parent is not acpi0", __func__,
22481bb99d2SJohn Baldwin 		    acpi_name(handle)));
22581bb99d2SJohn Baldwin 		device_delete_child(device_get_parent(child), child);
22681bb99d2SJohn Baldwin 	}
22781bb99d2SJohn Baldwin 
22881bb99d2SJohn Baldwin 	/*
22981bb99d2SJohn Baldwin 	 * Update ACPI-CA to use the PCI enumerated device_t for this handle.
23081bb99d2SJohn Baldwin 	 */
23181bb99d2SJohn Baldwin 	status = AcpiDetachData(handle, acpi_fake_objhandler);
23281bb99d2SJohn Baldwin 	if (ACPI_FAILURE(status))
23381bb99d2SJohn Baldwin 		printf("WARNING: Unable to detach object data from %s - %s\n",
23481bb99d2SJohn Baldwin 		    acpi_name(handle), AcpiFormatException(status));
23530d577a0SNate Lawson 	status = AcpiAttachData(handle, acpi_fake_objhandler, pci_child);
23681bb99d2SJohn Baldwin 	if (ACPI_FAILURE(status))
23781bb99d2SJohn Baldwin 		printf("WARNING: Unable to attach object data to %s - %s\n",
23881bb99d2SJohn Baldwin 		    acpi_name(handle), AcpiFormatException(status));
23981bb99d2SJohn Baldwin }
24081bb99d2SJohn Baldwin 
24128bf1fadSJohn Baldwin static ACPI_STATUS
24228bf1fadSJohn Baldwin acpi_pci_save_handle(ACPI_HANDLE handle, UINT32 level, void *context,
24328bf1fadSJohn Baldwin     void **status)
24428bf1fadSJohn Baldwin {
24528bf1fadSJohn Baldwin 	struct acpi_pci_devinfo *dinfo;
24628bf1fadSJohn Baldwin 	device_t *devlist;
24728bf1fadSJohn Baldwin 	int devcount, i, func, slot;
24828bf1fadSJohn Baldwin 	UINT32 address;
24928bf1fadSJohn Baldwin 
25028bf1fadSJohn Baldwin 	ACPI_FUNCTION_TRACE((char *)(uintptr_t)__func__);
25128bf1fadSJohn Baldwin 
252c310653eSNate Lawson 	if (ACPI_FAILURE(acpi_GetInteger(handle, "_ADR", &address)))
25328bf1fadSJohn Baldwin 		return_ACPI_STATUS (AE_OK);
25428bf1fadSJohn Baldwin 	slot = address >> 16;
25528bf1fadSJohn Baldwin 	func = address & 0xffff;
25628bf1fadSJohn Baldwin 	if (device_get_children((device_t)context, &devlist, &devcount) != 0)
25728bf1fadSJohn Baldwin 		return_ACPI_STATUS (AE_OK);
25828bf1fadSJohn Baldwin 	for (i = 0; i < devcount; i++) {
25928bf1fadSJohn Baldwin 		dinfo = device_get_ivars(devlist[i]);
26028bf1fadSJohn Baldwin 		if (dinfo->ap_dinfo.cfg.func == func &&
26128bf1fadSJohn Baldwin 		    dinfo->ap_dinfo.cfg.slot == slot) {
26228bf1fadSJohn Baldwin 			dinfo->ap_handle = handle;
26381bb99d2SJohn Baldwin 			acpi_pci_update_device(handle, devlist[i]);
26430d577a0SNate Lawson 			break;
26528bf1fadSJohn Baldwin 		}
26628bf1fadSJohn Baldwin 	}
26728bf1fadSJohn Baldwin 	free(devlist, M_TEMP);
26828bf1fadSJohn Baldwin 	return_ACPI_STATUS (AE_OK);
26928bf1fadSJohn Baldwin }
27028bf1fadSJohn Baldwin 
27128bf1fadSJohn Baldwin static int
27228bf1fadSJohn Baldwin acpi_pci_probe(device_t dev)
27328bf1fadSJohn Baldwin {
27428bf1fadSJohn Baldwin 
27528bf1fadSJohn Baldwin 	if (pcib_get_bus(dev) < 0)
27628bf1fadSJohn Baldwin 		return (ENXIO);
27728bf1fadSJohn Baldwin 	if (acpi_get_handle(dev) == NULL)
27828bf1fadSJohn Baldwin 		return (ENXIO);
2795acd0218SNate Lawson 	device_set_desc(dev, "ACPI PCI bus");
28028bf1fadSJohn Baldwin 	return (0);
28128bf1fadSJohn Baldwin }
28228bf1fadSJohn Baldwin 
28328bf1fadSJohn Baldwin static int
28428bf1fadSJohn Baldwin acpi_pci_attach(device_t dev)
28528bf1fadSJohn Baldwin {
28628bf1fadSJohn Baldwin 	int busno;
28728bf1fadSJohn Baldwin 
28828bf1fadSJohn Baldwin 	/*
28928bf1fadSJohn Baldwin 	 * Since there can be multiple independantly numbered PCI
29028bf1fadSJohn Baldwin 	 * busses on some large alpha systems, we can't use the unit
29128bf1fadSJohn Baldwin 	 * number to decide what bus we are probing. We ask the parent
29228bf1fadSJohn Baldwin 	 * pcib what our bus number is.
29328bf1fadSJohn Baldwin 	 */
29428bf1fadSJohn Baldwin 	busno = pcib_get_bus(dev);
29528bf1fadSJohn Baldwin 	if (bootverbose)
29628bf1fadSJohn Baldwin 		device_printf(dev, "physical bus=%d\n", busno);
29728bf1fadSJohn Baldwin 
29828bf1fadSJohn Baldwin 	/*
29928bf1fadSJohn Baldwin 	 * First, PCI devices are added as in the normal PCI bus driver.
30028bf1fadSJohn Baldwin 	 * Afterwards, the ACPI namespace under the bridge driver is
30128bf1fadSJohn Baldwin 	 * walked to save ACPI handles to all the devices that appear in
30228bf1fadSJohn Baldwin 	 * the ACPI namespace as immediate descendants of the bridge.
30328bf1fadSJohn Baldwin 	 *
30428bf1fadSJohn Baldwin 	 * XXX: Sometimes PCI devices show up in the ACPI namespace that
30528bf1fadSJohn Baldwin 	 * pci_add_children() doesn't find.  We currently just ignore
30628bf1fadSJohn Baldwin 	 * these devices.
30728bf1fadSJohn Baldwin 	 */
30828bf1fadSJohn Baldwin 	pci_add_children(dev, busno, sizeof(struct acpi_pci_devinfo));
3095acd0218SNate Lawson 	AcpiWalkNamespace(ACPI_TYPE_DEVICE, acpi_get_handle(dev), 1,
31028bf1fadSJohn Baldwin 	    acpi_pci_save_handle, dev, NULL);
31128bf1fadSJohn Baldwin 
31228bf1fadSJohn Baldwin 	return (bus_generic_attach(dev));
31328bf1fadSJohn Baldwin }
314