]> git.hungrycats.org Git - linux/commitdiff
[ACPI] ACPICA 20040816 update from Bob Moore
authorLen Brown <len.brown@intel.com>
Fri, 20 Aug 2004 14:18:05 +0000 (10:18 -0400)
committerLen Brown <lenb@dhcppc3.>
Fri, 20 Aug 2004 14:18:05 +0000 (10:18 -0400)
Designed and implemented support within the AML interpreter
for the so-called implicit return.  This support returns
the result of the last ASL operation within a control
method, in the absence of an explicit Return() operator.
A few machines depend on this behavior, even though it
is not explicitly supported by the ASL language.  It is
optional support that can be enabled at runtime via the
acpi_gbl_enable_interpreter_slack flag.

Removed support for the PCI_Config address space from the
internal low level hardware interfaces (acpi_hw_low_level_read
and acpi_hw_low_level_write).  This support was not used
internally, and would not work correctly anyway because
the PCI bus number and segment number were not supported.
There are separate interfaces for PCI configuration space
access because of the unique interface.
acpica-unix-20040816.patch

AE_CODE_AML_MAX fix from Bjorn Helgaas

13 files changed:
drivers/acpi/dispatcher/dsmethod.c
drivers/acpi/dispatcher/dsutils.c
drivers/acpi/events/evgpe.c
drivers/acpi/events/evmisc.c
drivers/acpi/events/evregion.c
drivers/acpi/events/evrgnini.c
drivers/acpi/events/evxface.c
drivers/acpi/hardware/hwgpe.c
drivers/acpi/hardware/hwregs.c
drivers/acpi/hardware/hwtimer.c
include/acpi/acconfig.h
include/acpi/acexcep.h
include/acpi/acmacros.h

index afe8f55c2d724ca8cbcb192d96e4cdbbe6435711..4fd369018ce20f0754169d9e809c924777cb0851 100644 (file)
  *
  * FUNCTION:    acpi_ds_parse_method
  *
- * PARAMETERS:  obj_handle      - Node of the method
- *              Level           - Current nesting level
- *              Context         - Points to a method counter
- *              return_value    - Not used
+ * PARAMETERS:  obj_handle      - Method node
  *
  * RETURN:      Status
  *
- * DESCRIPTION: Call the parser and parse the AML that is
- *              associated with the method.
+ * DESCRIPTION: Call the parser and parse the AML that is associated with the
+ *              method.
  *
  * MUTEX:       Assumes parser is locked
  *
