ACPICA: Update FACS waking vector interfaces

Split AcpiSetFirmwareWakingVector into two: one for the 32-bit
vector, another for the 64-bit vector. This is required because the
host OS must setup the wake much differently for each vector (real
vs. protected mode, etc.) and the interface should not be deciding
which vector to use.  Also eliminate the GetFirmwareWakingVector
interface, as it served no purpose (only the firmware reads the
vector, OS only writes the vector.)  ACPICA BZ 731.

http://www.acpica.org/bugzilla/show_bug.cgi?id=731

Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Lin Ming <ming.m.lin@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
This commit is contained in:
Bob Moore 2008-11-12 14:54:05 +08:00 committed by Len Brown
parent 6de4048a41
commit d85988fa02
2 changed files with 34 additions and 26 deletions

View File

@ -52,16 +52,16 @@ ACPI_MODULE_NAME("hwsleep")
* *
* FUNCTION: acpi_set_firmware_waking_vector * FUNCTION: acpi_set_firmware_waking_vector
* *
* PARAMETERS: physical_address - Physical address of ACPI real mode * PARAMETERS: physical_address - 32-bit physical address of ACPI real mode
* entry point. * entry point.
* *
* RETURN: Status * RETURN: Status
* *
* DESCRIPTION: Access function for the firmware_waking_vector field in FACS * DESCRIPTION: Sets the 32-bit firmware_waking_vector field of the FACS
* *
******************************************************************************/ ******************************************************************************/
acpi_status acpi_status
acpi_set_firmware_waking_vector(acpi_physical_address physical_address) acpi_set_firmware_waking_vector(u32 physical_address)
{ {
struct acpi_table_facs *facs; struct acpi_table_facs *facs;
acpi_status status; acpi_status status;
@ -85,10 +85,16 @@ acpi_set_firmware_waking_vector(acpi_physical_address physical_address)
* Protected Mode. Some systems (for example HP dv5-1004nr) are known * Protected Mode. Some systems (for example HP dv5-1004nr) are known
* to fail to resume if the 64-bit vector is used. * to fail to resume if the 64-bit vector is used.
*/ */
if (facs->version >= 1)
facs->xfirmware_waking_vector = 0;
facs->firmware_waking_vector = (u32)physical_address; /* Set the 32-bit vector */
facs->firmware_waking_vector = physical_address;
/* Clear the 64-bit vector if it exists */
if ((facs->length > 32) && (facs->version >= 1)) {
facs->xfirmware_waking_vector = 0;
}
return_ACPI_STATUS(AE_OK); return_ACPI_STATUS(AE_OK);
} }
@ -97,29 +103,25 @@ ACPI_EXPORT_SYMBOL(acpi_set_firmware_waking_vector)
/******************************************************************************* /*******************************************************************************
* *
* FUNCTION: acpi_get_firmware_waking_vector * FUNCTION: acpi_set_firmware_waking_vector64
* *
* PARAMETERS: *physical_address - Where the contents of * PARAMETERS: physical_address - 64-bit physical address of ACPI protected
* the firmware_waking_vector field of * mode entry point.
* the FACS will be returned.
* *
* RETURN: Status, vector * RETURN: Status
* *
* DESCRIPTION: Access function for the firmware_waking_vector field in FACS * DESCRIPTION: Sets the 64-bit X_firmware_waking_vector field of the FACS, if
* it exists in the table.
* *
******************************************************************************/ ******************************************************************************/
#ifdef ACPI_FUTURE_USAGE
acpi_status acpi_status
acpi_get_firmware_waking_vector(acpi_physical_address * physical_address) acpi_set_firmware_waking_vector64(u64 physical_address)
{ {
struct acpi_table_facs *facs; struct acpi_table_facs *facs;
acpi_status status; acpi_status status;
ACPI_FUNCTION_TRACE(acpi_get_firmware_waking_vector); ACPI_FUNCTION_TRACE(acpi_set_firmware_waking_vector64);
if (!physical_address) {
return_ACPI_STATUS(AE_BAD_PARAMETER);
}
/* Get the FACS */ /* Get the FACS */
@ -131,14 +133,22 @@ acpi_get_firmware_waking_vector(acpi_physical_address * physical_address)
return_ACPI_STATUS(status); return_ACPI_STATUS(status);
} }
/* Get the vector */ /* Determine if the 64-bit vector actually exists */
*physical_address = (acpi_physical_address)facs->firmware_waking_vector;
if ((facs->length <= 32) || (facs->version < 1)) {
return_ACPI_STATUS(AE_NOT_EXIST);
}
/* Clear 32-bit vector, set the 64-bit X_ vector */
facs->firmware_waking_vector = 0;
facs->xfirmware_waking_vector = physical_address;
return_ACPI_STATUS(AE_OK); return_ACPI_STATUS(AE_OK);
} }
ACPI_EXPORT_SYMBOL(acpi_get_firmware_waking_vector) ACPI_EXPORT_SYMBOL(acpi_set_firmware_waking_vector64)
#endif
/******************************************************************************* /*******************************************************************************
* *
* FUNCTION: acpi_enter_sleep_state_prep * FUNCTION: acpi_enter_sleep_state_prep

View File

@ -320,12 +320,10 @@ acpi_status acpi_get_register_unlocked(u32 register_id, u32 *return_value);
acpi_status acpi_set_register(u32 register_id, u32 value); acpi_status acpi_set_register(u32 register_id, u32 value);
acpi_status acpi_status
acpi_set_firmware_waking_vector(acpi_physical_address physical_address); acpi_set_firmware_waking_vector(u32 physical_address);
#ifdef ACPI_FUTURE_USAGE
acpi_status acpi_status
acpi_get_firmware_waking_vector(acpi_physical_address * physical_address); acpi_set_firmware_waking_vector64(u64 physical_address);
#endif
acpi_status acpi_status
acpi_get_sleep_type_data(u8 sleep_state, u8 * slp_typ_a, u8 * slp_typ_b); acpi_get_sleep_type_data(u8 sleep_state, u8 * slp_typ_a, u8 * slp_typ_b);