ACPICA support for ACPI 5.0
Subsystems (2)
Len Brown
reviewerHere is the ACPICA queue for linux-next. Most of the patches are to support ACPI 5.0 in Linux.
There are additional Linux-specific patches outside of ACPICA on the way.
cheers,
-Len
Len Brown
reviewerPATCH 1/18 ACPICA: Update for GCC 4
From: Bob Moore <[email protected]>
Fixes several issues with GCC 4.6 related to the new checks for unused variables.
http://www.acpica.org/bugzilla/show_bug.cgi?id=935
--- drivers/acpi/acpica/utmutex.c | 9 +++------ 1 files changed, 3 insertions(+), 6 deletions(-)
drivers/acpi/acpica/utmutex.c +3 -6
@@ -293,14 +293,10 @@ acpi_status acpi_ut_acquire_mutex(acpi_mutex_handle mutex_id)acpi_status acpi_ut_release_mutex (acpi_mutex_handle mutex_id) { -acpi_thread_id this_thread_id; -ACPI_FUNCTION_NAME (ut_release_mutex); - this_thread_id =acpi_os_get_thread_id (); -ACPI_DEBUG_PRINT ((ACPI_DB_MUTEX ,"Thread %u releasing Mutex [%s]\n" , - (u32 )this_thread_id, + (u32 )acpi_os_get_thread_id (),acpi_ut_get_mutex_name (mutex_id)));if (mutex_id >ACPI_MAX_MUTEX ) {
@@ -329,7 +325,8 @@ acpi_status acpi_ut_release_mutex(acpi_mutex_handle mutex_id)* the ACPI subsystem code. */ for (i = mutex_id; i <ACPI_NUM_MUTEX ; i++) { -if (acpi_gbl_mutex_info[i].thread_id == this_thread_id) { +if (acpi_gbl_mutex_info[i].thread_id == +acpi_os_get_thread_id ()) {if (i == mutex_id) {continue ; }
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 2/18 ACPICA: Fix to allow region arguments to reference other scopes
In reply to Len BrownFrom: Lin Ming <[email protected]>
Allow referenced objects to be in a different scope.
http://www.acpica.org/bugzilla/show_bug.cgi?id=937
http://marc.info/?l=linux-acpi&m=131636632718222&w=2
ACPI Error: [RAMB] Namespace lookup failure, AE_NOT_FOUND (20110112/psargs-359) ACPI Exception: AE_NOT_FOUND, Could not execute arguments for [RAMW] (Region) (20110112/nsinit-349)
Scope (_SB)
{
Name (RAMB, 0xDF5A1018)
OperationRegion (\RAMW, SystemMemory, RAMB, 0x00010000)
}For above ASL code, we need to save scope node(\_SB) to lookup the argument node(\_SB.RAMB).
--- drivers/acpi/acpica/acobject.h | 1 + drivers/acpi/acpica/dsargs.c | 2 +- drivers/acpi/acpica/excreate.c | 6 ++++++ 3 files changed, 8 insertions(+), 1 deletions(-)
drivers/acpi/acpica/acobject.h +1 -0
@@ -358,6 +358,7 @@ typedef enum {*/ struct acpi_object_extra { ACPI_OBJECT_COMMON_HEADER struct acpi_namespace_node *method_REG; /* _REG method for this region (if any) */ +struct acpi_namespace_node *scope_node;void *region_context;/* Region-specific data */ u8 *aml_start;u32 aml_length;
drivers/acpi/acpica/dsargs.c +1 -1
@@ -384,7 +384,7 @@ acpi_status acpi_ds_get_region_arguments(union acpi_operand_object *obj_desc)/* Execute the argument AML */ - status =acpi_ds_execute_arguments (node, node->parent, + status =acpi_ds_execute_arguments (node, extra_desc->extra.scope_node, extra_desc->extra.aml_length, extra_desc->extra.aml_start);return_ACPI_STATUS (status);
drivers/acpi/acpica/excreate.c +6 -0
@@ -330,6 +330,12 @@ acpi_ex_create_region(u8 * aml_start, region_obj2 = obj_desc->common.next_object; region_obj2->extra.aml_start = aml_start; region_obj2->extra.aml_length = aml_length; +if (walk_state->scope_info) { + region_obj2->extra.scope_node = + walk_state->scope_info->scope.node; + }else { + region_obj2->extra.scope_node = node; + }/* Init the region from the operands */
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 3/18 ACPICA: Add error msg for unsupported I/O requests (not 8/16/32 bit length)
In reply to Len BrownFrom: Bob Moore <[email protected]>
Found during ACPICA debugging.
--- drivers/acpi/acpica/hwvalid.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)
drivers/acpi/acpica/hwvalid.c +2 -0
@@ -134,6 +134,8 @@ acpi_hw_validate_io_request(acpi_io_address address, u32 bit_width)/* Supported widths are 8/16/32 */ if ((bit_width !=8 ) && (bit_width !=16 ) && (bit_width !=32 )) { +ACPI_ERROR ((AE_INFO , +"Bad BitWidth parameter: %8.8X" , bit_width));return AE_BAD_PARAMETER ; }
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 4/18 ACPICA: Add error msg for Index/Bank field registers out-of-range
In reply to Len BrownFrom: Bob Moore <[email protected]>
Supplement the exception code with an actual message.
Found during ACPICA debugging.
--- drivers/acpi/acpica/exfldio.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-)
drivers/acpi/acpica/exfldio.c +6 -0
@@ -316,6 +316,7 @@ acpi_ex_access_region(union acpi_operand_object *obj_desc,static u8 acpi_ex_register_overflow (union acpi_operand_object *obj_desc,u64 value) { +ACPI_FUNCTION_NAME (ex_register_overflow);if (obj_desc->common_field.bit_length >=ACPI_INTEGER_BIT_SIZE ) {/*
@@ -330,6 +331,11 @@ acpi_ex_register_overflow(union acpi_operand_object *obj_desc, u64 value)* The Value is larger than the maximum value that can fit into * the register. */ +ACPI_ERROR ((AE_INFO , +"Index value 0x%8.8X%8.8X overflows field width 0x%X" , +ACPI_FORMAT_UINT64 (value), + obj_desc->common_field.bit_length)); +return (TRUE); }
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 5/18 ACPICA: Do not abort table load on invalid space ID
In reply to Len BrownFrom: Bob Moore <[email protected]>
Ignore an invalid space ID during a table load. Instead, detect it if a control method attempts access - then abort the method.
http://www.acpica.org/bugzilla/show_bug.cgi?id=925
--- drivers/acpi/acpica/acinterp.h | 2 ++ drivers/acpi/acpica/excreate.c | 23 +++++++++++++---------- drivers/acpi/acpica/exfldio.c | 16 ++++++++++++++-- drivers/acpi/acpica/exutils.c | 25 +++++++++++++++++++++++++ 4 files changed, 54 insertions(+), 12 deletions(-)
drivers/acpi/acpica/acinterp.h +2 -0
@@ -468,6 +468,8 @@ void acpi_ex_eisa_id_to_string(char *dest, u64 compressed_id);void acpi_ex_integer_to_string (char *dest,u64 value); +u8 acpi_is_valid_space_id (u8 space_id); +/* * exregion - default op_region handlers */
drivers/acpi/acpica/excreate.c +13 -10
@@ -267,7 +267,7 @@ acpi_status acpi_ex_create_mutex(struct acpi_walk_state *walk_state)* * PARAMETERS: aml_start - Pointer to the region declaration AML * aml_length - Max length of the declaration AML -* region_space - space_iD for the region +* space_id - Address space ID for the region * walk_state - Current state * * RETURN: Status
@@ -279,7 +279,7 @@ acpi_status acpi_ex_create_mutex(struct acpi_walk_state *walk_state)acpi_status acpi_ex_create_region (u8 * aml_start,u32 aml_length, -u8 region_space,struct acpi_walk_state *walk_state) +u8 space_id,struct acpi_walk_state *walk_state) {acpi_status status;union acpi_operand_object *obj_desc;
@@ -304,16 +304,19 @@ acpi_ex_create_region(u8 * aml_start,* Space ID must be one of the predefined IDs, or in the user-defined * range */ -if ((region_space >=ACPI_NUM_PREDEFINED_REGIONS ) && - (region_space <ACPI_USER_REGION_BEGIN ) && - (region_space !=ACPI_ADR_SPACE_DATA_TABLE )) { -ACPI_ERROR ((AE_INFO ,"Invalid AddressSpace type 0x%X" , - region_space)); -return_ACPI_STATUS (AE_AML_INVALID_SPACE_ID ); +if (!acpi_is_valid_space_id (space_id)) { +/* +* Print an error message, but continue. We don't want to abort +* a table load for this exception. Instead, if the region is +* actually used at runtime, abort the executing method. +*/ +ACPI_ERROR ((AE_INFO , +"Invalid/unknown Address Space ID: 0x%2.2X" , + space_id)); }ACPI_DEBUG_PRINT ((ACPI_DB_LOAD ,"Region Type - %s (0x%X)\n" , -acpi_ut_get_region_name (region_space), region_space)); +acpi_ut_get_region_name (space_id), space_id));/* Create the region descriptor */
@@ -339,7 +342,7 @@ acpi_ex_create_region(u8 * aml_start,/* Init the region from the operands */ - obj_desc->region.space_id = region_space; + obj_desc->region.space_id = space_id; obj_desc->region.address =0 ; obj_desc->region.length =0 ; obj_desc->region.node = node;
drivers/acpi/acpica/exfldio.c +14 -2
@@ -86,6 +86,7 @@ acpi_ex_setup_region(union acpi_operand_object *obj_desc, {acpi_status status =AE_OK ;union acpi_operand_object *rgn_desc; +u8 space_id;ACPI_FUNCTION_TRACE_U32 (ex_setup_region, field_datum_byte_offset);
@@ -101,6 +102,17 @@ acpi_ex_setup_region(union acpi_operand_object *obj_desc,return_ACPI_STATUS (AE_AML_OPERAND_TYPE ); } + space_id = rgn_desc->region.space_id; + +/* Validate the Space ID */ + +if (!acpi_is_valid_space_id (space_id)) { +ACPI_ERROR ((AE_INFO , +"Invalid/unknown Address Space ID: 0x%2.2X" , + space_id)); +return_ACPI_STATUS (AE_AML_INVALID_SPACE_ID ); + } +/* * If the Region Address and Length have not been previously evaluated, * evaluate them now and save the results.
@@ -122,8 +134,8 @@ acpi_ex_setup_region(union acpi_operand_object *obj_desc,* Exit now for SMBus or IPMI address space, it has a non-linear * address space and the request cannot be directly validated */ -if (rgn_desc->region.space_id ==ACPI_ADR_SPACE_SMBUS || - rgn_desc->region.space_id ==ACPI_ADR_SPACE_IPMI ) { +if (space_id ==ACPI_ADR_SPACE_SMBUS || + space_id ==ACPI_ADR_SPACE_IPMI ) {/* SMBus or IPMI has a non-linear address space */
drivers/acpi/acpica/exutils.c +25 -0
@@ -435,4 +435,29 @@ void acpi_ex_integer_to_string(char *out_string, u64 value) } } +/******************************************************************************* +* +* FUNCTION: acpi_is_valid_space_id +* +* PARAMETERS: space_id - ID to be validated +* +* RETURN: TRUE if valid/supported ID. +* +* DESCRIPTION: Validate an operation region space_iD. +* +******************************************************************************/ + +u8 acpi_is_valid_space_id (u8 space_id) +{ + +if ((space_id >=ACPI_NUM_PREDEFINED_REGIONS ) && + (space_id <ACPI_USER_REGION_BEGIN ) && + (space_id !=ACPI_ADR_SPACE_DATA_TABLE ) && + (space_id !=ACPI_ADR_SPACE_FIXED_HARDWARE )) { +return (FALSE); + } + +return (TRUE); +} +#endif
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 6/18 ACPI 5.0: Basic support for FADT version 5
In reply to Len BrownFrom: Bob Moore <[email protected]>
--- drivers/acpi/acpica/tbfadt.c | 4 ++-- include/acpi/actbl.h | 22 ++++++++++++++++------ 2 files changed, 18 insertions(+), 8 deletions(-)
drivers/acpi/acpica/tbfadt.c +2 -2
@@ -277,12 +277,12 @@ void acpi_tb_create_local_fadt(struct acpi_table_header *table, u32 length) {/* * Check if the FADT is larger than the largest table that we expect -* (the ACPI 2.0/3.0 version). If so, truncate the table, and issue +* (the ACPI 5.0 version). If so, truncate the table, and issue * a warning. */ if (length >sizeof (struct acpi_table_fadt )) {ACPI_WARNING ((AE_INFO , -"FADT (revision %u) is longer than ACPI 2.0 version, " +"FADT (revision %u) is longer than ACPI 5.0 version, " "truncating length %u to %u" , table->revision, length, (u32 )sizeof (struct acpi_table_fadt )));
include/acpi/actbl.h +16 -6
@@ -255,6 +255,8 @@ struct acpi_table_fadt {struct acpi_generic_address xpm_timer_block;/* 64-bit Extended Power Mgt Timer Ctrl Reg Blk address */ struct acpi_generic_address xgpe0_block;/* 64-bit Extended General Purpose Event 0 Reg Blk address */ struct acpi_generic_address xgpe1_block;/* 64-bit Extended General Purpose Event 1 Reg Blk address */ +struct acpi_generic_address sleep_control;/* 64-bit Sleep Control register */ +struct acpi_generic_address sleep_status;/* 64-bit Sleep Status register */ };/* Masks for FADT Boot Architecture Flags (boot_flags) */
@@ -264,6 +266,7 @@ struct acpi_table_fadt {#define ACPI_FADT_NO_VGA (1<<2)/* 02: [V4] It is not safe to probe for VGA hardware */ #define ACPI_FADT_NO_MSI (1<<3)/* 03: [V4] Message Signaled Interrupts (MSI) must not be enabled */ #define ACPI_FADT_NO_ASPM (1<<4)/* 04: [V4] PCIe ASPM control must not be enabled */ +#define ACPI_FADT_NO_CMOS_RTC (1<<5)/* 05: [V5] No CMOS real-time clock present */ #define FADT2_REVISION_ID 3
@@ -289,6 +292,8 @@ struct acpi_table_fadt {#define ACPI_FADT_REMOTE_POWER_ON (1<<17)/* 17: [V4] System is compatible with remote power on (ACPI 3.0) */ #define ACPI_FADT_APIC_CLUSTER (1<<18)/* 18: [V4] All local APICs must use cluster model (ACPI 3.0) */ #define ACPI_FADT_APIC_PHYSICAL (1<<19)/* 19: [V4] All local x_aPICs must use physical dest mode (ACPI 3.0) */ +#define ACPI_FADT_HW_REDUCED (1<<20)/* 20: [V5] ACPI hardware is not implemented (ACPI 5.0) */ +#define ACPI_FADT_LOW_POWER_S0 (1<<21)/* 21: [V5] S0 power savings are equal or better than S3 (ACPI 5.0) */ /* Values for preferred_profile (Preferred Power Management Profiles) */
@@ -299,14 +304,16 @@ enum acpi_prefered_pm_profiles {PM_WORKSTATION =3 ,PM_ENTERPRISE_SERVER =4 ,PM_SOHO_SERVER =5 , - PM_APPLIANCE_PC = 6 +PM_APPLIANCE_PC =6 , +PM_PERFORMANCE_SERVER =7 , + PM_TABLET = 8 };/* Reset to default packing */ #pragma pack() -#define ACPI_FADT_OFFSET (f) (u8) ACPI_OFFSET (struct acpi_table_fadt, f) +#define ACPI_FADT_OFFSET (f) (u16) ACPI_OFFSET (struct acpi_table_fadt, f)/* * Internal table-related structures
@@ -351,12 +358,15 @@ struct acpi_table_desc {* FADT is the bottom line as to what the version really is. * * For reference, the values below are as follows: -* FADT V1 size: 0x74 -* FADT V2 size: 0x84 -* FADT V3+ size: 0xF4 +* FADT V1 size: 0x074 +* FADT V2 size: 0x084 +* FADT V3 size: 0x0F4 +* FADT V4 size: 0x0F4 +* FADT V5 size: 0x10C */ #define ACPI_FADT_V1_SIZE (u32) (ACPI_FADT_OFFSET (flags) + 4)#define ACPI_FADT_V2_SIZE (u32) (ACPI_FADT_OFFSET (reserved4[0]) + 3) -#define ACPI_FADT_V3_SIZE (u32) (sizeof (struct acpi_table_fadt)) +#define ACPI_FADT_V3_SIZE (u32) (ACPI_FADT_OFFSET (sleep_control)) +#define ACPI_FADT_V5_SIZE (u32) (sizeof (struct acpi_table_fadt))#endif /* __ACTBL_H__ */
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 7/18 ACPI 5.0: Implement hardware-reduced option
In reply to Len BrownFrom: Bob Moore <[email protected]>
If HW-reduced flag is set in the FADT, do not attempt to access or initialize any ACPI hardware, including SCI and global lock. No FACS will be present.
--- drivers/acpi/acpica/acglobal.h | 13 ++++++++++++- drivers/acpi/acpica/evevent.c | 12 ++++++++++++ drivers/acpi/acpica/evglock.c | 6 ++++++ drivers/acpi/acpica/tbfadt.c | 22 ++++++++++++++++++++-- drivers/acpi/acpica/tbutils.c | 7 +++++++ include/acpi/acpixf.h | 1 + 6 files changed, 58 insertions(+), 3 deletions(-)
drivers/acpi/acpica/acglobal.h +12 -1
@@ -140,8 +140,19 @@ u32 acpi_gbl_trace_flags;acpi_name acpi_gbl_trace_method_name;u8 acpi_gbl_system_awake_and_running; +/* +* ACPI 5.0 introduces the concept of a "reduced hardware platform", meaning +* that the ACPI hardware is no longer required. A flag in the FADT indicates +* a reduced HW machine, and that flag is duplicated here for convenience. +*/ +u8 acpi_gbl_reduced_hardware; +#endif +/* Do not disassemble buffers to resource descriptors */ + +ACPI_EXTERN u8 ACPI_INIT_GLOBAL(acpi_gbl_no_resource_disassembly, FALSE); +/***************************************************************************** * * Debug support
@@ -207,7 +218,7 @@ ACPI_EXTERN struct acpi_rw_lock acpi_gbl_namespace_rw_lock;/***************************************************************************** * -* Mutual exlusion within ACPICA subsystem +* Mutual exclusion within ACPICA subsystem * ****************************************************************************/
drivers/acpi/acpica/evevent.c +12 -0
@@ -71,6 +71,12 @@ acpi_status acpi_ev_initialize_events(void)ACPI_FUNCTION_TRACE (ev_initialize_events); +/* If Hardware Reduced flag is set, there are no fixed events */ + +if (acpi_gbl_reduced_hardware) { +return_ACPI_STATUS (AE_OK ); + } +/* * Initialize the Fixed and General Purpose Events. This is done prior to * enabling SCIs to prevent interrupts from occurring before the handlers
@@ -111,6 +117,12 @@ acpi_status acpi_ev_install_xrupt_handlers(void)ACPI_FUNCTION_TRACE (ev_install_xrupt_handlers); +/* If Hardware Reduced flag is set, there is no ACPI h/w */ + +if (acpi_gbl_reduced_hardware) { +return_ACPI_STATUS (AE_OK ); + } +/* Install the SCI handler */ status =acpi_ev_install_sci_handler ();
drivers/acpi/acpica/evglock.c +6 -0
@@ -70,6 +70,12 @@ acpi_status acpi_ev_init_global_lock_handler(void)ACPI_FUNCTION_TRACE (ev_init_global_lock_handler); +/* If Hardware Reduced flag is set, there is no global lock */ + +if (acpi_gbl_reduced_hardware) { +return_ACPI_STATUS (AE_OK ); + } +/* Attempt installation of the global lock handler */ status = acpi_install_fixed_event_handler(ACPI_EVENT_GLOBAL,
drivers/acpi/acpica/tbfadt.c +20 -2
@@ -253,8 +253,13 @@ void acpi_tb_parse_fadt(u32 table_index)acpi_tb_install_table ((acpi_physical_address ) acpi_gbl_FADT.Xdsdt,ACPI_SIG_DSDT ,ACPI_TABLE_INDEX_DSDT ); -acpi_tb_install_table ((acpi_physical_address ) acpi_gbl_FADT.Xfacs, -ACPI_SIG_FACS ,ACPI_TABLE_INDEX_FACS ); +/* If Hardware Reduced flag is set, there is no FACS */ + +if (!acpi_gbl_reduced_hardware) { +acpi_tb_install_table ((acpi_physical_address ) acpi_gbl_FADT. + Xfacs,ACPI_SIG_FACS , +ACPI_TABLE_INDEX_FACS ); + } }/*******************************************************************************
@@ -297,6 +302,13 @@ void acpi_tb_create_local_fadt(struct acpi_table_header *table, u32 length)ACPI_MEMCPY (&acpi_gbl_FADT, table,ACPI_MIN (length,sizeof (struct acpi_table_fadt ))); +/* Take a copy of the Hardware Reduced flag */ + + acpi_gbl_reduced_hardware = FALSE; +if (acpi_gbl_FADT.flags &ACPI_FADT_HW_REDUCED ) { + acpi_gbl_reduced_hardware = TRUE; + } +/* Convert the local copy of the FADT to the common internal format */ acpi_tb_convert_fadt ();
@@ -502,6 +514,12 @@ static void acpi_tb_validate_fadt(void) acpi_gbl_FADT.Xdsdt = (u64 ) acpi_gbl_FADT.dsdt; } +/* If Hardware Reduced flag is set, we are all done */ + +if (acpi_gbl_reduced_hardware) { +return ; + } +/* Examine all of the 64-bit extended address fields (X fields) */ for (i =0 ; i <ACPI_FADT_INFO_ENTRIES ; i++) {
drivers/acpi/acpica/tbutils.c +7 -0
@@ -135,6 +135,13 @@ acpi_status acpi_tb_initialize_facs(void) {acpi_status status; +/* If Hardware Reduced flag is set, there is no FACS */ + +if (acpi_gbl_reduced_hardware) { + acpi_gbl_FACS =NULL ; +return (AE_OK ); + } + status = acpi_get_table_by_index(ACPI_TABLE_INDEX_FACS, ACPI_CAST_INDIRECT_PTR(struct acpi_table_header,
include/acpi/acpixf.h +1 -0
@@ -74,6 +74,7 @@ extern u8 acpi_gbl_disable_auto_repair;extern u32 acpi_current_gpe_count;extern struct acpi_table_fadt acpi_gbl_FADT;extern u8 acpi_gbl_system_awake_and_running; +extern u8 acpi_gbl_reduced_hardware;/* ACPI 5.0 */ extern u32 acpi_rsdt_forced;/*
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 8/18 ACPI 5.0: Add new/changed tables to headers
In reply to Len BrownFrom: Bob Moore <[email protected]>
Adds new file, actbl3.h
--- include/acpi/actbl.h | 1 + include/acpi/actbl1.h | 57 +++++- include/acpi/actbl3.h | 552 +++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 606 insertions(+), 4 deletions(-) create mode 100644 include/acpi/actbl3.h
include/acpi/actbl.h +1 -0
@@ -349,6 +349,7 @@ struct acpi_table_desc {#include <acpi/actbl1.h> #include <acpi/actbl2.h> +#include <acpi/actbl3.h> /* * Sizes of the various flavors of FADT. We need to look closely
include/acpi/actbl1.h +53 -4
91 lines not on this page · raw message
include/acpi/actbl3.h new file +552 -0
552 lines not on this page · raw message
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 9/18 ACPI 5.0: New interfaces to allow driver access to AML mutex objects
In reply to Len BrownFrom: Lin Ming <[email protected]>
Adds acpi_acquire_mutex, acpi_release_mutex external interfaces. New file, utxfmutex.c.
--- drivers/acpi/acpica/Makefile | 2 +- drivers/acpi/acpica/utxfmutex.c | 187 +++++++++++++++++++++++++++++++++++++++ include/acpi/acpixf.h | 15 +++- 3 files changed, 201 insertions(+), 3 deletions(-) create mode 100644 drivers/acpi/acpica/utxfmutex.c
drivers/acpi/acpica/Makefile +1 -1
@@ -45,4 +45,4 @@ acpi-y += tbxface.o tbinstal.o tbutils.o tbfind.o tbfadt.o tbxfroot.o acpi-y += utalloc.o utdebug.o uteval.o utinit.o utmisc.o utxface.o \ utcopy.o utdelete.o utglobal.o utmath.o utobject.o \ utstate.o utmutex.o utobject.o utresrc.o utlock.o utids.o \ - utosi.o utxferror.o utdecode.o + utosi.o utxferror.o utdecode.o utxfmutex.o
drivers/acpi/acpica/utxfmutex.c new file +187 -0
187 lines not on this page · raw message
include/acpi/acpixf.h +13 -2
@@ -277,12 +277,23 @@ acpi_status acpi_install_exception_handler(acpi_exception_handler handler);acpi_status acpi_install_interface_handler (acpi_interface_handler handler);/* -* Event interfaces +* Global Lock interfaces */ acpi_status acpi_acquire_global_lock (u16 timeout,u32 * handle);acpi_status acpi_release_global_lock (u32 handle); +/* +* Interfaces to AML mutex objects +*/ +acpi_status +acpi_acquire_mutex (acpi_handle handle,acpi_string pathname,u16 timeout); + +acpi_status acpi_release_mutex (acpi_handle handle,acpi_string pathname); + +/* +* Fixed Event interfaces +*/ acpi_status acpi_enable_event (u32 event,u32 flags);acpi_status acpi_disable_event (u32 event,u32 flags);
@@ -292,7 +303,7 @@ acpi_status acpi_clear_event(u32 event);acpi_status acpi_get_event_status (u32 event,acpi_event_status * event_status);/* -* GPE Interfaces +* General Purpose Event (GPE) Interfaces */ acpi_status acpi_enable_gpe (acpi_handle gpe_device,u32 gpe_number);
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 10/18 ACPI 5.0: Implement Connection() and AccessAs() changes
In reply to Len BrownFrom: Bob Moore <[email protected]>
Support within the interpreter and operation region dispatch.
--- drivers/acpi/acpica/acevents.h | 1 + drivers/acpi/acpica/aclocal.h | 6 ++- drivers/acpi/acpica/acobject.h | 5 +- drivers/acpi/acpica/acopcode.h | 4 + drivers/acpi/acpica/amlcode.h | 10 +++ drivers/acpi/acpica/dsfield.c | 81 +++++++++++++++++++---- drivers/acpi/acpica/evregion.c | 29 +++++++- drivers/acpi/acpica/exconfig.c | 6 +- drivers/acpi/acpica/exdump.c | 7 ++- drivers/acpi/acpica/exfldio.c | 11 ++-- drivers/acpi/acpica/exprep.c | 25 +++++++ drivers/acpi/acpica/psargs.c | 141 ++++++++++++++++++++++++++++++++++++---- drivers/acpi/acpica/psopcode.c | 13 +++- drivers/acpi/acpica/pstree.c | 6 ++ include/acpi/actypes.h | 8 ++ 15 files changed, 308 insertions(+), 45 deletions(-)
drivers/acpi/acpica/acevents.h +1 -0
@@ -162,6 +162,7 @@ acpi_status acpi_ev_initialize_op_regions(void);acpi_status acpi_ev_address_space_dispatch (union acpi_operand_object *region_obj, +union acpi_operand_object *field_obj,u32 function,u32 region_offset,u32 bit_width,u64 *value);
drivers/acpi/acpica/aclocal.h +5 -1
@@ -53,7 +53,7 @@ typedef u32 acpi_mutex_handle;/* Total number of aml opcodes defined */ -#define AML_NUM_OPCODES 0x7F +#define AML_NUM_OPCODES 0x81/* Forward declarations */
@@ -249,12 +249,16 @@ struct acpi_create_field_info {struct acpi_namespace_node *field_node;struct acpi_namespace_node *register_node;struct acpi_namespace_node *data_register_node; +struct acpi_namespace_node *connection_node; +u8 *resource_buffer;u32 bank_value;u32 field_bit_position;u32 field_bit_length; +u16 resource_length;u8 field_flags;u8 attribute;u8 field_type; +u8 access_length; }; typedef
drivers/acpi/acpica/acobject.h +4 -1
@@ -254,6 +254,7 @@ ACPI_OBJECT_COMMON_HEADER ACPI_COMMON_NOTIFY_INFO};u32 base_byte_offset;/* Byte offset within containing object */ \u32 value;/* Value to store into the Bank or Index register */ \u8 start_field_bit_offset;/* Bit offset within first field datum (0-63) */ \ +u8 access_length;/* For serial regions/fields */ struct acpi_object_field_common {/* COMMON FIELD (for BUFFER, REGION, BANK, and INDEX fields) */
@@ -261,7 +262,9 @@ struct acpi_object_field_common { /* COMMON FIELD (for BUFFER, REGION, BANK, and };struct acpi_object_region_field { - ACPI_OBJECT_COMMON_HEADER ACPI_COMMON_FIELD_INFO union acpi_operand_object *region_obj; /* Containing op_region object */ + ACPI_OBJECT_COMMON_HEADER ACPI_COMMON_FIELD_INFO u16 resource_length; +union acpi_operand_object *region_obj;/* Containing op_region object */ +u8 *resource_buffer;/* resource_template for serial regions/fields */ };struct acpi_object_bank_field {
drivers/acpi/acpica/acopcode.h +4 -0
@@ -93,6 +93,7 @@#define ARGP_CONCAT_OP ARGP_LIST3 (ARGP_TERMARG, ARGP_TERMARG, ARGP_TARGET)#define ARGP_CONCAT_RES_OP ARGP_LIST3 (ARGP_TERMARG, ARGP_TERMARG, ARGP_TARGET)#define ARGP_COND_REF_OF_OP ARGP_LIST2 (ARGP_SUPERNAME, ARGP_SUPERNAME) +#define ARGP_CONNECTFIELD_OP ARGP_LIST1 (ARGP_NAMESTRING)#define ARGP_CONTINUE_OP ARG_NONE#define ARGP_COPY_OP ARGP_LIST2 (ARGP_TERMARG, ARGP_SIMPLENAME)#define ARGP_CREATE_BIT_FIELD_OP ARGP_LIST3 (ARGP_TERMARG, ARGP_TERMARG, ARGP_NAME)
@@ -164,6 +165,7 @@#define ARGP_RETURN_OP ARGP_LIST1 (ARGP_TERMARG)#define ARGP_REVISION_OP ARG_NONE#define ARGP_SCOPE_OP ARGP_LIST3 (ARGP_PKGLENGTH, ARGP_NAME, ARGP_TERMLIST) +#define ARGP_SERIALFIELD_OP ARGP_LIST1 (ARGP_NAMESTRING)#define ARGP_SHIFT_LEFT_OP ARGP_LIST3 (ARGP_TERMARG, ARGP_TERMARG, ARGP_TARGET)#define ARGP_SHIFT_RIGHT_OP ARGP_LIST3 (ARGP_TERMARG, ARGP_TERMARG, ARGP_TARGET)#define ARGP_SIGNAL_OP ARGP_LIST1 (ARGP_SUPERNAME)
@@ -223,6 +225,7 @@#define ARGI_CONCAT_OP ARGI_LIST3 (ARGI_COMPUTEDATA,ARGI_COMPUTEDATA, ARGI_TARGETREF)#define ARGI_CONCAT_RES_OP ARGI_LIST3 (ARGI_BUFFER, ARGI_BUFFER, ARGI_TARGETREF)#define ARGI_COND_REF_OF_OP ARGI_LIST2 (ARGI_OBJECT_REF, ARGI_TARGETREF) +#define ARGI_CONNECTFIELD_OP ARGI_INVALID_OPCODE#define ARGI_CONTINUE_OP ARGI_INVALID_OPCODE#define ARGI_COPY_OP ARGI_LIST2 (ARGI_ANYTYPE, ARGI_SIMPLE_TARGET)#define ARGI_CREATE_BIT_FIELD_OP ARGI_LIST3 (ARGI_BUFFER, ARGI_INTEGER, ARGI_REFERENCE)
@@ -294,6 +297,7 @@#define ARGI_RETURN_OP ARGI_INVALID_OPCODE#define ARGI_REVISION_OP ARG_NONE#define ARGI_SCOPE_OP ARGI_INVALID_OPCODE +#define ARGI_SERIALFIELD_OP ARGI_INVALID_OPCODE#define ARGI_SHIFT_LEFT_OP ARGI_LIST3 (ARGI_INTEGER, ARGI_INTEGER, ARGI_TARGETREF)#define ARGI_SHIFT_RIGHT_OP ARGI_LIST3 (ARGI_INTEGER, ARGI_INTEGER, ARGI_TARGETREF)#define ARGI_SIGNAL_OP ARGI_LIST1 (ARGI_EVENT)
drivers/acpi/acpica/amlcode.h +10 -0
@@ -189,6 +189,14 @@#define AML_LNOTEQUAL_OP (u16) 0x9293/* +* Opcodes for "Field" operators +*/ +#define AML_FIELD_OFFSET_OP (u8) 0x00 +#define AML_FIELD_ACCESS_OP (u8) 0x01 +#define AML_FIELD_CONNECTION_OP (u8) 0x02/* ACPI 5.0 */ +#define AML_FIELD_EXT_ACCESS_OP (u8) 0x03/* ACPI 5.0 */ + +/* * Internal opcodes * Use only "Unknown" AML opcodes, don't attempt to use * any valid ACPI ASCII values (A-Z, 0-9, '-')
@@ -202,6 +210,8 @@#define AML_INT_METHODCALL_OP (u16) 0x0035#define AML_INT_RETURN_VALUE_OP (u16) 0x0036#define AML_INT_EVAL_SUBTREE_OP (u16) 0x0037 +#define AML_INT_CONNECTION_OP (u16) 0x0038 +#define AML_INT_EXTACCESSFIELD_OP (u16) 0x0039#define ARG_NONE 0x0
drivers/acpi/acpica/dsfield.c +66 -15
130 lines not on this page · raw message
drivers/acpi/acpica/evregion.c +26 -3
@@ -329,6 +329,7 @@ acpi_ev_execute_reg_method(union acpi_operand_object *region_obj, u32 function)* FUNCTION: acpi_ev_address_space_dispatch * * PARAMETERS: region_obj - Internal region object +* field_obj - Corresponding field. Can be NULL. * Function - Read or Write operation * region_offset - Where in the region to read or write * bit_width - Field width in bits (8, 16, 32, or 64)
@@ -344,6 +345,7 @@ acpi_ev_execute_reg_method(union acpi_operand_object *region_obj, u32 function)acpi_status acpi_ev_address_space_dispatch (union acpi_operand_object *region_obj, +union acpi_operand_object *field_obj,u32 function,u32 region_offset,u32 bit_width,u64 *value) {
@@ -353,6 +355,7 @@ acpi_ev_address_space_dispatch(union acpi_operand_object *region_obj,union acpi_operand_object *handler_desc;union acpi_operand_object *region_obj2;void *region_context =NULL ; +struct acpi_connection_info *context;ACPI_FUNCTION_TRACE (ev_address_space_dispatch);
@@ -375,6 +378,8 @@ acpi_ev_address_space_dispatch(union acpi_operand_object *region_obj,return_ACPI_STATUS (AE_NOT_EXIST ); } + context = handler_desc->address_space.context; +/* * It may be the case that the region has never been initialized. * Some types of regions require special init code
@@ -404,8 +409,7 @@ acpi_ev_address_space_dispatch(union acpi_operand_object *region_obj,acpi_ex_exit_interpreter (); status =region_setup (region_obj,ACPI_REGION_ACTIVATE , - handler_desc->address_space.context, - ®ion_context); + context, ®ion_context);/* Re-enter the interpreter */
@@ -455,6 +459,25 @@ acpi_ev_address_space_dispatch(union acpi_operand_object *region_obj,acpi_ut_get_region_name (region_obj->region. space_id))); +/* +* Special handling for generic_serial_bus and general_purpose_io: +* There are three extra parameters that must be passed to the +* handler via the context: +* 1) Connection buffer, a resource template from Connection() op. +* 2) Length of the above buffer. +* 3) Actual access length from the access_as() op. +*/ +if (((region_obj->region.space_id ==ACPI_ADR_SPACE_GSBUS ) || + (region_obj->region.space_id ==ACPI_ADR_SPACE_GPIO )) && + context && field_obj) { + +/* Get the Connection (resource_template) buffer */ + + context->connection = field_obj->field.resource_buffer; + context->length = field_obj->field.resource_length; + context->access_length = field_obj->field.access_length; + } +if (!(handler_desc->address_space.handler_flags &ACPI_ADDR_HANDLER_DEFAULT_INSTALLED )) {/*
@@ -469,7 +492,7 @@ acpi_ev_address_space_dispatch(union acpi_operand_object *region_obj, status =handler (function, (region_obj->region.address + region_offset), - bit_width, value, handler_desc->address_space.context, + bit_width, value, context, region_obj2->extra.region_context);if (ACPI_FAILURE (status)) {
drivers/acpi/acpica/exconfig.c +3 -3
@@ -297,9 +297,9 @@ acpi_ex_region_read(union acpi_operand_object *obj_desc, u32 length, u8 *buffer)/* Bytewise reads */ for (i =0 ; i < length; i++) { - status =acpi_ev_address_space_dispatch (obj_desc,ACPI_READ , - region_offset,8 , - &value); + status = +acpi_ev_address_space_dispatch (obj_desc,NULL ,ACPI_READ , + region_offset,8 , &value);if (ACPI_FAILURE (status)) {return status; }
drivers/acpi/acpica/exdump.c +5 -2
@@ -192,10 +192,13 @@ static struct acpi_exdump_info acpi_ex_dump_buffer_field[3] = { "Buffer Object"} }; -static struct acpi_exdump_info acpi_ex_dump_region_field[3 ] = { +static struct acpi_exdump_info acpi_ex_dump_region_field[5 ] = { {ACPI_EXD_INIT ,ACPI_EXD_TABLE_SIZE (acpi_ex_dump_region_field),NULL }, {ACPI_EXD_FIELD ,0 ,NULL }, - {ACPI_EXD_POINTER ,ACPI_EXD_OFFSET (field.region_obj),"Region Object" } + {ACPI_EXD_UINT8 ,ACPI_EXD_OFFSET (field.access_length),"AccessLength" }, + {ACPI_EXD_POINTER ,ACPI_EXD_OFFSET (field.region_obj),"Region Object" }, + {ACPI_EXD_POINTER ,ACPI_EXD_OFFSET (field.resource_buffer), +"ResourceBuffer" } };static struct acpi_exdump_info acpi_ex_dump_bank_field[5 ] = {
drivers/acpi/acpica/exfldio.c +6 -5
@@ -283,11 +283,12 @@ acpi_ex_access_region(union acpi_operand_object *obj_desc,/* Invoke the appropriate address_space/op_region handler */ - status = -acpi_ev_address_space_dispatch (rgn_desc, function, region_offset, -ACPI_MUL_8 (obj_desc->common_field. - access_byte_width), - value); + status =acpi_ev_address_space_dispatch (rgn_desc, obj_desc, + function, region_offset, +ACPI_MUL_8 (obj_desc-> + common_field. + access_byte_width), + value);if (ACPI_FAILURE (status)) {if (status ==AE_NOT_IMPLEMENTED ) {
drivers/acpi/acpica/exprep.c +25 -0
@@ -47,6 +47,7 @@#include "acinterp.h" #include "amlcode.h" #include "acnamesp.h" +#include "acdispat.h" #define _COMPONENT ACPI_EXECUTERACPI_MODULE_NAME ("exprep" )
@@ -455,6 +456,30 @@ acpi_status acpi_ex_prep_field_value(struct acpi_create_field_info *info) obj_desc->field.region_obj =acpi_ns_get_attached_object (info->region_node); +/* Fields specific to generic_serial_bus fields */ + + obj_desc->field.access_length = info->access_length; + +if (info->connection_node) { + second_desc = info->connection_node->object; +if (!(second_desc->common.flags &AOPOBJ_DATA_VALID )) { + status = +acpi_ds_get_buffer_arguments (second_desc); +if (ACPI_FAILURE (status)) { +acpi_ut_delete_object_desc (obj_desc); +return_ACPI_STATUS (status); + } + } + + obj_desc->field.resource_buffer = + second_desc->buffer.pointer; + obj_desc->field.resource_length = + (u16 )second_desc->buffer.length; + }else if (info->resource_buffer) { + obj_desc->field.resource_buffer = info->resource_buffer; + obj_desc->field.resource_length = info->resource_length; + } +/* Allow full data read from EC address space */ if ((obj_desc->field.region_obj->region.space_id ==
drivers/acpi/acpica/psargs.c +128 -13
178 lines not on this page · raw message
drivers/acpi/acpica/psopcode.c +11 -2
@@ -638,7 +638,16 @@ const struct acpi_opcode_info acpi_gbl_aml_op_info[AML_NUM_OPCODES] = {/* 7E */ ACPI_OP ("Timer" ,ARGP_TIMER_OP ,ARGI_TIMER_OP ,ACPI_TYPE_ANY ,AML_CLASS_EXECUTE ,AML_TYPE_EXEC_0A_0T_1R , -AML_FLAGS_EXEC_0A_0T_1R ) +AML_FLAGS_EXEC_0A_0T_1R ), + +/* ACPI 5.0 opcodes */ + +/* 7F */ ACPI_OP ("-ConnectField-" ,ARGP_CONNECTFIELD_OP , +ARGI_CONNECTFIELD_OP ,ACPI_TYPE_ANY , +AML_CLASS_INTERNAL ,AML_TYPE_BOGUS ,AML_HAS_ARGS ), +/* 80 */ ACPI_OP ("-ExtAccessField-" ,ARGP_CONNECTFIELD_OP , +ARGI_CONNECTFIELD_OP ,ACPI_TYPE_ANY , +AML_CLASS_INTERNAL ,AML_TYPE_BOGUS ,0 ) /*! [End] no source code translation !*/ };
@@ -657,7 +666,7 @@ static const u8 acpi_gbl_short_op_index[256] = {/* 0x20 */ _UNK, _UNK, _UNK, _UNK, _UNK, _UNK, _UNK, _UNK,/* 0x28 */ _UNK, _UNK, _UNK, _UNK, _UNK,0x63 , _PFX, _PFX,/* 0x30 */ 0x67 ,0x66 ,0x68 ,0x65 ,0x69 ,0x64 ,0x6A ,0x7D , -/* 0x38 */ _UNK, _UNK, _UNK, _UNK, _UNK, _UNK, _UNK, _UNK, +/* 0x38 */ 0x7F ,0x80 , _UNK, _UNK, _UNK, _UNK, _UNK, _UNK,/* 0x40 */ _UNK, _ASC, _ASC, _ASC, _ASC, _ASC, _ASC, _ASC,/* 0x48 */ _ASC, _ASC, _ASC, _ASC, _ASC, _ASC, _ASC, _ASC, /* 0x50 */ _ASC, _ASC, _ASC, _ASC, _ASC, _ASC, _ASC, _ASC,
drivers/acpi/acpica/pstree.c +6 -0
@@ -74,6 +74,12 @@ union acpi_parse_object *acpi_ps_get_arg(union acpi_parse_object *op, u32 argn)ACPI_FUNCTION_ENTRY (); +/* +if (Op->Common.aml_opcode == AML_INT_CONNECTION_OP) +{ +return (Op->Common.Value.Arg); +} +*/ /* Get the info structure for this opcode */ op_info =acpi_ps_get_opcode_info (op->common.aml_opcode);
include/acpi/actypes.h +8 -0
@@ -957,6 +957,14 @@ acpi_status(*acpi_adr_space_handler) (u32 function,#define ACPI_DEFAULT_HANDLER NULL +/* Special Context data for generic_serial_bus/general_purpose_io (ACPI 5.0) */ + +struct acpi_connection_info { +u8 *connection; +u16 length; +u8 access_length; +}; +typedef acpi_status(*acpi_adr_space_setup) (acpi_handle region_handle, u32 function,
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 11/18 ACPI 5.0: Support for GeneralPurposeIo and GenericSerialBus operation region
In reply to Len BrownFrom: Bob Moore <[email protected]>
--- drivers/acpi/acpica/acconfig.h | 3 ++- drivers/acpi/acpica/amlcode.h | 17 ++++++++++------- drivers/acpi/acpica/exfield.c | 28 +++++++++++++++++++++------- drivers/acpi/acpica/exfldio.c | 3 ++- drivers/acpi/acpica/utdecode.c | 4 +++- include/acpi/actypes.h | 4 +++- 6 files changed, 41 insertions(+), 18 deletions(-)
drivers/acpi/acpica/acconfig.h +2 -1
@@ -202,9 +202,10 @@#define ACPI_RSDP_CHECKSUM_LENGTH 20#define ACPI_RSDP_XCHECKSUM_LENGTH 36 -/* SMBus and IPMI bidirectional buffer size */ +/* SMBus, GSBus and IPMI bidirectional buffer size */ #define ACPI_SMBUS_BUFFER_SIZE 34 +#define ACPI_GSBUS_BUFFER_SIZE 34#define ACPI_IPMI_BUFFER_SIZE 66/* _sx_d and _sx_w control methods */
drivers/acpi/acpica/amlcode.h +10 -7
@@ -466,13 +466,16 @@ typedef enum {* access_as keyword */ typedef enum { -AML_FIELD_ATTRIB_SMB_QUICK =0x02 , -AML_FIELD_ATTRIB_SMB_SEND_RCV =0x04 , -AML_FIELD_ATTRIB_SMB_BYTE =0x06 , -AML_FIELD_ATTRIB_SMB_WORD =0x08 , -AML_FIELD_ATTRIB_SMB_BLOCK =0x0A , -AML_FIELD_ATTRIB_SMB_WORD_CALL =0x0C , -AML_FIELD_ATTRIB_SMB_BLOCK_CALL =0x0D +AML_FIELD_ATTRIB_QUICK =0x02 , +AML_FIELD_ATTRIB_SEND_RCV =0x04 , +AML_FIELD_ATTRIB_BYTE =0x06 , +AML_FIELD_ATTRIB_WORD =0x08 , +AML_FIELD_ATTRIB_BLOCK =0x0A , +AML_FIELD_ATTRIB_MULTIBYTE =0x0B , +AML_FIELD_ATTRIB_WORD_CALL =0x0C , +AML_FIELD_ATTRIB_BLOCK_CALL =0x0D , +AML_FIELD_ATTRIB_RAW_BYTES =0x0E , +AML_FIELD_ATTRIB_RAW_PROCESS =0x0F }AML_ACCESS_ATTRIBUTE ;/* Bit fields in the AML method_flags byte */
drivers/acpi/acpica/exfield.c +21 -7
@@ -100,18 +100,25 @@ acpi_ex_read_data_from_field(struct acpi_walk_state *walk_state, (obj_desc->field.region_obj->region.space_id == ACPI_ADR_SPACE_SMBUS || obj_desc->field.region_obj->region.space_id == + ACPI_ADR_SPACE_GSBUS + || obj_desc->field.region_obj->region.space_id == ACPI_ADR_SPACE_IPMI)) {/* -* This is an SMBus or IPMI read. We must create a buffer to hold +* This is an SMBus, GSBus or IPMI read. We must create a buffer to hold * the data and then directly access the region handler. * -* Note: Smbus protocol value is passed in upper 16-bits of Function +* Note: SMBus and GSBus protocol value is passed in upper 16-bits of Function */ if (obj_desc->field.region_obj->region.space_id ==ACPI_ADR_SPACE_SMBUS ) { length =ACPI_SMBUS_BUFFER_SIZE ; function =ACPI_READ | (obj_desc->field.attribute <<16 ); + }else if (obj_desc->field.region_obj->region.space_id == +ACPI_ADR_SPACE_GSBUS ) { + length =ACPI_GSBUS_BUFFER_SIZE ; + function = +ACPI_READ | (obj_desc->field.attribute <<16 ); }else {/* IPMI */ length =ACPI_IPMI_BUFFER_SIZE ;
@@ -248,21 +255,23 @@ acpi_ex_write_data_to_field(union acpi_operand_object *source_desc, (obj_desc->field.region_obj->region.space_id == ACPI_ADR_SPACE_SMBUS || obj_desc->field.region_obj->region.space_id == + ACPI_ADR_SPACE_GSBUS + || obj_desc->field.region_obj->region.space_id == ACPI_ADR_SPACE_IPMI)) {/* -* This is an SMBus or IPMI write. We will bypass the entire field +* This is an SMBus, GSBus or IPMI write. We will bypass the entire field * mechanism and handoff the buffer directly to the handler. For * these address spaces, the buffer is bi-directional; on a write, * return data is returned in the same buffer. * * Source must be a buffer of sufficient size: -* ACPI_SMBUS_BUFFER_SIZE or ACPI_IPMI_BUFFER_SIZE. +* ACPI_SMBUS_BUFFER_SIZE, ACPI_GSBUS_BUFFER_SIZE, or ACPI_IPMI_BUFFER_SIZE. * -* Note: SMBus protocol type is passed in upper 16-bits of Function +* Note: SMBus and GSBus protocol type is passed in upper 16-bits of Function */ if (source_desc->common.type !=ACPI_TYPE_BUFFER ) {ACPI_ERROR ((AE_INFO , -"SMBus or IPMI write requires Buffer, found type %s" , +"SMBus/IPMI/GenericSerialBus write requires Buffer, found type %s" ,acpi_ut_get_object_type_name (source_desc)));return_ACPI_STATUS (AE_AML_OPERAND_TYPE );
@@ -273,6 +282,11 @@ acpi_ex_write_data_to_field(union acpi_operand_object *source_desc, length =ACPI_SMBUS_BUFFER_SIZE ; function =ACPI_WRITE | (obj_desc->field.attribute <<16 ); + }else if (obj_desc->field.region_obj->region.space_id == +ACPI_ADR_SPACE_GSBUS ) { + length =ACPI_GSBUS_BUFFER_SIZE ; + function = +ACPI_WRITE | (obj_desc->field.attribute <<16 ); }else {/* IPMI */ length =ACPI_IPMI_BUFFER_SIZE ;
@@ -281,7 +295,7 @@ acpi_ex_write_data_to_field(union acpi_operand_object *source_desc,if (source_desc->buffer.length < length) {ACPI_ERROR ((AE_INFO , -"SMBus or IPMI write requires Buffer of length %u, found length %u" , +"SMBus/IPMI/GenericSerialBus write requires Buffer of length %u, found length %u" , length, source_desc->buffer.length));return_ACPI_STATUS (AE_AML_BUFFER_LIMIT );
drivers/acpi/acpica/exfldio.c +2 -1
@@ -131,10 +131,11 @@ acpi_ex_setup_region(union acpi_operand_object *obj_desc, }/* -* Exit now for SMBus or IPMI address space, it has a non-linear +* Exit now for SMBus, GSBus or IPMI address space, it has a non-linear * address space and the request cannot be directly validated */ if (space_id ==ACPI_ADR_SPACE_SMBUS || + space_id ==ACPI_ADR_SPACE_GSBUS || space_id ==ACPI_ADR_SPACE_IPMI ) {/* SMBus or IPMI has a non-linear address space */
drivers/acpi/acpica/utdecode.c +3 -1
@@ -171,7 +171,9 @@ const char *acpi_gbl_region_types[ACPI_NUM_PREDEFINED_REGIONS] = {"SMBus" ,"SystemCMOS" ,"PCIBARTarget" , - "IPMI" +"IPMI" , +"GeneralPurposeIo" , + "GenericSerialBus" };char *acpi_ut_get_region_name (u8 space_id)
include/acpi/actypes.h +3 -1
@@ -712,8 +712,10 @@ typedef u8 acpi_adr_space_type;#define ACPI_ADR_SPACE_CMOS (acpi_adr_space_type) 5#define ACPI_ADR_SPACE_PCI_BAR_TARGET (acpi_adr_space_type) 6#define ACPI_ADR_SPACE_IPMI (acpi_adr_space_type) 7 +#define ACPI_ADR_SPACE_GPIO (acpi_adr_space_type) 8 +#define ACPI_ADR_SPACE_GSBUS (acpi_adr_space_type) 9 -#define ACPI_NUM_PREDEFINED_REGIONS 8 +#define ACPI_NUM_PREDEFINED_REGIONS 10/* * Special Address Spaces
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 12/18 ACPI 5.0: Support for all new resource descriptors
In reply to Len BrownFrom: Lin Ming <[email protected]>
FixedDMA, GPIO descriptors, SerialBus descriptors
--- drivers/acpi/acpica/Makefile | 2 +- drivers/acpi/acpica/aclocal.h | 6 +- drivers/acpi/acpica/acresrc.h | 109 +++++++---- drivers/acpi/acpica/acutils.h | 1 + drivers/acpi/acpica/amlresrc.h | 136 ++++++++++++ drivers/acpi/acpica/rscalc.c | 87 ++++++++- drivers/acpi/acpica/rscreate.c | 3 +- drivers/acpi/acpica/rsdump.c | 194 +++++++++++++++++- drivers/acpi/acpica/rsinfo.c | 56 +++++- drivers/acpi/acpica/rsirq.c | 31 +++ drivers/acpi/acpica/rslist.c | 75 ++++++-- drivers/acpi/acpica/rsmisc.c | 267 ++++++++++++++++++++++++- drivers/acpi/acpica/rsserial.c | 441 ++++++++++++++++++++++++++++++++++++++++ drivers/acpi/acpica/rsutils.c | 4 + drivers/acpi/acpica/utresrc.c | 276 ++++++++++++++++++++++--- include/acpi/acrestyp.h | 207 ++++++++++++++++++- 16 files changed, 1776 insertions(+), 119 deletions(-) create mode 100644 drivers/acpi/acpica/rsserial.c
drivers/acpi/acpica/Makefile +1 -1
@@ -36,7 +36,7 @@ acpi-y += psargs.o psparse.o psloop.o pstree.o pswalk.o \ psopcode.o psscope.o psutils.o psxface.o acpi-y += rsaddr.o rscreate.o rsinfo.o rsio.o rslist.o rsmisc.o rsxface.o \ - rscalc.o rsirq.o rsmemory.o rsutils.o + rscalc.o rsirq.o rsmemory.o rsutils.o rsserial.o acpi-$(ACPI_FUTURE_USAGE) += rsdump.o
drivers/acpi/acpica/aclocal.h +4 -2
@@ -955,7 +955,7 @@ struct acpi_port_info {#define ACPI_RESOURCE_NAME_END_DEPENDENT 0x38#define ACPI_RESOURCE_NAME_IO 0x40#define ACPI_RESOURCE_NAME_FIXED_IO 0x48 -#define ACPI_RESOURCE_NAME_RESERVED_S1 0x50 +#define ACPI_RESOURCE_NAME_FIXED_DMA 0x50#define ACPI_RESOURCE_NAME_RESERVED_S2 0x58#define ACPI_RESOURCE_NAME_RESERVED_S3 0x60#define ACPI_RESOURCE_NAME_RESERVED_S4 0x68
@@ -977,7 +977,9 @@ struct acpi_port_info {#define ACPI_RESOURCE_NAME_EXTENDED_IRQ 0x89#define ACPI_RESOURCE_NAME_ADDRESS64 0x8A#define ACPI_RESOURCE_NAME_EXTENDED_ADDRESS64 0x8B -#define ACPI_RESOURCE_NAME_LARGE_MAX 0x8B +#define ACPI_RESOURCE_NAME_GPIO 0x8C +#define ACPI_RESOURCE_NAME_SERIAL_BUS 0x8E +#define ACPI_RESOURCE_NAME_LARGE_MAX 0x8E/***************************************************************************** *
drivers/acpi/acpica/acresrc.h +73 -36
158 lines not on this page · raw message
drivers/acpi/acpica/acutils.h +1 -0
@@ -45,6 +45,7 @@#define _ACUTILS_Hextern const u8 acpi_gbl_resource_aml_sizes[]; +extern const u8 acpi_gbl_resource_aml_serial_bus_sizes[];/* Strings used by the disassembler and debugger resource dump routines */
drivers/acpi/acpica/amlresrc.h +136 -0
195 lines not on this page · raw message
drivers/acpi/acpica/rscalc.c +80 -7
130 lines not on this page · raw message
drivers/acpi/acpica/rscreate.c +2 -1
@@ -66,9 +66,10 @@ ACPI_MODULE_NAME("rscreate")* of device resources. * ******************************************************************************/ +acpi_status acpi_rs_create_resource_list (union acpi_operand_object *aml_buffer, -struct acpi_buffer *output_buffer) +struct acpi_buffer * output_buffer) {acpi_status status;
drivers/acpi/acpica/rsdump.c +185 -9
245 lines not on this page · raw message
drivers/acpi/acpica/rsinfo.c +50 -6
96 lines not on this page · raw message
drivers/acpi/acpica/rsirq.c +31 -0
@@ -264,3 +264,34 @@ struct acpi_rsconvert_info acpi_rs_convert_dma[6] = {AML_OFFSET (dma.dma_channel_mask), ACPI_RS_OFFSET(data.dma.channel_count)} }; + +/******************************************************************************* +* +* acpi_rs_convert_fixed_dma +* +******************************************************************************/ + +struct acpi_rsconvert_info acpi_rs_convert_fixed_dma[4 ] = { + {ACPI_RSC_INITGET ,ACPI_RESOURCE_TYPE_FIXED_DMA , + ACPI_RS_SIZE(struct acpi_resource_fixed_dma), +ACPI_RSC_TABLE_SIZE (acpi_rs_convert_fixed_dma)}, + + {ACPI_RSC_INITSET ,ACPI_RESOURCE_NAME_FIXED_DMA , +sizeof (struct aml_resource_fixed_dma ), +0 }, + +/* +* These fields are contiguous in both the source and destination: +* request_lines +* Channels +*/ + + {ACPI_RSC_MOVE16 ,ACPI_RS_OFFSET (data.fixed_dma.request_lines), +AML_OFFSET (fixed_dma.request_lines), +2 }, + + {ACPI_RSC_MOVE8 ,ACPI_RS_OFFSET (data.fixed_dma.width), +AML_OFFSET (fixed_dma.width), +1 }, + +};
drivers/acpi/acpica/rslist.c +61 -14
114 lines not on this page · raw message
drivers/acpi/acpica/rsmisc.c +262 -5
336 lines not on this page · raw message
drivers/acpi/acpica/rsserial.c new file +441 -0
441 lines not on this page · raw message
drivers/acpi/acpica/rsutils.c +4 -0
@@ -144,6 +144,9 @@ acpi_rs_move_data(void *destination, void *source, u16 item_count, u8 move_type)* since there are no alignment or endian issues */ case ACPI_RSC_MOVE8 : +case ACPI_RSC_MOVE_GPIO_RES : +case ACPI_RSC_MOVE_SERIAL_VEN : +case ACPI_RSC_MOVE_SERIAL_RES :ACPI_MEMCPY (destination, source, item_count);return ;
@@ -153,6 +156,7 @@ acpi_rs_move_data(void *destination, void *source, u16 item_count, u8 move_type)* misaligned memory transfers */ case ACPI_RSC_MOVE16 : +case ACPI_RSC_MOVE_GPIO_PIN :ACPI_MOVE_16_TO_16 (&ACPI_CAST_PTR (u16, destination)[i], &ACPI_CAST_PTR (u16, source)[i]);break ;
drivers/acpi/acpica/utresrc.c +242 -34
392 lines not on this page · raw message
include/acpi/acrestyp.h +203 -4
267 lines not on this page · raw message
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 13/18 ACPI 5.0: New interface, acpi_get_event_resources
In reply to Len BrownFrom: Bob Moore <[email protected]>
Executes _AEI and formats the result, similar to acpi_get_current_resources, etc.
--- drivers/acpi/acpica/acresrc.h | 4 +++ drivers/acpi/acpica/rsutils.c | 50 +++++++++++++++++++++++++++++++++++++++++ drivers/acpi/acpica/rsxface.c | 40 ++++++++++++++++++++++++++++++++ include/acpi/acpixf.h | 4 +++ 4 files changed, 98 insertions(+), 0 deletions(-)
drivers/acpi/acpica/acresrc.h +4 -0
@@ -215,6 +215,10 @@ acpi_status acpi_rs_set_srs_method_data(struct acpi_namespace_node *node, struct acpi_buffer *ret_buffer); +acpi_status +acpi_rs_get_aei_method_data (struct acpi_namespace_node *node, +struct acpi_buffer *ret_buffer); +/* * rscalc */
drivers/acpi/acpica/rsutils.c +50 -0
56 lines not on this page · raw message
drivers/acpi/acpica/rsxface.c +40 -0
@@ -307,6 +307,46 @@ acpi_set_current_resources(acpi_handle device_handle,ACPI_EXPORT_SYMBOL (acpi_set_current_resources) +/******************************************************************************* +* +* FUNCTION: acpi_get_event_resources +* +* PARAMETERS: device_handle - Handle to the device object for the +* device we are getting resources +* in_buffer - Pointer to a buffer containing the +* resources to be set for the device +* +* RETURN: Status +* +* DESCRIPTION: This function is called to get the event resources for a +* specific device. The caller must first acquire a handle for +* the desired device. The resource data is passed to the routine +* the buffer pointed to by the in_buffer variable. Uses the +* _AEI method. +* +******************************************************************************/ +acpi_status +acpi_get_event_resources (acpi_handle device_handle, +struct acpi_buffer *ret_buffer) +{ +acpi_status status; +struct acpi_namespace_node *node; + +ACPI_FUNCTION_TRACE (acpi_get_event_resources); + +/* Validate parameters then dispatch to internal routine */ + + status =acpi_rs_validate_parameters (device_handle, ret_buffer, &node); +if (ACPI_FAILURE (status)) { +return_ACPI_STATUS (status); + } + + status =acpi_rs_get_aei_method_data (node, ret_buffer); +return_ACPI_STATUS (status); +} + +ACPI_EXPORT_SYMBOL (acpi_get_event_resources) +/****************************************************************************** * * FUNCTION: acpi_resource_to_address64
include/acpi/acpixf.h +4 -0
@@ -358,6 +358,10 @@ acpi_get_possible_resources(acpi_handle device, struct acpi_buffer *ret_buffer);#endif acpi_status +acpi_get_event_resources (acpi_handle device_handle, +struct acpi_buffer *ret_buffer); + +acpi_status acpi_walk_resources (acpi_handle device,char *name,acpi_walk_resource_callback user_function,void *context);
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 14/18 ACPI 5.0: New interface, acpi_buffer_to_resource
In reply to Len BrownFrom: Bob Moore <[email protected]>
This interface converts an AML buffer to an internal ACPI_RESOURCE.
--- drivers/acpi/acpica/rscreate.c | 64 ++++++++++++++++++++++++++++++++++++++++ include/acpi/acpixf.h | 5 +++ 2 files changed, 69 insertions(+), 0 deletions(-)
drivers/acpi/acpica/rscreate.c +64 -0
@@ -51,6 +51,70 @@ ACPI_MODULE_NAME("rscreate")/******************************************************************************* * +* FUNCTION: acpi_buffer_to_resource +* +* PARAMETERS: aml_buffer - Pointer to the resource byte stream +* aml_buffer_length - Length of the aml_buffer +* resource_ptr - Where the converted resource is returned +* +* RETURN: Status +* +* DESCRIPTION: Convert a raw AML buffer to a resource list +* +******************************************************************************/ +acpi_status +acpi_buffer_to_resource (u8 *aml_buffer, +u16 aml_buffer_length, +struct acpi_resource **resource_ptr) +{ +acpi_status status; +acpi_size list_size_needed; +void *resource; +void *current_resource_ptr; + +/* +* Note: we allow AE_AML_NO_RESOURCE_END_TAG, since an end tag +* is not required here. +*/ + +/* Get the required length for the converted resource */ + + status =acpi_rs_get_list_length (aml_buffer, aml_buffer_length, + &list_size_needed); +if (status ==AE_AML_NO_RESOURCE_END_TAG ) { + status =AE_OK ; + } +if (ACPI_FAILURE (status)) { +return (status); + } + +/* Allocate a buffer for the converted resource */ + + resource =ACPI_ALLOCATE_ZEROED (list_size_needed); + current_resource_ptr = resource; +if (!resource) { +return (AE_NO_MEMORY ); + } + +/* Perform the AML-to-Resource conversion */ + + status =acpi_ut_walk_aml_resources (aml_buffer, aml_buffer_length, + acpi_rs_convert_aml_to_resources, + ¤t_resource_ptr); +if (status ==AE_AML_NO_RESOURCE_END_TAG ) { + status =AE_OK ; + } +if (ACPI_FAILURE (status)) { +ACPI_FREE (resource); + }else { + *resource_ptr = resource; + } + +return (status); +} + +/******************************************************************************* +* * FUNCTION: acpi_rs_create_resource_list * * PARAMETERS: aml_buffer - Pointer to the resource byte stream
include/acpi/acpixf.h +5 -0
@@ -376,6 +376,11 @@ acpi_status acpi_resource_to_address64(struct acpi_resource *resource, struct acpi_resource_address64 *out); +acpi_status +acpi_buffer_to_resource (u8 *aml_buffer, +u16 aml_buffer_length, +struct acpi_resource **resource_ptr); +/* * Hardware (ACPI device) interfaces */
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 15/18 ACPI 5.0: Add new predefined names
In reply to Len BrownFrom: Bob Moore <[email protected]>
--- drivers/acpi/acpica/aclocal.h | 3 ++- drivers/acpi/acpica/acpredef.h | 39 ++++++++++++++++++++++++++++++++++++++- drivers/acpi/acpica/nspredef.c | 29 +++++++++++++++++++++++++++++ drivers/acpi/acpica/nsrepair.c | 1 + drivers/acpi/acpica/nsrepair2.c | 5 +++-- 5 files changed, 73 insertions(+), 4 deletions(-)
drivers/acpi/acpica/aclocal.h +2 -1
@@ -319,7 +319,8 @@ struct acpi_name_info {/* * Used for ACPI_PTYPE1_FIXED, ACPI_PTYPE1_VAR, ACPI_PTYPE2, -* ACPI_PTYPE2_MIN, ACPI_PTYPE2_PKG_COUNT, ACPI_PTYPE2_COUNT +* ACPI_PTYPE2_MIN, ACPI_PTYPE2_PKG_COUNT, ACPI_PTYPE2_COUNT, +* ACPI_PTYPE2_FIX_VAR */ struct acpi_package_info {u8 type;
drivers/acpi/acpica/acpredef.h +38 -1
@@ -94,6 +94,14 @@* ACPI_PTYPE2_REV_FIXED: Revision at start, each subpackage is Fixed-length * (Used for _ART, _FPS) * +* ACPI_PTYPE2_FIX_VAR: Each subpackage consists of some fixed-length elements +* followed by an optional element +* object type +* count +* object type +* count = 0 (optional) +* (Used for _DLM) +* *****************************************************************************/ enum acpi_return_package_types {
@@ -105,7 +113,8 @@ enum acpi_return_package_types {ACPI_PTYPE2_PKG_COUNT =6 ,ACPI_PTYPE2_FIXED =7 ,ACPI_PTYPE2_MIN =8 , - ACPI_PTYPE2_REV_FIXED = 9 +ACPI_PTYPE2_REV_FIXED =9 , + ACPI_PTYPE2_FIX_VAR = 10 };#ifdef ACPI_CREATE_PREDEFINED_TABLE
@@ -154,6 +163,7 @@ static const union acpi_predefined_info predefined_names[] = {{"_AC8", 0, ACPI_RTYPE_INTEGER}}, {{"_AC9", 0, ACPI_RTYPE_INTEGER}}, {{"_ADR", 0, ACPI_RTYPE_INTEGER}}, + {{"_AEI", 0, ACPI_RTYPE_BUFFER}}, {{"_AL0", 0, ACPI_RTYPE_PACKAGE}}, /* Variable-length (Refs) */ {{{ACPI_PTYPE1_VAR, ACPI_RTYPE_REFERENCE, 0,0}, 0,0}},
@@ -229,6 +239,13 @@ static const union acpi_predefined_info predefined_names[] = {{"_CID", 0, ACPI_RTYPE_INTEGER | ACPI_RTYPE_STRING | ACPI_RTYPE_PACKAGE}}, /* Variable-length (Ints/Strs) */ {{{ACPI_PTYPE1_VAR, ACPI_RTYPE_INTEGER | ACPI_RTYPE_STRING, 0,0}, 0,0}}, + {{"_CLS", 0, ACPI_RTYPE_PACKAGE}}, /* Fixed-length (3 Int) */ + {{{ACPI_PTYPE1_FIXED, ACPI_RTYPE_INTEGER, 3, 0}, 0, 0}}, + + {{"_CPC", 0, ACPI_RTYPE_PACKAGE}}, /* Variable-length (Ints/Bufs) */ + {{{ACPI_PTYPE1_VAR, ACPI_RTYPE_INTEGER | ACPI_RTYPE_BUFFER, 0, 0}, 0, + 0}}, + {{"_CRS", 0, ACPI_RTYPE_BUFFER}}, {{"_CRT", 0, ACPI_RTYPE_INTEGER}}, {{"_CSD", 0, ACPI_RTYPE_PACKAGE}}, /* Variable-length (1 Int(n), n-1 Int) */
@@ -237,12 +254,21 @@ static const union acpi_predefined_info predefined_names[] = {{"_CST", 0, ACPI_RTYPE_PACKAGE}}, /* Variable-length (1 Int(n), n Pkg (1 Buf/3 Int) */ {{{ACPI_PTYPE2_PKG_COUNT,ACPI_RTYPE_BUFFER, 1, ACPI_RTYPE_INTEGER}, 3,0}}, + {{"_CWS", 1, ACPI_RTYPE_INTEGER}}, {{"_DCK", 1, ACPI_RTYPE_INTEGER}}, {{"_DCS", 0, ACPI_RTYPE_INTEGER}}, {{"_DDC", 1, ACPI_RTYPE_INTEGER | ACPI_RTYPE_BUFFER}}, {{"_DDN", 0, ACPI_RTYPE_STRING}}, + {{"_DEP", 0, ACPI_RTYPE_PACKAGE}}, /* Variable-length (Refs) */ + {{{ACPI_PTYPE1_VAR, ACPI_RTYPE_REFERENCE, 0, 0}, 0, 0}}, + {{"_DGS", 0, ACPI_RTYPE_INTEGER}}, {{"_DIS", 0, 0}}, + + {{"_DLM", 0, ACPI_RTYPE_PACKAGE}}, /* Variable-length (Pkgs) each (1 Ref, 0/1 Optional Buf/Ref) */ + {{{ACPI_PTYPE2_FIX_VAR, ACPI_RTYPE_REFERENCE, 1, + ACPI_RTYPE_REFERENCE | ACPI_RTYPE_BUFFER}, 0, 0}}, + {{"_DMA", 0, ACPI_RTYPE_BUFFER}}, {{"_DOD", 0, ACPI_RTYPE_PACKAGE}}, /* Variable-length (Ints) */ {{{ACPI_PTYPE1_VAR, ACPI_RTYPE_INTEGER, 0,0}, 0,0}},
@@ -262,6 +288,7 @@ static const union acpi_predefined_info predefined_names[] = {{"_EJ3", 1, 0}}, {{"_EJ4", 1, 0}}, {{"_EJD", 0, ACPI_RTYPE_STRING}}, + {{"_EVT", 1, 0}}, {{"_FDE", 0, ACPI_RTYPE_BUFFER}}, {{"_FDI", 0, ACPI_RTYPE_PACKAGE}}, /* Fixed-length (16 Int) */ {{{ACPI_PTYPE1_FIXED, ACPI_RTYPE_INTEGER, 16,0}, 0,0}},
@@ -281,14 +308,17 @@ static const union acpi_predefined_info predefined_names[] = {{{ACPI_PTYPE1_FIXED, ACPI_RTYPE_INTEGER, 3, 0}, 0, 0}}, {{"_GAI", 0, ACPI_RTYPE_INTEGER}}, + {{"_GCP", 0, ACPI_RTYPE_INTEGER}}, {{"_GHL", 0, ACPI_RTYPE_INTEGER}}, {{"_GLK", 0, ACPI_RTYPE_INTEGER}}, {{"_GPD", 0, ACPI_RTYPE_INTEGER}}, {{"_GPE", 0, ACPI_RTYPE_INTEGER}}, /* _GPE method, not _GPE scope */ + {{"_GRT", 0, ACPI_RTYPE_BUFFER}}, {{"_GSB", 0, ACPI_RTYPE_INTEGER}}, {{"_GTF", 0, ACPI_RTYPE_BUFFER}}, {{"_GTM", 0, ACPI_RTYPE_BUFFER}}, {{"_GTS", 1, 0}}, + {{"_GWS", 1, ACPI_RTYPE_INTEGER}}, {{"_HID", 0, ACPI_RTYPE_INTEGER | ACPI_RTYPE_STRING}}, {{"_HOT", 0, ACPI_RTYPE_INTEGER}}, {{"_HPP", 0, ACPI_RTYPE_PACKAGE}}, /* Fixed-length (4 Int) */
@@ -303,6 +333,7 @@ static const union acpi_predefined_info predefined_names[] = {{"_HPX", 0, ACPI_RTYPE_PACKAGE}}, /* Variable-length (Pkgs) each (var Ints) */ {{{ACPI_PTYPE2_MIN, ACPI_RTYPE_INTEGER, 5,0}, 0,0}}, + {{"_HRV", 0, ACPI_RTYPE_INTEGER}}, {{"_IFT", 0, ACPI_RTYPE_INTEGER}}, /* See IPMI spec */ {{"_INI", 0, 0}}, {{"_IRC", 0, 0}},
@@ -361,6 +392,9 @@ static const union acpi_predefined_info predefined_names[] = {{"_PR3", 0, ACPI_RTYPE_PACKAGE}}, /* Variable-length (Refs) */ {{{ACPI_PTYPE1_VAR, ACPI_RTYPE_REFERENCE, 0, 0}, 0, 0}}, + {{"_PRE", 0, ACPI_RTYPE_PACKAGE}}, /* Variable-length (Refs) */ + {{{ACPI_PTYPE1_VAR, ACPI_RTYPE_REFERENCE, 0, 0}, 0, 0}}, + {{"_PRL", 0, ACPI_RTYPE_PACKAGE}}, /* Variable-length (Refs) */ {{{ACPI_PTYPE1_VAR, ACPI_RTYPE_REFERENCE, 0, 0}, 0, 0}},
@@ -391,6 +425,7 @@ static const union acpi_predefined_info predefined_names[] = {{"_PSD", 0, ACPI_RTYPE_PACKAGE}}, /* Variable-length (Pkgs) each (5 Int) with count */ {{{ACPI_PTYPE2_COUNT, ACPI_RTYPE_INTEGER,0,0}, 0,0}}, + {{"_PSE", 1, 0}}, {{"_PSL", 0, ACPI_RTYPE_PACKAGE}}, /* Variable-length (Refs) */ {{{ACPI_PTYPE1_VAR, ACPI_RTYPE_REFERENCE, 0,0}, 0,0}},
@@ -457,6 +492,7 @@ static const union acpi_predefined_info predefined_names[] = {{"_SLI", 0, ACPI_RTYPE_BUFFER}}, {{"_SPD", 1, ACPI_RTYPE_INTEGER}}, {{"_SRS", 1, 0}}, + {{"_SRT", 1, ACPI_RTYPE_INTEGER}}, {{"_SRV", 0, ACPI_RTYPE_INTEGER}}, /* See IPMI spec */ {{"_SST", 1, 0}}, {{"_STA", 0, ACPI_RTYPE_INTEGER}},
@@ -464,6 +500,7 @@ static const union acpi_predefined_info predefined_names[] = {{"_STP", 2, ACPI_RTYPE_INTEGER}}, {{"_STR", 0, ACPI_RTYPE_BUFFER}}, {{"_STV", 2, ACPI_RTYPE_INTEGER}}, + {{"_SUB", 0, ACPI_RTYPE_STRING}}, {{"_SUN", 0, ACPI_RTYPE_INTEGER}}, {{"_SWS", 0, ACPI_RTYPE_INTEGER}}, {{"_TC1", 0, ACPI_RTYPE_INTEGER}},
drivers/acpi/acpica/nspredef.c +29 -0
@@ -620,6 +620,7 @@ acpi_ns_check_package(struct acpi_predefined_data *data,case ACPI_PTYPE2_FIXED :case ACPI_PTYPE2_MIN :case ACPI_PTYPE2_COUNT : +case ACPI_PTYPE2_FIX_VAR :/* * These types all return a single Package that consists of a
@@ -759,6 +760,34 @@ acpi_ns_check_package_list(struct acpi_predefined_data *data, }break ; +case ACPI_PTYPE2_FIX_VAR : +/* +* Each subpackage has a fixed number of elements and an +* optional element +*/ + expected_count = + package->ret_info.count1 + package->ret_info.count2; +if (sub_package->package.count < expected_count) { + goto package_too_small; + } + + status = +acpi_ns_check_package_elements (data, sub_elements, + package->ret_info. + object_type1, + package->ret_info. + count1, + package->ret_info. + object_type2, + sub_package->package. + count - + package->ret_info. + count1,0 ); +if (ACPI_FAILURE (status)) { +return (status); + } +break ; +case ACPI_PTYPE2_FIXED :/* Each sub-package has a fixed length */
drivers/acpi/acpica/nsrepair.c +1 -0
@@ -634,6 +634,7 @@ acpi_ns_remove_null_elements(struct acpi_predefined_data *data,case ACPI_PTYPE2_FIXED :case ACPI_PTYPE2_MIN :case ACPI_PTYPE2_REV_FIXED : +case ACPI_PTYPE2_FIX_VAR :break ;default :
drivers/acpi/acpica/nsrepair2.c +3 -2
@@ -467,11 +467,12 @@ acpi_ns_repair_HID(struct acpi_predefined_data *data, }/* -* Copy and uppercase the string. From the ACPI specification: +* Copy and uppercase the string. From the ACPI 5.0 specification: * * A valid PNP ID must be of the form "AAA####" where A is an uppercase * letter and # is a hex digit. A valid ACPI ID must be of the form -* "ACPI####" where # is a hex digit. +* "NNNN####" where N is an uppercase letter or decimal digit, and +* # is a hex digit. */ for (dest = new_string->string.pointer; *source; dest++, source++) { *dest = (char )ACPI_TOUPPER (*source);
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 16/18 ACPI 5.0: Allow _AEI method in walk resources
In reply to Len BrownFrom: Zhang Rui <[email protected]>
_AEI contains a resource template, this change adds support for the walk resources function.
--- drivers/acpi/acpica/rsxface.c | 10 ++++++---- include/acpi/acnames.h | 1 + 2 files changed, 7 insertions(+), 4 deletions(-)
drivers/acpi/acpica/rsxface.c +6 -4
@@ -526,8 +526,9 @@ acpi_rs_match_vendor_resource(struct acpi_resource *resource, void *context)* * PARAMETERS: device_handle - Handle to the device object for the * device we are querying -* Name - Method name of the resources we want -* (METHOD_NAME__CRS or METHOD_NAME__PRS) +* Name - Method name of the resources we want. +* (METHOD_NAME__CRS, METHOD_NAME__PRS, or +* METHOD_NAME__AEI) * user_function - Called for each resource * Context - Passed to user_function *
@@ -554,11 +555,12 @@ acpi_walk_resources(acpi_handle device_handle,if (!device_handle || !user_function || !name || (!ACPI_COMPARE_NAME (name,METHOD_NAME__CRS ) && - !ACPI_COMPARE_NAME (name,METHOD_NAME__PRS ))) { + !ACPI_COMPARE_NAME (name,METHOD_NAME__PRS ) && + !ACPI_COMPARE_NAME (name,METHOD_NAME__AEI ))) {return_ACPI_STATUS (AE_BAD_PARAMETER ); } -/* Get the _CRS or _PRS resource list */ +/* Get the _CRS/_PRS/_AEI resource list */ buffer.length =ACPI_ALLOCATE_LOCAL_BUFFER ; status =acpi_rs_get_method_data (device_handle, name, &buffer);
include/acpi/acnames.h +1 -0
@@ -58,6 +58,7 @@#define METHOD_NAME__PRT "_PRT"#define METHOD_NAME__CRS "_CRS"#define METHOD_NAME__PRS "_PRS" +#define METHOD_NAME__AEI "_AEI"#define METHOD_NAME__PRW "_PRW"#define METHOD_NAME__SRS "_SRS"
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 17/18 ACPICA: Clean up Makefile
In reply to Len BrownFrom: Lin Ming <[email protected]>
Make the various files in alphabetical order to simplify addition of new files.
--- drivers/acpi/acpica/Makefile | 157 +++++++++++++++++++++++++++++++++++------- 1 files changed, 133 insertions(+), 24 deletions(-)
drivers/acpi/acpica/Makefile +133 -24
@@ -8,41 +8,150 @@ ccflags-$(CONFIG_ACPI_DEBUG) += -DACPI_DEBUG_OUTPUT # use acpi.o to put all files here into acpi.o modparam namespace obj-y += acpi.o -acpi-y := dsfield.o dsmthdat.o dsopcode.o dswexec.o dswscope.o \ - dsmethod.o dsobject.o dsutils.o dswload.o dswstate.o \ - dsinit.o dsargs.o dscontrol.o dswload2.o +acpi-y := \ + dsargs.o \ + dscontrol.o \ + dsfield.o \ + dsinit.o \ + dsmethod.o \ + dsmthdat.o \ + dsobject.o \ + dsopcode.o \ + dsutils.o \ + dswexec.o \ + dswload.o \ + dswload2.o \ + dswscope.o \ + dswstate.o -acpi-y += evevent.o evregion.o evsci.o evxfevnt.o \ - evmisc.o evrgnini.o evxface.o evxfregn.o \ - evgpe.o evgpeblk.o evgpeinit.o evgpeutil.o evxfgpe.o evglock.o +acpi-y += \ + evevent.o \ + evgpe.o \ + evgpeblk.o \ + evgpeinit.o \ + evgpeutil.o \ + evglock.o \ + evmisc.o \ + evregion.o \ + evrgnini.o \ + evsci.o \ + evxface.o \ + evxfevnt.o \ + evxfgpe.o \ + evxfregn.o -acpi-y += exconfig.o exfield.o exnames.o exoparg6.o exresolv.o exstorob.o\ - exconvrt.o exfldio.o exoparg1.o exprep.o exresop.o exsystem.o\ - excreate.o exmisc.o exoparg2.o exregion.o exstore.o exutils.o \ - exdump.o exmutex.o exoparg3.o exresnte.o exstoren.o exdebug.o +acpi-y += \ + exconfig.o \ + exconvrt.o \ + excreate.o \ + exdebug.o \ + exdump.o \ + exfield.o \ + exfldio.o \ + exmutex.o \ + exnames.o \ + exoparg1.o \ + exoparg2.o \ + exoparg3.o \ + exoparg6.o \ + exprep.o \ + exmisc.o \ + exregion.o \ + exresnte.o \ + exresolv.o \ + exresop.o \ + exstore.o \ + exstoren.o \ + exstorob.o \ + exsystem.o \ + exutils.o -acpi-y += hwacpi.o hwgpe.o hwregs.o hwsleep.o hwxface.o hwvalid.o hwpci.o +acpi-y += \ + hwacpi.o \ + hwgpe.o \ + hwpci.o \ + hwregs.o \ + hwsleep.o \ + hwvalid.o \ + hwxface.o acpi-$(ACPI_FUTURE_USAGE) += hwtimer.o -acpi-y += nsaccess.o nsload.o nssearch.o nsxfeval.o \ - nsalloc.o nseval.o nsnames.o nsutils.o nsxfname.o \ - nsdump.o nsinit.o nsobject.o nswalk.o nsxfobj.o \ - nsparse.o nspredef.o nsrepair.o nsrepair2.o +acpi-y += \ + nsaccess.o \ + nsalloc.o \ + nsdump.o \ + nseval.o \ + nsinit.o \ + nsload.o \ + nsnames.o \ + nsobject.o \ + nsparse.o \ + nspredef.o \ + nsrepair.o \ + nsrepair2.o \ + nssearch.o \ + nsutils.o \ + nswalk.o \ + nsxfeval.o \ + nsxfname.o \ + nsxfobj.o acpi-$(ACPI_FUTURE_USAGE) += nsdumpdv.o -acpi-y += psargs.o psparse.o psloop.o pstree.o pswalk.o \ - psopcode.o psscope.o psutils.o psxface.o +acpi-y += \ + psargs.o \ + psloop.o \ + psopcode.o \ + psparse.o \ + psscope.o \ + pstree.o \ + psutils.o \ + pswalk.o \ + psxface.o -acpi-y += rsaddr.o rscreate.o rsinfo.o rsio.o rslist.o rsmisc.o rsxface.o \ - rscalc.o rsirq.o rsmemory.o rsutils.o rsserial.o +acpi-y += \ + rsaddr.o \ + rscalc.o \ + rscreate.o \ + rsinfo.o \ + rsio.o \ + rsirq.o \ + rslist.o \ + rsmemory.o \ + rsmisc.o \ + rsserial.o \ + rsutils.o \ + rsxface.o acpi-$(ACPI_FUTURE_USAGE) += rsdump.o -acpi-y += tbxface.o tbinstal.o tbutils.o tbfind.o tbfadt.o tbxfroot.o +acpi-y += \ + tbfadt.o \ + tbfind.o \ + tbinstal.o \ + tbutils.o \ + tbxface.o \ + tbxfroot.o -acpi-y += utalloc.o utdebug.o uteval.o utinit.o utmisc.o utxface.o \ - utcopy.o utdelete.o utglobal.o utmath.o utobject.o \ - utstate.o utmutex.o utobject.o utresrc.o utlock.o utids.o \ - utosi.o utxferror.o utdecode.o utxfmutex.o +acpi-y += \ + utalloc.o \ + utcopy.o \ + utdebug.o \ + utdecode.o \ + utdelete.o \ + uteval.o \ + utglobal.o \ + utids.o \ + utinit.o \ + utlock.o \ + utmath.o \ + utmisc.o \ + utmutex.o \ + utobject.o \ + utosi.o \ + utresrc.o \ + utstate.o \ + utxface.o \ + utxferror.o \ + utxfmutex.o
Signature · 1 line
1.7.7.rc0.72.g4b5ea
Len Brown
reviewerPATCH 18/18 ACPICA: Update to version 20111123
In reply to Len BrownFrom: Bob Moore <[email protected]>
This version contains full support for the ACPI 5.0 specification.
--- include/acpi/acpixf.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
include/acpi/acpixf.h +1 -1
@@ -47,7 +47,7 @@/* Current ACPICA subsystem version in YYYYMMDD format */ -#define ACPI_CA_VERSION 0x20110623 +#define ACPI_CA_VERSION 0x20111123#include "actypes.h" #include "actbl.h"
Signature · 1 line
1.7.7.rc0.72.g4b5ea