@@ -191,8 +188,6 @@ acpi_ds_parse_method (
  *              increments the thread count, and waits at the method semaphore
  *              for clearance to execute.
  *
- * MUTEX:       Locks/unlocks parser.
- *
  ******************************************************************************/
 
 acpi_status
@@ -251,7 +246,8 @@ acpi_ds_begin_method_execution (
  *
  * FUNCTION:    acpi_ds_call_control_method
  *
- * PARAMETERS:  walk_state          - Current state of the walk
+ * PARAMETERS:  Thread              - Info for this thread
+ *              this_walk_state     - Current walk state
  *              Op                  - Current Op to be walked
  *
  * RETURN:      Status
@@ -401,12 +397,13 @@ cleanup:
  *
  * FUNCTION:    acpi_ds_restart_control_method
  *
- * PARAMETERS:  walk_state          - State of the method when it was preempted
- *              Op                  - Pointer to new current op
+ * PARAMETERS:  walk_state          - State for preempted method (caller)
+ *              return_desc         - Return value from the called method
  *
  * RETURN:      Status
  *
- * DESCRIPTION: Restart a method that was preempted
+ * DESCRIPTION: Restart a method that was preempted by another (nested) method
+ *              invocation.  Handle the return value (if any) from the callee.
  *
  ******************************************************************************/
 
@@ -421,17 +418,35 @@ acpi_ds_restart_control_method (
        ACPI_FUNCTION_TRACE_PTR ("ds_restart_control_method", walk_state);
 
 
+       ACPI_DEBUG_PRINT ((ACPI_DB_DISPATCH,
+               "****Restart [%4.4s] Op %p return_value_from_callee %p\n",
+               (char *) &walk_state->method_node->name, walk_state->method_call_op,
+               return_desc));
+
+       ACPI_DEBUG_PRINT ((ACPI_DB_DISPATCH,
+               "    return_from_this_method_used?=%X res_stack %p Walk %p\n",
+               walk_state->return_used,
+               walk_state->results, walk_state));
+
+       /* Did the called method return a value? */
+
        if (return_desc) {
+               /* Are we actually going to use the return value? */
+
                if (walk_state->return_used) {
-                       /*
-                        * Get the return value (if any) from the previous method.
-                        * NULL if no return value
-                        */
+                       /* Save the return value from the previous method */
+
                        status = acpi_ds_result_push (return_desc, walk_state);
                        if (ACPI_FAILURE (status)) {
                                acpi_ut_remove_reference (return_desc);
                                return_ACPI_STATUS (status);
                        }
+
+                       /*
+                        * Save as THIS method's return value in case it is returned
+                        * immediately to yet another method
+                        */
+                       walk_state->return_desc = return_desc;
                }
                else {
                        /*
@@ -442,11 +457,6 @@ acpi_ds_restart_control_method (
                }
        }
 
-       ACPI_DEBUG_PRINT ((ACPI_DB_DISPATCH,
-               "Method=%p Return=%p return_used?=%X res_stack=%p State=%p\n",
-               walk_state->method_call_op, return_desc, walk_state->return_used,
-               walk_state->results, walk_state));
-
        return_ACPI_STATUS (AE_OK);
 }
 
index 5c6a37e057dc039996eca2aa40e2f3b8f2cadf5e..fc99dc0ba510c5e2eda5ae1e57c547e708732d20 100644 (file)
  *
  * FUNCTION:    acpi_ds_is_result_used
  *
- * PARAMETERS:  Op
- *              result_obj
- *              walk_state
+ * PARAMETERS:  Op                  - Current Op
+ *              walk_state          - Current State
  *
- * RETURN:      Status
+ * RETURN:      TRUE if result is used, FALSE otherwise
  *
  * DESCRIPTION: Check if a result object will be used by the parent
  *
@@ -89,18 +88,39 @@ acpi_ds_is_result_used (
        }
 
        /*
-        * If there is no parent, the result can't possibly be used!
-        * (An executing method typically has no parent, since each
-        * method is parsed separately)  However, a method that is
-        * invoked from another method has a parent.
+        * If there is no parent, we are executing at the method level.
+        * An executing method typically has no parent, since each method
+        * is parsed separately.
         */
        if (!op->common.parent) {
+               /*
+                * If this is the last statement in the method, we know it is not a
+                * Return() operator (would not come here.) The following code is the
+                * optional support for a so-called "implicit return". Some AML code
+                * assumes that the last value of the method is "implicitly" returned
+                * to the caller. Just save the last result as the return value.
+                * NOTE: this is optional because the ASL language does not actually
+                * support this behavior.
+                */
+               if ((acpi_gbl_enable_interpeter_slack) &&
+                       (walk_state->parser_state.aml >= walk_state->parser_state.aml_end)) {
+                       ACPI_DEBUG_PRINT ((ACPI_DB_DISPATCH,
+                                       "Result of [%s] will be implicitly returned\n",
+                                       acpi_ps_get_opcode_name (op->common.aml_opcode)));
+
+                       /* Use the top of the result stack as the implicit return value */
+
+                       walk_state->return_desc = walk_state->results->results.obj_desc[0];
+                       return_VALUE (TRUE);
+               }
+
+               /* No parent, the return value cannot possibly be used */
+
                return_VALUE (FALSE);
        }
 
-       /*
-        * Get info on the parent.  The root Op is AML_SCOPE
-        */
+       /* Get info on the parent. The root_op is AML_SCOPE */
+
        parent_info = acpi_ps_get_opcode_info (op->common.parent->common.aml_opcode);
        if (parent_info->class == AML_CLASS_UNKNOWN) {
                ACPI_DEBUG_PRINT ((ACPI_DB_ERROR, "Unknown parent opcode. Op=%p\n", op));
@@ -204,9 +224,9 @@ result_not_used:
  *
  * FUNCTION:    acpi_ds_delete_result_if_not_used
  *
- * PARAMETERS:  Op
- *              result_obj
- *              walk_state
+ * PARAMETERS:  Op              - Current parse Op
+ *              result_obj      - Result of the operation
+ *              walk_state      - Current state
  *
  * RETURN:      Status
  *
@@ -338,8 +358,9 @@ acpi_ds_clear_operands (
  *
  * FUNCTION:    acpi_ds_create_operand
  *
- * PARAMETERS:  walk_state
- *              Arg
+ * PARAMETERS:  walk_state      - Current walk state
+ *              Arg             - Parse object for the argument
+ *              arg_index       - Which argument (zero based)
  *
  * RETURN:      Status
  *
index 5a1191c671bbc7667ae2133255742b7f86aea04e..afb36886bfacb5c70c47402146bcc3186802256e 100644 (file)
@@ -102,6 +102,8 @@ acpi_ev_set_gpe_type (
  * FUNCTION:    acpi_ev_update_gpe_enable_masks
  *
  * PARAMETERS:  gpe_event_info          - GPE to update
+ *              Type                    - What to do: ACPI_GPE_DISABLE or
+ *                                        ACPI_GPE_ENABLE
  *
  * RETURN:      Status
  *
@@ -166,6 +168,8 @@ acpi_ev_update_gpe_enable_masks (
  * FUNCTION:    acpi_ev_enable_gpe
  *
  * PARAMETERS:  gpe_event_info          - GPE to enable
+ *              write_to_hardware       - Enable now, or just mark data structs
+ *                                        (WAKE GPEs should be deferred)
  *
  * RETURN:      Status
  *
@@ -707,7 +711,7 @@ acpi_ev_gpe_dispatch (
 #ifdef ACPI_GPE_NOTIFY_CHECK
 
 /*******************************************************************************
- * NOT USED, PROTOTYPE ONLY AND WILL PROBABLY BE REMOVED
+ * TBD: NOT USED, PROTOTYPE ONLY AND WILL PROBABLY BE REMOVED
  *
  * FUNCTION:    acpi_ev_check_for_wake_only_gpe
  *
index 888d5cec5d7be514c7123f34d05ac0b05e5a65fa..35e61d5221a4740bd79707ba7499fcd05f5c01b9 100644 (file)
@@ -88,9 +88,10 @@ acpi_ev_is_notify_object (
  *
  * FUNCTION:    acpi_ev_queue_notify_request
  *
- * PARAMETERS:
+ * PARAMETERS:  Node            - NS node for the notified object
+ *              notify_value    - Value from the Notify() request
  *
- * RETURN:      None.
+ * RETURN:      Status
  *
  * DESCRIPTION: Dispatch a device notification event to a previously
  *              installed handler.
@@ -143,9 +144,8 @@ acpi_ev_queue_notify_request (
                                notify_value));
        }
 
-       /*
-        * Get the notify object attached to the NS Node
-        */
+       /* Get the notify object attached to the NS Node */
+
        obj_desc = acpi_ns_get_attached_object (node);
        if (obj_desc) {
                /* We have the notify object, Get the right handler */
@@ -193,8 +193,10 @@ acpi_ev_queue_notify_request (
        }
 
        if (!handler_obj) {
-               /* There is no per-device notify handler for this device */
-
+               /*
+                * There is no per-device notify handler for this device.
+                * This may or may not be a problem.
+                */
                ACPI_DEBUG_PRINT ((ACPI_DB_INFO,
                        "No notify handler for Notify(%4.4s, %X) node %p\n",
                        acpi_ut_get_node_name (node), notify_value, node));
@@ -208,7 +210,7 @@ acpi_ev_queue_notify_request (
  *
  * FUNCTION:    acpi_ev_notify_dispatch
  *
- * PARAMETERS:
+ * PARAMETERS:  Context         - To be passsed to the notify handler
  *
  * RETURN:      None.
  *
@@ -275,6 +277,8 @@ acpi_ev_notify_dispatch (
  *
  * FUNCTION:    acpi_ev_global_lock_thread
  *
+ * PARAMETERS:  Context         - From thread interface, not used
+ *
  * RETURN:      None
  *
  * DESCRIPTION: Invoked by SCI interrupt handler upon acquisition of the
@@ -308,7 +312,9 @@ acpi_ev_global_lock_thread (
  *
  * FUNCTION:    acpi_ev_global_lock_handler
  *
- * RETURN:      Status
+ * PARAMETERS:  Context         - From thread interface, not used
+ *
+ * RETURN:      ACPI_INTERRUPT_HANDLED or ACPI_INTERRUPT_NOT_HANDLED
  *
  * DESCRIPTION: Invoked directly from the SCI handler when a global lock
  *              release interrupt occurs.  Grab the global lock and queue
@@ -355,6 +361,8 @@ acpi_ev_global_lock_handler (
  *
  * FUNCTION:    acpi_ev_init_global_lock_handler
  *
+ * PARAMETERS:  None
+ *
  * RETURN:      Status
  *
  * DESCRIPTION: Install a handler for the global lock release event
@@ -394,6 +402,8 @@ acpi_ev_init_global_lock_handler (void)
  *
  * FUNCTION:    acpi_ev_acquire_global_lock
  *
+ * PARAMETERS:  Timeout         - Max time to wait for the lock, in millisec.
+ *
  * RETURN:      Status
  *
  * DESCRIPTION: Attempt to gain ownership of the Global Lock.
@@ -461,6 +471,10 @@ acpi_ev_acquire_global_lock (
  *
  * FUNCTION:    acpi_ev_release_global_lock
  *
+ * PARAMETERS:  None
+ *
+ * RETURN:      Status
+ *
  * DESCRIPTION: Releases ownership of the Global Lock.
  *
  ******************************************************************************/
index 1a98937586da97c75899994492a62a1e70e52881..b7ff3b1629444ff0ed8b6cee8b5111da50a4b829 100644 (file)
@@ -180,7 +180,7 @@ acpi_ev_initialize_op_regions (
  * FUNCTION:    acpi_ev_execute_reg_method
  *
  * PARAMETERS:  region_obj          - Object structure
- *              Function            - On (1) or Off (0)
+ *              Function            - Passed to _REG:  On (1) or Off (0)
  *
  * RETURN:      Status
  *
@@ -232,7 +232,7 @@ acpi_ev_execute_reg_method (
                goto cleanup;
        }
 
-       /* Set up the parameter objects */
+       /* Setup the parameter objects */
 
        params[0]->integer.value = region_obj->region.space_id;
        params[1]->integer.value = function;
@@ -262,7 +262,6 @@ cleanup:
  * FUNCTION:    acpi_ev_address_space_dispatch
  *
  * PARAMETERS:  region_obj          - Internal region object
- *              space_id            - ID of the address space (0-255)
  *              Function            - Read or Write operation
  *              Address             - Where in the space to read or write
  *              bit_width           - Field width in bits (8, 16, 32, or 64)
@@ -425,8 +424,8 @@ acpi_ev_address_space_dispatch (
  *
  * FUNCTION:    acpi_ev_detach_region
  *
- * PARAMETERS:  region_obj      - Region Object
- *              acpi_ns_is_locked - Namespace Region Already Locked?
+ * PARAMETERS:  region_obj          - Region Object
+ *              acpi_ns_is_locked   - Namespace Region Already Locked?
  *
  * RETURN:      None
  *
@@ -560,9 +559,9 @@ acpi_ev_detach_region(
  *
  * FUNCTION:    acpi_ev_attach_region
  *
- * PARAMETERS:  handler_obj     - Handler Object
- *              region_obj      - Region Object
- *              acpi_ns_is_locked - Namespace Region Already Locked?
+ * PARAMETERS:  handler_obj         - Handler Object
+ *              region_obj          - Region Object
+ *              acpi_ns_is_locked   - Namespace Region Already Locked?
  *
  * RETURN:      None
  *
@@ -971,7 +970,7 @@ unlock_and_exit:
  *
  * RETURN:      Status
  *
- * DESCRIPTION: Run _REG methods for the Space ID;
+ * DESCRIPTION: Run all _REG methods for the input Space ID;
  *              Note: assumes namespace is locked, or system init time.
  *
  ******************************************************************************/
index 2e480d301f714f4683c625931107af9abd0eb983..e19d822977112d54f36ebb535ab037bfa614439a 100644 (file)
@@ -54,7 +54,7 @@
  *
  * FUNCTION:    acpi_ev_system_memory_region_setup
  *
- * PARAMETERS:  region_obj          - Region we are interested in
+ * PARAMETERS:  Handle              - Region we are interested in
  *              Function            - Start or stop
  *              handler_context     - Address space handler context
  *              region_context      - Region specific context
@@ -108,7 +108,7 @@ acpi_ev_system_memory_region_setup (
  *
  * FUNCTION:    acpi_ev_io_space_region_setup
  *
- * PARAMETERS:  region_obj          - Region we are interested in
+ * PARAMETERS:  Handle              - Region we are interested in
  *              Function            - Start or stop
  *              handler_context     - Address space handler context
  *              region_context      - Region specific context
@@ -144,7 +144,7 @@ acpi_ev_io_space_region_setup (
  *
  * FUNCTION:    acpi_ev_pci_config_region_setup
  *
- * PARAMETERS:  region_obj          - Region we are interested in
+ * PARAMETERS:  Handle             - Region we are interested in
  *              Function            - Start or stop
  *              handler_context     - Address space handler context
  *              region_context      - Region specific context
@@ -317,7 +317,7 @@ acpi_ev_pci_config_region_setup (
  *
  * FUNCTION:    acpi_ev_pci_bar_region_setup
  *
- * PARAMETERS:  region_obj          - Region we are interested in
+ * PARAMETERS:  Handle              - Region we are interested in
  *              Function            - Start or stop
  *              handler_context     - Address space handler context
  *              region_context      - Region specific context
@@ -348,7 +348,7 @@ acpi_ev_pci_bar_region_setup (
  *
  * FUNCTION:    acpi_ev_cmos_region_setup
  *
- * PARAMETERS:  region_obj          - Region we are interested in
+ * PARAMETERS:  Handle              - Region we are interested in
  *              Function            - Start or stop
  *              handler_context     - Address space handler context
  *              region_context      - Region specific context
@@ -379,7 +379,7 @@ acpi_ev_cmos_region_setup (
  *
  * FUNCTION:    acpi_ev_default_region_setup
  *
- * PARAMETERS:  region_obj          - Region we are interested in
+ * PARAMETERS:  Handle              - Region we are interested in
  *              Function            - Start or stop
  *              handler_context     - Address space handler context
  *              region_context      - Region specific context
index 15ff704911c69041c6232fcc668f498300ec1171..26e9b0e41565a297e913a165e593c4b1717ef957 100644 (file)
@@ -359,6 +359,7 @@ unlock_and_exit:
  *                                  ACPI_DEVICE_NOTIFY: driver_handler (80-ff)
  *                                  ACPI_ALL_NOTIFY:  both system and device
  *              Handler         - Address of the handler
+ *
  * RETURN:      Status
  *
  * DESCRIPTION: Remove a handler for notifies on an ACPI device
@@ -401,9 +402,8 @@ acpi_remove_notify_handler (
                goto unlock_and_exit;
        }
 
-       /*
-        * Root Object
-        */
+       /* Root Object */
+
        if (device == ACPI_ROOT_OBJECT) {
                ACPI_DEBUG_PRINT ((ACPI_DB_INFO, "Removing notify handler for ROOT object.\n"));
 
@@ -437,9 +437,8 @@ acpi_remove_notify_handler (
                }
        }
 
-       /*
-        * All Other Objects
-        */
+       /* All Other Objects */
+
        else {
                /* Notifies allowed on this object? */
 
index dacef55cf54879b25b528bff4e65455c6076c974..3910b5ec9f3a2eba86372ac33d4bc0d61975793d 100644 (file)
@@ -96,7 +96,7 @@ acpi_hw_write_gpe_enable_reg (
  *
  * PARAMETERS:  gpe_event_info      - Info block for the GPE to be cleared
  *
- * RETURN:      status_status
+ * RETURN:      Status
  *
  * DESCRIPTION: Clear the status bit for a single GPE.
  *
index 40a18c8513fa3270d5eff36db563df539e679150..cb998fe9e9700ef494380b8b5ad75c9d38d878e3 100644 (file)
@@ -249,8 +249,8 @@ acpi_hw_get_bit_register_info (
  *              return_value    - Value that was read from the register
  *              Flags           - Lock the hardware or not
  *
- * RETURN:      Value is read from specified Register.  Value returned is
- *              normalized to bit0 (is shifted all the way right)
+ * RETURN:      Status and the value read from specified Register.  Value
+ *              returned is normalized to bit0 (is shifted all the way right)
  *
  * DESCRIPTION: ACPI bit_register read function.
  *
@@ -284,6 +284,8 @@ acpi_get_register (
                }
        }
 
+       /* Read from the register */
+
        status = acpi_hw_register_read (ACPI_MTX_DO_NOT_LOCK,
                          bit_reg_info->parent_register, &register_value);
 
@@ -313,10 +315,10 @@ acpi_get_register (
  *
  * PARAMETERS:  register_id     - ID of ACPI bit_register to access
  *              Value           - (only used on write) value to write to the
- *                                Register, NOT pre-normalized to the bit pos.
+ *                                Register, NOT pre-normalized to the bit pos
  *              Flags           - Lock the hardware or not
  *
- * RETURN:      None
+ * RETURN:      Status
  *
  * DESCRIPTION: ACPI Bit Register write function.
  *
@@ -461,10 +463,11 @@ unlock_and_exit:
  *
  * FUNCTION:    acpi_hw_register_read
  *
- * PARAMETERS:  use_lock               - Mutex hw access.
- *              register_id            - register_iD + Offset.
+ * PARAMETERS:  use_lock            - Mutex hw access
+ *              register_id         - register_iD + Offset
+ *              return_value        - Value that was read from the register
  *
- * RETURN:      Value read or written.
+ * RETURN:      Status and the value read.
  *
  * DESCRIPTION: Acpi register read function.  Registers are read at the
  *              given offset.
@@ -572,10 +575,11 @@ unlock_and_exit:
  *
  * FUNCTION:    acpi_hw_register_write
  *
- * PARAMETERS:  use_lock               - Mutex hw access.
- *              register_id            - register_iD + Offset.
+ * PARAMETERS:  use_lock            - Mutex hw access
+ *              register_id         - register_iD + Offset
+ *              Value               - The value to write
  *
- * RETURN:      Value read or written.
+ * RETURN:      Status
  *
  * DESCRIPTION: Acpi register Write function.  Registers are written at the
  *              given offset.
@@ -691,11 +695,11 @@ unlock_and_exit:
  *
  * PARAMETERS:  Width               - 8, 16, or 32
  *              Value               - Where the value is returned
- *              Register            - GAS register structure
+ *              Reg                 - GAS register structure
  *
  * RETURN:      Status
  *
- * DESCRIPTION: Read from either memory, IO, or PCI config space.
+ * DESCRIPTION: Read from either memory or IO space.
  *
  ******************************************************************************/
 
@@ -705,8 +709,6 @@ acpi_hw_low_level_read (
        u32                             *value,
        struct acpi_generic_address     *reg)
 {
-       struct acpi_pci_id              pci_id;
-       u16                             pci_register;
        acpi_status                     status;
 
 
@@ -725,8 +727,8 @@ acpi_hw_low_level_read (
        *value = 0;
 
        /*
-        * Three address spaces supported:
-        * Memory, IO, or PCI_Config.
+        * Two address spaces supported: Memory or IO.
+        * PCI_Config is not supported here because the GAS struct is insufficient
         */
        switch (reg->address_space_id) {
        case ACPI_ADR_SPACE_SYSTEM_MEMORY:
@@ -744,19 +746,6 @@ acpi_hw_low_level_read (
                break;
 
 
-       case ACPI_ADR_SPACE_PCI_CONFIG:
-
-               pci_id.segment = 0;
-               pci_id.bus     = 0;
-               pci_id.device  = ACPI_PCI_DEVICE (reg->address);
-               pci_id.function = ACPI_PCI_FUNCTION (reg->address);
-               pci_register   = (u16) ACPI_PCI_REGISTER (reg->address);
-
-               status = acpi_os_read_pci_configuration (&pci_id, pci_register,
-                                value, width);
-               break;
-
-
        default:
                ACPI_DEBUG_PRINT ((ACPI_DB_ERROR,
                        "Unsupported address space: %X\n", reg->address_space_id));
@@ -778,11 +767,11 @@ acpi_hw_low_level_read (
  *
  * PARAMETERS:  Width               - 8, 16, or 32
  *              Value               - To be written
- *              Register            - GAS register structure
+ *              Reg                 - GAS register structure
  *
  * RETURN:      Status
  *
- * DESCRIPTION: Write to either memory, IO, or PCI config space.
+ * DESCRIPTION: Write to either memory or IO space.
  *
  ******************************************************************************/
 
@@ -792,8 +781,6 @@ acpi_hw_low_level_write (
        u32                             value,
        struct acpi_generic_address     *reg)
 {
-       struct acpi_pci_id              pci_id;
-       u16                             pci_register;
        acpi_status                     status;
 
 
@@ -811,8 +798,8 @@ acpi_hw_low_level_write (
        }
 
        /*
-        * Three address spaces supported:
-        * Memory, IO, or PCI_Config.
+        * Two address spaces supported: Memory or IO.
+        * PCI_Config is not supported here because the GAS struct is insufficient
         */
        switch (reg->address_space_id) {
        case ACPI_ADR_SPACE_SYSTEM_MEMORY:
@@ -830,19 +817,6 @@ acpi_hw_low_level_write (
                break;
 
 
-       case ACPI_ADR_SPACE_PCI_CONFIG:
-
-               pci_id.segment = 0;
-               pci_id.bus     = 0;
-               pci_id.device  = ACPI_PCI_DEVICE (reg->address);
-               pci_id.function = ACPI_PCI_FUNCTION (reg->address);
-               pci_register   = (u16) ACPI_PCI_REGISTER (reg->address);
-
-               status = acpi_os_write_pci_configuration (&pci_id, pci_register,
-                                (acpi_integer) value, width);
-               break;
-
-
        default:
                ACPI_DEBUG_PRINT ((ACPI_DB_ERROR,
                        "Unsupported address space: %X\n", reg->address_space_id));
index 8878aff9d33617758ca2ee312bb6db3d71ae0063..1f94d6ffa9f88908befc92978424fbef57bf3373 100644 (file)
  *
  * FUNCTION:    acpi_get_timer_resolution
  *
- * PARAMETERS:  none
+ * PARAMETERS:  Resolution          - Where the resolution is returned
  *
- * RETURN:      Number of bits of resolution in the PM Timer (24 or 32).
+ * RETURN:      Status and timer resolution
  *
- * DESCRIPTION: Obtains resolution of the ACPI PM Timer.
+ * DESCRIPTION: Obtains resolution of the ACPI PM Timer (24 or 32 bits).
  *
  ******************************************************************************/
 
@@ -86,11 +86,11 @@ acpi_get_timer_resolution (
  *
  * FUNCTION:    acpi_get_timer
  *
- * PARAMETERS:  none
+ * PARAMETERS:  Ticks               - Where the timer value is returned
  *
- * RETURN:      Current value of the ACPI PM Timer (in ticks).
+ * RETURN:      Status and current ticks
  *
- * DESCRIPTION: Obtains current value of ACPI PM Timer.
+ * DESCRIPTION: Obtains current value of ACPI PM Timer (in ticks).
  *
  ******************************************************************************/
 
@@ -118,11 +118,11 @@ acpi_get_timer (
  *
  * FUNCTION:    acpi_get_timer_duration
  *
- * PARAMETERS:  start_ticks
- *              end_ticks
- *              time_elapsed
+ * PARAMETERS:  start_ticks         - Starting timestamp
+ *              end_ticks           - End timestamp
+ *              time_elapsed        - Where the elapsed time is returned
  *
- * RETURN:      time_elapsed
+ * RETURN:      Status and time_elapsed
  *
  * DESCRIPTION: Computes the time elapsed (in microseconds) between two
  *              PM Timer time stamps, taking into account the possibility of
@@ -136,7 +136,7 @@ acpi_get_timer (
  *              Note that this function accommodates only a single timer
  *              rollover.  Thus for 24-bit timers, this function should only
  *              be used for calculating durations less than ~4.6 seconds
- *              (~20 minutes for 32-bit timers) -- calculations below
+ *              (~20 minutes for 32-bit timers) -- calculations below:
  *
  *              2**24 Ticks / 3,600,000 Ticks/Sec = 4.66 sec
  *              2**32 Ticks / 3,600,000 Ticks/Sec = 1193 sec or 19.88 minutes
@@ -164,7 +164,6 @@ acpi_get_timer_duration (
 
        /*
         * Compute Tick Delta:
-        * -------------------
         * Handle (max one) timer rollovers on 24- versus 32-bit timers.
         */
        if (start_ticks < end_ticks) {
@@ -188,10 +187,7 @@ acpi_get_timer_duration (
        }
 
        /*
-        * Compute Duration:
-        * -----------------
-        *
-        * Requires a 64-bit divide:
+        * Compute Duration (Requires a 64-bit divide):
         *
         * time_elapsed = (delta_ticks * 1000000) / PM_TIMER_FREQUENCY;
         */
index f18824862e96179495e3f6490076f32735728dad..e4a0f1e6f814d09ad5c41c3939ddf825816c1667 100644 (file)
@@ -64,7 +64,7 @@
 
 /* Version string */
 
-#define ACPI_CA_VERSION                 0x20040715
+#define ACPI_CA_VERSION                 0x20040816
 
 /*
  * OS name, used for the _OS object.  The _OS object is essentially obsolete,
index abda2357e543223b66debbe4d27d2ae11988c640..1022cc2da61d040d1895e7f0793b3e383de3a77b 100644 (file)
 #define AE_AML_CIRCULAR_REFERENCE       (acpi_status) (0x0020 | AE_CODE_AML)
 #define AE_AML_BAD_RESOURCE_LENGTH      (acpi_status) (0x0021 | AE_CODE_AML)
 
-#define AE_CODE_AML_MAX                 0x0020
+#define AE_CODE_AML_MAX                 0x0021
 
 /*
  * Internal exceptions used for control
index eb2c5ebbb0e10e565c2eb08871d09273e5d00c2b..43c474ab83406b3d3d670648b9e9a8ec188b7695 100644 (file)
 
 #define ACPI_IS_OCTAL_DIGIT(d)               (((char)(d) >= '0') && ((char)(d) <= '7'))
 
-/* Macros for GAS addressing */
-
-#if ACPI_MACHINE_WIDTH != 16
-
-#define ACPI_PCI_DEVICE(a)              (u16) ((ACPI_HIDWORD ((a))) & 0x0000FFFF)
-#define ACPI_PCI_FUNCTION(a)            (u16) ((ACPI_LODWORD ((a))) >> 16)
-#define ACPI_PCI_REGISTER(a)            (u16) ((ACPI_LODWORD ((a))) & 0x0000FFFF)
-
-#else
-
-/* No support for GAS and PCI IDs in 16-bit mode  */
-
-#define ACPI_PCI_FUNCTION(a)            (u16) ((a) & 0xFFFF0000)
-#define ACPI_PCI_DEVICE(a)              (u16) ((a) & 0x0000FFFF)
-#define ACPI_PCI_REGISTER(a)            (u16) ((a) & 0x0000FFFF)
-
-#endif
-
 
 /* Bitfields within ACPI registers */