[src/trunk]: src/sys/dev/acpi/acpica ACPICA 20100915:

[jruoho <jruoho%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/6577ab9da083
branches:  trunk
changeset: 762186:6577ab9da083
user:      jruoho <jruoho%NetBSD.org@localhost>
date:      Thu Feb 17 10:20:32 2011 +0000

description:
ACPICA 20100915:

Removed the AcpiOsDerivePciId OSL interface. The various host
implementations of this function were not OS-dependent and are now obsolete
and can be removed from all host OSLs.  This function has been replaced by
AcpiHwDerivePciId, which is now part of the ACPICA core code.
AcpiHwDerivePciId has been implemented without recursion.  Adds one new
module, hwpci.c.  ACPICA BZ 857.

diffstat:

 sys/dev/acpi/acpica/OsdHardware.c |  118 +-------------------------------------
 1 files changed, 2 insertions(+), 116 deletions(-)

diffs (136 lines):

diff -r 0fc84b62cdbf -r 6577ab9da083 sys/dev/acpi/acpica/OsdHardware.c
--- a/sys/dev/acpi/acpica/OsdHardware.c Thu Feb 17 10:18:05 2011 +0000
+++ b/sys/dev/acpi/acpica/OsdHardware.c Thu Feb 17 10:20:32 2011 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: OsdHardware.c,v 1.6 2010/07/10 21:31:00 gsutre Exp $   */
+/*     $NetBSD: OsdHardware.c,v 1.7 2011/02/17 10:20:32 jruoho Exp $   */
 
 /*
  * Copyright 2001 Wasabi Systems, Inc.
@@ -44,7 +44,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: OsdHardware.c,v 1.6 2010/07/10 21:31:00 gsutre Exp $");
+__KERNEL_RCSID(0, "$NetBSD: OsdHardware.c,v 1.7 2011/02/17 10:20:32 jruoho Exp $");
 
 #include <sys/param.h>
 #include <sys/device.h>
@@ -276,117 +276,3 @@
 
        return AE_OK;
 }
-
-/*
- * acpi_os_derive_pciid_rec:
- *
- *     Helper function for AcpiOsDerivePciId.  The parameters are:
- *     - chandle:      a handle to the node whose PCI id shall be derived.
- *     - rhandle:      a handle the PCI root bridge upstream of chandle.
- *     - pciid:        where the derived PCI id is returned.
- *
- *     This function assumes that rhandle is a proper ancestor of chandle,
- *     and that pciid has already been filled by ACPICA:
- *     - segment# and bus# obtained from _SEG and _BBN on rhandle,
- *     - device# and function# obtained from _ADR on the ACPI device node
- *       whose scope chandle is in).
- */
-static ACPI_STATUS
-acpi_os_derive_pciid_rec(ACPI_HANDLE chandle, ACPI_HANDLE rhandle, ACPI_PCI_ID *pciid)
-{
-       ACPI_HANDLE phandle;
-       ACPI_INTEGER address;
-       ACPI_OBJECT_TYPE objtype;
-       ACPI_STATUS rv;
-       uint16_t valb;
-
-       KASSERT(chandle != rhandle);
-
-       /*
-        * Get parent device node.  This is should not fail since chandle has
-        * at least one ancestor that is a device node: rhandle.
-        */
-       phandle = chandle;
-       do {
-               rv = AcpiGetParent(phandle, &phandle);
-               if (ACPI_FAILURE(rv))
-                       return rv;
-               rv = AcpiGetType(phandle, &objtype);
-               if (ACPI_FAILURE(rv))
-                       return rv;
-       }
-       while (objtype != ACPI_TYPE_DEVICE);
-
-       /*
-        * If the parent is rhandle then we have nothing to do since ACPICA
-        * has pre-filled the PCI id to the best it could.
-        */
-       if (phandle == rhandle)
-               return AE_OK;
-
-       /* Recursive call to get PCI id of the parent */
-       rv = acpi_os_derive_pciid_rec(phandle, rhandle, pciid);
-       if (ACPI_FAILURE(rv))
-               return rv;
-
-       /*
-        * If this is not an ACPI device, return the PCI id of its parent.
-        */
-       rv = AcpiGetType(chandle, &objtype);
-       if (ACPI_FAILURE(rv))
-               return rv;
-       if (objtype != ACPI_TYPE_DEVICE)
-               return AE_OK;
-
-       /*
-        * This is an ACPI device node.  Its parent device node is not a PCI
-        * root bridge.  Check that it is a PCI-to-PCI bridge and get its
-        * secondary bus#.
-        */
-       rv = acpi_pcidev_ppb_downbus(pciid->Segment, pciid->Bus, pciid->Device,
-           pciid->Function, &valb);
-       if (ACPI_FAILURE(rv))
-               return rv;
-
-       /* Get address (contains dev# and fun# for PCI devices). */
-       rv = acpi_eval_integer(chandle, METHOD_NAME__ADR, &address);
-       if (ACPI_FAILURE(rv))
-               return rv;
-
-       pciid->Bus = valb;
-       pciid->Device = ACPI_HIWORD(ACPI_LODWORD(address));
-       pciid->Function = ACPI_LOWORD(ACPI_LODWORD(address));
-       return AE_OK;
-}
-
-/*
- * AcpiOsDerivePciId:
- *
- *     Derive correct PCI bus# by traversing bridges.
- *
- *     In ACPICA release 20100331 (as well as older versions), the interface
- *     of this function is not correctly documented in the ACPICA programmer
- *     reference.  The correct interface parameters to this function are:
- *     - rhandle:      a handle the PCI root bridge upstream of handle.
- *     - chandle:      a handle to the PCI_Config operation region.
- *     - PciId:        where the derived PCI id is returned.
- */
-void
-AcpiOsDerivePciId(
-    ACPI_HANDLE        rhandle,
-    ACPI_HANDLE        chandle,
-    ACPI_PCI_ID        **PciId)
-{
-       ACPI_PCI_ID pciid;
-       ACPI_STATUS rv;
-
-       if (chandle == rhandle)
-               return;
-
-       pciid = **PciId;
-       rv = acpi_os_derive_pciid_rec(chandle, rhandle, &pciid);
-       if (ACPI_FAILURE(rv))
-               return;
-
-       (*PciId)->Bus = pciid.Bus;
-}