ACPICA 20121018 Release
Subsystems (2)
cover ACPICA 20121018 Release
Here are the 20121018 ACPICA linuxized patches based on the acpi/next branch, please review.
Bob Moore (18):
Cleanup of invalid ACPI name handling and repair
ACPICA: Audit/update for ACPICA return macros and debug depth counter
ACPICA: ACPICA core: Cleanup empty lines at file start and end
ACPICA: Fix some typos in comments
ACPICA: Update local C library module comments for ASCII table
ACPICA: Remove extra spaces after periods within comments
ACPICA: Remove extra spaces after periods in the Intel license
ACPICA: Add debug print message for mutex objects that are
force-released
ACPICA: AcpiExec: Improve algorithm for tracking memory leaks
ACPICA: Add ACPI_MOVE_NAME macro to optimize 4-byte ACPI_NAME copies
ACPICA: Enhance error reporting for invalid opcodes and bad
ACPI_NAMEs
ACPICA: Update support for ACPI 5 MPST table
ACPICA: Deploy ACPI_MOVE_NAME across ACPICA source base
ACPICA: Add starting offset parameter to common dump buffer routine
ACPICA: Fix externalize name to complete migration to ACPI_MOVE_NAME
ACPICA: Update for 64-bit generation of recent error message changes
ACPICA: AcpiGetObjectInfo: Add support for ACPI 5 _SUB method
ACPICA: Update version to 20121018
Feng Tang (1):
ACPICA: Resource Mgr: Small fix for buffer size calculation
Lv Zheng (6):
ACPICA: Fix unmerged utility divergences.
ACPICA: Fix unmerged debugger divergences.
ACPICA: Fix divergences of definition conflicts.
ACPICA: Fix AcpiSrc caused divergences.
ACPICA: Fix indent caused divergences.
ACPICA: Fix unmerged acmacros.h divergences.
Robert Moore (1):
ACPICA: Fix for predefined name loop during ACPICA initialization
drivers/acpi/acpica/Makefile | 3 + drivers/acpi/acpica/acdebug.h | 94 +++-- drivers/acpi/acpica/acdispat.h | 11 +- drivers/acpi/acpica/acevents.h | 6 +- drivers/acpi/acpica/acglobal.h | 73 ++-- drivers/acpi/acpica/aclocal.h | 16 +- drivers/acpi/acpica/acmacros.h | 163 +++------ drivers/acpi/acpica/acobject.h | 7 +- drivers/acpi/acpica/acopcode.h | 6 +- drivers/acpi/acpica/acparser.h | 3 +- drivers/acpi/acpica/acpredef.h | 11 +- drivers/acpi/acpica/acstruct.h | 2 +- drivers/acpi/acpica/acutils.h | 58 ++- drivers/acpi/acpica/amlresrc.h | 1 - drivers/acpi/acpica/dscontrol.c | 2 +- drivers/acpi/acpica/dsfield.c | 2 +- drivers/acpi/acpica/dsmethod.c | 6 +- drivers/acpi/acpica/dsmthdat.c | 14 +- drivers/acpi/acpica/dsobject.c | 6 +- drivers/acpi/acpica/dsopcode.c | 3 +- drivers/acpi/acpica/dsutils.c | 33 +- drivers/acpi/acpica/dswexec.c | 10 +- drivers/acpi/acpica/dswload2.c | 4 +- drivers/acpi/acpica/dswstate.c | 26 +- drivers/acpi/acpica/evgpe.c | 20 +- drivers/acpi/acpica/evgpeblk.c | 3 +- drivers/acpi/acpica/evgpeutil.c | 3 +- drivers/acpi/acpica/evrgnini.c | 7 +- drivers/acpi/acpica/evxface.c | 2 +- drivers/acpi/acpica/evxfgpe.c | 13 +- drivers/acpi/acpica/exconvrt.c | 4 +- drivers/acpi/acpica/excreate.c | 9 +- drivers/acpi/acpica/exdebug.c | 10 +- drivers/acpi/acpica/exdump.c | 20 +- drivers/acpi/acpica/exfield.c | 4 +- drivers/acpi/acpica/exfldio.c | 15 +- drivers/acpi/acpica/exmisc.c | 5 +- drivers/acpi/acpica/exmutex.c | 9 +- drivers/acpi/acpica/exnames.c | 9 +- drivers/acpi/acpica/exoparg1.c | 11 +- drivers/acpi/acpica/exoparg2.c | 2 +- drivers/acpi/acpica/exoparg3.c | 3 +- drivers/acpi/acpica/exoparg6.c | 5 +- drivers/acpi/acpica/exprep.c | 13 +- drivers/acpi/acpica/exregion.c | 3 +- drivers/acpi/acpica/exresnte.c | 9 +- drivers/acpi/acpica/exresolv.c | 3 +- drivers/acpi/acpica/exresop.c | 8 +- drivers/acpi/acpica/exstore.c | 4 +- drivers/acpi/acpica/exstoren.c | 11 +- drivers/acpi/acpica/exstorob.c | 5 +- drivers/acpi/acpica/exsystem.c | 9 +- drivers/acpi/acpica/exutils.c | 5 +- drivers/acpi/acpica/hwacpi.c | 3 +- drivers/acpi/acpica/hwgpe.c | 4 +- drivers/acpi/acpica/hwpci.c | 4 +- drivers/acpi/acpica/hwregs.c | 1 - drivers/acpi/acpica/hwtimer.c | 6 +- drivers/acpi/acpica/hwvalid.c | 1 - drivers/acpi/acpica/hwxface.c | 1 - drivers/acpi/acpica/hwxfsleep.c | 12 +- drivers/acpi/acpica/nsaccess.c | 7 +- drivers/acpi/acpica/nsalloc.c | 4 +- drivers/acpi/acpica/nsdump.c | 10 +- drivers/acpi/acpica/nsinit.c | 4 +- drivers/acpi/acpica/nsload.c | 10 +- drivers/acpi/acpica/nsnames.c | 2 +- drivers/acpi/acpica/nsobject.c | 8 +- drivers/acpi/acpica/nsparse.c | 8 +- drivers/acpi/acpica/nssearch.c | 17 +- drivers/acpi/acpica/nsutils.c | 18 +- drivers/acpi/acpica/nswalk.c | 10 +- drivers/acpi/acpica/nsxfeval.c | 20 +- drivers/acpi/acpica/nsxfname.c | 66 ++-- drivers/acpi/acpica/nsxfobj.c | 4 +- drivers/acpi/acpica/psargs.c | 8 +- drivers/acpi/acpica/psloop.c | 59 ++- drivers/acpi/acpica/psopcode.c | 29 +- drivers/acpi/acpica/psparse.c | 13 +- drivers/acpi/acpica/psutils.c | 4 +- drivers/acpi/acpica/rscalc.c | 14 +- drivers/acpi/acpica/rslist.c | 4 +- drivers/acpi/acpica/tbfind.c | 2 +- drivers/acpi/acpica/tbinstal.c | 2 + drivers/acpi/acpica/tbutils.c | 2 +- drivers/acpi/acpica/tbxface.c | 4 +- drivers/acpi/acpica/tbxfload.c | 2 +- drivers/acpi/acpica/tbxfroot.c | 3 +- drivers/acpi/acpica/utcache.c | 323 +++++++++++++++++ drivers/acpi/acpica/utclib.c | 749 +++++++++++++++++++++++++++++++++++++++ drivers/acpi/acpica/utdebug.c | 37 +- drivers/acpi/acpica/utids.c | 104 +++++- drivers/acpi/acpica/utmath.c | 2 +- drivers/acpi/acpica/utmisc.c | 150 ++++++-- drivers/acpi/acpica/utmutex.c | 14 +- drivers/acpi/acpica/utobject.c | 8 +- drivers/acpi/acpica/utstate.c | 2 +- drivers/acpi/acpica/uttrack.c | 692 ++++++++++++++++++++++++++++++++++++ drivers/acpi/acpica/utxface.c | 5 +- drivers/acpi/acpica/utxferror.c | 2 +- drivers/acpi/scan.c | 2 +- drivers/acpi/sysfs.c | 4 +- include/acpi/acconfig.h | 1 + include/acpi/acexcep.h | 2 +- include/acpi/acnames.h | 1 + include/acpi/acpiosxf.h | 3 +- include/acpi/acpixf.h | 18 +- include/acpi/actbl3.h | 22 +- include/acpi/actypes.h | 42 ++- 109 files changed, 2669 insertions(+), 664 deletions(-) create mode 100644 drivers/acpi/acpica/utcache.c create mode 100644 drivers/acpi/acpica/utclib.c create mode 100644 drivers/acpi/acpica/uttrack.c
Signature · 1 line
1.7.10
PATCH 1/26 Cleanup of invalid ACPI name handling and repair
From: Bob Moore <[email protected]>
Implemented a change/cleanup for the handling of invalid ACPI names. Names are now validated and repaired only when
1) entering a new name into the namespace and
2) disassembling a named AML opcode. A warning is only displayed in debug mode or when the interpreter is in "strict" mode, since some working machines do in fact contain invalid ACPI names.
--- drivers/acpi/acpica/acutils.h | 2 +- drivers/acpi/acpica/nsdump.c | 8 -------- drivers/acpi/acpica/nssearch.c | 17 +---------------- drivers/acpi/acpica/utmisc.c | 34 ++++++++++++++++++++++++++-------- 4 files changed, 28 insertions(+), 33 deletions(-)
drivers/acpi/acpica/acutils.h +1 -1
@@ -483,7 +483,7 @@ void acpi_ut_print_string(char *string, u8 max_length);u8 acpi_ut_valid_acpi_name (u32 name); -acpi_name acpi_ut_repair_name (char *name); +void acpi_ut_repair_name (char *name);u8 acpi_ut_valid_acpi_char (char character,u32 position);
drivers/acpi/acpica/nsdump.c +0 -8
@@ -209,14 +209,6 @@ acpi_ns_dump_one_object(acpi_handle obj_handle, "Invalid ACPI Object Type 0x%08X", type)); } -if (!acpi_ut_valid_acpi_name (this_node->name.integer)) { - this_node->name.integer = -acpi_ut_repair_name (this_node->name.ascii); - -ACPI_WARNING ((AE_INFO ,"Invalid ACPI Name %08X" , - this_node->name.integer)); - } -acpi_os_printf ("%4.4s" ,acpi_ut_get_node_name (this_node)); }
drivers/acpi/acpica/nssearch.c +1 -16
@@ -314,22 +314,7 @@ acpi_ns_search_and_enter(u32 target_name,* this problem, and we want to be able to enable ACPI support for them, * even though there are a few bad names. */ -if (!acpi_ut_valid_acpi_name (target_name)) { - target_name = -acpi_ut_repair_name (ACPI_CAST_PTR (char, &target_name)); - -/* Report warning only if in strict mode or debug mode */ - -if (!acpi_gbl_enable_interpreter_slack) { -ACPI_WARNING ((AE_INFO , -"Found bad character(s) in name, repaired: [%4.4s]\n" , -ACPI_CAST_PTR (char, &target_name))); - }else { -ACPI_DEBUG_PRINT ((ACPI_DB_INFO , -"Found bad character(s) in name, repaired: [%4.4s]\n" , -ACPI_CAST_PTR (char, &target_name))); - } - } +acpi_ut_repair_name (ACPI_CAST_PTR (char, &target_name));/* Try to find the name in the namespace level specified by the caller */
drivers/acpi/acpica/utmisc.c +26 -8
@@ -642,25 +642,43 @@ u8 acpi_ut_valid_acpi_name(u32 name)* ******************************************************************************/ -acpi_name acpi_ut_repair_name (char *name) +void acpi_ut_repair_name (char *name) { -u32 i; -char new_name[ACPI_NAME_SIZE ]; +u32 i; +u8 found_bad_char = FALSE; + +ACPI_FUNCTION_NAME (ut_repair_name); + +/* Check each character in the name */ for (i =0 ; i <ACPI_NAME_SIZE ; i++) { - new_name[i] = name[i]; +if (acpi_ut_valid_acpi_char (name[i], i)) { +continue ; + }/* * Replace a bad character with something printable, yet technically * still invalid. This prevents any collisions with existing "good" * names in the namespace. */ -if (!acpi_ut_valid_acpi_char (name[i], i)) { - new_name[i] ='*' ; - } + name[i] ='*' ; + found_bad_char = TRUE; } -return (*(u32 *) new_name); +if (found_bad_char) { + +/* Report warning only if in strict mode or debug mode */ + +if (!acpi_gbl_enable_interpreter_slack) { +ACPI_WARNING ((AE_INFO , +"Found bad character(s) in name, repaired: [%4.4s]\n" , + name)); + }else { +ACPI_DEBUG_PRINT ((ACPI_DB_WARN , +"Found bad character(s) in name, repaired: [%4.4s]\n" , + name)); + } + } }/*******************************************************************************
Signature · 1 line
1.7.10
PATCH 2/26 ACPICA: Fix unmerged utility divergences.
In reply to Lv ZhengUtility improvements in ACPICA are partial ignored by ACPICA Linux release. This will lead to divergences between Linux and ACPICA. This patch ports the entire "utility" into Linux and makes them igored in the compilation stage by "ACPI_FUTURE_USAGE". The following "Utility" files have been ported into the Linux:
drivers/acpi/uttrack.c
drivers/acpi/utcache.c
drivers/acpi/utids.c
This patch will not affect the generated vmlinx binary. This will decrease 274 lines of 20120913 divergence.diff.
--- drivers/acpi/acpica/Makefile | 3 + drivers/acpi/acpica/acutils.h | 51 ++- drivers/acpi/acpica/utcache.c | 323 ++++++++++++++++++ drivers/acpi/acpica/utclib.c | 748 +++++++++++++++++++++++++++++++++++++++++ drivers/acpi/acpica/utdebug.c | 9 +- drivers/acpi/acpica/utmisc.c | 99 +++++- drivers/acpi/acpica/uttrack.c | 650 +++++++++++++++++++++++++++++++++++ 7 files changed, 1852 insertions(+), 31 deletions(-) create mode 100644 drivers/acpi/acpica/utcache.c create mode 100644 drivers/acpi/acpica/utclib.c create mode 100644 drivers/acpi/acpica/uttrack.c
drivers/acpi/acpica/Makefile +3 -0
@@ -161,3 +161,6 @@ acpi-y += \ utxfinit.o \ utxferror.o \ utxfmutex.o + +acpi-$(ACPI_FUTURE_USAGE) += uttrack.o utcache.o utclib.o +
drivers/acpi/acpica/acutils.h +36 -15
131 lines not on this page · raw message
drivers/acpi/acpica/utcache.c new file +323 -0
323 lines not on this page · raw message
drivers/acpi/acpica/utclib.c new file +748 -0
748 lines not on this page · raw message
drivers/acpi/acpica/utdebug.c +5 -4
@@ -47,8 +47,9 @@#define _COMPONENT ACPI_UTILITIES ACPI_MODULE_NAME("utdebug") +#ifdef ACPI_DEBUG_OUTPUT -static acpi_thread_id acpi_gbl_prev_thread_id; +static acpi_thread_id acpi_gbl_prev_thread_id = (acpi_thread_id )0xFFFFFFFF ;static char *acpi_gbl_fn_entry_str ="----Entry" ;static char *acpi_gbl_fn_exit_str ="----Exit-" ;
@@ -109,7 +110,7 @@ void acpi_ut_track_stack_ptr(void)* RETURN: Updated pointer to the function name * * DESCRIPTION: Remove the "Acpi" prefix from the function name, if present. -* This allows compiler macros such as __func__ to be used +* This allows compiler macros such as __FUNCTION__ to be used * with no change to the debug output. * ******************************************************************************/
@@ -519,7 +520,7 @@ acpi_ut_ptr_exit(u32 line_number,* ******************************************************************************/ -void acpi_ut_dump_buffer2 (u8 * buffer,u32 count,u32 display) +void acpi_ut_dump_buffer2 (u8 *buffer,u32 count,u32 display) {u32 i =0 ;u32 j;
@@ -636,7 +637,7 @@ void acpi_ut_dump_buffer2(u8 * buffer, u32 count, u32 display)* ******************************************************************************/ -void acpi_ut_dump_buffer (u8 * buffer,u32 count,u32 display,u32 component_id) +void acpi_ut_dump_buffer (u8 *buffer,u32 count,u32 display,u32 component_id) {/* Only dump the buffer if tracing is enabled */
drivers/acpi/acpica/utmisc.c +87 -12
159 lines not on this page · raw message
drivers/acpi/acpica/uttrack.c new file +650 -0
650 lines not on this page · raw message
Signature · 1 line
1.7.10
PATCH 3/26 ACPICA: Fix unmerged debugger divergences.
In reply to Lv ZhengDebugger improvements in ACPICA are always ignored by ACPICA Linux release. This will lead to divergences between Linux and ACPICA. This patch fixes such unmerged debugger updates.
Following patches are included:
1. Fixed a couple compiler warnings for extra extern Wed, 14 Mar 2007 21:12:19 +0000
2. Cleanup for internal Reference Object.
Wed, 27 Aug 2008 10:11:30 -0700
3. Debugger: Lock method args for multithread command. Fri, 24 Apr 2009 12:28:49 -0700
4. Debugger: Add max count argument for Batch command. Tue, 29 Sep 2009 12:31:58 -0700
5. Add new host interfaces for _OSI support.
Thu, 5 Aug 2010 14:18:28 -0700
6. Increase debugger buffer size for method return objects. Wed, 17 Nov 2010 13:48:30 -0800
7. Debugger: Add command to display status of global handlers. Tue, 25 Jan 2011 13:47:58 -0800
8. Debugger: Split large dbcmds.c file.
Wed, 26 Jan 2011 13:03:41 -0800
9. Debugger/AcpiExec: Add support to pass complex args to methods. Tue, 17 May 2011 13:33:39 -0700
10.Debugger: Add Template command to dump resource templates.
Fri, 28 Oct 2011 14:18:51 -0700
11.Support for custom ACPICA build for ACPI 5.0 reduced hardware.
Wed, 1 Feb 2012 13:18:17 -0800
12.Debugger: Improve command help support.
Wed, 15 Feb 2012 07:59:26 -0800
13.Update ACPI_HW_DEPENDENT* macro invocations.
Wed, 15 Feb 2012 08:14:08 -0800
14.Debugger: Rename function to simplify source code conversion.
Wed, 13 Jun 2012 14:23:06 -0700
15.Debugger: Enhance "Tables" and "Unload" commands.
Fri, 29 Jun 2012 13:10:58 -0700
16.Debugger: update prototype for AcpiDbSleep function.
Fri, 17 Aug 2012 13:43:02 -0700
This patch will not affect the generated vmlinx binary. This will decrease 264 lines of 20120913 divergence.diff.
--- drivers/acpi/acpica/acdebug.h | 94 ++++++++++++++++++++++++++-------------- drivers/acpi/acpica/acglobal.h | 69 +++++++++++++++-------------- drivers/acpi/acpica/aclocal.h | 2 + include/acpi/acconfig.h | 1 + 4 files changed, 101 insertions(+), 65 deletions(-)
drivers/acpi/acpica/acdebug.h +62 -32
@@ -44,17 +44,28 @@#ifndef __ACDEBUG_H__#define __ACDEBUG_H__ -#define ACPI_DEBUG_BUFFER_SIZE 4196 +#define ACPI_DEBUG_BUFFER_SIZE 0x4000/* 16K buffer for return objects */ -struct command_info { +struct acpi_db_command_info {char *name;/* Command Name */ u8 min_args;/* Minimum arguments required */ }; -struct argument_info { +struct acpi_db_command_help { +u8 line_count;/* Number of help lines */ +char *invocation;/* Command Invocation */ +char *description;/* Command Description */ +}; + +struct acpi_db_argument_info {char *name;/* Argument Name */ }; +struct acpi_db_execute_walk { +u32 count; +u32 max_count; +}; +#define PARAM_LIST (pl) pl#define DBTEST_OUTPUT_LEVEL (lvl) if (acpi_gbl_db_opt_verbose)#define VERBOSE_PRINT (fp) DBTEST_OUTPUT_LEVEL(lvl) {\
@@ -77,59 +88,71 @@ acpi_db_single_step(struct acpi_walk_state *walk_state,/* * dbcmds - debug commands and output routines */ -acpi_status acpi_db_disassemble_method (char *name); +struct acpi_namespace_node *acpi_db_convert_to_node (char *in_string);void acpi_db_display_table_info (char *table_arg); -void acpi_db_unload_acpi_table (char *table_arg,char *instance_arg); +void acpi_db_display_template (char *buffer_arg); -void -acpi_db_set_method_breakpoint (char *location, -struct acpi_walk_state *walk_state, -union acpi_parse_object *op); +void acpi_db_unload_acpi_table (char *name); -void acpi_db_set_method_call_breakpoint (union acpi_parse_object *op); +void acpi_db_send_notify (char *name,u32 value); -void acpi_db_get_bus_info (void ); +void acpi_db_display_interfaces (char *action_arg,char *interface_name_arg); -void acpi_db_disassemble_aml (char *statements,union acpi_parse_object *op); +acpi_status acpi_db_sleep (char *object_arg); -void acpi_db_dump_namespace (char *start_arg,char *depth_arg); +void acpi_db_display_locks (void ); -void acpi_db_dump_namespace_by_owner (char *owner_arg,char *depth_arg); +void acpi_db_display_resources (char *object_arg); -void acpi_db_send_notify (char *name,u32 value); +ACPI_HW_DEPENDENT_RETURN_VOID(void acpi_db_display_gpes(void)) + +void acpi_db_display_handlers(void); + +ACPI_HW_DEPENDENT_RETURN_VOID(void + acpi_db_generate_gpe(char *gpe_arg, +char *block_arg)) + +/* +* dbmethod - control method commands +*/ +void +acpi_db_set_method_breakpoint (char *location, +struct acpi_walk_state *walk_state, +union acpi_parse_object *op); + +void acpi_db_set_method_call_breakpoint (union acpi_parse_object *op);void acpi_db_set_method_data (char *type_arg,char *index_arg,char *value_arg); -acpi_status -acpi_db_display_objects (char *obj_type_arg,char *display_count_arg); +acpi_status acpi_db_disassemble_method (char *name); -void acpi_db_display_interfaces (char *action_arg,char *interface_name_arg); +void acpi_db_disassemble_aml (char *statements,union acpi_parse_object *op); -acpi_status acpi_db_find_name_in_namespace (char *name_arg); +void acpi_db_batch_execute (char *count_arg); +/* +* dbnames - namespace commands +*/ void acpi_db_set_scope (char *name); -ACPI_HW_DEPENDENT_RETURN_OK(acpi_status acpi_db_sleep(char *object_arg)) +void acpi_db_dump_namespace (char *start_arg,char *depth_arg); -void acpi_db_find_references(char *object_arg); +void acpi_db_dump_namespace_by_owner (char *owner_arg,char *depth_arg); -void acpi_db_display_locks (void ); +acpi_status acpi_db_find_name_in_namespace (char *name_arg); -void acpi_db_display_resources (char *object_arg); +void acpi_db_check_predefined_names (void ); -ACPI_HW_DEPENDENT_RETURN_VOID(void acpi_db_display_gpes(void)) +acpi_status +acpi_db_display_objects (char *obj_type_arg,char *display_count_arg);void acpi_db_check_integrity (void ); -ACPI_HW_DEPENDENT_RETURN_VOID(void - acpi_db_generate_gpe(char *gpe_arg, -char *block_arg)) - -void acpi_db_check_predefined_names(void); +void acpi_db_find_references (char *object_arg); -void acpi_db_batch_execute (void ); +void acpi_db_get_bus_info (void );/* * dbdisply - debug display commands
@@ -161,7 +184,8 @@ acpi_db_display_argument_object(union acpi_operand_object *obj_desc,/* * dbexec - debugger control method execution */ -void acpi_db_execute (char *name,char **args,u32 flags); +void +acpi_db_execute (char *name,char **args,acpi_object_type * types,u32 flags); void acpi_db_create_execution_threads(char *num_threads_arg,
@@ -175,7 +199,8 @@ u32 acpi_db_get_cache_info(struct acpi_memory_list *cache);* dbfileio - Debugger file I/O commands */ acpi_object_type -acpi_db_match_argument (char *user_argument,struct argument_info *arguments); +acpi_db_match_argument (char *user_argument, +struct acpi_db_argument_info *arguments);void acpi_db_close_debug_file (void );
@@ -208,6 +233,11 @@ acpi_db_command_dispatch(char *input_buffer, void ACPI_SYSTEM_XFACE acpi_db_execute_thread(void *context); +acpi_status acpi_db_user_commands (char prompt,union acpi_parse_object *op); + +char *acpi_db_get_next_token (char *string, +char **next,acpi_object_type * return_type); +/* * dbstats - Generation and display of ACPI table statistics */
drivers/acpi/acpica/acglobal.h +36 -33
99 lines not on this page · raw message
drivers/acpi/acpica/aclocal.h +2 -0
@@ -1031,6 +1031,7 @@ struct acpi_db_method_info {acpi_handle method;acpi_handle main_thread_gate;acpi_handle thread_complete_gate; +acpi_handle info_gate;acpi_thread_id *threads;u32 num_threads;u32 num_created;
@@ -1041,6 +1042,7 @@ struct acpi_db_method_info {u32 num_loops;char pathname[128 ];char **args; +acpi_object_type *types;/* * Arguments to be passed to method for the command
include/acpi/acconfig.h +1 -0
@@ -241,6 +241,7 @@*****************************************************************************/ #define ACPI_DEBUGGER_MAX_ARGS 8/* Must be max method args + 1 */ +#define ACPI_DB_LINE_BUFFER_SIZE 512#define ACPI_DEBUGGER_COMMAND_PROMPT '-'#define ACPI_DEBUGGER_EXECUTE_PROMPT '%'
Signature · 1 line
1.7.10
PATCH 4/26 ACPICA: Fix divergences of definition conflicts.
In reply to Lv ZhengThere are conflicts in the "acpi_device_id*" definitions between the Linux and the ACPICA. The definitions of acpi_device_id* in ACPICA have been changed to the "acpi_pnp_device_id*". This patch changes the corresponding "acpica_device_id*" definitiions in the Linux.
This patch will not affect the generated vmlinx binary. This will decrease 298 lines of 20120913 divergence.diff.
--- drivers/acpi/acpica/acutils.h | 6 +++--- drivers/acpi/acpica/evrgnini.c | 4 ++-- drivers/acpi/acpica/nsxfeval.c | 4 ++-- drivers/acpi/acpica/nsxfname.c | 28 ++++++++++++++-------------- drivers/acpi/acpica/utids.c | 37 ++++++++++++++++++++----------------- drivers/acpi/scan.c | 2 +- include/acpi/actypes.h | 12 ++++++------ 7 files changed, 48 insertions(+), 45 deletions(-)
drivers/acpi/acpica/acutils.h +3 -3
@@ -355,15 +355,15 @@ acpi_ut_execute_power_methods(struct acpi_namespace_node *device_node,*/ acpi_status acpi_ut_execute_HID (struct acpi_namespace_node *device_node, -struct acpica_device_id ** return_id); +struct acpi_pnp_device_id ** return_id);acpi_status acpi_ut_execute_UID (struct acpi_namespace_node *device_node, -struct acpica_device_id ** return_id); +struct acpi_pnp_device_id ** return_id);acpi_status acpi_ut_execute_CID (struct acpi_namespace_node *device_node, -struct acpica_device_id_list ** return_cid_list); +struct acpi_pnp_device_id_list ** return_cid_list);/* * utlock - reader/writer locks
drivers/acpi/acpica/evrgnini.c +2 -2
@@ -350,8 +350,8 @@ acpi_ev_pci_config_region_setup(acpi_handle handle,static u8 acpi_ev_is_pci_root_bridge (struct acpi_namespace_node *node) {acpi_status status; -struct acpica_device_id *hid; -struct acpica_device_id_list *cid; +struct acpi_pnp_device_id *hid; +struct acpi_pnp_device_id_list *cid;u32 i;u8 match;
drivers/acpi/acpica/nsxfeval.c +2 -2
@@ -542,8 +542,8 @@ acpi_ns_get_device_callback(acpi_handle obj_handle,acpi_status status;struct acpi_namespace_node *node;u32 flags; -struct acpica_device_id *hid; -struct acpica_device_id_list *cid; +struct acpi_pnp_device_id *hid; +struct acpi_pnp_device_id_list *cid;u32 i;u8 found;int no_match;
drivers/acpi/acpica/nsxfname.c +14 -14
@@ -53,8 +53,8 @@ACPI_MODULE_NAME ("nsxfname" ) /* Local prototypes */ -static char *acpi_ns_copy_device_id (struct acpica_device_id *dest, -struct acpica_device_id *source, +static char *acpi_ns_copy_device_id (struct acpi_pnp_device_id *dest, +struct acpi_pnp_device_id *source,char *string_area);/******************************************************************************
@@ -219,20 +219,20 @@ ACPI_EXPORT_SYMBOL(acpi_get_name)* * FUNCTION: acpi_ns_copy_device_id * -* PARAMETERS: dest - Pointer to the destination DEVICE_ID -* source - Pointer to the source DEVICE_ID +* PARAMETERS: dest - Pointer to the destination PNP_DEVICE_ID +* source - Pointer to the source PNP_DEVICE_ID * string_area - Pointer to where to copy the dest string * * RETURN: Pointer to the next string area * -* DESCRIPTION: Copy a single DEVICE_ID, including the string data. +* DESCRIPTION: Copy a single PNP_DEVICE_ID, including the string data. * ******************************************************************************/ -static char *acpi_ns_copy_device_id (struct acpica_device_id *dest, -struct acpica_device_id *source, +static char *acpi_ns_copy_device_id (struct acpi_pnp_device_id *dest, +struct acpi_pnp_device_id *source,char *string_area) { -/* Create the destination DEVICE_ID */ +/* Create the destination PNP_DEVICE_ID */ dest->string = string_area; dest->length = source->length;
@@ -269,9 +269,9 @@ acpi_get_object_info(acpi_handle handle, {struct acpi_namespace_node *node;struct acpi_device_info *info; -struct acpica_device_id_list *cid_list =NULL ; -struct acpica_device_id *hid =NULL ; -struct acpica_device_id *uid =NULL ; +struct acpi_pnp_device_id_list *cid_list =NULL ; +struct acpi_pnp_device_id *hid =NULL ; +struct acpi_pnp_device_id *uid =NULL ;char *next_id_string;acpi_object_type type;acpi_name name;
@@ -348,7 +348,7 @@ acpi_get_object_info(acpi_handle handle, info_size += (cid_list->list_size - -sizeof (struct acpica_device_id_list )); +sizeof (struct acpi_pnp_device_id_list )); valid |=ACPI_VALID_CID ; } }
@@ -418,11 +418,11 @@ acpi_get_object_info(acpi_handle handle, next_id_string =ACPI_CAST_PTR (char, info->compatible_id_list.ids);if (cid_list) { -/* Point past the CID DEVICE_ID array */ +/* Point past the CID PNP_DEVICE_ID array */ next_id_string += ((acpi_size ) cid_list->count * -sizeof (struct acpica_device_id )); +sizeof (struct acpi_pnp_device_id )); }/*
drivers/acpi/acpica/utids.c +20 -17
@@ -67,10 +67,10 @@ ACPI_MODULE_NAME("utids")******************************************************************************/ acpi_status acpi_ut_execute_HID (struct acpi_namespace_node *device_node, -struct acpica_device_id **return_id) +struct acpi_pnp_device_id **return_id) {union acpi_operand_object *obj_desc; -struct acpica_device_id *hid; +struct acpi_pnp_device_id *hid;u32 length;acpi_status status;
@@ -94,16 +94,17 @@ acpi_ut_execute_HID(struct acpi_namespace_node *device_node,/* Allocate a buffer for the HID */ hid = -ACPI_ALLOCATE_ZEROED (sizeof (struct acpica_device_id ) + +ACPI_ALLOCATE_ZEROED (sizeof (struct acpi_pnp_device_id ) + (acpi_size ) length);if (!hid) { status =AE_NO_MEMORY ; goto cleanup; } -/* Area for the string starts after DEVICE_ID struct */ +/* Area for the string starts after PNP_DEVICE_ID struct */ - hid->string =ACPI_ADD_PTR (char, hid,sizeof (struct acpica_device_id )); + hid->string = +ACPI_ADD_PTR (char, hid,sizeof (struct acpi_pnp_device_id ));/* Convert EISAID to a string or simply copy existing string */
@@ -144,10 +145,10 @@ cleanup:acpi_status acpi_ut_execute_UID (struct acpi_namespace_node *device_node, -struct acpica_device_id **return_id) +struct acpi_pnp_device_id **return_id) {union acpi_operand_object *obj_desc; -struct acpica_device_id *uid; +struct acpi_pnp_device_id *uid;u32 length;acpi_status status;
@@ -171,16 +172,17 @@ acpi_ut_execute_UID(struct acpi_namespace_node *device_node,/* Allocate a buffer for the UID */ uid = -ACPI_ALLOCATE_ZEROED (sizeof (struct acpica_device_id ) + +ACPI_ALLOCATE_ZEROED (sizeof (struct acpi_pnp_device_id ) + (acpi_size ) length);if (!uid) { status =AE_NO_MEMORY ; goto cleanup; } -/* Area for the string starts after DEVICE_ID struct */ +/* Area for the string starts after PNP_DEVICE_ID struct */ - uid->string =ACPI_ADD_PTR (char, uid,sizeof (struct acpica_device_id )); + uid->string = +ACPI_ADD_PTR (char, uid,sizeof (struct acpi_pnp_device_id ));/* Convert an Integer to string, or just copy an existing string */
@@ -226,11 +228,11 @@ cleanup:acpi_status acpi_ut_execute_CID (struct acpi_namespace_node *device_node, -struct acpica_device_id_list **return_cid_list) +struct acpi_pnp_device_id_list **return_cid_list) {union acpi_operand_object **cid_objects;union acpi_operand_object *obj_desc; -struct acpica_device_id_list *cid_list; +struct acpi_pnp_device_id_list *cid_list;char *next_id_string;u32 string_area_size;u32 length;
@@ -288,11 +290,12 @@ acpi_ut_execute_CID(struct acpi_namespace_node *device_node,/* * Now that we know the length of the CIDs, allocate return buffer: * 1) Size of the base structure + -* 2) Size of the CID DEVICE_ID array + +* 2) Size of the CID PNP_DEVICE_ID array + * 3) Size of the actual CID strings */ - cid_list_size =sizeof (struct acpica_device_id_list ) + - ((count -1 ) *sizeof (struct acpica_device_id )) + string_area_size; + cid_list_size =sizeof (struct acpi_pnp_device_id_list ) + + ((count -1 ) *sizeof (struct acpi_pnp_device_id )) + + string_area_size; cid_list =ACPI_ALLOCATE_ZEROED (cid_list_size);if (!cid_list) {
@@ -300,10 +303,10 @@ acpi_ut_execute_CID(struct acpi_namespace_node *device_node, goto cleanup; } -/* Area for CID strings starts after the CID DEVICE_ID array */ +/* Area for CID strings starts after the CID PNP_DEVICE_ID array */ next_id_string =ACPI_CAST_PTR (char, cid_list->ids) + - ((acpi_size ) count *sizeof (struct acpica_device_id )); + ((acpi_size ) count *sizeof (struct acpi_pnp_device_id ));/* Copy/convert the CIDs to the return buffer */
drivers/acpi/scan.c +1 -1
@@ -1185,7 +1185,7 @@ static void acpi_device_set_id(struct acpi_device *device) {acpi_status status;struct acpi_device_info *info; -struct acpica_device_id_list *cid_list; +struct acpi_pnp_device_id_list *cid_list;int i;switch (device->device_type) {
include/acpi/actypes.h +6 -6
@@ -1020,15 +1020,15 @@ u32 (*acpi_interface_handler) (acpi_string interface_name, u32 supported);/* Structures used for device/processor HID, UID, CID */ -struct acpica_device_id { +struct acpi_pnp_device_id {u32 length;/* Length of string + null */ char *string; }; -struct acpica_device_id_list { +struct acpi_pnp_device_id_list {u32 count;/* Number of IDs in Ids array */ u32 list_size;/* Size of list, including ID strings */ -struct acpica_device_id ids[1 ];/* ID array */ +struct acpi_pnp_device_id ids[1 ];/* ID array */ };/*
@@ -1046,9 +1046,9 @@ struct acpi_device_info {u8 lowest_dstates[5 ];/* _sx_w values: 0xFF indicates not valid */ u32 current_status;/* _STA value */ u64 address;/* _ADR value */ -struct acpica_device_id hardware_id;/* _HID value */ -struct acpica_device_id unique_id;/* _UID value */ -struct acpica_device_id_list compatible_id_list;/* _CID list <must be last> */ +struct acpi_pnp_device_id hardware_id;/* _HID value */ +struct acpi_pnp_device_id unique_id;/* _UID value */ +struct acpi_pnp_device_id_list compatible_id_list;/* _CID list <must be last> */ };/* Values for Flags field above (acpi_get_object_info) */
Signature · 1 line
1.7.10
PATCH 5/26 ACPICA: Fix AcpiSrc caused divergences.
In reply to Lv ZhengThere are definitions that can been converted into new styles by the recent AcpiSrc while they remain the old styles in the Linux. This patch fixes those definitions that will be converted by the AcpiSrc.
This patch will not affect the generated vmlinux binary. This will decrease 97 lines of 20120913 divergence.diff.
--- drivers/acpi/acpica/acglobal.h | 2 +- drivers/acpi/acpica/aclocal.h | 4 ++-- drivers/acpi/acpica/acobject.h | 2 +- drivers/acpi/acpica/dswexec.c | 2 +- drivers/acpi/acpica/evxface.c | 2 +- drivers/acpi/sysfs.c | 4 ++-- include/acpi/acpixf.h | 2 +- include/acpi/actypes.h | 8 ++++---- 8 files changed, 13 insertions(+), 13 deletions(-)
drivers/acpi/acpica/acglobal.h +1 -1
@@ -376,7 +376,7 @@ ACPI_EXTERN struct acpi_gpe_block_info#if (!ACPI_REDUCED_HARDWARE ) ACPI_EXTERN u8 acpi_gbl_all_gpes_initialized; -ACPI_EXTERN ACPI_GBL_EVENT_HANDLER acpi_gbl_global_event_handler; +ACPI_EXTERN acpi_gbl_event_handler acpi_gbl_global_event_handler; ACPI_EXTERN void *acpi_gbl_global_event_handler_context;#endif /* !ACPI_REDUCED_HARDWARE */
drivers/acpi/acpica/aclocal.h +2 -2
@@ -262,7 +262,7 @@ struct acpi_create_field_info { };typedef -acpi_status (*ACPI_INTERNAL_METHOD ) (struct acpi_walk_state * walk_state); +acpi_status (*acpi_internal_method ) (struct acpi_walk_state * walk_state);/* * Bitmapped ACPI types. Used internally only
@@ -645,7 +645,7 @@ union acpi_generic_state {* ****************************************************************************/ -typedef acpi_status (*ACPI_EXECUTE_OP ) (struct acpi_walk_state * walk_state); +typedef acpi_status (*acpi_execute_op ) (struct acpi_walk_state * walk_state);/* Address Range info block */
drivers/acpi/acpica/acobject.h +1 -1
@@ -179,7 +179,7 @@ struct acpi_object_method {union acpi_operand_object *mutex;u8 *aml_start;union { -ACPI_INTERNAL_METHOD implementation; +acpi_internal_method implementation;union acpi_operand_object *handler; } dispatch;
drivers/acpi/acpica/dswexec.c +1 -1
@@ -57,7 +57,7 @@ ACPI_MODULE_NAME("dswexec")/* * Dispatch table for opcode classes */ -static ACPI_EXECUTE_OP acpi_gbl_op_type_dispatch[] = { +static acpi_execute_op acpi_gbl_op_type_dispatch[] = { acpi_ex_opcode_0A_0T_1R, acpi_ex_opcode_1A_0T_0R, acpi_ex_opcode_1A_0T_1R,
drivers/acpi/acpica/evxface.c +1 -1
@@ -398,7 +398,7 @@ ACPI_EXPORT_SYMBOL(acpi_install_exception_handler)* ******************************************************************************/ acpi_status -acpi_install_global_event_handler (ACPI_GBL_EVENT_HANDLER handler,void *context) +acpi_install_global_event_handler (acpi_gbl_event_handler handler,void *context) {acpi_status status;
drivers/acpi/sysfs.c +2 -2
@@ -476,7 +476,7 @@ static void fixed_event_count(u32 event_number)return ; } -static void acpi_gbl_event_handler (u32 event_type,acpi_handle device, +static void acpi_global_event_handler (u32 event_type,acpi_handle device,u32 event_number,void *context) {if (event_type ==ACPI_EVENT_TYPE_GPE )
@@ -638,7 +638,7 @@ void acpi_irq_stats_init(void)if (all_counters ==NULL ) goto fail; - status =acpi_install_global_event_handler (acpi_gbl_event_handler,NULL ); + status =acpi_install_global_event_handler (acpi_global_event_handler,NULL );if (ACPI_FAILURE (status)) goto fail;
include/acpi/acpixf.h +1 -1
@@ -274,7 +274,7 @@ acpi_install_initialization_handler(acpi_init_handler handler, u32 function); ACPI_HW_DEPENDENT_RETURN_STATUS(acpi_status acpi_install_global_event_handler - (ACPI_GBL_EVENT_HANDLER handler, void *context)) + (acpi_gbl_event_handler handler, void *context)) ACPI_HW_DEPENDENT_RETURN_STATUS(acpi_status acpi_install_fixed_event_handler(u32
include/acpi/actypes.h +4 -4
@@ -796,11 +796,11 @@ typedef u8 acpi_adr_space_type;/* Sleep function dispatch */ -typedef acpi_status (*ACPI_SLEEP_FUNCTION ) (u8 sleep_state); +typedef acpi_status (*acpi_sleep_function ) (u8 sleep_state);struct acpi_sleep_functions { -ACPI_SLEEP_FUNCTION legacy_function; -ACPI_SLEEP_FUNCTION extended_function; +acpi_sleep_function legacy_function; +acpi_sleep_function extended_function; };/*
@@ -931,7 +931,7 @@ typedef void* Various handlers and callback procedures */ typedef -void (*ACPI_GBL_EVENT_HANDLER ) (u32 event_type, +void (*acpi_gbl_event_handler ) (u32 event_type,acpi_handle device,u32 event_number,void *context);
Signature · 1 line
1.7.10
PATCH 6/26 ACPICA: Fix indent caused divergences.
In reply to Lv ZhengNew version of "indent" program will generate different outputs that will lead to the divergences between the Linux and the ACPICA. This patch fixes such divergences caused by the "indent" program. The version of the "indent" used for this patch is "GNU indent 2.2.11".
This patch will not affect the generated vmlinux binary. This will decrease 581 lines of 20120913 divergence.diff.
--- drivers/acpi/acpica/acdispat.h | 11 +++++++---- drivers/acpi/acpica/acevents.h | 6 ++++-- drivers/acpi/acpica/aclocal.h | 6 ++++-- drivers/acpi/acpica/acparser.h | 3 +-- drivers/acpi/acpica/acpredef.h | 11 ++++++----- drivers/acpi/acpica/dsopcode.c | 3 ++- drivers/acpi/acpica/dsutils.c | 3 ++- drivers/acpi/acpica/dswstate.c | 14 +++++++++----- drivers/acpi/acpica/evgpe.c | 20 ++++++++++++-------- drivers/acpi/acpica/evgpeblk.c | 3 ++- drivers/acpi/acpica/evgpeutil.c | 3 ++- drivers/acpi/acpica/evrgnini.c | 3 +-- drivers/acpi/acpica/evxfgpe.c | 9 +++++---- drivers/acpi/acpica/exconvrt.c | 4 ++-- drivers/acpi/acpica/excreate.c | 3 +-- drivers/acpi/acpica/exdump.c | 9 ++++++--- drivers/acpi/acpica/exfldio.c | 3 +-- drivers/acpi/acpica/exnames.c | 6 +++--- drivers/acpi/acpica/exresop.c | 3 ++- drivers/acpi/acpica/hwgpe.c | 3 ++- drivers/acpi/acpica/hwtimer.c | 3 +-- drivers/acpi/acpica/hwxfsleep.c | 12 ++++-------- drivers/acpi/acpica/nsaccess.c | 3 +-- drivers/acpi/acpica/nsinit.c | 4 ++-- drivers/acpi/acpica/psopcode.c | 23 ++++++++++++++--------- drivers/acpi/acpica/psparse.c | 5 +++-- drivers/acpi/acpica/rscalc.c | 3 +-- include/acpi/acpixf.h | 13 +++++-------- include/acpi/actypes.h | 6 ++++-- 29 files changed, 109 insertions(+), 89 deletions(-)
drivers/acpi/acpica/acdispat.h +7 -4
@@ -309,10 +309,13 @@ acpi_ds_obj_stack_push(void *object, struct acpi_walk_state *walk_state);acpi_status acpi_ds_obj_stack_pop (u32 pop_count,struct acpi_walk_state *walk_state); -struct acpi_walk_state *acpi_ds_create_walk_state (acpi_owner_id owner_id,union acpi_parse_object - *origin,union acpi_operand_object - *mth_desc,struct acpi_thread_state - *thread); +struct acpi_walk_state *acpi_ds_create_walk_state (acpi_owner_id owner_id, +union acpi_parse_object + *origin, +union acpi_operand_object + *mth_desc, +struct acpi_thread_state + *thread); acpi_status acpi_ds_init_aml_walk(struct acpi_walk_state *walk_state,
drivers/acpi/acpica/acevents.h +4 -2
@@ -84,9 +84,11 @@ acpi_ev_update_gpe_enable_mask(struct acpi_gpe_event_info *gpe_event_info);acpi_status acpi_ev_enable_gpe (struct acpi_gpe_event_info *gpe_event_info); -acpi_status acpi_ev_add_gpe_reference (struct acpi_gpe_event_info *gpe_event_info); +acpi_status +acpi_ev_add_gpe_reference (struct acpi_gpe_event_info *gpe_event_info); -acpi_status acpi_ev_remove_gpe_reference (struct acpi_gpe_event_info *gpe_event_info); +acpi_status +acpi_ev_remove_gpe_reference (struct acpi_gpe_event_info *gpe_event_info);struct acpi_gpe_event_info *acpi_ev_get_gpe_event_info (acpi_handle gpe_device,u32 gpe_number);
drivers/acpi/acpica/aclocal.h +4 -2
@@ -486,8 +486,10 @@ struct acpi_gpe_device_info {struct acpi_namespace_node *gpe_device; }; -typedef acpi_status (*acpi_gpe_callback ) (struct acpi_gpe_xrupt_info *gpe_xrupt_info, -struct acpi_gpe_block_info *gpe_block,void *context); +typedef acpi_status (*acpi_gpe_callback ) (struct acpi_gpe_xrupt_info * + gpe_xrupt_info, +struct acpi_gpe_block_info *gpe_block, +void *context);/* Information about each particular fixed event */
drivers/acpi/acpica/acparser.h +1 -2
@@ -150,8 +150,7 @@ u8 acpi_ps_has_completed_scope(struct acpi_parse_state *parser_state);void acpi_ps_pop_scope (struct acpi_parse_state *parser_state, -union acpi_parse_object **op, -u32 * arg_list,u32 * arg_count); +union acpi_parse_object **op,u32 *arg_list,u32 *arg_count); acpi_status acpi_ps_push_scope(struct acpi_parse_state *parser_state,
drivers/acpi/acpica/acpredef.h +6 -5
@@ -150,8 +150,7 @@ enum acpi_return_package_types {* is saved here (rather than in a separate table) in order to minimize the * overall size of the stored data. */ -static const union acpi_predefined_info predefined_names[] = -{ +static const union acpi_predefined_info predefined_names[] = { {{"_AC0" ,0 ,ACPI_RTYPE_INTEGER }}, {{"_AC1" ,0 ,ACPI_RTYPE_INTEGER }}, {{"_AC2" ,0 ,ACPI_RTYPE_INTEGER }},
@@ -538,7 +537,8 @@ static const union acpi_predefined_info predefined_names[] =/* Acpi 1.0 defined _WAK with no return value. Later, it was changed to return a package */ - {{"_WAK", 1, ACPI_RTYPE_NONE | ACPI_RTYPE_INTEGER | ACPI_RTYPE_PACKAGE}}, + {{"_WAK" ,1 , + ACPI_RTYPE_NONE | ACPI_RTYPE_INTEGER | ACPI_RTYPE_PACKAGE}}, {{{ACPI_PTYPE1_FIXED, ACPI_RTYPE_INTEGER, 2,0}, 0,0}}, /* Fixed-length (2 Int), but is optional *//* _WDG/_WED are MS extensions defined by "Windows Instrumentation" */
@@ -551,11 +551,12 @@ static const union acpi_predefined_info predefined_names[] = };#if 0 +/* This is an internally implemented control method, no need to check */ - {{"_OSI", 1, ACPI_RTYPE_INTEGER}}, +{ { +"_OSI", 1, ACPI_RTYPE_INTEGER}},/* TBD: */ - _PRT - currently ignore reversed entries. attempt to fix here? think about possibly fixing package elements like _BIF, etc.#endif
drivers/acpi/acpica/dsopcode.c +2 -1
@@ -649,7 +649,8 @@ acpi_ds_eval_data_object_operands(struct acpi_walk_state *walk_state, ((op->common.parent->common.aml_opcode !=AML_PACKAGE_OP ) && (op->common.parent->common.aml_opcode !=AML_VAR_PACKAGE_OP ) - && (op->common.parent->common.aml_opcode != AML_NAME_OP))) { + && (op->common.parent->common.aml_opcode != + AML_NAME_OP))) { walk_state->result_obj = obj_desc; } }
drivers/acpi/acpica/dsutils.c +2 -1
@@ -560,7 +560,8 @@ acpi_ds_create_operand(struct acpi_walk_state *walk_state,* indicate this to the interpreter, set the * object to the root */ - obj_desc = ACPI_CAST_PTR(union + obj_desc = + ACPI_CAST_PTR(union acpi_operand_object, acpi_gbl_root_node); status =AE_OK ;
drivers/acpi/acpica/dswstate.c +9 -5
@@ -51,8 +51,9 @@ACPI_MODULE_NAME ("dswstate" ) /* Local prototypes */ -static acpi_status acpi_ds_result_stack_push (struct acpi_walk_state *ws); -static acpi_status acpi_ds_result_stack_pop (struct acpi_walk_state *ws); +static acpi_status +acpi_ds_result_stack_push (struct acpi_walk_state *walk_state); +static acpi_status acpi_ds_result_stack_pop (struct acpi_walk_state *walk_state);/******************************************************************************* *
@@ -536,9 +537,12 @@ struct acpi_walk_state *acpi_ds_pop_walk_state(struct acpi_thread_state *thread)* ******************************************************************************/ -struct acpi_walk_state *acpi_ds_create_walk_state (acpi_owner_id owner_id,union acpi_parse_object - *origin,union acpi_operand_object - *method_desc,struct acpi_thread_state +struct acpi_walk_state *acpi_ds_create_walk_state (acpi_owner_id owner_id, +union acpi_parse_object + *origin, +union acpi_operand_object + *method_desc, +struct acpi_thread_state *thread) {struct acpi_walk_state *walk_state;
drivers/acpi/acpica/evgpe.c +12 -8
@@ -89,7 +89,8 @@ acpi_ev_update_gpe_enable_mask(struct acpi_gpe_event_info *gpe_event_info)/* Set the mask bit only if there are references to this GPE */ if (gpe_event_info->runtime_count) { -ACPI_SET_BIT (gpe_register_info->enable_for_run, (u8 )register_bit); +ACPI_SET_BIT (gpe_register_info->enable_for_run, + (u8 )register_bit); }return_ACPI_STATUS (AE_OK );
@@ -106,8 +107,7 @@ acpi_ev_update_gpe_enable_mask(struct acpi_gpe_event_info *gpe_event_info)* DESCRIPTION: Clear a GPE of stale events and enable it. * ******************************************************************************/ -acpi_status -acpi_ev_enable_gpe (struct acpi_gpe_event_info *gpe_event_info) +acpi_status acpi_ev_enable_gpe (struct acpi_gpe_event_info *gpe_event_info) {acpi_status status;
@@ -131,8 +131,8 @@ acpi_ev_enable_gpe(struct acpi_gpe_event_info *gpe_event_info) }/* Enable the requested GPE */ - status =acpi_hw_low_set_gpe (gpe_event_info,ACPI_GPE_ENABLE ); + status =acpi_hw_low_set_gpe (gpe_event_info,ACPI_GPE_ENABLE );return_ACPI_STATUS (status); }
@@ -150,7 +150,8 @@ acpi_ev_enable_gpe(struct acpi_gpe_event_info *gpe_event_info)* ******************************************************************************/ -acpi_status acpi_ev_add_gpe_reference (struct acpi_gpe_event_info *gpe_event_info) +acpi_status +acpi_ev_add_gpe_reference (struct acpi_gpe_event_info *gpe_event_info) {acpi_status status =AE_OK ;
@@ -191,7 +192,8 @@ acpi_status acpi_ev_add_gpe_reference(struct acpi_gpe_event_info *gpe_event_info* ******************************************************************************/ -acpi_status acpi_ev_remove_gpe_reference (struct acpi_gpe_event_info *gpe_event_info) +acpi_status +acpi_ev_remove_gpe_reference (struct acpi_gpe_event_info *gpe_event_info) {acpi_status status =AE_OK ;
@@ -208,7 +210,8 @@ acpi_status acpi_ev_remove_gpe_reference(struct acpi_gpe_event_info *gpe_event_i status =acpi_ev_update_gpe_enable_mask (gpe_event_info);if (ACPI_SUCCESS (status)) { - status =acpi_hw_low_set_gpe (gpe_event_info, + status = +acpi_hw_low_set_gpe (gpe_event_info,ACPI_GPE_DISABLE ); }
@@ -306,7 +309,8 @@ struct acpi_gpe_event_info *acpi_ev_get_gpe_event_info(acpi_handle gpe_device,/* A Non-NULL gpe_device means this is a GPE Block Device */ - obj_desc =acpi_ns_get_attached_object ((struct acpi_namespace_node *) + obj_desc = +acpi_ns_get_attached_object ((struct acpi_namespace_node *) gpe_device);if (!obj_desc || !obj_desc->device.gpe_block) {return (NULL );
drivers/acpi/acpica/evgpeblk.c +2 -1
@@ -486,7 +486,8 @@ acpi_ev_initialize_gpe_block(struct acpi_gpe_xrupt_info *gpe_xrupt_info,if (ACPI_FAILURE (status)) {ACPI_EXCEPTION ((AE_INFO , status,"Could not enable GPE 0x%02X" , - gpe_index + gpe_block->block_base_number)); + gpe_index + + gpe_block->block_base_number));continue ; }
drivers/acpi/acpica/evgpeutil.c +2 -1
@@ -374,7 +374,8 @@ acpi_ev_delete_gpe_handlers(struct acpi_gpe_xrupt_info *gpe_xrupt_info, gpe_event_info->dispatch.handler =NULL ; gpe_event_info->flags &= ~ACPI_GPE_DISPATCH_MASK ; - }else if ((gpe_event_info-> + }else +if ((gpe_event_info-> flags &ACPI_GPE_DISPATCH_MASK ) ==ACPI_GPE_DISPATCH_NOTIFY ) {
drivers/acpi/acpica/evrgnini.c +1 -2
@@ -227,8 +227,7 @@ acpi_ev_pci_config_region_setup(acpi_handle handle,/* Install a handler for this PCI root bridge */ - status = -acpi_install_address_space_handler ((acpi_handle ) pci_root_node,ACPI_ADR_SPACE_PCI_CONFIG ,ACPI_DEFAULT_HANDLER ,NULL ,NULL ); + status =acpi_install_address_space_handler ((acpi_handle ) pci_root_node,ACPI_ADR_SPACE_PCI_CONFIG ,ACPI_DEFAULT_HANDLER ,NULL ,NULL );if (ACPI_FAILURE (status)) {if (status ==AE_SAME_HANDLER ) {/*
drivers/acpi/acpica/evxfgpe.c +5 -4
@@ -221,7 +221,8 @@ acpi_setup_gpe_for_wake(acpi_handle wake_device,if (wake_device ==ACPI_ROOT_OBJECT ) { device_node = acpi_gbl_root_node; }else { - device_node = ACPI_CAST_PTR(struct acpi_namespace_node, wake_device); + device_node = + ACPI_CAST_PTR(struct acpi_namespace_node, wake_device); }/* Validate WakeDevice is of type Device */
@@ -324,7 +325,8 @@ ACPI_EXPORT_SYMBOL(acpi_setup_gpe_for_wake)* ******************************************************************************/ -acpi_status acpi_set_gpe_wake_mask (acpi_handle gpe_device,u32 gpe_number,u8 action) +acpi_status +acpi_set_gpe_wake_mask (acpi_handle gpe_device,u32 gpe_number,u8 action) {acpi_status status =AE_OK ;struct acpi_gpe_event_info *gpe_event_info;
@@ -694,8 +696,7 @@ ACPI_EXPORT_SYMBOL(acpi_remove_gpe_block)* the FADT-defined gpe blocks. Otherwise, the GPE block device. * ******************************************************************************/ -acpi_status -acpi_get_gpe_device (u32 index,acpi_handle *gpe_device) +acpi_status acpi_get_gpe_device (u32 index,acpi_handle * gpe_device) {struct acpi_gpe_device_info info;acpi_status status;
drivers/acpi/acpica/exconvrt.c +2 -2
@@ -516,8 +516,8 @@ acpi_ex_convert_to_string(union acpi_operand_object * obj_desc, string_length--; } - return_desc =acpi_ut_create_string_object ((acpi_size ) - string_length); + return_desc = +acpi_ut_create_string_object ((acpi_size ) string_length);if (!return_desc) {return_ACPI_STATUS (AE_NO_MEMORY ); }
drivers/acpi/acpica/excreate.c +1 -2
@@ -243,8 +243,7 @@ acpi_status acpi_ex_create_mutex(struct acpi_walk_state *walk_state)/* Init object and attach to NS node */ - obj_desc->mutex.sync_level = - (u8 ) walk_state->operands[1 ]->integer.value; + obj_desc->mutex.sync_level = (u8 )walk_state->operands[1 ]->integer.value; obj_desc->mutex.node = (struct acpi_namespace_node *)walk_state->operands[0 ];
drivers/acpi/acpica/exdump.c +6 -3
@@ -464,7 +464,8 @@ void acpi_ex_dump_operand(union acpi_operand_object *obj_desc, u32 depth) ACPI_FUNCTION_NAME(ex_dump_operand) - if (!((ACPI_LV_EXEC & acpi_dbg_level) + if (! + ((ACPI_LV_EXEC & acpi_dbg_level) && (_COMPONENT & acpi_dbg_layer))) {return ; }
@@ -810,7 +811,8 @@ void acpi_ex_dump_namespace_node(struct acpi_namespace_node *node, u32 flags)ACPI_FUNCTION_ENTRY ();if (!flags) { -if (!((ACPI_LV_OBJECTS & acpi_dbg_level) +if (! + ((ACPI_LV_OBJECTS & acpi_dbg_level) && (_COMPONENT & acpi_dbg_layer))) {return ; }
@@ -996,7 +998,8 @@ acpi_ex_dump_object_descriptor(union acpi_operand_object *obj_desc, u32 flags) }if (!flags) { -if (!((ACPI_LV_OBJECTS & acpi_dbg_level) +if (! + ((ACPI_LV_OBJECTS & acpi_dbg_level) && (_COMPONENT & acpi_dbg_layer))) { return_VOID; }
drivers/acpi/acpica/exfldio.c +1 -2
@@ -54,8 +54,7 @@ ACPI_MODULE_NAME("exfldio")/* Local prototypes */ static acpi_status acpi_ex_field_datum_io (union acpi_operand_object *obj_desc, -u32 field_datum_byte_offset, -u64 *value,u32 read_write); +u32 field_datum_byte_offset,u64 *value,u32 read_write);static u8 acpi_ex_register_overflow (union acpi_operand_object *obj_desc,u64 value);
drivers/acpi/acpica/exnames.c +3 -3
@@ -53,8 +53,7 @@ ACPI_MODULE_NAME("exnames")/* Local prototypes */ static char *acpi_ex_allocate_name_string (u32 prefix_count,u32 num_name_segs); -static acpi_status -acpi_ex_name_segment (u8 ** in_aml_address,char *name_string); +static acpi_status acpi_ex_name_segment (u8 **in_aml_address,char *name_string);/******************************************************************************* *
@@ -178,7 +177,8 @@ static acpi_status acpi_ex_name_segment(u8 ** in_aml_address, char *name_string)ACPI_DEBUG_PRINT ((ACPI_DB_LOAD ,"Bytes from stream:\n" )); -for (index =0 ; (index <ACPI_NAME_SIZE ) +for (index =0 ; + (index <ACPI_NAME_SIZE ) && (acpi_ut_valid_acpi_char (*aml_address,0 )); index++) { char_buf[index] = *aml_address++;ACPI_DEBUG_PRINT ((ACPI_DB_LOAD ,"%c\n" , char_buf[index]));
drivers/acpi/acpica/exresop.c +2 -1
@@ -337,7 +337,8 @@ acpi_ex_resolve_operands(u16 opcode,if ((opcode ==AML_STORE_OP ) && ((*stack_ptr)->common.type ==ACPI_TYPE_LOCAL_REFERENCE ) - && ((*stack_ptr)->reference.class ==ACPI_REFCLASS_INDEX )) { + && ((*stack_ptr)->reference.class == +ACPI_REFCLASS_INDEX )) { goto next_operand; }break ;
drivers/acpi/acpica/hwgpe.c +2 -1
@@ -339,7 +339,8 @@ acpi_hw_clear_gpe_block(struct acpi_gpe_xrupt_info *gpe_xrupt_info,acpi_status acpi_hw_enable_runtime_gpe_block (struct acpi_gpe_xrupt_info *gpe_xrupt_info, -struct acpi_gpe_block_info *gpe_block,void *context) +struct acpi_gpe_block_info * gpe_block, +void *context) {u32 i;acpi_status status;
drivers/acpi/acpica/hwtimer.c +1 -2
@@ -101,8 +101,7 @@ acpi_status acpi_get_timer(u32 * ticks)return_ACPI_STATUS (AE_BAD_PARAMETER ); } - status = -acpi_hw_read (ticks, &acpi_gbl_FADT.xpm_timer_block); + status =acpi_hw_read (ticks, &acpi_gbl_FADT.xpm_timer_block);return_ACPI_STATUS (status); }
drivers/acpi/acpica/hwxfsleep.c +4 -8
@@ -49,8 +49,7 @@ACPI_MODULE_NAME ("hwxfsleep" ) /* Local prototypes */ -static acpi_status -acpi_hw_sleep_dispatch (u8 sleep_state,u32 function_id); +static acpi_status acpi_hw_sleep_dispatch (u8 sleep_state,u32 function_id);/* * Dispatch table used to efficiently branch to the various sleep
@@ -234,8 +233,7 @@ ACPI_EXPORT_SYMBOL(acpi_enter_sleep_state_s4bios)* function. * ******************************************************************************/ -static acpi_status -acpi_hw_sleep_dispatch (u8 sleep_state,u32 function_id) +static acpi_status acpi_hw_sleep_dispatch (u8 sleep_state,u32 function_id) {acpi_status status; struct acpi_sleep_functions *sleep_functions =
@@ -369,8 +367,7 @@ acpi_status asmlinkage acpi_enter_sleep_state(u8 sleep_state)return_ACPI_STATUS (AE_AML_OPERAND_VALUE ); } - status = -acpi_hw_sleep_dispatch (sleep_state,ACPI_SLEEP_FUNCTION_ID ); + status =acpi_hw_sleep_dispatch (sleep_state,ACPI_SLEEP_FUNCTION_ID );return_ACPI_STATUS (status); }
@@ -396,8 +393,7 @@ acpi_status acpi_leave_sleep_state_prep(u8 sleep_state)ACPI_FUNCTION_TRACE (acpi_leave_sleep_state_prep); status = -acpi_hw_sleep_dispatch (sleep_state, -ACPI_WAKE_PREP_FUNCTION_ID ); +acpi_hw_sleep_dispatch (sleep_state,ACPI_WAKE_PREP_FUNCTION_ID );return_ACPI_STATUS (status); }
drivers/acpi/acpica/nsaccess.c +1 -2
@@ -179,8 +179,7 @@ acpi_status acpi_ns_root_initialize(void)/* Build an object around the static string */ - obj_desc->string.length = - (u32 )ACPI_STRLEN (val); + obj_desc->string.length = (u32 )ACPI_STRLEN (val); obj_desc->string.pointer = val; obj_desc->common.flags |=AOPOBJ_STATIC_POINTER ;break ;
drivers/acpi/acpica/nsinit.c +2 -2
@@ -96,8 +96,8 @@ acpi_status acpi_ns_initialize_objects(void)/* Walk entire namespace from the supplied root */ status =acpi_walk_namespace (ACPI_TYPE_ANY ,ACPI_ROOT_OBJECT , -ACPI_UINT32_MAX , acpi_ns_init_one_object,NULL , - &info,NULL ); +ACPI_UINT32_MAX , acpi_ns_init_one_object, +NULL , &info,NULL );if (ACPI_FAILURE (status)) {ACPI_EXCEPTION ((AE_INFO , status,"During WalkNamespace" )); }
drivers/acpi/acpica/psopcode.c +14 -9
@@ -392,10 +392,12 @@ const struct acpi_opcode_info acpi_gbl_aml_op_info[AML_NUM_OPCODES] = { AML_FLAGS_EXEC_1A_0T_1R | AML_NO_OPERAND_RESOLVE),/* 38 */ ACPI_OP ("LAnd" ,ARGP_LAND_OP ,ARGI_LAND_OP ,ACPI_TYPE_ANY ,AML_CLASS_EXECUTE ,AML_TYPE_EXEC_2A_0T_1R , -AML_FLAGS_EXEC_2A_0T_1R |AML_LOGICAL_NUMERIC |AML_CONSTANT ), +AML_FLAGS_EXEC_2A_0T_1R |AML_LOGICAL_NUMERIC | +AML_CONSTANT ),/* 39 */ ACPI_OP ("LOr" ,ARGP_LOR_OP ,ARGI_LOR_OP ,ACPI_TYPE_ANY ,AML_CLASS_EXECUTE ,AML_TYPE_EXEC_2A_0T_1R , -AML_FLAGS_EXEC_2A_0T_1R |AML_LOGICAL_NUMERIC |AML_CONSTANT ), +AML_FLAGS_EXEC_2A_0T_1R |AML_LOGICAL_NUMERIC | +AML_CONSTANT ),/* 3A */ ACPI_OP ("LNot" ,ARGP_LNOT_OP ,ARGI_LNOT_OP ,ACPI_TYPE_ANY ,AML_CLASS_EXECUTE ,AML_TYPE_EXEC_1A_0T_1R , AML_FLAGS_EXEC_1A_0T_1R | AML_CONSTANT),
@@ -495,7 +497,8 @@ const struct acpi_opcode_info acpi_gbl_aml_op_info[AML_NUM_OPCODES] = { AML_NSNODE | AML_NAMED | AML_DEFER),/* 59 */ ACPI_OP ("Field" ,ARGP_FIELD_OP ,ARGI_FIELD_OP ,ACPI_TYPE_ANY ,AML_CLASS_NAMED_OBJECT ,AML_TYPE_NAMED_FIELD , -AML_HAS_ARGS |AML_NSOBJECT |AML_NSOPCODE |AML_FIELD ), +AML_HAS_ARGS |AML_NSOBJECT |AML_NSOPCODE | +AML_FIELD ), /* 5A */ ACPI_OP("Device", ARGP_DEVICE_OP, ARGI_DEVICE_OP, ACPI_TYPE_DEVICE, AML_CLASS_NAMED_OBJECT, AML_TYPE_NAMED_NO_OBJ,
@@ -519,12 +522,13 @@ const struct acpi_opcode_info acpi_gbl_aml_op_info[AML_NUM_OPCODES] = {/* 5E */ ACPI_OP ("IndexField" ,ARGP_INDEX_FIELD_OP ,ARGI_INDEX_FIELD_OP ,ACPI_TYPE_ANY ,AML_CLASS_NAMED_OBJECT ,AML_TYPE_NAMED_FIELD , -AML_HAS_ARGS |AML_NSOBJECT |AML_NSOPCODE |AML_FIELD ), +AML_HAS_ARGS |AML_NSOBJECT |AML_NSOPCODE | +AML_FIELD ),/* 5F */ ACPI_OP ("BankField" ,ARGP_BANK_FIELD_OP ,ARGI_BANK_FIELD_OP , -ACPI_TYPE_LOCAL_BANK_FIELD ,AML_CLASS_NAMED_OBJECT , -AML_TYPE_NAMED_FIELD , -AML_HAS_ARGS |AML_NSOBJECT |AML_NSOPCODE |AML_FIELD | - AML_DEFER), +ACPI_TYPE_LOCAL_BANK_FIELD , +AML_CLASS_NAMED_OBJECT ,AML_TYPE_NAMED_FIELD , +AML_HAS_ARGS |AML_NSOBJECT |AML_NSOPCODE | + AML_FIELD | AML_DEFER),/* Internal opcodes that map to invalid AML opcodes */
@@ -632,7 +636,8 @@ const struct acpi_opcode_info acpi_gbl_aml_op_info[AML_NUM_OPCODES] = {/* 7D */ ACPI_OP ("[EvalSubTree]" ,ARGP_SCOPE_OP ,ARGI_SCOPE_OP ,ACPI_TYPE_ANY ,AML_CLASS_NAMED_OBJECT ,AML_TYPE_NAMED_NO_OBJ , - AML_HAS_ARGS | AML_NSOBJECT | AML_NSOPCODE | AML_NSNODE), +AML_HAS_ARGS |AML_NSOBJECT |AML_NSOPCODE | + AML_NSNODE),/* ACPI 3.0 opcodes */
drivers/acpi/acpica/psparse.c +3 -2
@@ -459,8 +459,9 @@ acpi_status acpi_ps_parse_aml(struct acpi_walk_state *walk_state)/* Executing a control method - additional cleanup */ -acpi_ds_terminate_control_method ( - walk_state->method_desc, walk_state); +acpi_ds_terminate_control_method (walk_state-> + method_desc, + walk_state); }acpi_ds_delete_walk_state (walk_state);
drivers/acpi/acpica/rscalc.c +1 -2
@@ -664,8 +664,7 @@ acpi_rs_get_pci_routing_table_length(union acpi_operand_object *package_object, (*sub_object_list)->string. length + 1); }else { - temp_size_needed += -acpi_ns_get_pathname_length ((*sub_object_list)->reference.node); + temp_size_needed +=acpi_ns_get_pathname_length ((*sub_object_list)->reference.node); } }else {/*
include/acpi/acpixf.h +5 -8
@@ -178,8 +178,7 @@ acpi_status acpi_unload_table_id(acpi_owner_id id);acpi_status acpi_get_table_header (acpi_string signature, -u32 instance, -struct acpi_table_header *out_table_header); +u32 instance,struct acpi_table_header *out_table_header); acpi_status acpi_get_table_with_size(acpi_string signature,
@@ -190,8 +189,7 @@ acpi_get_table(acpi_string signature, u32 instance, struct acpi_table_header **out_table);acpi_status -acpi_get_table_by_index (u32 table_index, -struct acpi_table_header **out_table); +acpi_get_table_by_index (u32 table_index,struct acpi_table_header **out_table);acpi_status acpi_install_table_handler (acpi_tbl_handler handler,void *context);
@@ -300,10 +298,9 @@ ACPI_HW_DEPENDENT_RETURN_STATUS(acpi_statusu32 gpe_number, acpi_gpe_handler address)) -acpi_status -acpi_install_notify_handler (acpi_handle device, -u32 handler_type, -acpi_notify_handler handler,void *context); +acpi_status acpi_install_notify_handler(acpi_handle device, u32 handler_type, +acpi_notify_handler handler, +void *context); acpi_status acpi_remove_notify_handler(acpi_handle device,
include/acpi/actypes.h +4 -2
@@ -922,7 +922,8 @@ struct acpi_system_info {/* * Types specific to the OS service interfaces */ -typedef u32(ACPI_SYSTEM_XFACE * acpi_osd_handler) (void *context); +typedef u32 + (ACPI_SYSTEM_XFACE * acpi_osd_handler) (void *context);typedef void (ACPI_SYSTEM_XFACE * acpi_osd_exec_callback) (void *context);
@@ -938,7 +939,8 @@ void (*acpi_gbl_event_handler) (u32 event_type,#define ACPI_EVENT_TYPE_GPE 0#define ACPI_EVENT_TYPE_FIXED 1 -typedef u32 (*acpi_event_handler ) (void *context); +typedef +u32 (*acpi_event_handler ) (void *context);typedef u32 (*acpi_gpe_handler ) (acpi_handle gpe_device,u32 gpe_number,void *context);
Signature · 1 line
1.7.10
PATCH 7/26 ACPICA: Fix unmerged acmacros.h divergences.
In reply to Lv ZhengThe 20121018 release depends on some unmerged acmaros.h fixes. This patch includes the fixes made on acmaros.h that will not affect the generated vmlinux binary.
This patch will not affect the generated vmlinux binary. This will decrease 157 lines of 20120913 divergence.diff.
--- drivers/acpi/acpica/acmacros.h | 161 ++++++++++++++-------------------------- 1 file changed, 57 insertions(+), 104 deletions(-)
drivers/acpi/acpica/acmacros.h +57 -104
@@ -84,29 +84,29 @@/* These macros reverse the bytes during the move, converting little-endian to big endian */ -/* Big Endian <== Little Endian */ -/* Hi...Lo Lo...Hi */ +/* Big Endian <== Little Endian */ +/* Hi...Lo Lo...Hi */ /* 16-bit source, 16/32/64 destination */ #define ACPI_MOVE_16_TO_16 (d, s) {(( u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[1];\ - (( u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[0];} + (( u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[0];}#define ACPI_MOVE_16_TO_32 (d, s) {(*(u32 *)(void *)(d))=0;\ - ((u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[1];\ - ((u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[0];} + ((u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[1];\ + ((u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[0];}#define ACPI_MOVE_16_TO_64 (d, s) {(*(u64 *)(void *)(d))=0;\ - ((u8 *)(void *)(d))[6] = ((u8 *)(void *)(s))[1];\ - ((u8 *)(void *)(d))[7] = ((u8 *)(void *)(s))[0];} + ((u8 *)(void *)(d))[6] = ((u8 *)(void *)(s))[1];\ + ((u8 *)(void *)(d))[7] = ((u8 *)(void *)(s))[0];}/* 32-bit source, 16/32/64 destination */ #define ACPI_MOVE_32_TO_16 (d, s) ACPI_MOVE_16_TO_16(d, s)/* Truncate to 16 */ #define ACPI_MOVE_32_TO_32 (d, s) {(( u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[3];\ - (( u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[2];\ - (( u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[1];\ - (( u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[0];} + (( u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[2];\ + (( u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[1];\ + (( u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[0];}#define ACPI_MOVE_32_TO_64 (d, s) {(*(u64 *)(void *)(d))=0;\ ((u8 *)(void *)(d))[4] = ((u8 *)(void *)(s))[3];\
@@ -196,24 +196,12 @@#endif #endif -/* Macros based on machine integer width */ - -#if ACPI_MACHINE_WIDTH ==32 -#define ACPI_MOVE_SIZE_TO_16 (d, s) ACPI_MOVE_32_TO_16(d, s) - -#elif ACPI_MACHINE_WIDTH ==64 -#define ACPI_MOVE_SIZE_TO_16 (d, s) ACPI_MOVE_64_TO_16(d, s) - -#else -#error unknown ACPI_MACHINE_WIDTH -#endif -/* * Fast power-of-two math macros for non-optimized compilers */ -#define _ACPI_DIV (value, power_of2) ((u32) ((value) >> (power_of2))) -#define _ACPI_MUL (value, power_of2) ((u32) ((value) << (power_of2))) -#define _ACPI_MOD (value, divisor) ((u32) ((value) & ((divisor) -1))) +#define _ACPI_DIV (value, power_of2) ((u32) ((value) >> (power_of2))) +#define _ACPI_MUL (value, power_of2) ((u32) ((value) << (power_of2))) +#define _ACPI_MOD (value, divisor) ((u32) ((value) & ((divisor) -1)))#define ACPI_DIV_2 (a) _ACPI_DIV(a, 1)#define ACPI_MUL_2 (a) _ACPI_MUL(a, 1)
@@ -238,12 +226,12 @@/* * Rounding macros (Power of two boundaries only) */ -#define ACPI_ROUND_DOWN (value, boundary) (((acpi_size)(value)) & \ - (~(((acpi_size) boundary)-1))) +#define ACPI_ROUND_DOWN (value, boundary) (((acpi_size)(value)) & \ + (~(((acpi_size) boundary)-1))) -#define ACPI_ROUND_UP (value, boundary) ((((acpi_size)(value)) + \ - (((acpi_size) boundary)-1)) & \ - (~(((acpi_size) boundary)-1))) +#define ACPI_ROUND_UP (value, boundary) ((((acpi_size)(value)) + \ + (((acpi_size) boundary)-1)) & \ + (~(((acpi_size) boundary)-1)))/* Note: sizeof(acpi_size) evaluates to either 4 or 8 (32- vs 64-bit mode) */
@@ -264,7 +252,7 @@#define ACPI_ROUND_UP_TO (value, boundary) (((value) + ((boundary)-1)) / (boundary)) -#define ACPI_IS_MISALIGNED (value) (((acpi_size) value) & (sizeof(acpi_size)-1)) +#define ACPI_IS_MISALIGNED (value) (((acpi_size) value) & (sizeof(acpi_size)-1))/* * Bitmask creation
@@ -355,7 +343,6 @@* Ascii error messages can be configured out */ #ifndef ACPI_NO_ERROR_MESSAGES -/* * Error reporting. Callers module and line number are inserted by AE_INFO, * the plist contains a set of parens to allow variable-length lists.
@@ -375,18 +362,15 @@#define ACPI_WARN_PREDEFINED (plist)#define ACPI_INFO_PREDEFINED (plist) -#endif /* ACPI_NO_ERROR_MESSAGES */ +#endif /* ACPI_NO_ERROR_MESSAGES */ /* * Debug macros that are conditionally compiled */ #ifdef ACPI_DEBUG_OUTPUT -/* * Function entry tracing */ -#ifdef CONFIG_ACPI_DEBUG_FUNC_TRACE -#define ACPI_FUNCTION_TRACE (a) ACPI_FUNCTION_NAME(a) \ acpi_ut_trace(ACPI_DEBUG_PARAMETERS)#define ACPI_FUNCTION_TRACE_PTR (a, b) ACPI_FUNCTION_NAME(a) \
@@ -464,44 +448,18 @@#endif /* ACPI_SIMPLE_RETURN_MACROS */ -#else /* !CONFIG_ACPI_DEBUG_FUNC_TRACE */ - -#define ACPI_FUNCTION_TRACE (a) -#define ACPI_FUNCTION_TRACE_PTR (a,b) -#define ACPI_FUNCTION_TRACE_U32 (a,b) -#define ACPI_FUNCTION_TRACE_STR (a,b) -#define ACPI_FUNCTION_EXIT -#define ACPI_FUNCTION_STATUS_EXIT (s) -#define ACPI_FUNCTION_VALUE_EXIT (s) -#define ACPI_FUNCTION_TRACE (a) -#define ACPI_FUNCTION_ENTRY () - -#define return_VOID return -#define return_ACPI_STATUS (s) return(s) -#define return_VALUE (s) return(s) -#define return_UINT8 (s) return(s) -#define return_UINT32 (s) return(s) -#define return_PTR (s) return(s) - -#endif /* CONFIG_ACPI_DEBUG_FUNC_TRACE */ -/* Conditional execution */ #define ACPI_DEBUG_EXEC (a) a -#define ACPI_NORMAL_EXEC (a) - -#define ACPI_DEBUG_DEFINE (a) a;#define ACPI_DEBUG_ONLY_MEMBERS (a) a;#define _VERBOSE_STRUCTURES -/* Stack and buffer dumping */ +/* Various object display routines for debug */ #define ACPI_DUMP_STACK_ENTRY (a) acpi_ex_dump_operand((a), 0) -#define ACPI_DUMP_OPERANDS (a, b, c) acpi_ex_dump_operands(a, b, c) - +#define ACPI_DUMP_OPERANDS (a, b ,c) acpi_ex_dump_operands(a, b, c)#define ACPI_DUMP_ENTRY (a, b) acpi_ns_dump_entry (a, b)#define ACPI_DUMP_PATHNAME (a, b, c, d) acpi_ns_dump_pathname(a, b, c, d) -#define ACPI_DUMP_RESOURCE_LIST (a) acpi_rs_dump_resource_list(a)#define ACPI_DUMP_BUFFER (a, b) acpi_ut_dump_buffer((u8 *) a, b, DB_BYTE_DISPLAY, _COMPONENT)#else
@@ -510,25 +468,23 @@* leaving no executable debug code! */ #define ACPI_DEBUG_EXEC (a) -#define ACPI_NORMAL_EXEC (a) a; - -#define ACPI_DEBUG_DEFINE (a) do { } while(0) -#define ACPI_DEBUG_ONLY_MEMBERS (a) do { } while(0) -#define ACPI_FUNCTION_TRACE (a) do { } while(0) -#define ACPI_FUNCTION_TRACE_PTR (a, b) do { } while(0) -#define ACPI_FUNCTION_TRACE_U32 (a, b) do { } while(0) -#define ACPI_FUNCTION_TRACE_STR (a, b) do { } while(0) -#define ACPI_FUNCTION_EXIT do { } while(0) -#define ACPI_FUNCTION_STATUS_EXIT (s) do { } while(0) -#define ACPI_FUNCTION_VALUE_EXIT (s) do { } while(0) -#define ACPI_FUNCTION_ENTRY () do { } while(0) -#define ACPI_DUMP_STACK_ENTRY (a) do { } while(0) -#define ACPI_DUMP_OPERANDS (a, b, c) do { } while(0) -#define ACPI_DUMP_ENTRY (a, b) do { } while(0) -#define ACPI_DUMP_TABLES (a, b) do { } while(0) -#define ACPI_DUMP_PATHNAME (a, b, c, d) do { } while(0) -#define ACPI_DUMP_RESOURCE_LIST (a) do { } while(0) -#define ACPI_DUMP_BUFFER (a, b) do { } while(0) +#define ACPI_DEBUG_ONLY_MEMBERS (a) +#define ACPI_FUNCTION_TRACE (a) +#define ACPI_FUNCTION_TRACE_PTR (a, b) +#define ACPI_FUNCTION_TRACE_U32 (a, b) +#define ACPI_FUNCTION_TRACE_STR (a, b) +#define ACPI_FUNCTION_EXIT +#define ACPI_FUNCTION_STATUS_EXIT (s) +#define ACPI_FUNCTION_VALUE_EXIT (s) +#define ACPI_FUNCTION_ENTRY () +#define ACPI_DUMP_STACK_ENTRY (a) +#define ACPI_DUMP_OPERANDS (a, b, c) +#define ACPI_DUMP_ENTRY (a, b) +#define ACPI_DUMP_TABLES (a, b) +#define ACPI_DUMP_PATHNAME (a, b, c, d) +#define ACPI_DUMP_BUFFER (a, b) +#define ACPI_DEBUG_PRINT (pl) +#define ACPI_DEBUG_PRINT_RAW (pl)#define return_VOID return#define return_ACPI_STATUS (s) return(s)
@@ -556,18 +512,6 @@#define ACPI_DEBUGGER_EXEC (a)#endif -#ifdef ACPI_DEBUG_OUTPUT -/* -* 1) Set name to blanks -* 2) Copy the object name -*/ -#define ACPI_ADD_OBJECT_NAME (a,b) ACPI_MEMSET (a->common.name, ' ', sizeof (a->common.name));\ - ACPI_STRNCPY (a->common.name, acpi_gbl_ns_type_names[b], sizeof (a->common.name)) -#else - -#define ACPI_ADD_OBJECT_NAME (a,b) -#endif -/* * Memory allocation tracking (DEBUG ONLY) */
@@ -578,13 +522,13 @@/* Memory allocation */ #ifndef ACPI_ALLOCATE -#define ACPI_ALLOCATE (a) acpi_ut_allocate((acpi_size)(a), ACPI_MEM_PARAMETERS) +#define ACPI_ALLOCATE (a) acpi_ut_allocate((acpi_size) (a), ACPI_MEM_PARAMETERS)#endif #ifndef ACPI_ALLOCATE_ZEROED -#define ACPI_ALLOCATE_ZEROED (a) acpi_ut_allocate_zeroed((acpi_size)(a), ACPI_MEM_PARAMETERS) +#define ACPI_ALLOCATE_ZEROED (a) acpi_ut_allocate_zeroed((acpi_size) (a), ACPI_MEM_PARAMETERS)#endif #ifndef ACPI_FREE -#define ACPI_FREE (a) acpio_os_free(a) +#define ACPI_FREE (a) acpi_os_free(a)#endif #define ACPI_MEM_TRACKING (a)
@@ -592,16 +536,25 @@/* Memory allocation */ -#define ACPI_ALLOCATE (a) acpi_ut_allocate_and_track((acpi_size)(a), ACPI_MEM_PARAMETERS) -#define ACPI_ALLOCATE_ZEROED (a) acpi_ut_allocate_zeroed_and_track((acpi_size)(a), ACPI_MEM_PARAMETERS) +#define ACPI_ALLOCATE (a) acpi_ut_allocate_and_track((acpi_size) (a), ACPI_MEM_PARAMETERS) +#define ACPI_ALLOCATE_ZEROED (a) acpi_ut_allocate_zeroed_and_track((acpi_size) (a), ACPI_MEM_PARAMETERS)#define ACPI_FREE (a) acpi_ut_free_and_track(a, ACPI_MEM_PARAMETERS)#define ACPI_MEM_TRACKING (a) a#endif /* ACPI_DBG_TRACK_ALLOCATIONS */ -/* Preemption point */ -#ifndef ACPI_PREEMPTION_POINT -#define ACPI_PREEMPTION_POINT ()/* no preemption */ -#endif +/* +* Macros used for ACPICA utilities only +*/ + +/* Generate a UUID */ + +#define ACPI_INIT_UUID (a, b, c, d0, d1, d2, d3, d4, d5, d6, d7) \ + (a) & 0xFF, ((a) >> 8) & 0xFF, ((a) >> 16) & 0xFF, ((a) >> 24) & 0xFF, \ + (b) & 0xFF, ((b) >> 8) & 0xFF, \ + (c) & 0xFF, ((c) >> 8) & 0xFF, \ + (d0), (d1), (d2), (d3), (d4), (d5), (d6), (d7) + +#define ACPI_IS_OCTAL_DIGIT (d) (((char)(d) >= '0') && ((char)(d) <= '7'))#endif /* ACMACROS_H */
Signature · 1 line
1.7.10
PATCH 8/26 ACPICA: Audit/update for ACPICA return macros and debug depth counter
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
1) Ensure that all functions that use the various TRACE macros also use the appropriate ACPICA return macros.
2) Ensure that all normal return statements surround the return expression (value) with parens.
--- drivers/acpi/acpica/dswload2.c | 4 ++-- drivers/acpi/acpica/dswstate.c | 4 ++-- drivers/acpi/acpica/evxfgpe.c | 4 ++-- drivers/acpi/acpica/exdebug.c | 2 +- drivers/acpi/acpica/exmutex.c | 2 +- drivers/acpi/acpica/exoparg6.c | 2 +- drivers/acpi/acpica/hwpci.c | 4 ++-- drivers/acpi/acpica/nsnames.c | 2 +- drivers/acpi/acpica/nsxfeval.c | 2 +- drivers/acpi/acpica/rslist.c | 4 ++-- drivers/acpi/acpica/tbinstal.c | 2 ++ drivers/acpi/acpica/tbutils.c | 2 +- drivers/acpi/acpica/tbxface.c | 4 ++-- drivers/acpi/acpica/tbxfroot.c | 1 - drivers/acpi/acpica/utdebug.c | 1 + drivers/acpi/acpica/utmutex.c | 2 ++ drivers/acpi/acpica/uttrack.c | 4 ++-- drivers/acpi/acpica/utxface.c | 3 ++- 18 files changed, 27 insertions(+), 22 deletions(-)
drivers/acpi/acpica/dswload2.c +2 -2
@@ -254,7 +254,7 @@ acpi_ds_load2_begin_op(struct acpi_walk_state *walk_state,acpi_ut_get_type_name (node->type), acpi_ut_get_node_name(node))); -return (AE_AML_OPERAND_TYPE ); +return_ACPI_STATUS (AE_AML_OPERAND_TYPE ); }break ;
@@ -602,7 +602,7 @@ acpi_status acpi_ds_load2_end_op(struct acpi_walk_state *walk_state) region_space, walk_state);if (ACPI_FAILURE (status)) { -return (status); +return_ACPI_STATUS (status); }acpi_ex_exit_interpreter ();
drivers/acpi/acpica/dswstate.c +2 -2
@@ -708,13 +708,13 @@ void acpi_ds_delete_walk_state(struct acpi_walk_state *walk_state)ACPI_FUNCTION_TRACE_PTR (ds_delete_walk_state, walk_state);if (!walk_state) { -return ; + return_VOID; }if (walk_state->descriptor_type !=ACPI_DESC_TYPE_WALK ) {ACPI_ERROR ((AE_INFO ,"%p is not a valid walk state" , walk_state)); -return ; + return_VOID; }/* There should not be any open scopes */
drivers/acpi/acpica/evxfgpe.c +2 -2
@@ -569,7 +569,7 @@ acpi_install_gpe_block(acpi_handle gpe_device, status =acpi_ut_acquire_mutex (ACPI_MTX_NAMESPACE );if (ACPI_FAILURE (status)) { -return (status); +return_ACPI_STATUS (status); } node =acpi_ns_validate_handle (gpe_device);
@@ -652,7 +652,7 @@ acpi_status acpi_remove_gpe_block(acpi_handle gpe_device) status =acpi_ut_acquire_mutex (ACPI_MTX_NAMESPACE );if (ACPI_FAILURE (status)) { -return (status); +return_ACPI_STATUS (status); } node =acpi_ns_validate_handle (gpe_device);
drivers/acpi/acpica/exdebug.c +1 -1
@@ -190,7 +190,7 @@ acpi_ex_do_debug_object(union acpi_operand_object *source_desc,acpi_os_printf ("Table Index 0x%X\n" , source_desc->reference.value); -return ; + return_VOID;default :break ;
drivers/acpi/acpica/exmutex.c +1 -1
@@ -305,7 +305,7 @@ acpi_status acpi_ex_release_mutex_object(union acpi_operand_object *obj_desc)ACPI_FUNCTION_TRACE (ex_release_mutex_object);if (obj_desc->mutex.acquisition_depth ==0 ) { -return (AE_NOT_ACQUIRED ); +return_ACPI_STATUS (AE_NOT_ACQUIRED ); }/* Match multiple Acquires with multiple Releases */
drivers/acpi/acpica/exoparg6.c +1 -1
@@ -198,7 +198,7 @@ acpi_ex_do_match(u32 match_op,return (FALSE); } -return logical_result; +return (logical_result); }/*******************************************************************************
drivers/acpi/acpica/hwpci.c +2 -2
@@ -259,7 +259,7 @@ acpi_hw_process_pci_list(struct acpi_pci_id *pci_id, status =acpi_hw_get_pci_device_info (pci_id, info->device, &bus_number, &is_bridge);if (ACPI_FAILURE (status)) { -return_ACPI_STATUS (status); +return (status); } info = info->next;
@@ -271,7 +271,7 @@ acpi_hw_process_pci_list(struct acpi_pci_id *pci_id, pci_id->segment, pci_id->bus, pci_id->device, pci_id->function, status, bus_number, is_bridge)); -return_ACPI_STATUS (AE_OK ); +return (AE_OK ); }/*******************************************************************************
drivers/acpi/acpica/nsnames.c +1 -1
@@ -195,7 +195,7 @@ acpi_size acpi_ns_get_pathname_length(struct acpi_namespace_node *node)ACPI_ERROR ((AE_INFO ,"Invalid Namespace Node (%p) while traversing namespace" , next_node)); -return 0 ; +return (0 ); } size +=ACPI_PATH_SEGMENT_LENGTH ; next_node = next_node->parent;
drivers/acpi/acpica/nsxfeval.c +1 -1
@@ -550,7 +550,7 @@ acpi_ns_get_device_callback(acpi_handle obj_handle, status =acpi_ut_acquire_mutex (ACPI_MTX_NAMESPACE );if (ACPI_FAILURE (status)) { -return (status); +return_ACPI_STATUS (status); } node =acpi_ns_validate_handle (obj_handle);
drivers/acpi/acpica/rslist.c +2 -2
@@ -109,7 +109,7 @@ acpi_rs_convert_aml_to_resources(u8 * aml,ACPI_ERROR ((AE_INFO ,"Invalid/unsupported resource descriptor: Type 0x%2.2X" , resource_index)); -return (AE_AML_INVALID_RESOURCE_TYPE ); +return_ACPI_STATUS (AE_AML_INVALID_RESOURCE_TYPE ); }/* Convert the AML byte stream resource to a local resource struct */
@@ -200,7 +200,7 @@ acpi_rs_convert_resources_to_aml(struct acpi_resource *resource,ACPI_ERROR ((AE_INFO ,"Invalid/unsupported resource descriptor: Type 0x%2.2X" , resource->type)); -return (AE_AML_INVALID_RESOURCE_TYPE ); +return_ACPI_STATUS (AE_AML_INVALID_RESOURCE_TYPE ); } status = acpi_rs_convert_resource_to_aml(resource,
drivers/acpi/acpica/tbinstal.c +2 -0
@@ -526,6 +526,8 @@ void acpi_tb_terminate(void)ACPI_DEBUG_PRINT ((ACPI_DB_INFO ,"ACPI Tables freed\n" )); (void )acpi_ut_release_mutex (ACPI_MTX_TABLES ); + + return_VOID; }/*******************************************************************************
drivers/acpi/acpica/tbutils.c +1 -1
@@ -354,7 +354,7 @@ u8 acpi_tb_checksum(u8 *buffer, u32 length) sum = (u8) (sum + *(buffer++)); } -return sum; +return (sum); }/*******************************************************************************
drivers/acpi/acpica/tbxface.c +2 -2
@@ -236,7 +236,7 @@ acpi_get_table_header(char *signature,sizeof (struct acpi_table_header));if (!header) { -return AE_NO_MEMORY ; +return (AE_NO_MEMORY ); }ACPI_MEMCPY (out_table_header, header,sizeof (struct acpi_table_header ));
@@ -244,7 +244,7 @@ acpi_get_table_header(char *signature,sizeof (struct acpi_table_header)); }else { -return AE_NOT_FOUND ; +return (AE_NOT_FOUND ); } }else { ACPI_MEMCPY(out_table_header,
drivers/acpi/acpica/tbxfroot.c +0 -1
@@ -67,7 +67,6 @@ static acpi_status acpi_tb_validate_rsdp(struct acpi_table_rsdp *rsdp);static acpi_status acpi_tb_validate_rsdp (struct acpi_table_rsdp *rsdp) { -ACPI_FUNCTION_ENTRY ();/* * The signature and checksum must both be correct
drivers/acpi/acpica/utdebug.c +1 -0
@@ -300,6 +300,7 @@ acpi_ut_trace_ptr(u32 line_number,const char *function_name, const char *module_name, u32 component_id, void *pointer) { + acpi_gbl_nesting_level++;acpi_ut_track_stack_ptr ();
drivers/acpi/acpica/utmutex.c +2 -0
@@ -193,6 +193,8 @@ static void acpi_ut_delete_mutex(acpi_mutex_handle mutex_id) acpi_gbl_mutex_info[mutex_id].mutex =NULL ; acpi_gbl_mutex_info[mutex_id].thread_id =ACPI_MUTEX_NOT_ACQUIRED ; + + return_VOID; }/*******************************************************************************
drivers/acpi/acpica/uttrack.c +2 -2
@@ -517,14 +517,14 @@ void acpi_ut_dump_allocations(u32 component, const char *module)ACPI_FUNCTION_TRACE (ut_dump_allocations);if (acpi_gbl_disable_mem_tracking) { -return ; + return_VOID; }/* * Walk the allocation list. */ if (ACPI_FAILURE (acpi_ut_acquire_mutex (ACPI_MTX_MEMORY ))) { -return ; + return_VOID; } element = acpi_gbl_global_list->list_head;
drivers/acpi/acpica/utxface.c +2 -1
@@ -238,7 +238,7 @@ acpi_install_initialization_handler(acpi_init_handler handler, u32 function) } acpi_gbl_init_handler = handler; -return AE_OK ; +return (AE_OK ); }ACPI_EXPORT_SYMBOL (acpi_install_initialization_handler)
@@ -263,6 +263,7 @@ acpi_status acpi_purge_cached_objects(void) (void )acpi_os_purge_cache (acpi_gbl_operand_cache); (void )acpi_os_purge_cache (acpi_gbl_ps_node_cache); (void )acpi_os_purge_cache (acpi_gbl_ps_node_ext_cache); +return_ACPI_STATUS (AE_OK ); }
Signature · 1 line
1.7.10
PATCH 9/26 ACPICA: ACPICA core: Cleanup empty lines at file start and end
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
Maintenance for source code consistency.
--- drivers/acpi/acpica/acobject.h | 1 - drivers/acpi/acpica/amlresrc.h | 1 - drivers/acpi/acpica/exmisc.c | 1 - drivers/acpi/acpica/exmutex.c | 1 - drivers/acpi/acpica/exnames.c | 1 - drivers/acpi/acpica/exoparg1.c | 1 - drivers/acpi/acpica/exoparg3.c | 1 - drivers/acpi/acpica/exoparg6.c | 1 - drivers/acpi/acpica/exprep.c | 1 - drivers/acpi/acpica/exregion.c | 1 - drivers/acpi/acpica/exresnte.c | 1 - drivers/acpi/acpica/exresolv.c | 1 - drivers/acpi/acpica/exresop.c | 1 - drivers/acpi/acpica/exstoren.c | 1 - drivers/acpi/acpica/exstorob.c | 1 - drivers/acpi/acpica/exsystem.c | 1 - drivers/acpi/acpica/exutils.c | 1 - drivers/acpi/acpica/hwacpi.c | 1 - drivers/acpi/acpica/hwgpe.c | 1 - drivers/acpi/acpica/hwregs.c | 1 - drivers/acpi/acpica/hwtimer.c | 1 - drivers/acpi/acpica/hwvalid.c | 1 - drivers/acpi/acpica/hwxface.c | 1 - include/acpi/acpiosxf.h | 1 - include/acpi/acpixf.h | 1 - 25 files changed, 25 deletions(-)
drivers/acpi/acpica/acobject.h +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Name: acobject.h - Definition of union acpi_operand_object (Internal object only)
drivers/acpi/acpica/amlresrc.h +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: amlresrc.h - AML resource descriptors
drivers/acpi/acpica/exmisc.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exmisc - ACPI AML (p-code) execution - specific opcodes
drivers/acpi/acpica/exmutex.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exmutex - ASL Mutex Acquire/Release functions
drivers/acpi/acpica/exnames.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exnames - interpreter/scanner name load/execute
drivers/acpi/acpica/exoparg1.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exoparg1 - AML execution - opcodes with 1 argument
drivers/acpi/acpica/exoparg3.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exoparg3 - AML execution - opcodes with 3 arguments
drivers/acpi/acpica/exoparg6.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exoparg6 - AML execution - opcodes with 6 arguments
drivers/acpi/acpica/exprep.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exprep - ACPI AML (p-code) execution - field prep utilities
drivers/acpi/acpica/exregion.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exregion - ACPI default op_region (address space) handlers
drivers/acpi/acpica/exresnte.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exresnte - AML Interpreter object resolution
drivers/acpi/acpica/exresolv.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exresolv - AML Interpreter object resolution
drivers/acpi/acpica/exresop.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exresop - AML Interpreter operand/object resolution
drivers/acpi/acpica/exstoren.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exstoren - AML Interpreter object store support,
drivers/acpi/acpica/exstorob.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exstorob - AML Interpreter object store support, store to object
drivers/acpi/acpica/exsystem.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exsystem - Interface to OS services
drivers/acpi/acpica/exutils.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: exutils - interpreter/scanner utilities
drivers/acpi/acpica/hwacpi.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: hwacpi - ACPI Hardware Initialization/Mode Interface
drivers/acpi/acpica/hwgpe.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: hwgpe - Low level GPE enable/disable/clear functions
drivers/acpi/acpica/hwregs.c +0 -1
@@ -1,4 +1,3 @@ -/******************************************************************************* * * Module Name: hwregs - Read/write access functions for the various ACPI
drivers/acpi/acpica/hwtimer.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Name: hwtimer.c - ACPI Power Management Timer Interface
drivers/acpi/acpica/hwvalid.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: hwvalid - I/O request validation
drivers/acpi/acpica/hwxface.c +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Module Name: hwxface - Public ACPICA hardware interfaces
include/acpi/acpiosxf.h +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Name: acpiosxf.h - All interfaces to the OS Services Layer (OSL). These
include/acpi/acpixf.h +0 -1
@@ -1,4 +1,3 @@ -/****************************************************************************** * * Name: acpixf.h - External interfaces to the ACPI subsystem
Signature · 1 line
1.7.10
PATCH 10/26 ACPICA: Fix some typos in comments
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
No functional changes. Some small fixes within commments. Colin Ian King.
--- drivers/acpi/acpica/tbxfload.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/acpi/acpica/tbxfload.c +1 -1
@@ -211,7 +211,7 @@ static acpi_status acpi_tb_load_namespace(void)* DESCRIPTION: Dynamically load an ACPI table from the caller's buffer. Must * be a valid ACPI table with a valid ACPI table header. * Note1: Mainly intended to support hotplug addition of SSDTs. -* Note2: Does not copy the incoming table. User is reponsible +* Note2: Does not copy the incoming table. User is responsible * to ensure that the table is not deleted or unmapped. * ******************************************************************************/
Signature · 1 line
1.7.10
PATCH 11/26 ACPICA: Fix for predefined name loop during ACPICA initialization
In reply to Lv ZhengFrom: Robert Moore <[email protected]>
If a name cannot be created, simply continue on to the next name. Do not attempt to use the name, do not abort. With assistance from Colin Ian King.
--- drivers/acpi/acpica/nsaccess.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/acpi/acpica/nsaccess.c +2 -2
@@ -110,11 +110,11 @@ acpi_status acpi_ns_root_initialize(void) status =acpi_ns_lookup (NULL , init_val->name, init_val->type,ACPI_IMODE_LOAD_PASS2 ,ACPI_NS_NO_UPSEARCH ,NULL , &new_node); - -if (ACPI_FAILURE (status) || (!new_node)) {/* Must be on same line for code converter */ +if (ACPI_FAILURE (status)) {ACPI_EXCEPTION ((AE_INFO , status,"Could not create predefined name %s" , init_val->name)); +continue ; }/*
Signature · 1 line
1.7.10
PATCH 12/26 ACPICA: Update local C library module comments for ASCII table
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
Improve the commenting of the table.
--- drivers/acpi/acpica/utclib.c | 261 +++++++++++++++++++++--------------------- 1 file changed, 131 insertions(+), 130 deletions(-)
drivers/acpi/acpica/utclib.c +131 -130
@@ -46,7 +46,7 @@/* * These implementations of standard C Library routines can optionally be -* used if a C library is not available. In general, they are less efficient +* used if a C library is not available. In general, they are less efficient * than an inline or assembly implementation */
@@ -606,134 +606,134 @@ int acpi_ut_to_lower(int c)******************************************************************************/ const u8 _acpi_ctype[257 ] = { - _ACPI_CN,/* 0x0 0. */ - _ACPI_CN,/* 0x1 1. */ - _ACPI_CN,/* 0x2 2. */ - _ACPI_CN,/* 0x3 3. */ - _ACPI_CN,/* 0x4 4. */ - _ACPI_CN,/* 0x5 5. */ - _ACPI_CN,/* 0x6 6. */ - _ACPI_CN,/* 0x7 7. */ - _ACPI_CN,/* 0x8 8. */ - _ACPI_CN | _ACPI_SP,/* 0x9 9. */ - _ACPI_CN | _ACPI_SP,/* 0xA 10. */ - _ACPI_CN | _ACPI_SP,/* 0xB 11. */ - _ACPI_CN | _ACPI_SP,/* 0xC 12. */ - _ACPI_CN | _ACPI_SP,/* 0xD 13. */ - _ACPI_CN,/* 0xE 14. */ - _ACPI_CN,/* 0xF 15. */ - _ACPI_CN,/* 0x10 16. */ - _ACPI_CN,/* 0x11 17. */ - _ACPI_CN,/* 0x12 18. */ - _ACPI_CN,/* 0x13 19. */ - _ACPI_CN,/* 0x14 20. */ - _ACPI_CN,/* 0x15 21. */ - _ACPI_CN,/* 0x16 22. */ - _ACPI_CN,/* 0x17 23. */ - _ACPI_CN,/* 0x18 24. */ - _ACPI_CN,/* 0x19 25. */ - _ACPI_CN,/* 0x1A 26. */ - _ACPI_CN,/* 0x1B 27. */ - _ACPI_CN,/* 0x1C 28. */ - _ACPI_CN,/* 0x1D 29. */ - _ACPI_CN,/* 0x1E 30. */ - _ACPI_CN,/* 0x1F 31. */ - _ACPI_XS | _ACPI_SP,/* 0x20 32. ' ' */ - _ACPI_PU,/* 0x21 33. '!' */ - _ACPI_PU,/* 0x22 34. '"' */ - _ACPI_PU,/* 0x23 35. '#' */ - _ACPI_PU,/* 0x24 36. '$' */ - _ACPI_PU,/* 0x25 37. '%' */ - _ACPI_PU,/* 0x26 38. '&' */ - _ACPI_PU,/* 0x27 39. ''' */ - _ACPI_PU,/* 0x28 40. '(' */ - _ACPI_PU,/* 0x29 41. ')' */ - _ACPI_PU,/* 0x2A 42. '*' */ - _ACPI_PU,/* 0x2B 43. '+' */ - _ACPI_PU,/* 0x2C 44. ',' */ - _ACPI_PU,/* 0x2D 45. '-' */ - _ACPI_PU,/* 0x2E 46. '.' */ - _ACPI_PU,/* 0x2F 47. '/' */ - _ACPI_XD | _ACPI_DI,/* 0x30 48. '0' */ - _ACPI_XD | _ACPI_DI,/* 0x31 49. '1' */ - _ACPI_XD | _ACPI_DI,/* 0x32 50. '2' */ - _ACPI_XD | _ACPI_DI,/* 0x33 51. '3' */ - _ACPI_XD | _ACPI_DI,/* 0x34 52. '4' */ - _ACPI_XD | _ACPI_DI,/* 0x35 53. '5' */ - _ACPI_XD | _ACPI_DI,/* 0x36 54. '6' */ - _ACPI_XD | _ACPI_DI,/* 0x37 55. '7' */ - _ACPI_XD | _ACPI_DI,/* 0x38 56. '8' */ - _ACPI_XD | _ACPI_DI,/* 0x39 57. '9' */ - _ACPI_PU,/* 0x3A 58. ':' */ - _ACPI_PU,/* 0x3B 59. ';' */ - _ACPI_PU,/* 0x3C 60. '<' */ - _ACPI_PU,/* 0x3D 61. '=' */ - _ACPI_PU,/* 0x3E 62. '>' */ - _ACPI_PU,/* 0x3F 63. '?' */ - _ACPI_PU,/* 0x40 64. '@' */ - _ACPI_XD | _ACPI_UP,/* 0x41 65. 'A' */ - _ACPI_XD | _ACPI_UP,/* 0x42 66. 'B' */ - _ACPI_XD | _ACPI_UP,/* 0x43 67. 'C' */ - _ACPI_XD | _ACPI_UP,/* 0x44 68. 'D' */ - _ACPI_XD | _ACPI_UP,/* 0x45 69. 'E' */ - _ACPI_XD | _ACPI_UP,/* 0x46 70. 'F' */ - _ACPI_UP,/* 0x47 71. 'G' */ - _ACPI_UP,/* 0x48 72. 'H' */ - _ACPI_UP,/* 0x49 73. 'I' */ - _ACPI_UP,/* 0x4A 74. 'J' */ - _ACPI_UP,/* 0x4B 75. 'K' */ - _ACPI_UP,/* 0x4C 76. 'L' */ - _ACPI_UP,/* 0x4D 77. 'M' */ - _ACPI_UP,/* 0x4E 78. 'N' */ - _ACPI_UP,/* 0x4F 79. 'O' */ - _ACPI_UP,/* 0x50 80. 'P' */ - _ACPI_UP,/* 0x51 81. 'Q' */ - _ACPI_UP,/* 0x52 82. 'R' */ - _ACPI_UP,/* 0x53 83. 'S' */ - _ACPI_UP,/* 0x54 84. 'T' */ - _ACPI_UP,/* 0x55 85. 'U' */ - _ACPI_UP,/* 0x56 86. 'V' */ - _ACPI_UP,/* 0x57 87. 'W' */ - _ACPI_UP,/* 0x58 88. 'X' */ - _ACPI_UP,/* 0x59 89. 'Y' */ - _ACPI_UP,/* 0x5A 90. 'Z' */ - _ACPI_PU,/* 0x5B 91. '[' */ - _ACPI_PU,/* 0x5C 92. '\' */ - _ACPI_PU,/* 0x5D 93. ']' */ - _ACPI_PU,/* 0x5E 94. '^' */ - _ACPI_PU,/* 0x5F 95. '_' */ - _ACPI_PU,/* 0x60 96. '`' */ - _ACPI_XD | _ACPI_LO,/* 0x61 97. 'a' */ - _ACPI_XD | _ACPI_LO,/* 0x62 98. 'b' */ - _ACPI_XD | _ACPI_LO,/* 0x63 99. 'c' */ - _ACPI_XD | _ACPI_LO,/* 0x64 100. 'd' */ - _ACPI_XD | _ACPI_LO,/* 0x65 101. 'e' */ - _ACPI_XD | _ACPI_LO,/* 0x66 102. 'f' */ - _ACPI_LO,/* 0x67 103. 'g' */ - _ACPI_LO,/* 0x68 104. 'h' */ - _ACPI_LO,/* 0x69 105. 'i' */ - _ACPI_LO,/* 0x6A 106. 'j' */ - _ACPI_LO,/* 0x6B 107. 'k' */ - _ACPI_LO,/* 0x6C 108. 'l' */ - _ACPI_LO,/* 0x6D 109. 'm' */ - _ACPI_LO,/* 0x6E 110. 'n' */ - _ACPI_LO,/* 0x6F 111. 'o' */ - _ACPI_LO,/* 0x70 112. 'p' */ - _ACPI_LO,/* 0x71 113. 'q' */ - _ACPI_LO,/* 0x72 114. 'r' */ - _ACPI_LO,/* 0x73 115. 's' */ - _ACPI_LO,/* 0x74 116. 't' */ - _ACPI_LO,/* 0x75 117. 'u' */ - _ACPI_LO,/* 0x76 118. 'v' */ - _ACPI_LO,/* 0x77 119. 'w' */ - _ACPI_LO,/* 0x78 120. 'x' */ - _ACPI_LO,/* 0x79 121. 'y' */ - _ACPI_LO,/* 0x7A 122. 'z' */ - _ACPI_PU,/* 0x7B 123. '{' */ - _ACPI_PU,/* 0x7C 124. '|' */ - _ACPI_PU,/* 0x7D 125. '}' */ - _ACPI_PU,/* 0x7E 126. '~' */ - _ACPI_CN,/* 0x7F 127. */ + _ACPI_CN,/* 0x00 0 NUL */ + _ACPI_CN,/* 0x01 1 SOH */ + _ACPI_CN,/* 0x02 2 STX */ + _ACPI_CN,/* 0x03 3 ETX */ + _ACPI_CN,/* 0x04 4 EOT */ + _ACPI_CN,/* 0x05 5 ENQ */ + _ACPI_CN,/* 0x06 6 ACK */ + _ACPI_CN,/* 0x07 7 BEL */ + _ACPI_CN,/* 0x08 8 BS */ + _ACPI_CN | _ACPI_SP,/* 0x09 9 TAB */ + _ACPI_CN | _ACPI_SP,/* 0x0A 10 LF */ + _ACPI_CN | _ACPI_SP,/* 0x0B 11 VT */ + _ACPI_CN | _ACPI_SP,/* 0x0C 12 FF */ + _ACPI_CN | _ACPI_SP,/* 0x0D 13 CR */ + _ACPI_CN,/* 0x0E 14 SO */ + _ACPI_CN,/* 0x0F 15 SI */ + _ACPI_CN,/* 0x10 16 DLE */ + _ACPI_CN,/* 0x11 17 DC1 */ + _ACPI_CN,/* 0x12 18 DC2 */ + _ACPI_CN,/* 0x13 19 DC3 */ + _ACPI_CN,/* 0x14 20 DC4 */ + _ACPI_CN,/* 0x15 21 NAK */ + _ACPI_CN,/* 0x16 22 SYN */ + _ACPI_CN,/* 0x17 23 ETB */ + _ACPI_CN,/* 0x18 24 CAN */ + _ACPI_CN,/* 0x19 25 EM */ + _ACPI_CN,/* 0x1A 26 SUB */ + _ACPI_CN,/* 0x1B 27 ESC */ + _ACPI_CN,/* 0x1C 28 FS */ + _ACPI_CN,/* 0x1D 29 GS */ + _ACPI_CN,/* 0x1E 30 RS */ + _ACPI_CN,/* 0x1F 31 US */ + _ACPI_XS | _ACPI_SP,/* 0x20 32 ' ' */ + _ACPI_PU,/* 0x21 33 '!' */ + _ACPI_PU,/* 0x22 34 '"' */ + _ACPI_PU,/* 0x23 35 '#' */ + _ACPI_PU,/* 0x24 36 '$' */ + _ACPI_PU,/* 0x25 37 '%' */ + _ACPI_PU,/* 0x26 38 '&' */ + _ACPI_PU,/* 0x27 39 ''' */ + _ACPI_PU,/* 0x28 40 '(' */ + _ACPI_PU,/* 0x29 41 ')' */ + _ACPI_PU,/* 0x2A 42 '*' */ + _ACPI_PU,/* 0x2B 43 '+' */ + _ACPI_PU,/* 0x2C 44 ',' */ + _ACPI_PU,/* 0x2D 45 '-' */ + _ACPI_PU,/* 0x2E 46 '.' */ + _ACPI_PU,/* 0x2F 47 '/' */ + _ACPI_XD | _ACPI_DI,/* 0x30 48 '0' */ + _ACPI_XD | _ACPI_DI,/* 0x31 49 '1' */ + _ACPI_XD | _ACPI_DI,/* 0x32 50 '2' */ + _ACPI_XD | _ACPI_DI,/* 0x33 51 '3' */ + _ACPI_XD | _ACPI_DI,/* 0x34 52 '4' */ + _ACPI_XD | _ACPI_DI,/* 0x35 53 '5' */ + _ACPI_XD | _ACPI_DI,/* 0x36 54 '6' */ + _ACPI_XD | _ACPI_DI,/* 0x37 55 '7' */ + _ACPI_XD | _ACPI_DI,/* 0x38 56 '8' */ + _ACPI_XD | _ACPI_DI,/* 0x39 57 '9' */ + _ACPI_PU,/* 0x3A 58 ':' */ + _ACPI_PU,/* 0x3B 59 ';' */ + _ACPI_PU,/* 0x3C 60 '<' */ + _ACPI_PU,/* 0x3D 61 '=' */ + _ACPI_PU,/* 0x3E 62 '>' */ + _ACPI_PU,/* 0x3F 63 '?' */ + _ACPI_PU,/* 0x40 64 '@' */ + _ACPI_XD | _ACPI_UP,/* 0x41 65 'A' */ + _ACPI_XD | _ACPI_UP,/* 0x42 66 'B' */ + _ACPI_XD | _ACPI_UP,/* 0x43 67 'C' */ + _ACPI_XD | _ACPI_UP,/* 0x44 68 'D' */ + _ACPI_XD | _ACPI_UP,/* 0x45 69 'E' */ + _ACPI_XD | _ACPI_UP,/* 0x46 70 'F' */ + _ACPI_UP,/* 0x47 71 'G' */ + _ACPI_UP,/* 0x48 72 'H' */ + _ACPI_UP,/* 0x49 73 'I' */ + _ACPI_UP,/* 0x4A 74 'J' */ + _ACPI_UP,/* 0x4B 75 'K' */ + _ACPI_UP,/* 0x4C 76 'L' */ + _ACPI_UP,/* 0x4D 77 'M' */ + _ACPI_UP,/* 0x4E 78 'N' */ + _ACPI_UP,/* 0x4F 79 'O' */ + _ACPI_UP,/* 0x50 80 'P' */ + _ACPI_UP,/* 0x51 81 'Q' */ + _ACPI_UP,/* 0x52 82 'R' */ + _ACPI_UP,/* 0x53 83 'S' */ + _ACPI_UP,/* 0x54 84 'T' */ + _ACPI_UP,/* 0x55 85 'U' */ + _ACPI_UP,/* 0x56 86 'V' */ + _ACPI_UP,/* 0x57 87 'W' */ + _ACPI_UP,/* 0x58 88 'X' */ + _ACPI_UP,/* 0x59 89 'Y' */ + _ACPI_UP,/* 0x5A 90 'Z' */ + _ACPI_PU,/* 0x5B 91 '[' */ + _ACPI_PU,/* 0x5C 92 '\' */ + _ACPI_PU,/* 0x5D 93 ']' */ + _ACPI_PU,/* 0x5E 94 '^' */ + _ACPI_PU,/* 0x5F 95 '_' */ + _ACPI_PU,/* 0x60 96 '`' */ + _ACPI_XD | _ACPI_LO,/* 0x61 97 'a' */ + _ACPI_XD | _ACPI_LO,/* 0x62 98 'b' */ + _ACPI_XD | _ACPI_LO,/* 0x63 99 'c' */ + _ACPI_XD | _ACPI_LO,/* 0x64 100 'd' */ + _ACPI_XD | _ACPI_LO,/* 0x65 101 'e' */ + _ACPI_XD | _ACPI_LO,/* 0x66 102 'f' */ + _ACPI_LO,/* 0x67 103 'g' */ + _ACPI_LO,/* 0x68 104 'h' */ + _ACPI_LO,/* 0x69 105 'i' */ + _ACPI_LO,/* 0x6A 106 'j' */ + _ACPI_LO,/* 0x6B 107 'k' */ + _ACPI_LO,/* 0x6C 108 'l' */ + _ACPI_LO,/* 0x6D 109 'm' */ + _ACPI_LO,/* 0x6E 110 'n' */ + _ACPI_LO,/* 0x6F 111 'o' */ + _ACPI_LO,/* 0x70 112 'p' */ + _ACPI_LO,/* 0x71 113 'q' */ + _ACPI_LO,/* 0x72 114 'r' */ + _ACPI_LO,/* 0x73 115 's' */ + _ACPI_LO,/* 0x74 116 't' */ + _ACPI_LO,/* 0x75 117 'u' */ + _ACPI_LO,/* 0x76 118 'v' */ + _ACPI_LO,/* 0x77 119 'w' */ + _ACPI_LO,/* 0x78 120 'x' */ + _ACPI_LO,/* 0x79 121 'y' */ + _ACPI_LO,/* 0x7A 122 'z' */ + _ACPI_PU,/* 0x7B 123 '{' */ + _ACPI_PU,/* 0x7C 124 '|' */ + _ACPI_PU,/* 0x7D 125 '}' */ + _ACPI_PU,/* 0x7E 126 '~' */ + _ACPI_CN,/* 0x7F 127 DEL */ 0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,/* 0x80 to 0x8F */ 0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,/* 0x90 to 0x9F */
@@ -742,7 +742,8 @@ const u8 _acpi_ctype[257] = {0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,/* 0xC0 to 0xCF */ 0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,/* 0xD0 to 0xDF */ 0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,/* 0xE0 to 0xEF */ -0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 /* 0xF0 to 0x100 */ +0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,0 ,/* 0xF0 to 0xFF */ +0 /* 0x100 */ };#endif /* ACPI_USE_SYSTEM_CLIBRARY */
Signature · 1 line
1.7.10
PATCH 13/26 ACPICA: Remove extra spaces after periods within comments
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
This makes all comments consistent.
--- drivers/acpi/acpica/acglobal.h | 2 +- drivers/acpi/acpica/aclocal.h | 4 ++-- drivers/acpi/acpica/acobject.h | 4 ++-- drivers/acpi/acpica/acopcode.h | 6 +++--- drivers/acpi/acpica/acstruct.h | 2 +- drivers/acpi/acpica/dscontrol.c | 2 +- drivers/acpi/acpica/dsfield.c | 2 +- drivers/acpi/acpica/dsmethod.c | 6 +++--- drivers/acpi/acpica/dsmthdat.c | 14 +++++++------- drivers/acpi/acpica/dsobject.c | 6 +++--- drivers/acpi/acpica/dsutils.c | 30 +++++++++++++++--------------- drivers/acpi/acpica/dswexec.c | 8 ++++---- drivers/acpi/acpica/dswstate.c | 8 ++++---- drivers/acpi/acpica/excreate.c | 6 +++--- drivers/acpi/acpica/exdump.c | 2 +- drivers/acpi/acpica/exfield.c | 4 ++-- drivers/acpi/acpica/exfldio.c | 12 ++++++------ drivers/acpi/acpica/exmisc.c | 4 ++-- drivers/acpi/acpica/exnames.c | 2 +- drivers/acpi/acpica/exoparg1.c | 10 +++++----- drivers/acpi/acpica/exoparg2.c | 2 +- drivers/acpi/acpica/exoparg3.c | 2 +- drivers/acpi/acpica/exoparg6.c | 2 +- drivers/acpi/acpica/exprep.c | 12 ++++++------ drivers/acpi/acpica/exregion.c | 2 +- drivers/acpi/acpica/exresnte.c | 8 ++++---- drivers/acpi/acpica/exresolv.c | 2 +- drivers/acpi/acpica/exresop.c | 4 ++-- drivers/acpi/acpica/exstore.c | 4 ++-- drivers/acpi/acpica/exstoren.c | 10 +++++----- drivers/acpi/acpica/exstorob.c | 4 ++-- drivers/acpi/acpica/exsystem.c | 8 ++++---- drivers/acpi/acpica/exutils.c | 4 ++-- drivers/acpi/acpica/hwacpi.c | 2 +- drivers/acpi/acpica/hwtimer.c | 2 +- drivers/acpi/acpica/nsalloc.c | 4 ++-- drivers/acpi/acpica/nsdump.c | 2 +- drivers/acpi/acpica/nsload.c | 10 +++++----- drivers/acpi/acpica/nsobject.c | 8 ++++---- drivers/acpi/acpica/nsparse.c | 8 ++++---- drivers/acpi/acpica/nsutils.c | 6 +++--- drivers/acpi/acpica/nswalk.c | 10 +++++----- drivers/acpi/acpica/nsxfeval.c | 14 +++++++------- drivers/acpi/acpica/nsxfname.c | 6 +++--- drivers/acpi/acpica/nsxfobj.c | 4 ++-- drivers/acpi/acpica/psargs.c | 8 ++++---- drivers/acpi/acpica/psopcode.c | 6 +++--- drivers/acpi/acpica/psparse.c | 8 ++++---- drivers/acpi/acpica/psutils.c | 4 ++-- drivers/acpi/acpica/rscalc.c | 2 +- drivers/acpi/acpica/tbxfroot.c | 2 +- drivers/acpi/acpica/utcache.c | 6 +++--- drivers/acpi/acpica/utdebug.c | 18 +++++++++--------- drivers/acpi/acpica/utmath.c | 2 +- drivers/acpi/acpica/utmisc.c | 14 +++++++------- drivers/acpi/acpica/utmutex.c | 12 ++++++------ drivers/acpi/acpica/utobject.c | 8 ++++---- drivers/acpi/acpica/utstate.c | 2 +- drivers/acpi/acpica/uttrack.c | 6 +++--- drivers/acpi/acpica/utxface.c | 2 +- include/acpi/acexcep.h | 2 +- 61 files changed, 188 insertions(+), 188 deletions(-)
drivers/acpi/acpica/acglobal.h +1 -1
@@ -70,7 +70,7 @@/* * Enable "slack" in the AML interpreter? Default is FALSE, and the -* interpreter strictly follows the ACPI specification. Setting to TRUE +* interpreter strictly follows the ACPI specification. Setting to TRUE * allows the interpreter to ignore certain errors and/or bad AML constructs. * * Currently, these features are enabled by this flag:
drivers/acpi/acpica/aclocal.h +2 -2
@@ -265,7 +265,7 @@ typedefacpi_status (*acpi_internal_method) (struct acpi_walk_state * walk_state);/* -* Bitmapped ACPI types. Used internally only +* Bitmapped ACPI types. Used internally only */ #define ACPI_BTYPE_ANY 0x00000000#define ACPI_BTYPE_INTEGER 0x00000001
@@ -584,7 +584,7 @@ struct acpi_pscope_state { };/* -* Thread state - one per thread across multiple walk states. Multiple walk +* Thread state - one per thread across multiple walk states. Multiple walk * states are created when there are nested control methods executing. */ struct acpi_thread_state {
drivers/acpi/acpica/acobject.h +2 -2
@@ -197,7 +197,7 @@ struct acpi_object_method {/****************************************************************************** * -* Objects that can be notified. All share a common notify_info area. +* Objects that can be notified. All share a common notify_info area. * *****************************************************************************/
@@ -234,7 +234,7 @@ ACPI_OBJECT_COMMON_HEADER ACPI_COMMON_NOTIFY_INFO};/****************************************************************************** * -* Fields. All share a common header/info field. +* Fields. All share a common header/info field. * *****************************************************************************/
drivers/acpi/acpica/acopcode.h +3 -3
@@ -54,7 +54,7 @@#define _UNK 0x6B/* -* Reserved ASCII characters. Do not use any of these for +* Reserved ASCII characters. Do not use any of these for * internal opcodes, since they are used to differentiate * name strings from AML opcodes */
@@ -63,7 +63,7 @@#define _PFX 0x6D/* -* All AML opcodes and the parse-time arguments for each. Used by the AML +* All AML opcodes and the parse-time arguments for each. Used by the AML * parser Each list is compressed into a 32-bit number and stored in the * master opcode table (in psopcode.c). */
@@ -193,7 +193,7 @@#define ARGP_ZERO_OP ARG_NONE/* -* All AML opcodes and the runtime arguments for each. Used by the AML +* All AML opcodes and the runtime arguments for each. Used by the AML * interpreter Each list is compressed into a 32-bit number and stored * in the master opcode table (in psopcode.c). *
drivers/acpi/acpica/acstruct.h +1 -1
@@ -53,7 +53,7 @@****************************************************************************/ /* -* Walk state - current state of a parse tree walk. Used for both a leisurely +* Walk state - current state of a parse tree walk. Used for both a leisurely * stroll through the tree (for whatever reason), and for control method * execution. */
drivers/acpi/acpica/dscontrol.c +1 -1
@@ -280,7 +280,7 @@ acpi_ds_exec_end_control_op(struct acpi_walk_state * walk_state,/* * Get the return value and save as the last result -* value. This is the only place where walk_state->return_desc +* value. This is the only place where walk_state->return_desc * is set to anything other than zero! */ walk_state->return_desc = walk_state->operands[0 ];
drivers/acpi/acpica/dsfield.c +1 -1
@@ -277,7 +277,7 @@ acpi_ds_create_buffer_field(union acpi_parse_object *op,* * RETURN: Status * -* DESCRIPTION: Process all named fields in a field declaration. Names are +* DESCRIPTION: Process all named fields in a field declaration. Names are * entered into the namespace. * ******************************************************************************/
drivers/acpi/acpica/dsmethod.c +3 -3
@@ -170,7 +170,7 @@ acpi_ds_create_method_mutex(union acpi_operand_object *method_desc)* * RETURN: Status * -* DESCRIPTION: Prepare a method for execution. Parses the method if necessary, +* DESCRIPTION: Prepare a method for execution. Parses the method if necessary, * increments the thread count, and waits at the method semaphore * for clearance to execute. *
@@ -444,7 +444,7 @@ acpi_ds_call_control_method(struct acpi_thread_state *thread,* RETURN: Status * * DESCRIPTION: Restart a method that was preempted by another (nested) method -* invocation. Handle the return value (if any) from the callee. +* invocation. Handle the return value (if any) from the callee. * ******************************************************************************/
@@ -530,7 +530,7 @@ acpi_ds_restart_control_method(struct acpi_walk_state *walk_state,* * RETURN: None * -* DESCRIPTION: Terminate a control method. Delete everything that the method +* DESCRIPTION: Terminate a control method. Delete everything that the method * created, delete all locals and arguments, and delete the parse * tree if requested. *
drivers/acpi/acpica/dsmthdat.c +7 -7
@@ -76,7 +76,7 @@ acpi_ds_method_data_get_type(u16 opcode,* RETURN: Status * * DESCRIPTION: Initialize the data structures that hold the method's arguments -* and locals. The data struct is an array of namespace nodes for +* and locals. The data struct is an array of namespace nodes for * each - this allows ref_of and de_ref_of to work properly for these * special data types. *
@@ -129,7 +129,7 @@ void acpi_ds_method_data_init(struct acpi_walk_state *walk_state)* * RETURN: None * -* DESCRIPTION: Delete method locals and arguments. Arguments are only +* DESCRIPTION: Delete method locals and arguments. Arguments are only * deleted if this method was called from another method. * ******************************************************************************/
@@ -183,7 +183,7 @@ void acpi_ds_method_data_delete_all(struct acpi_walk_state *walk_state)* * RETURN: Status * -* DESCRIPTION: Initialize arguments for a method. The parameter list is a list +* DESCRIPTION: Initialize arguments for a method. The parameter list is a list * of ACPI operand objects, either null terminated or whose length * is defined by max_param_count. *
@@ -401,7 +401,7 @@ acpi_ds_method_data_get_value(u8 type,* This means that either 1) The expected argument was * not passed to the method, or 2) A local variable * was referenced by the method (via the ASL) -* before it was initialized. Either case is an error. +* before it was initialized. Either case is an error. */ /* If slack enabled, init the local_x/arg_x to an Integer of value zero */
@@ -465,7 +465,7 @@ acpi_ds_method_data_get_value(u8 type,* * RETURN: None * -* DESCRIPTION: Delete the entry at Opcode:Index. Inserts +* DESCRIPTION: Delete the entry at Opcode:Index. Inserts * a null into the stack slot after the object is deleted. * ******************************************************************************/
@@ -523,7 +523,7 @@ acpi_ds_method_data_delete_value(u8 type,* * RETURN: Status * -* DESCRIPTION: Store a value in an Arg or Local. The obj_desc is installed +* DESCRIPTION: Store a value in an Arg or Local. The obj_desc is installed * as the new value for the Arg or Local and the reference count * for obj_desc is incremented. *
@@ -566,7 +566,7 @@ acpi_ds_store_object_to_local(u8 type,/* * If the reference count on the object is more than one, we must -* take a copy of the object before we store. A reference count +* take a copy of the object before we store. A reference count * of exactly 1 means that the object was just created during the * evaluation of an expression, and we can safely use it since it * is not used anywhere else.
drivers/acpi/acpica/dsobject.c +3 -3
@@ -293,7 +293,7 @@ acpi_ds_build_internal_buffer_obj(struct acpi_walk_state *walk_state,/* * Second arg is the buffer data (optional) byte_list can be either -* individual bytes or a string initializer. In either case, a +* individual bytes or a string initializer. In either case, a * byte_list appears in the AML. */ arg = op->common.value.arg;/* skip first arg */
@@ -568,7 +568,7 @@ acpi_ds_create_node(struct acpi_walk_state *walk_state,/* * Because of the execution pass through the non-control-method -* parts of the table, we can arrive here twice. Only init +* parts of the table, we can arrive here twice. Only init * the named object node the first time through */ if (acpi_ns_get_attached_object (node)) {
@@ -618,7 +618,7 @@ acpi_ds_create_node(struct acpi_walk_state *walk_state,* RETURN: Status * * DESCRIPTION: Initialize a namespace object from a parser Op and its -* associated arguments. The namespace object is a more compact +* associated arguments. The namespace object is a more compact * representation of the Op and its arguments. * ******************************************************************************/
drivers/acpi/acpica/dsutils.c +15 -15
@@ -61,7 +61,7 @@ ACPI_MODULE_NAME("dsutils")* * RETURN: None. * -* DESCRIPTION: Clear and remove a reference on an implicit return value. Used +* DESCRIPTION: Clear and remove a reference on an implicit return value. Used * to delete "stale" return values (if enabled, the return value * from every operator is saved at least momentarily, in case the * parent method exits.)
@@ -107,7 +107,7 @@ void acpi_ds_clear_implicit_return(struct acpi_walk_state *walk_state)* * DESCRIPTION: Implements the optional "implicit return". We save the result * of every ASL operator and control method invocation in case the -* parent method exit. Before storing a new return value, we +* parent method exit. Before storing a new return value, we * delete the previous return value. * ******************************************************************************/
@@ -198,7 +198,7 @@ acpi_ds_is_result_used(union acpi_parse_object * op,* * If there is no parent, or the parent is a scope_op, we are executing * at the method level. An executing method typically has no parent, -* since each method is parsed separately. A method invoked externally +* since each method is parsed separately. A method invoked externally * via execute_control_method has a scope_op as the parent. */ if ((!op->common.parent) ||
@@ -223,7 +223,7 @@ acpi_ds_is_result_used(union acpi_parse_object * op, }/* -* Decide what to do with the result based on the parent. If +* Decide what to do with the result based on the parent. If * the parent opcode will not use the result, delete the object. * Otherwise leave it as is, it will be deleted when it is used * as an operand later.
@@ -266,7 +266,7 @@ acpi_ds_is_result_used(union acpi_parse_object * op,/* * These opcodes allow term_arg(s) as operands and therefore -* the operands can be method calls. The result is used. +* the operands can be method calls. The result is used. */ goto result_used;
@@ -284,7 +284,7 @@ acpi_ds_is_result_used(union acpi_parse_object * op, AML_BANK_FIELD_OP)) {/* * These opcodes allow term_arg(s) as operands and therefore -* the operands can be method calls. The result is used. +* the operands can be method calls. The result is used. */ goto result_used; }
@@ -329,9 +329,9 @@ acpi_ds_is_result_used(union acpi_parse_object * op,* * RETURN: Status * -* DESCRIPTION: Used after interpretation of an opcode. If there is an internal +* DESCRIPTION: Used after interpretation of an opcode. If there is an internal * result descriptor, check if the parent opcode will actually use -* this result. If not, delete the result now so that it will +* this result. If not, delete the result now so that it will * not become orphaned. * ******************************************************************************/
@@ -376,7 +376,7 @@ acpi_ds_delete_result_if_not_used(union acpi_parse_object *op,* * RETURN: Status * -* DESCRIPTION: Resolve all operands to their values. Used to prepare +* DESCRIPTION: Resolve all operands to their values. Used to prepare * arguments to a control method invocation (a call from one * method to another.) *
@@ -391,7 +391,7 @@ acpi_status acpi_ds_resolve_operands(struct acpi_walk_state *walk_state)/* * Attempt to resolve each of the valid operands -* Method arguments are passed by reference, not by value. This means +* Method arguments are passed by reference, not by value. This means * that the actual objects are passed, not copies of the objects. */ for (i =0 ; i < walk_state->num_operands; i++) {
@@ -451,7 +451,7 @@ void acpi_ds_clear_operands(struct acpi_walk_state *walk_state)* RETURN: Status * * DESCRIPTION: Translate a parse tree object that is an argument to an AML -* opcode to the equivalent interpreter object. This may include +* opcode to the equivalent interpreter object. This may include * looking up a name or entering a new name into the internal * namespace. *
@@ -496,9 +496,9 @@ acpi_ds_create_operand(struct acpi_walk_state *walk_state,/* * Special handling for buffer_field declarations. This is a deferred * opcode that unfortunately defines the field name as the last -* parameter instead of the first. We get here when we are performing +* parameter instead of the first. We get here when we are performing * the deferred execution, so the actual name of the field is already -* in the namespace. We don't want to attempt to look it up again +* in the namespace. We don't want to attempt to look it up again * because we may be executing in a different scope than where the * actual opcode exists. */
@@ -605,8 +605,8 @@ acpi_ds_create_operand(struct acpi_walk_state *walk_state,/* * If the name is null, this means that this is an * optional result parameter that was not specified -* in the original ASL. Create a Zero Constant for a -* placeholder. (Store to a constant is a Noop.) +* in the original ASL. Create a Zero Constant for a +* placeholder. (Store to a constant is a Noop.) */ opcode =AML_ZERO_OP ;/* Has no arguments! */
drivers/acpi/acpica/dswexec.c +4 -4
@@ -204,7 +204,7 @@ acpi_ds_get_predicate_value(struct acpi_walk_state *walk_state,* RETURN: Status * * DESCRIPTION: Descending callback used during the execution of control -* methods. This is where most operators and operands are +* methods. This is where most operators and operands are * dispatched to the interpreter. * ****************************************************************************/
@@ -297,7 +297,7 @@ acpi_ds_exec_begin_op(struct acpi_walk_state *walk_state,if (walk_state->walk_type &ACPI_WALK_METHOD ) {/* * Found a named object declaration during method execution; -* we must enter this object into the namespace. The created +* we must enter this object into the namespace. The created * object is temporary and will be deleted upon completion of * the execution of this method. *
@@ -348,7 +348,7 @@ acpi_ds_exec_begin_op(struct acpi_walk_state *walk_state,* RETURN: Status * * DESCRIPTION: Ascending callback used during the execution of control -* methods. The only thing we really need to do here is to +* methods. The only thing we really need to do here is to * notice the beginning of IF, ELSE, and WHILE blocks. * ****************************************************************************/
@@ -432,7 +432,7 @@ acpi_status acpi_ds_exec_end_op(struct acpi_walk_state *walk_state)if (ACPI_SUCCESS (status)) {/* * Dispatch the request to the appropriate interpreter handler -* routine. There is one routine per opcode "type" based upon the +* routine. There is one routine per opcode "type" based upon the * number of opcode arguments and return type. */ status =
drivers/acpi/acpica/dswstate.c +4 -4
@@ -348,7 +348,7 @@ acpi_ds_obj_stack_push(void *object, struct acpi_walk_state * walk_state)* * RETURN: Status * -* DESCRIPTION: Pop this walk's object stack. Objects on the stack are NOT +* DESCRIPTION: Pop this walk's object stack. Objects on the stack are NOT * deleted by this routine. * ******************************************************************************/
@@ -492,7 +492,7 @@ acpi_ds_push_walk_state(struct acpi_walk_state *walk_state,* RETURN: A walk_state object popped from the thread's stack * * DESCRIPTION: Remove and return the walkstate object that is at the head of -* the walk stack for the given walk list. NULL indicates that +* the walk stack for the given walk list. NULL indicates that * the list is empty. * ******************************************************************************/
@@ -532,7 +532,7 @@ struct acpi_walk_state *acpi_ds_pop_walk_state(struct acpi_thread_state *thread)* * RETURN: Pointer to the new walk state. * -* DESCRIPTION: Allocate and initialize a new walk state. The current walk +* DESCRIPTION: Allocate and initialize a new walk state. The current walk * state is set to this new state. * ******************************************************************************/
@@ -657,7 +657,7 @@ acpi_ds_init_aml_walk(struct acpi_walk_state *walk_state,/* * Setup the current scope. * Find a Named Op that has a namespace node associated with it. -* search upwards from this Op. Current scope is the first +* search upwards from this Op. Current scope is the first * Op with a namespace node. */ extra_op = parser_state->start_op;
drivers/acpi/acpica/excreate.c +3 -3
@@ -78,7 +78,7 @@ acpi_status acpi_ex_create_alias(struct acpi_walk_state *walk_state) (target_node->type == ACPI_TYPE_LOCAL_METHOD_ALIAS)) {/* * Dereference an existing alias so that we don't create a chain -* of aliases. With this code, we guarantee that an alias is +* of aliases. With this code, we guarantee that an alias is * always exactly one level of indirection away from the * actual aliased name. */
@@ -90,7 +90,7 @@ acpi_status acpi_ex_create_alias(struct acpi_walk_state *walk_state)/* * For objects that can never change (i.e., the NS node will * permanently point to the same object), we can simply attach -* the object to the new NS node. For other objects (such as +* the object to the new NS node. For other objects (such as * Integers, buffers, etc.), we have to point the Alias node * to the original Node. */
@@ -139,7 +139,7 @@ acpi_status acpi_ex_create_alias(struct acpi_walk_state *walk_state)/* * The new alias assumes the type of the target, and it points -* to the same object. The reference count of the object has an +* to the same object. The reference count of the object has an * additional reference to prevent deletion out from under either the * target node or the alias Node */
drivers/acpi/acpica/exdump.c +1 -1
@@ -778,7 +778,7 @@ acpi_ex_dump_operands(union acpi_operand_object **operands,* PARAMETERS: title - Descriptive text * value - Value to be displayed * -* DESCRIPTION: Object dump output formatting functions. These functions +* DESCRIPTION: Object dump output formatting functions. These functions * reduce the number of format strings required and keeps them * all in one place for easy modification. *
drivers/acpi/acpica/exfield.c +2 -2
@@ -59,7 +59,7 @@ ACPI_MODULE_NAME("exfield")* * RETURN: Status * -* DESCRIPTION: Read from a named field. Returns either an Integer or a +* DESCRIPTION: Read from a named field. Returns either an Integer or a * Buffer, depending on the size of the field. * ******************************************************************************/
@@ -149,7 +149,7 @@ acpi_ex_read_data_from_field(struct acpi_walk_state *walk_state,* Allocate a buffer for the contents of the field. * * If the field is larger than the current integer width, create -* a BUFFER to hold it. Otherwise, use an INTEGER. This allows +* a BUFFER to hold it. Otherwise, use an INTEGER. This allows * the use of arithmetic operators on the returned value if the * field size is equal or smaller than an Integer. *
drivers/acpi/acpica/exfldio.c +6 -6
@@ -154,7 +154,7 @@ acpi_ex_setup_region(union acpi_operand_object *obj_desc,#endif /* -* Validate the request. The entire request from the byte offset for a +* Validate the request. The entire request from the byte offset for a * length of one field datum (access width) must fit within the region. * (Region length is specified in bytes) */
@@ -182,7 +182,7 @@ acpi_ex_setup_region(union acpi_operand_object *obj_desc, obj_desc->common_field.access_byte_width) {/* * This is the case where the access_type (acc_word, etc.) is wider -* than the region itself. For example, a region of length one +* than the region itself. For example, a region of length one * byte, and a field with Dword access specified. */ ACPI_ERROR((AE_INFO,
@@ -320,7 +320,7 @@ acpi_ex_access_region(union acpi_operand_object *obj_desc,* * DESCRIPTION: Check if a value is out of range of the field being written. * Used to check if the values written to Index and Bank registers -* are out of range. Normally, the value is simply truncated +* are out of range. Normally, the value is simply truncated * to fit the field, but this case is most likely a serious * coding error in the ASL. *
@@ -369,7 +369,7 @@ acpi_ex_register_overflow(union acpi_operand_object *obj_desc, u64 value)* * RETURN: Status * -* DESCRIPTION: Read or Write a single datum of a field. The field_type is +* DESCRIPTION: Read or Write a single datum of a field. The field_type is * demultiplexed here to handle the different types of fields * (buffer_field, region_field, index_field, bank_field) *
@@ -859,7 +859,7 @@ acpi_ex_insert_into_field(union acpi_operand_object *obj_desc,ACPI_ROUND_BITS_UP_TO_BYTES (obj_desc->common_field.bit_length);/* * We must have a buffer that is at least as long as the field -* we are writing to. This is because individual fields are +* we are writing to. This is because individual fields are * indivisible and partial writes are not supported -- as per * the ACPI specification. */
@@ -874,7 +874,7 @@ acpi_ex_insert_into_field(union acpi_operand_object *obj_desc,/* * Copy the original data to the new buffer, starting -* at Byte zero. All unused (upper) bytes of the +* at Byte zero. All unused (upper) bytes of the * buffer will be 0. */ ACPI_MEMCPY ((char *)new_buffer, (char *)buffer, buffer_length);
drivers/acpi/acpica/exmisc.c +2 -2
@@ -253,7 +253,7 @@ acpi_ex_do_concatenate(union acpi_operand_object *operand0,ACPI_FUNCTION_TRACE (ex_do_concatenate);/* -* Convert the second operand if necessary. The first operand +* Convert the second operand if necessary. The first operand * determines the type of the second operand, (See the Data Types * section of the ACPI specification.) Both object types are * guaranteed to be either Integer/String/Buffer by the operand
@@ -572,7 +572,7 @@ acpi_ex_do_logical_op(u16 opcode,ACPI_FUNCTION_TRACE (ex_do_logical_op);/* -* Convert the second operand if necessary. The first operand +* Convert the second operand if necessary. The first operand * determines the type of the second operand, (See the Data Types * section of the ACPI 3.0+ specification.) Both object types are * guaranteed to be either Integer/String/Buffer by the operand
drivers/acpi/acpica/exnames.c +1 -1
@@ -62,7 +62,7 @@ static acpi_status acpi_ex_name_segment(u8 **in_aml_address, char *name_string);* (-1)==root, 0==none * num_name_segs - count of 4-character name segments * -* RETURN: A pointer to the allocated string segment. This segment must +* RETURN: A pointer to the allocated string segment. This segment must * be deleted by the caller. * * DESCRIPTION: Allocate a buffer for a name string. Ensure allocated name
drivers/acpi/acpica/exoparg1.c +5 -5
@@ -605,7 +605,7 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state) }/* -* Set result to ONES (TRUE) if Value == 0. Note: +* Set result to ONES (TRUE) if Value == 0. Note: * return_desc->Integer.Value is initially == 0 (FALSE) from above. */ if (!operand[0 ]->integer.value) {
@@ -617,7 +617,7 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state)case AML_INCREMENT_OP :/* Increment (Operand) */ /* -* Create a new integer. Can't just get the base integer and +* Create a new integer. Can't just get the base integer and * increment it because it may be an Arg or Field. */ return_desc =acpi_ut_create_internal_object (ACPI_TYPE_INTEGER );
@@ -685,7 +685,7 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state)/* * Note: The operand is not resolved at this point because we want to -* get the associated object, not its value. For example, we don't +* get the associated object, not its value. For example, we don't * want to resolve a field_unit to its value, we want the actual * field_unit object. */
@@ -726,7 +726,7 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state)/* * The type of the base object must be integer, buffer, string, or -* package. All others are not supported. +* package. All others are not supported. * * NOTE: Integer is not specifically supported by the ACPI spec, * but is supported implicitly via implicit operand conversion.
@@ -964,7 +964,7 @@ acpi_status acpi_ex_opcode_1A_0T_1R(struct acpi_walk_state *walk_state)case ACPI_TYPE_PACKAGE :/* -* Return the referenced element of the package. We must +* Return the referenced element of the package. We must * add another reference to the referenced object, however. */ return_desc =
drivers/acpi/acpica/exoparg2.c +1 -1
@@ -123,7 +123,7 @@ acpi_status acpi_ex_opcode_2A_0T_0R(struct acpi_walk_state *walk_state)/* * Dispatch the notify to the appropriate handler * NOTE: the request is queued for execution after this method -* completes. The notify handlers are NOT invoked synchronously +* completes. The notify handlers are NOT invoked synchronously * from this thread -- because handlers may in turn run other * control methods. */
drivers/acpi/acpica/exoparg3.c +1 -1
@@ -157,7 +157,7 @@ acpi_status acpi_ex_opcode_3A_1T_1R(struct acpi_walk_state *walk_state)case AML_MID_OP :/* Mid (Source[0], Index[1], Length[2], Result[3]) */ /* -* Create the return object. The Source operand is guaranteed to be +* Create the return object. The Source operand is guaranteed to be * either a String or a Buffer, so just use its type. */ return_desc = acpi_ut_create_internal_object((operand[0])->
drivers/acpi/acpica/exoparg6.c +1 -1
@@ -268,7 +268,7 @@ acpi_status acpi_ex_opcode_6A_0T_1R(struct acpi_walk_state * walk_state)* and the next should be examined. * * Upon finding a match, the loop will terminate via "break" at -* the bottom. If it terminates "normally", match_value will be +* the bottom. If it terminates "normally", match_value will be * ACPI_UINT64_MAX (Ones) (its initial value) indicating that no * match was found. */
drivers/acpi/acpica/exprep.c +6 -6
@@ -77,8 +77,8 @@ acpi_ex_generate_access(u32 field_bit_offset,* any_acc keyword. * * NOTE: Need to have the region_length in order to check for boundary -* conditions (end-of-region). However, the region_length is a deferred -* operation. Therefore, to complete this implementation, the generation +* conditions (end-of-region). However, the region_length is a deferred +* operation. Therefore, to complete this implementation, the generation * of this access width must be deferred until the region length has * been evaluated. *
@@ -307,7 +307,7 @@ acpi_ex_decode_field_access(union acpi_operand_object *obj_desc,* RETURN: Status * * DESCRIPTION: Initialize the areas of the field object that are common -* to the various types of fields. Note: This is very "sensitive" +* to the various types of fields. Note: This is very "sensitive" * code because we are solving the general case for field * alignment. *
@@ -335,13 +335,13 @@ acpi_ex_prep_common_field_object(union acpi_operand_object *obj_desc, obj_desc->common_field.bit_length = field_bit_length;/* -* Decode the access type so we can compute offsets. The access type gives +* Decode the access type so we can compute offsets. The access type gives * two pieces of information - the width of each field access and the * necessary byte_alignment (address granularity) of the access. * * For any_acc, the access_bit_width is the largest width that is both * necessary and possible in an attempt to access the whole field in one -* I/O operation. However, for any_acc, the byte_alignment is always one +* I/O operation. However, for any_acc, the byte_alignment is always one * byte. * * For all Buffer Fields, the byte_alignment is always one byte.
@@ -362,7 +362,7 @@ acpi_ex_prep_common_field_object(union acpi_operand_object *obj_desc,/* * base_byte_offset is the address of the start of the field within the -* region. It is the byte address of the first *datum* (field-width data +* region. It is the byte address of the first *datum* (field-width data * unit) of the field. (i.e., the first datum that contains at least the * first *bit* of the field.) *
drivers/acpi/acpica/exregion.c +1 -1
@@ -201,7 +201,7 @@ acpi_ex_system_memory_space_handler(u32 function,* Perform the memory read or write * * Note: For machines that do not support non-aligned transfers, the target -* address was checked for alignment above. We do not attempt to break the +* address was checked for alignment above. We do not attempt to break the * transfer up into smaller (byte-size) chunks because the AML specifically * asked for a transfer width that the hardware may require. */
drivers/acpi/acpica/exresnte.c +4 -4
@@ -57,8 +57,8 @@ ACPI_MODULE_NAME("exresnte")* PARAMETERS: object_ptr - Pointer to a location that contains * a pointer to a NS node, and will receive a * pointer to the resolved object. -* walk_state - Current state. Valid only if executing AML -* code. NULL if simply resolving an object +* walk_state - Current state. Valid only if executing AML +* code. NULL if simply resolving an object * * RETURN: Status *
@@ -66,7 +66,7 @@ ACPI_MODULE_NAME("exresnte")* * Note: for some of the data types, the pointer attached to the Node * can be either a pointer to an actual internal object or a pointer into the -* AML stream itself. These types are currently: +* AML stream itself. These types are currently: * * ACPI_TYPE_INTEGER * ACPI_TYPE_STRING
@@ -88,7 +88,7 @@ acpi_ex_resolve_node_to_value(struct acpi_namespace_node **object_ptr,ACPI_FUNCTION_TRACE (ex_resolve_node_to_value);/* -* The stack pointer points to a struct acpi_namespace_node (Node). Get the +* The stack pointer points to a struct acpi_namespace_node (Node). Get the * object that is attached to the Node. */ node = *object_ptr;
drivers/acpi/acpica/exresolv.c +1 -1
@@ -326,7 +326,7 @@ acpi_ex_resolve_object_to_value(union acpi_operand_object **stack_ptr,* * RETURN: Status * -* DESCRIPTION: Return the base object and type. Traverse a reference list if +* DESCRIPTION: Return the base object and type. Traverse a reference list if * necessary to get to the base object. * ******************************************************************************/
drivers/acpi/acpica/exresop.c +2 -2
@@ -86,7 +86,7 @@ acpi_ex_check_object_type(acpi_object_type type_needed,if (type_needed ==ACPI_TYPE_LOCAL_REFERENCE ) {/* * Allow the AML "Constant" opcodes (Zero, One, etc.) to be reference -* objects and thus allow them to be targets. (As per the ACPI +* objects and thus allow them to be targets. (As per the ACPI * specification, a store to a constant is a noop.) */ if ((this_type == ACPI_TYPE_INTEGER) &&
@@ -638,7 +638,7 @@ acpi_ex_resolve_operands(u16 opcode,if (acpi_gbl_enable_interpreter_slack) {/* * Enable original behavior of Store(), allowing any and all -* objects as the source operand. The ACPI spec does not +* objects as the source operand. The ACPI spec does not * allow this, however. */ break ;
drivers/acpi/acpica/exstore.c +2 -2
@@ -374,7 +374,7 @@ acpi_ex_store_object_to_index(union acpi_operand_object *source_desc,* with the input value. * * When storing into an object the data is converted to the -* target object type then stored in the object. This means +* target object type then stored in the object. This means * that the target object type (for an initialized target) will * not be changed by a store operation. *
@@ -491,7 +491,7 @@ acpi_ex_store_object_to_node(union acpi_operand_object *source_desc,acpi_ut_get_object_type_name (source_desc), source_desc, node)); -/* No conversions for all other types. Just attach the source object */ +/* No conversions for all other types. Just attach the source object */ status =acpi_ns_attach_object (node, source_desc, source_desc->common.type);
drivers/acpi/acpica/exstoren.c +5 -5
@@ -60,7 +60,7 @@ ACPI_MODULE_NAME("exstoren")* * RETURN: Status, resolved object in source_desc_ptr. * -* DESCRIPTION: Resolve an object. If the object is a reference, dereference +* DESCRIPTION: Resolve an object. If the object is a reference, dereference * it and return the actual object in the source_desc_ptr. * ******************************************************************************/
@@ -92,7 +92,7 @@ acpi_ex_resolve_object(union acpi_operand_object **source_desc_ptr,/* * Stores into a Field/Region or into a Integer/Buffer/String -* are all essentially the same. This case handles the +* are all essentially the same. This case handles the * "interchangeable" types Integer, String, and Buffer. */ if (source_desc->common.type ==ACPI_TYPE_LOCAL_REFERENCE ) {
@@ -166,7 +166,7 @@ acpi_ex_resolve_object(union acpi_operand_object **source_desc_ptr,* * RETURN: Status * -* DESCRIPTION: "Store" an object to another object. This may include +* DESCRIPTION: "Store" an object to another object. This may include * converting the source type to the target type (implicit * conversion), and a copy of the value of the source to * the target.
@@ -177,14 +177,14 @@ acpi_ex_resolve_object(union acpi_operand_object **source_desc_ptr,* with the input value. * * When storing into an object the data is converted to the -* target object type then stored in the object. This means +* target object type then stored in the object. This means * that the target object type (for an initialized target) will * not be changed by a store operation. * * This module allows destination types of Number, String, * Buffer, and Package. * -* Assumes parameters are already validated. NOTE: source_desc +* Assumes parameters are already validated. NOTE: source_desc * resolution (from a reference object) must be performed by * the caller if necessary. *
drivers/acpi/acpica/exstorob.c +2 -2
@@ -107,7 +107,7 @@ acpi_ex_store_buffer_to_buffer(union acpi_operand_object *source_desc,#ifdef ACPI_OBSOLETE_BEHAVIOR /* * NOTE: ACPI versions up to 3.0 specified that the buffer must be -* truncated if the string is smaller than the buffer. However, "other" +* truncated if the string is smaller than the buffer. However, "other" * implementations of ACPI never did this and thus became the defacto * standard. ACPI 3.0A changes this behavior such that the buffer * is no longer truncated.
@@ -116,7 +116,7 @@ acpi_ex_store_buffer_to_buffer(union acpi_operand_object *source_desc,/* * OBSOLETE BEHAVIOR: * If the original source was a string, we must truncate the buffer, -* according to the ACPI spec. Integer-to-Buffer and Buffer-to-Buffer +* according to the ACPI spec. Integer-to-Buffer and Buffer-to-Buffer * copy must not truncate the original buffer. */ if (original_src_type ==ACPI_TYPE_STRING ) {
drivers/acpi/acpica/exsystem.c +4 -4
@@ -58,7 +58,7 @@ ACPI_MODULE_NAME("exsystem")* RETURN: Status * * DESCRIPTION: Implements a semaphore wait with a check to see if the -* semaphore is available immediately. If it is not, the +* semaphore is available immediately. If it is not, the * interpreter is released before waiting. * ******************************************************************************/
@@ -103,7 +103,7 @@ acpi_status acpi_ex_system_wait_semaphore(acpi_semaphore semaphore, u16 timeout)* RETURN: Status * * DESCRIPTION: Implements a mutex wait with a check to see if the -* mutex is available immediately. If it is not, the +* mutex is available immediately. If it is not, the * interpreter is released before waiting. * ******************************************************************************/
@@ -151,7 +151,7 @@ acpi_status acpi_ex_system_wait_mutex(acpi_mutex mutex, u16 timeout)* DESCRIPTION: Suspend running thread for specified amount of time. * Note: ACPI specification requires that Stall() does not * relinquish the processor, and delays longer than 100 usec -* should use Sleep() instead. We allow stalls up to 255 usec +* should use Sleep() instead. We allow stalls up to 255 usec * for compatibility with other interpreters and existing BIOSs. * ******************************************************************************/
@@ -253,7 +253,7 @@ acpi_status acpi_ex_system_signal_event(union acpi_operand_object * obj_desc)* RETURN: Status * * DESCRIPTION: Provides an access point to perform synchronization operations -* within the AML. This operation is a request to wait for an +* within the AML. This operation is a request to wait for an * event. * ******************************************************************************/
drivers/acpi/acpica/exutils.c +2 -2
@@ -44,12 +44,12 @@/* * DEFINE_AML_GLOBALS is tested in amlcode.h * to determine whether certain global names should be "defined" or only -* "declared" in the current compilation. This enhances maintainability +* "declared" in the current compilation. This enhances maintainability * by enabling a single header file to embody all knowledge of the names * in question. * * Exactly one module of any executable should #define DEFINE_GLOBALS -* before #including the header files which use this convention. The +* before #including the header files which use this convention. The * names in question will be defined and initialized in that module, * and declared as extern in all other modules which #include those * header files.
drivers/acpi/acpica/hwacpi.c +1 -1
@@ -135,7 +135,7 @@ acpi_status acpi_hw_set_mode(u32 mode)* * RETURN: SYS_MODE_ACPI or SYS_MODE_LEGACY * -* DESCRIPTION: Return current operating state of system. Determined by +* DESCRIPTION: Return current operating state of system. Determined by * querying the SCI_EN bit. * ******************************************************************************/
drivers/acpi/acpica/hwtimer.c +1 -1
@@ -127,7 +127,7 @@ ACPI_EXPORT_SYMBOL(acpi_get_timer)* a versatile and accurate timer. * * Note that this function accommodates only a single timer -* rollover. Thus for 24-bit timers, this function should only +* 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: *
drivers/acpi/acpica/nsalloc.c +2 -2
@@ -332,7 +332,7 @@ void acpi_ns_delete_children(struct acpi_namespace_node *parent_node)* * RETURN: None. * -* DESCRIPTION: Delete a subtree of the namespace. This includes all objects +* DESCRIPTION: Delete a subtree of the namespace. This includes all objects * stored within the subtree. * ******************************************************************************/
@@ -418,7 +418,7 @@ void acpi_ns_delete_namespace_subtree(struct acpi_namespace_node *parent_node)* RETURN: Status * * DESCRIPTION: Delete entries within the namespace that are owned by a -* specific ID. Used to delete entire ACPI tables. All +* specific ID. Used to delete entire ACPI tables. All * reference counts are updated. * * MUTEX: Locks namespace during deletion walk.
drivers/acpi/acpica/nsdump.c +1 -1
@@ -692,7 +692,7 @@ void acpi_ns_dump_entry(acpi_handle handle, u32 debug_level)* * PARAMETERS: search_base - Root of subtree to be dumped, or * NS_ALL to dump the entire namespace -* max_depth - Maximum depth of dump. Use INT_MAX +* max_depth - Maximum depth of dump. Use INT_MAX * for an effectively unlimited depth. * * RETURN: None
drivers/acpi/acpica/nsload.c +5 -5
@@ -80,8 +80,8 @@ acpi_ns_load_table(u32 table_index, struct acpi_namespace_node *node)/* * Parse the table and load the namespace with all named -* objects found within. Control methods are NOT parsed -* at this time. In fact, the control methods cannot be +* objects found within. Control methods are NOT parsed +* at this time. In fact, the control methods cannot be * parsed until the entire namespace is loaded, because * if a control method makes a forward reference (call) * to another control method, we can't continue parsing
@@ -122,7 +122,7 @@ acpi_ns_load_table(u32 table_index, struct acpi_namespace_node *node) }/* -* Now we can parse the control methods. We always parse +* Now we can parse the control methods. We always parse * them here for a sanity check, and if configured for * just-in-time parsing, we delete the control method * parse trees.
@@ -166,7 +166,7 @@ acpi_status acpi_ns_load_namespace(void) }/* -* Load the namespace. The DSDT is required, +* Load the namespace. The DSDT is required, * but the SSDT and PSDT tables are optional. */ status =acpi_ns_load_table_by_type (ACPI_TABLE_ID_DSDT );
@@ -283,7 +283,7 @@ static acpi_status acpi_ns_delete_subtree(acpi_handle start_handle)* RETURN: Status * * DESCRIPTION: Shrinks the namespace, typically in response to an undocking -* event. Deletes an entire subtree starting from (and +* event. Deletes an entire subtree starting from (and * including) the given handle. * ******************************************************************************/
drivers/acpi/acpica/nsobject.c +4 -4
@@ -61,7 +61,7 @@ ACPI_MODULE_NAME("nsobject")* RETURN: Status * * DESCRIPTION: Record the given object as the value associated with the -* name whose acpi_handle is passed. If Object is NULL +* name whose acpi_handle is passed. If Object is NULL * and Type is ACPI_TYPE_ANY, set the name as having no value. * Note: Future may require that the Node->Flags field be passed * as a parameter.
@@ -133,7 +133,7 @@ acpi_ns_attach_object(struct acpi_namespace_node *node, ((struct acpi_namespace_node *)object)->object) {/* * Value passed is a name handle and that name has a -* non-null value. Use that name's value and type. +* non-null value. Use that name's value and type. */ obj_desc = ((struct acpi_namespace_node *)object)->object; object_type = ((struct acpi_namespace_node *)object)->type;
@@ -321,7 +321,7 @@ union acpi_operand_object *acpi_ns_get_secondary_object(union* * RETURN: Status * -* DESCRIPTION: Low-level attach data. Create and attach a Data object. +* DESCRIPTION: Low-level attach data. Create and attach a Data object. * ******************************************************************************/
@@ -377,7 +377,7 @@ acpi_ns_attach_data(struct acpi_namespace_node *node,* * RETURN: Status * -* DESCRIPTION: Low-level detach data. Delete the data node, but the caller +* DESCRIPTION: Low-level detach data. Delete the data node, but the caller * is responsible for the actual data. * ******************************************************************************/
drivers/acpi/acpica/nsparse.c +4 -4
@@ -168,11 +168,11 @@ acpi_ns_parse_table(u32 table_index, struct acpi_namespace_node *start_node)/* * AML Parse, pass 1 * -* In this pass, we load most of the namespace. Control methods -* are not parsed until later. A parse tree is not created. Instead, -* each Parser Op subtree is deleted when it is finished. This saves +* In this pass, we load most of the namespace. Control methods +* are not parsed until later. A parse tree is not created. Instead, +* each Parser Op subtree is deleted when it is finished. This saves * a great deal of memory, and allows a small cache of parse objects -* to service the entire parse. The second pass of the parse then +* to service the entire parse. The second pass of the parse then * performs another complete parse of the AML. */ ACPI_DEBUG_PRINT ((ACPI_DB_PARSE ,"**** Start pass 1\n" ));
drivers/acpi/acpica/nsutils.c +3 -3
@@ -530,7 +530,7 @@ acpi_ns_externalize_name(u32 internal_name_length, ((num_segments >0 ) ? (num_segments -1 ) :0 ) +1 ;/* -* Check to see if we're still in bounds. If not, there's a problem +* Check to see if we're still in bounds. If not, there's a problem * with internal_name (invalid format). */ if (required_length > internal_name_length) {
@@ -681,7 +681,7 @@ u32 acpi_ns_opens_scope(acpi_object_type type)* \ (backslash) and ^ (carat) prefixes, and the * . (period) to separate segments are supported. * prefix_node - Root of subtree to be searched, or NS_ALL for the -* root of the name space. If Name is fully +* root of the name space. If Name is fully * qualified (first s8 is '\'), the passed value * of Scope will not be accessed. * flags - Used to indicate whether to perform upsearch or
@@ -689,7 +689,7 @@ u32 acpi_ns_opens_scope(acpi_object_type type)* return_node - Where the Node is returned * * DESCRIPTION: Look up a name relative to a given scope and return the -* corresponding Node. NOTE: Scope can be null. +* corresponding Node. NOTE: Scope can be null. * * MUTEX: Locks namespace *
drivers/acpi/acpica/nswalk.c +5 -5
@@ -60,8 +60,8 @@ ACPI_MODULE_NAME("nswalk")* RETURN: struct acpi_namespace_node - Pointer to the NEXT child or NULL if * none is found. * -* DESCRIPTION: Return the next peer node within the namespace. If Handle -* is valid, Scope is ignored. Otherwise, the first node +* DESCRIPTION: Return the next peer node within the namespace. If Handle +* is valid, Scope is ignored. Otherwise, the first node * within Scope is returned. * ******************************************************************************/
@@ -97,8 +97,8 @@ struct acpi_namespace_node *acpi_ns_get_next_node(struct acpi_namespace_node* RETURN: struct acpi_namespace_node - Pointer to the NEXT child or NULL if * none is found. * -* DESCRIPTION: Return the next peer node within the namespace. If Handle -* is valid, Scope is ignored. Otherwise, the first node +* DESCRIPTION: Return the next peer node within the namespace. If Handle +* is valid, Scope is ignored. Otherwise, the first node * within Scope is returned. * ******************************************************************************/
@@ -305,7 +305,7 @@ acpi_ns_walk_namespace(acpi_object_type type,/* * Depth first search: Attempt to go down another level in the -* namespace if we are allowed to. Don't go any further if we have +* namespace if we are allowed to. Don't go any further if we have * reached the caller specified maximum depth or if the user * function has specified that the maximum depth has been reached. */
drivers/acpi/acpica/nsxfeval.c +7 -7
@@ -61,16 +61,16 @@ static void acpi_ns_resolve_references(struct acpi_evaluate_info *info);* PARAMETERS: handle - Object handle (optional) * pathname - Object pathname (optional) * external_params - List of parameters to pass to method, -* terminated by NULL. May be NULL +* terminated by NULL. May be NULL * if no parameters are being passed. * return_buffer - Where to put method's return value (if -* any). If NULL, no value is returned. +* any). If NULL, no value is returned. * return_type - Expected type of return object * * RETURN: Status * * DESCRIPTION: Find and evaluate the given object, passing the given -* parameters if necessary. One of "Handle" or "Pathname" must +* parameters if necessary. One of "Handle" or "Pathname" must * be valid (non-null) * ******************************************************************************/
@@ -155,15 +155,15 @@ ACPI_EXPORT_SYMBOL(acpi_evaluate_object_typed)* PARAMETERS: handle - Object handle (optional) * pathname - Object pathname (optional) * external_params - List of parameters to pass to method, -* terminated by NULL. May be NULL +* terminated by NULL. May be NULL * if no parameters are being passed. * return_buffer - Where to put method's return value (if -* any). If NULL, no value is returned. +* any). If NULL, no value is returned. * * RETURN: Status * * DESCRIPTION: Find and evaluate the given object, passing the given -* parameters if necessary. One of "Handle" or "Pathname" must +* parameters if necessary. One of "Handle" or "Pathname" must * be valid (non-null) * ******************************************************************************/
@@ -656,7 +656,7 @@ acpi_ns_get_device_callback(acpi_handle obj_handle,* DESCRIPTION: Performs a modified depth-first walk of the namespace tree, * starting (and ending) at the object specified by start_handle. * The user_function is called whenever an object of type -* Device is found. If the user function returns +* Device is found. If the user function returns * a non-zero value, the search is terminated immediately and this * value is returned to the caller. *
drivers/acpi/acpica/nsxfname.c +3 -3
@@ -69,8 +69,8 @@ static char *acpi_ns_copy_device_id(struct acpi_pnp_device_id *dest,* RETURN: Status * * DESCRIPTION: This routine will search for a caller specified name in the -* name space. The caller can restrict the search region by -* specifying a non NULL parent. The parent value is itself a +* name space. The caller can restrict the search region by +* specifying a non NULL parent. The parent value is itself a * namespace handle. * ******************************************************************************/
@@ -149,7 +149,7 @@ ACPI_EXPORT_SYMBOL(acpi_get_handle)* RETURN: Pointer to a string containing the fully qualified Name. * * DESCRIPTION: This routine returns the fully qualified name associated with -* the Handle parameter. This and the acpi_pathname_to_handle are +* the Handle parameter. This and the acpi_pathname_to_handle are * complementary functions. * ******************************************************************************/
drivers/acpi/acpica/nsxfobj.c +2 -2
@@ -220,8 +220,8 @@ ACPI_EXPORT_SYMBOL(acpi_get_parent)* * RETURN: Status * -* DESCRIPTION: Return the next peer object within the namespace. If Handle is -* valid, Scope is ignored. Otherwise, the first object within +* DESCRIPTION: Return the next peer object within the namespace. If Handle is +* valid, Scope is ignored. Otherwise, the first object within * Scope is returned. * ******************************************************************************/
drivers/acpi/acpica/psargs.c +4 -4
@@ -120,7 +120,7 @@ acpi_ps_get_next_package_length(struct acpi_parse_state *parser_state)* RETURN: Pointer to end-of-package +1 * * DESCRIPTION: Get next package length and return a pointer past the end of -* the package. Consumes the package length field +* the package. Consumes the package length field * ******************************************************************************/
@@ -147,8 +147,8 @@ u8 *acpi_ps_get_next_package_end(struct acpi_parse_state *parser_state)* RETURN: Pointer to the start of the name string (pointer points into * the AML. * -* DESCRIPTION: Get next raw namestring within the AML stream. Handles all name -* prefix characters. Set parser state to point past the string. +* DESCRIPTION: Get next raw namestring within the AML stream. Handles all name +* prefix characters. Set parser state to point past the string. * (Name is consumed from the AML.) * ******************************************************************************/
@@ -220,7 +220,7 @@ char *acpi_ps_get_next_namestring(struct acpi_parse_state *parser_state)* * DESCRIPTION: Get next name (if method call, return # of required args). * Names are looked up in the internal namespace to determine -* if the name represents a control method. If a method +* if the name represents a control method. If a method * is found, the number of arguments to the method is returned. * This information is critical for parsing to continue correctly. *
drivers/acpi/acpica/psopcode.c +3 -3
@@ -59,7 +59,7 @@ static const u8 acpi_gbl_argument_count[] =* * DESCRIPTION: Opcode table. Each entry contains <opcode, type, name, operands> * The name is a simple ascii string, the operand specifier is an -* ascii string with one letter per operand. The letter specifies +* ascii string with one letter per operand. The letter specifies * the operand type. * ******************************************************************************/
@@ -183,7 +183,7 @@ static const u8 acpi_gbl_argument_count[] =******************************************************************************/ /* -* Master Opcode information table. A summary of everything we know about each +* Master Opcode information table. A summary of everything we know about each * opcode, all in one place. */ const struct acpi_opcode_info acpi_gbl_aml_op_info[AML_NUM_OPCODES ] = {
@@ -700,7 +700,7 @@ static const u8 acpi_gbl_short_op_index[256] = {/* * This table is indexed by the second opcode of the extended opcode -* pair. It returns an index into the opcode table (acpi_gbl_aml_op_info) +* pair. It returns an index into the opcode table (acpi_gbl_aml_op_info) */ static const u8 acpi_gbl_long_op_index[NUM_EXTENDED_OPCODE ] = {/* 0 1 2 3 4 5 6 7 */
drivers/acpi/acpica/psparse.c +4 -4
@@ -43,9 +43,9 @@/* * Parse the AML and build an operation tree as most interpreters, -* like Perl, do. Parsing is done by hand rather than with a YACC +* like Perl, do. Parsing is done by hand rather than with a YACC * generated parser to tightly constrain stack and dynamic memory -* usage. At the same time, parsing is kept flexible and the code +* usage. At the same time, parsing is kept flexible and the code * fairly compact by parsing based on a list of AML opcode * templates in aml_op_info[] */
@@ -379,7 +379,7 @@ acpi_ps_next_parse_state(struct acpi_walk_state *walk_state,case AE_CTRL_FALSE :/* * Either an IF/WHILE Predicate was false or we encountered a BREAK -* opcode. In both cases, we do not execute the rest of the +* opcode. In both cases, we do not execute the rest of the * package; We simply close out the parent (finishing the walk of * this branch of the tree) and continue execution at the parent * level.
@@ -488,7 +488,7 @@ acpi_status acpi_ps_parse_aml(struct acpi_walk_state *walk_state) acpi_gbl_current_walk_list = thread;/* -* Execute the walk loop as long as there is a valid Walk State. This +* Execute the walk loop as long as there is a valid Walk State. This * handles nested control method invocations without recursion. */ ACPI_DEBUG_PRINT ((ACPI_DB_PARSE ,"State=%p\n" , walk_state));
drivers/acpi/acpica/psutils.c +2 -2
@@ -108,7 +108,7 @@ void acpi_ps_init_op(union acpi_parse_object *op, u16 opcode)* RETURN: Pointer to the new Op, null on failure * * DESCRIPTION: Allocate an acpi_op, choose op type (and thus size) based on -* opcode. A cache of opcodes is available for the pure +* opcode. A cache of opcodes is available for the pure * GENERIC_OP, since this is by far the most commonly used. * ******************************************************************************/
@@ -164,7 +164,7 @@ union acpi_parse_object *acpi_ps_alloc_op(u16 opcode)* * RETURN: None. * -* DESCRIPTION: Free an Op object. Either put it on the GENERIC_OP cache list +* DESCRIPTION: Free an Op object. Either put it on the GENERIC_OP cache list * or actually free it. * ******************************************************************************/
drivers/acpi/acpica/rscalc.c +1 -1
@@ -601,7 +601,7 @@ acpi_rs_get_pci_routing_table_length(union acpi_operand_object *package_object,/* * Calculate the size of the return buffer. * The base size is the number of elements * the sizes of the -* structures. Additional space for the strings is added below. +* structures. Additional space for the strings is added below. * The minus one is to subtract the size of the u8 Source[1] * member because it is added below. *
drivers/acpi/acpica/tbxfroot.c +1 -1
@@ -107,7 +107,7 @@ static acpi_status acpi_tb_validate_rsdp(struct acpi_table_rsdp *rsdp)* RETURN: Status, RSDP physical address * * DESCRIPTION: Search lower 1Mbyte of memory for the root system descriptor -* pointer structure. If it is found, set *RSDP to point to it. +* pointer structure. If it is found, set *RSDP to point to it. * * NOTE1: The RSDP must be either in the first 1K of the Extended * BIOS Data Area or between E0000 and FFFFF (From ACPI Spec.)
drivers/acpi/acpica/utcache.c +3 -3
@@ -183,7 +183,7 @@ acpi_status acpi_os_delete_cache(struct acpi_memory_list * cache)* * RETURN: None * -* DESCRIPTION: Release an object to the specified cache. If cache is full, +* DESCRIPTION: Release an object to the specified cache. If cache is full, * the object is deleted. * ******************************************************************************/
@@ -240,9 +240,9 @@ acpi_os_release_object(struct acpi_memory_list * cache, void *object)* * PARAMETERS: cache - Handle to cache object * -* RETURN: the acquired object. NULL on error +* RETURN: the acquired object. NULL on error * -* DESCRIPTION: Get an object from the specified cache. If cache is empty, +* DESCRIPTION: Get an object from the specified cache. If cache is empty, * the object is allocated. * ******************************************************************************/
drivers/acpi/acpica/utdebug.c +9 -9
@@ -223,7 +223,7 @@ ACPI_EXPORT_SYMBOL(acpi_debug_print)* * RETURN: None * -* DESCRIPTION: Print message with no headers. Has same interface as +* DESCRIPTION: Print message with no headers. Has same interface as * debug_print so that the same macros can be used. * ******************************************************************************/
@@ -259,7 +259,7 @@ ACPI_EXPORT_SYMBOL(acpi_debug_print_raw)* * RETURN: None * -* DESCRIPTION: Function entry trace. Prints only if TRACE_FUNCTIONS bit is +* DESCRIPTION: Function entry trace. Prints only if TRACE_FUNCTIONS bit is * set in debug_level * ******************************************************************************/
@@ -291,7 +291,7 @@ ACPI_EXPORT_SYMBOL(acpi_ut_trace)* * RETURN: None * -* DESCRIPTION: Function entry trace. Prints only if TRACE_FUNCTIONS bit is +* DESCRIPTION: Function entry trace. Prints only if TRACE_FUNCTIONS bit is * set in debug_level * ******************************************************************************/
@@ -321,7 +321,7 @@ acpi_ut_trace_ptr(u32 line_number,* * RETURN: None * -* DESCRIPTION: Function entry trace. Prints only if TRACE_FUNCTIONS bit is +* DESCRIPTION: Function entry trace. Prints only if TRACE_FUNCTIONS bit is * set in debug_level * ******************************************************************************/
@@ -352,7 +352,7 @@ acpi_ut_trace_str(u32 line_number,* * RETURN: None * -* DESCRIPTION: Function entry trace. Prints only if TRACE_FUNCTIONS bit is +* DESCRIPTION: Function entry trace. Prints only if TRACE_FUNCTIONS bit is * set in debug_level * ******************************************************************************/
@@ -382,7 +382,7 @@ acpi_ut_trace_u32(u32 line_number,* * RETURN: None * -* DESCRIPTION: Function exit trace. Prints only if TRACE_FUNCTIONS bit is +* DESCRIPTION: Function exit trace. Prints only if TRACE_FUNCTIONS bit is * set in debug_level * ******************************************************************************/
@@ -414,7 +414,7 @@ ACPI_EXPORT_SYMBOL(acpi_ut_exit)* * RETURN: None * -* DESCRIPTION: Function exit trace. Prints only if TRACE_FUNCTIONS bit is +* DESCRIPTION: Function exit trace. Prints only if TRACE_FUNCTIONS bit is * set in debug_level. Prints exit status also. * ******************************************************************************/
@@ -455,7 +455,7 @@ ACPI_EXPORT_SYMBOL(acpi_ut_status_exit)* * RETURN: None * -* DESCRIPTION: Function exit trace. Prints only if TRACE_FUNCTIONS bit is +* DESCRIPTION: Function exit trace. Prints only if TRACE_FUNCTIONS bit is * set in debug_level. Prints exit value also. * ******************************************************************************/
@@ -487,7 +487,7 @@ ACPI_EXPORT_SYMBOL(acpi_ut_value_exit)* * RETURN: None * -* DESCRIPTION: Function exit trace. Prints only if TRACE_FUNCTIONS bit is +* DESCRIPTION: Function exit trace. Prints only if TRACE_FUNCTIONS bit is * set in debug_level. Prints exit value also. * ******************************************************************************/
drivers/acpi/acpica/utmath.c +1 -1
@@ -81,7 +81,7 @@ typedef union uint64_overlay {* RETURN: Status (Checks for divide-by-zero) * * DESCRIPTION: Perform a short (maximum 64 bits divided by 32 bits) -* divide and modulo. The result is a 64-bit quotient and a +* divide and modulo. The result is a 64-bit quotient and a * 32-bit remainder. * ******************************************************************************/
drivers/acpi/acpica/utmisc.c +7 -7
@@ -250,7 +250,7 @@ acpi_status acpi_ut_allocate_owner_id(acpi_owner_id * owner_id)* control method or unloading a table. Either way, we would * ignore any error anyway. * -* DESCRIPTION: Release a table or method owner ID. Valid IDs are 1 - 255 +* DESCRIPTION: Release a table or method owner ID. Valid IDs are 1 - 255 * ******************************************************************************/
@@ -534,8 +534,8 @@ u32 acpi_ut_dword_byte_swap(u32 value)* RETURN: None * * DESCRIPTION: Set the global integer bit width based upon the revision -* of the DSDT. For Revision 1 and 0, Integers are 32 bits. -* For Revision 2 and above, Integers are 64 bits. Yes, this +* of the DSDT. For Revision 1 and 0, Integers are 32 bits. +* For Revision 2 and above, Integers are 64 bits. Yes, this * makes a difference. * ******************************************************************************/
@@ -671,7 +671,7 @@ u8 acpi_ut_valid_acpi_char(char character, u32 position)* * RETURN: TRUE if the name is valid, FALSE otherwise * -* DESCRIPTION: Check for a valid ACPI name. Each character must be one of: +* DESCRIPTION: Check for a valid ACPI name. Each character must be one of: * 1) Upper case alpha * 2) numeric * 3) underscore
@@ -1010,10 +1010,10 @@ acpi_ut_walk_package_tree(union acpi_operand_object *source_object,/* * Check for: -* 1) An uninitialized package element. It is completely +* 1) An uninitialized package element. It is completely * legal to declare a package and leave it uninitialized * 2) Not an internal object - can be a namespace node instead -* 3) Any type other than a package. Packages are handled in else +* 3) Any type other than a package. Packages are handled in else * case below. */ if ((!this_source_obj) ||
@@ -1032,7 +1032,7 @@ acpi_ut_walk_package_tree(union acpi_operand_object *source_object, state->pkg.source_object->package.count) {/* * We've handled all of the objects at this level, This means -* that we have just completed a package. That package may +* that we have just completed a package. That package may * have contained one or more packages itself. * * Delete this state and pop the previous state (package).
drivers/acpi/acpica/utmutex.c +6 -6
@@ -228,9 +228,9 @@ acpi_status acpi_ut_acquire_mutex(acpi_mutex_handle mutex_id)/* * Mutex debug code, for internal debugging only. * -* Deadlock prevention. Check if this thread owns any mutexes of value -* greater than or equal to this one. If so, the thread has violated -* the mutex ordering rule. This indicates a coding error somewhere in +* Deadlock prevention. Check if this thread owns any mutexes of value +* greater than or equal to this one. If so, the thread has violated +* the mutex ordering rule. This indicates a coding error somewhere in * the ACPI subsystem code. */ for (i = mutex_id; i <ACPI_NUM_MUTEX ; i++) {
@@ -321,9 +321,9 @@ acpi_status acpi_ut_release_mutex(acpi_mutex_handle mutex_id)/* * Mutex debug code, for internal debugging only. * -* Deadlock prevention. Check if this thread owns any mutexes of value -* greater than this one. If so, the thread has violated the mutex -* ordering rule. This indicates a coding error somewhere in +* Deadlock prevention. Check if this thread owns any mutexes of value +* greater than this one. If so, the thread has violated the mutex +* ordering rule. This indicates a coding error somewhere in * the ACPI subsystem code. */ for (i = mutex_id; i <ACPI_NUM_MUTEX ; i++) {
drivers/acpi/acpica/utobject.c +4 -4
@@ -77,7 +77,7 @@ acpi_ut_get_element_length(u8 object_type,* * NOTE: We always allocate the worst-case object descriptor because * these objects are cached, and we want them to be -* one-size-satisifies-any-request. This in itself may not be +* one-size-satisifies-any-request. This in itself may not be * the most memory efficient, but the efficiency of the object * cache should more than make up for this! *
@@ -370,9 +370,9 @@ u8 acpi_ut_valid_internal_object(void *object)* line_number - Caller's line number (for error output) * component_id - Caller's component ID (for error output) * -* RETURN: Pointer to newly allocated object descriptor. Null on error +* RETURN: Pointer to newly allocated object descriptor. Null on error * -* DESCRIPTION: Allocate a new object descriptor. Gracefully handle +* DESCRIPTION: Allocate a new object descriptor. Gracefully handle * error conditions. * ******************************************************************************/
@@ -554,7 +554,7 @@ acpi_ut_get_simple_object_size(union acpi_operand_object *internal_object,/* * Account for the space required by the object rounded up to the next -* multiple of the machine word size. This keeps each object aligned +* multiple of the machine word size. This keeps each object aligned * on a machine word boundary. (preventing alignment faults on some * machines.) */
drivers/acpi/acpica/utstate.c +1 -1
@@ -147,7 +147,7 @@ union acpi_generic_state *acpi_ut_pop_generic_state(union acpi_generic_state* * RETURN: The new state object. NULL on failure. * -* DESCRIPTION: Create a generic state object. Attempt to obtain one from +* DESCRIPTION: Create a generic state object. Attempt to obtain one from * the global state cache; If none available, create a new one. * ******************************************************************************/
drivers/acpi/acpica/uttrack.c +3 -3
@@ -45,9 +45,9 @@* These procedures are used for tracking memory leaks in the subsystem, and * they get compiled out when the ACPI_DBG_TRACK_ALLOCATIONS is not set. * -* Each memory allocation is tracked via a doubly linked list. Each +* Each memory allocation is tracked via a doubly linked list. Each * element contains the caller's component, module name, function name, and -* line number. acpi_ut_allocate and acpi_ut_allocate_zeroed call +* line number. acpi_ut_allocate and acpi_ut_allocate_zeroed call * acpi_ut_track_allocation to add an element to the list; deletion * occurs in the body of acpi_ut_free. */
@@ -499,7 +499,7 @@ void acpi_ut_dump_allocation_info(void)* FUNCTION: acpi_ut_dump_allocations * * PARAMETERS: component - Component(s) to dump info for. -* module - Module to dump info for. NULL means all. +* module - Module to dump info for. NULL means all. * * RETURN: None *
drivers/acpi/acpica/utxface.c +1 -1
@@ -147,7 +147,7 @@ ACPI_EXPORT_SYMBOL(acpi_subsystem_status)* RETURN: status - the status of the call * * DESCRIPTION: This function is called to get information about the current -* state of the ACPI subsystem. It will return system information +* state of the ACPI subsystem. It will return system information * in the out_buffer. * * If the function fails an appropriate status will be returned
include/acpi/acexcep.h +1 -1
@@ -122,7 +122,7 @@#define AE_CODE_TBL_MAX 0x0005/* -* AML exceptions. These are caused by problems with +* AML exceptions. These are caused by problems with * the actual AML byte stream */ #define AE_AML_BAD_OPCODE (acpi_status) (0x0001 | AE_CODE_AML)
Signature · 1 line
1.7.10
PATCH 14/26 ACPICA: Remove extra spaces after periods in the Intel license
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
For consistency with the rest of the source code.
--- include/acpi/acpiosxf.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
include/acpi/acpiosxf.h +1 -1
@@ -1,6 +1,6 @@/****************************************************************************** * -* Name: acpiosxf.h - All interfaces to the OS Services Layer (OSL). These +* Name: acpiosxf.h - All interfaces to the OS Services Layer (OSL). These * interfaces must be implemented by OSL to interface the * ACPI components to the host operating system. *
Signature · 1 line
1.7.10
PATCH 15/26 ACPICA: Resource Mgr: Small fix for buffer size calculation
In reply to Lv ZhengFrom: Feng Tang <[email protected]>
Fixes a one byte error in the output buffer calculation. Feng Tang - ACPICA BZ 849:
https://www.acpica.org/bugzilla/show_bug.cgi?id=849
--- drivers/acpi/acpica/rscalc.c | 9 +++++++++ 1 file changed, 9 insertions(+)
drivers/acpi/acpica/rscalc.c +9 -0
@@ -457,6 +457,15 @@ acpi_rs_get_list_length(u8 * aml_buffer,* Get the number of vendor data bytes */ extra_struct_bytes = resource_length; + +/* +* There is already one byte included in the minimum +* descriptor size. If there are extra struct bytes, +* subtract one from the count. +*/ +if (extra_struct_bytes) { + extra_struct_bytes--; + }break ;case ACPI_RESOURCE_NAME_END_TAG :
Signature · 1 line
1.7.10
PATCH 16/26 ACPICA: Add debug print message for mutex objects that are force-released
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
At control method termination, any currently acquired mutex objects are force-released. Add a new message for each one that is released.
--- drivers/acpi/acpica/exmutex.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)
drivers/acpi/acpica/exmutex.c +5 -1
@@ -461,7 +461,7 @@ void acpi_ex_release_all_mutexes(struct acpi_thread_state *thread)union acpi_operand_object *next = thread->acquired_mutex_list;union acpi_operand_object *obj_desc; -ACPI_FUNCTION_ENTRY (); +ACPI_FUNCTION_NAME (ex_release_all_mutexes);/* Traverse the list of owned mutexes, releasing each one */
@@ -473,6 +473,10 @@ void acpi_ex_release_all_mutexes(struct acpi_thread_state *thread) obj_desc->mutex.next =NULL ; obj_desc->mutex.acquisition_depth =0 ; +ACPI_DEBUG_PRINT ((ACPI_DB_EXEC , +"Force-releasing held mutex: %p\n" , + obj_desc)); +/* Release the mutex, special case for Global Lock */ if (obj_desc == acpi_gbl_global_lock_mutex) {
Signature · 1 line
1.7.10
PATCH 17/26 ACPICA: AcpiExec: Improve algorithm for tracking memory leaks
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
Add some intelligence to the code that maintains the global list of allocated memory. The list is now ordered by allocated memory address, significantly improving performance. When running AcpiExec on the ASLTS test suite, speed improvements of 3X to 5X are seen, depending on the platform and/or the environment.
--- drivers/acpi/acpica/uttrack.c | 100 +++++++++++++++++++++++++++++------------ 1 file changed, 71 insertions(+), 29 deletions(-)
drivers/acpi/acpica/uttrack.c +71 -29
@@ -61,7 +61,9 @@ACPI_MODULE_NAME ("uttrack" ) /* Local prototypes */ -static struct acpi_debug_mem_block *acpi_ut_find_allocation (void *allocation); +static struct acpi_debug_mem_block *acpi_ut_find_allocation (struct +acpi_debug_mem_block + *allocation); static acpi_status acpi_ut_track_allocation(struct acpi_debug_mem_block *address,
@@ -263,31 +265,61 @@ acpi_ut_free_and_track(void *allocation,* * PARAMETERS: allocation - Address of allocated memory * -* RETURN: A list element if found; NULL otherwise. +* RETURN: Three cases: +* 1) List is empty, NULL is returned. +* 2) Element was found. Returns Allocation parameter. +* 3) Element was not found. Returns position where it should be +* inserted into the list. * * DESCRIPTION: Searches for an element in the global allocation tracking list. +* If the element is not found, returns the location within the +* list where the element should be inserted. +* +* Note: The list is ordered by larger-to-smaller addresses. +* +* This global list is used to detect memory leaks in ACPICA as +* well as other issues such as an attempt to release the same +* internal object more than once. Although expensive as far +* as cpu time, this list is much more helpful for finding these +* types of issues than using memory leak detectors outside of +* the ACPICA code. * ******************************************************************************/ -static struct acpi_debug_mem_block *acpi_ut_find_allocation (void *allocation) +static struct acpi_debug_mem_block *acpi_ut_find_allocation (struct +acpi_debug_mem_block + *allocation) {struct acpi_debug_mem_block *element; -ACPI_FUNCTION_ENTRY (); - element = acpi_gbl_global_list->list_head; +if (!element) { +return (NULL ); + } + +/* +* Search for the address. +* +* Note: List is ordered by larger-to-smaller addresses, on the +* assumption that a new allocation usually has a larger address +* than previous allocations. +*/ +while (element > allocation) { -/* Search for the address. */ +/* Check for end-of-list */ -while (element) { -if (element == allocation) { +if (!element->next) {return (element); } element = element->next; } -return (NULL ); +if (element == allocation) { +return (element); + } + +return (element->previous); }/*******************************************************************************
@@ -301,7 +333,7 @@ static struct acpi_debug_mem_block *acpi_ut_find_allocation(void *allocation)* module - Source file name of caller * line - Line number of caller * -* RETURN: None. +* RETURN: Status * * DESCRIPTION: Inserts an element into the global allocation tracking list. *
@@ -330,22 +362,18 @@ acpi_ut_track_allocation(struct acpi_debug_mem_block *allocation, }/* -* Search list for this address to make sure it is not already on the list. -* This will catch several kinds of problems. +* Search the global list for this address to make sure it is not +* already present. This will catch several kinds of problems. */ element =acpi_ut_find_allocation (allocation); -if (element) { +if (element == allocation) {ACPI_ERROR ((AE_INFO , -"UtTrackAllocation: Allocation already present in list! (%p)" , +"UtTrackAllocation: Allocation (%p) already present in global list!" , allocation)); - -ACPI_ERROR ((AE_INFO ,"Element %p Address %p" , - element, allocation)); - goto unlock_and_exit; } -/* Fill in the instance data. */ +/* Fill in the instance data */ allocation->size = (u32 )size; allocation->alloc_type = alloc_type;
@@ -355,17 +383,31 @@ acpi_ut_track_allocation(struct acpi_debug_mem_block *allocation,ACPI_STRNCPY (allocation->module, module,ACPI_MAX_MODULE_NAME ); allocation->module[ACPI_MAX_MODULE_NAME -1 ] =0 ; -/* Insert at list head */ +if (!element) { -if (mem_list->list_head) { - ((struct acpi_debug_mem_block *)(mem_list->list_head))-> - previous = allocation; - } +/* Insert at list head */ + +if (mem_list->list_head) { + ((struct acpi_debug_mem_block *)(mem_list->list_head))-> + previous = allocation; + } + + allocation->next = mem_list->list_head; + allocation->previous =NULL ; - allocation->next = mem_list->list_head; - allocation->previous =NULL ; + mem_list->list_head = allocation; + }else { +/* Insert after element */ + + allocation->next = element->next; + allocation->previous = element; + +if (element->next) { + (element->next)->previous = allocation; + } - mem_list->list_head = allocation; + element->next = allocation; + } unlock_and_exit: status =acpi_ut_release_mutex (ACPI_MTX_MEMORY );
@@ -381,7 +423,7 @@ acpi_ut_track_allocation(struct acpi_debug_mem_block *allocation,* module - Source file name of caller * line - Line number of caller * -* RETURN: +* RETURN: Status * * DESCRIPTION: Deletes an element from the global allocation tracking list. *
@@ -443,7 +485,7 @@ acpi_ut_remove_allocation(struct acpi_debug_mem_block *allocation,* * FUNCTION: acpi_ut_dump_allocation_info * -* PARAMETERS: +* PARAMETERS: None * * RETURN: None *
Signature · 1 line
1.7.10
PATCH 18/26 ACPICA: Add ACPI_MOVE_NAME macro to optimize 4-byte ACPI_NAME copies
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
Resolves to a 32-bit move for the normal case, strncpy on machines that do not support misaligned transfers.
--- include/acpi/actypes.h | 4 ++++ 1 file changed, 4 insertions(+)
include/acpi/actypes.h +4 -0
@@ -453,10 +453,14 @@ typedef u64 acpi_integer;#define ACPI_PHYSADDR_TO_PTR (i) ACPI_TO_POINTER(i)#define ACPI_PTR_TO_PHYSADDR (i) ACPI_TO_INTEGER(i) +/* Optimizations for 4-character (32-bit) acpi_name manipulation */ +#ifndef ACPI_MISALIGNMENT_NOT_SUPPORTED #define ACPI_COMPARE_NAME (a,b) (*ACPI_CAST_PTR (u32, (a)) == *ACPI_CAST_PTR (u32, (b))) +#define ACPI_MOVE_NAME (dest,src) (*ACPI_CAST_PTR (u32, (dest)) = *ACPI_CAST_PTR (u32, (src)))#else #define ACPI_COMPARE_NAME (a,b) (!ACPI_STRNCMP (ACPI_CAST_PTR (char, (a)), ACPI_CAST_PTR (char, (b)), ACPI_NAME_SIZE)) +#define ACPI_MOVE_NAME (dest,src) (ACPI_STRNCPY (ACPI_CAST_PTR (char, (dest)), ACPI_CAST_PTR (char, (src)), ACPI_NAME_SIZE))#endif /*******************************************************************************
Signature · 1 line
1.7.10
PATCH 19/26 ACPICA: Enhance error reporting for invalid opcodes and bad ACPI_NAMEs
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
For disassembler, dump the 48 bytes surrounding the invalid opcode. Fix incorrect table offset reported for invalid opcodes. Report original 32-bit value for bad ACPI_NAMEs.
--- drivers/acpi/acpica/nsutils.c | 9 ++++--- drivers/acpi/acpica/psloop.c | 51 ++++++++++++++++++++++++++++++--------- drivers/acpi/acpica/utmisc.c | 3 +++ drivers/acpi/acpica/utxferror.c | 2 +- 4 files changed, 49 insertions(+), 16 deletions(-)
drivers/acpi/acpica/nsutils.c +5 -4
@@ -557,10 +557,11 @@ acpi_ns_externalize_name(u32 internal_name_length, (*converted_name)[j++] ='.' ; } - (*converted_name)[j++] = internal_name[names_index++]; - (*converted_name)[j++] = internal_name[names_index++]; - (*converted_name)[j++] = internal_name[names_index++]; - (*converted_name)[j++] = internal_name[names_index++]; +ACPI_MOVE_NAME (*converted_name, internal_name); +acpi_ut_repair_name (*converted_name); + + j +=ACPI_NAME_SIZE ; + names_index +=ACPI_NAME_SIZE ; } }
drivers/acpi/acpica/psloop.c +40 -11
67 lines not on this page · raw message
drivers/acpi/acpica/utmisc.c +3 -0
@@ -720,9 +720,12 @@ void acpi_ut_repair_name(char *name) {u32 i;u8 found_bad_char = FALSE; +u32 original_name;ACPI_FUNCTION_NAME (ut_repair_name); +ACPI_MOVE_NAME (&original_name, name); +/* Check each character in the name */ for (i =0 ; i <ACPI_NAME_SIZE ; i++) {
drivers/acpi/acpica/utxferror.c +1 -1
@@ -408,7 +408,7 @@ acpi_ut_namespace_error(const char *module_name,ACPI_MOVE_32_TO_32 (&bad_name,ACPI_CAST_PTR (u32, internal_name)); -acpi_os_printf ("[0x%4.4X] (NON-ASCII)" , bad_name); +acpi_os_printf ("[0x%.8X] (NON-ASCII)" , bad_name); }else {/* Convert path to external format */
Signature · 1 line
1.7.10
PATCH 20/26 ACPICA: Update support for ACPI 5 MPST table
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
Fixes some problems introduced by late changes to the table as it was added to the ACPI 5.0 specification. Both the table compiler and the disassembler and the main header support for the table.
--- include/acpi/actbl3.h | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-)
include/acpi/actbl3.h +11 -11
@@ -277,10 +277,10 @@ struct acpi_table_gtdt {******************************************************************************/ #define ACPI_MPST_CHANNEL_INFO \ - u16 reserved1; \ u8 channel_id; \ - u8 reserved2; \ - u16 power_node_count; + u8 reserved1[3]; \ + u16 power_node_count; \ + u16 reserved2;/* Main table */
@@ -304,9 +304,8 @@ struct acpi_mpst_power_node {u32 length;u64 range_address;u64 range_length; -u8 num_power_states; -u8 num_physical_components; -u16 reserved2; +u32 num_power_states; +u32 num_physical_components; };/* Values for Flags field above */
@@ -332,10 +331,11 @@ struct acpi_mpst_component {struct acpi_mpst_data_hdr {u16 characteristics_count; +u16 reserved; };struct acpi_mpst_power_data { -u8 revision; +u8 structure_id;u8 flags;u16 reserved1;u32 average_power;
@@ -356,10 +356,10 @@ struct acpi_mpst_shared {u32 signature;u16 pcc_command;u16 pcc_status; -u16 command_register; -u16 status_register; -u16 power_state_id; -u16 power_node_id; +u32 command_register; +u32 status_register; +u32 power_state_id; +u32 power_node_id;u64 energy_consumed;u64 average_power; };
Signature · 1 line
1.7.10
PATCH 21/26 ACPICA: Deploy ACPI_MOVE_NAME across ACPICA source base
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
Replaces instances of strncpy(...,4) for ACPI_NAMEs. ACPI_MOVE_NAME optimizes these to a single 32-bit copy on machines that support misaligned transfers.
--- drivers/acpi/acpica/nsxfname.c | 3 +-- drivers/acpi/acpica/tbfind.c | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-)
drivers/acpi/acpica/nsxfname.c +1 -2
@@ -202,8 +202,7 @@ acpi_get_name(acpi_handle handle, u32 name_type, struct acpi_buffer * buffer)/* Just copy the ACPI name from the Node and zero terminate it */ -ACPI_STRNCPY (buffer->pointer,acpi_ut_get_node_name (node), -ACPI_NAME_SIZE ); +ACPI_MOVE_NAME (buffer->pointer,acpi_ut_get_node_name (node)); ((char *)buffer->pointer)[ACPI_NAME_SIZE ] =0 ; status =AE_OK ;
drivers/acpi/acpica/tbfind.c +1 -1
@@ -77,7 +77,7 @@ acpi_tb_find_table(char *signature,/* Normalize the input strings */ ACPI_MEMSET (&header,0 ,sizeof (struct acpi_table_header )); -ACPI_STRNCPY (header.signature, signature,ACPI_NAME_SIZE ); +ACPI_MOVE_NAME (header.signature, signature);ACPI_STRNCPY (header.oem_id, oem_id,ACPI_OEM_ID_SIZE );ACPI_STRNCPY (header.oem_table_id, oem_table_id,ACPI_OEM_TABLE_ID_SIZE );
Signature · 1 line
1.7.10
PATCH 22/26 ACPICA: Add starting offset parameter to common dump buffer routine
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
Rename the dump buffer routines. Offset parameter can specify the buffer starting offset that is used when displaying each line of the buffer.
--- drivers/acpi/acpica/acmacros.h | 2 +- drivers/acpi/acpica/acutils.h | 5 +++-- drivers/acpi/acpica/exdebug.c | 8 ++++---- drivers/acpi/acpica/exdump.c | 9 +++++---- drivers/acpi/acpica/psloop.c | 22 ++++++++++++++-------- drivers/acpi/acpica/utdebug.c | 13 +++++++------ 6 files changed, 34 insertions(+), 25 deletions(-)
drivers/acpi/acpica/acmacros.h +1 -1
@@ -460,7 +460,7 @@#define ACPI_DUMP_OPERANDS (a, b ,c) acpi_ex_dump_operands(a, b, c)#define ACPI_DUMP_ENTRY (a, b) acpi_ns_dump_entry (a, b)#define ACPI_DUMP_PATHNAME (a, b, c, d) acpi_ns_dump_pathname(a, b, c, d) -#define ACPI_DUMP_BUFFER (a, b) acpi_ut_dump_buffer((u8 *) a, b, DB_BYTE_DISPLAY, _COMPONENT) +#define ACPI_DUMP_BUFFER (a, b) acpi_ut_debug_dump_buffer((u8 *) a, b, DB_BYTE_DISPLAY, _COMPONENT)#else /*
drivers/acpi/acpica/acutils.h +3 -2
@@ -305,9 +305,10 @@ acpi_ut_ptr_exit(u32 line_number,const char *function_name, const char *module_name, u32 component_id, u8 *ptr); -void acpi_ut_dump_buffer (u8 *buffer,u32 count,u32 display,u32 component_id); +void +acpi_ut_debug_dump_buffer (u8 *buffer,u32 count,u32 display,u32 component_id); -void acpi_ut_dump_buffer2 (u8 *buffer,u32 count,u32 display); +void acpi_ut_dump_buffer (u8 *buffer,u32 count,u32 display,u32 offset);void acpi_ut_report_error (char *module_name,u32 line_number);
drivers/acpi/acpica/exdebug.c +4 -4
@@ -145,10 +145,10 @@ acpi_ex_do_debug_object(union acpi_operand_object *source_desc,case ACPI_TYPE_BUFFER :acpi_os_printf ("[0x%.2X]\n" , (u32 )source_desc->buffer.length); -acpi_ut_dump_buffer2 (source_desc->buffer.pointer, - (source_desc->buffer.length <256 ) ? - source_desc->buffer.length :256 , -DB_BYTE_DISPLAY ); +acpi_ut_dump_buffer (source_desc->buffer.pointer, + (source_desc->buffer.length <256 ) ? + source_desc->buffer.length :256 , +DB_BYTE_DISPLAY ,0 );break ;case ACPI_TYPE_STRING :
drivers/acpi/acpica/exdump.c +5 -4
@@ -942,10 +942,11 @@ acpi_ex_dump_package_obj(union acpi_operand_object *obj_desc,acpi_os_printf ("[Buffer] Length %.2X = " , obj_desc->buffer.length);if (obj_desc->buffer.length) { -acpi_ut_dump_buffer (ACPI_CAST_PTR - (u8, obj_desc->buffer.pointer), - obj_desc->buffer.length, -DB_DWORD_DISPLAY , _COMPONENT); +acpi_ut_debug_dump_buffer (ACPI_CAST_PTR + (u8, + obj_desc->buffer.pointer), + obj_desc->buffer.length, +DB_DWORD_DISPLAY , _COMPONENT); }else {acpi_os_printf ("\n" ); }
drivers/acpi/acpica/psloop.c +14 -8
@@ -133,7 +133,7 @@ static acpi_status acpi_ps_get_aml_opcode(struct acpi_walk_state *walk_state)case AML_CLASS_UNKNOWN : -/* The opcode is unrecognized. Just skip unknown opcodes */ +/* The opcode is unrecognized. Complain and skip unknown opcodes */ if (walk_state->pass_number ==2 ) { ACPI_ERROR((AE_INFO,
@@ -142,28 +142,34 @@ static acpi_status acpi_ps_get_aml_opcode(struct acpi_walk_state *walk_state) walk_state->aml_offset + sizeof(struct acpi_table_header))); -ACPI_DUMP_BUFFER (walk_state->parser_state.aml,128 ); +ACPI_DUMP_BUFFER (walk_state->parser_state.aml -16 ,48 );#ifdef ACPI_ASL_COMPILER - +/* +* This is executed for the disassembler only. Output goes +* to the disassembled ASL output file. +*/ acpi_os_printf ("/*\nError: Unknown opcode 0x%.2X at table offset 0x%.4X, context:\n" , walk_state->opcode, walk_state->aml_offset +sizeof (struct acpi_table_header )); -/* TBD: Pass current offset to dump_buffer */ +/* Dump the context surrounding the invalid opcode */ -acpi_ut_dump_buffer2 (((u8 *)walk_state->parser_state. - aml -16 ),48 ,DB_BYTE_DISPLAY ); +acpi_ut_dump_buffer (((u8 *)walk_state->parser_state. + aml -16 ),48 ,DB_BYTE_DISPLAY , + walk_state->aml_offset + +sizeof (struct acpi_table_header ) - +16 );acpi_os_printf (" */\n" );#endif } -/* Increment past one or two-byte opcode */ +/* Increment past one-byte or two-byte opcode */ walk_state->parser_state.aml++; -if (walk_state->opcode >0xFF ) { +if (walk_state->opcode >0xFF ) {/* Can only happen if first byte is 0x5B */ walk_state->parser_state.aml++; }
drivers/acpi/acpica/utdebug.c +7 -6
@@ -513,7 +513,7 @@ acpi_ut_ptr_exit(u32 line_number,* PARAMETERS: buffer - Buffer to dump * count - Amount to dump, in bytes * display - BYTE, WORD, DWORD, or QWORD display -* component_ID - Caller's component ID +* offset - Beginning buffer offset (display only) * * RETURN: None *
@@ -521,7 +521,7 @@ acpi_ut_ptr_exit(u32 line_number,* ******************************************************************************/ -void acpi_ut_dump_buffer2 (u8 *buffer,u32 count,u32 display) +void acpi_ut_dump_buffer (u8 *buffer,u32 count,u32 display,u32 base_offset) {u32 i =0 ;u32 j;
@@ -543,7 +543,7 @@ void acpi_ut_dump_buffer2(u8 *buffer, u32 count, u32 display)/* Print current offset */ -acpi_os_printf ("%6.4X: " , i); +acpi_os_printf ("%6.4X: " , (base_offset + i));/* Print 16 hex chars */
@@ -625,7 +625,7 @@ void acpi_ut_dump_buffer2(u8 *buffer, u32 count, u32 display)/******************************************************************************* * -* FUNCTION: acpi_ut_dump_buffer +* FUNCTION: acpi_ut_debug_dump_buffer * * PARAMETERS: buffer - Buffer to dump * count - Amount to dump, in bytes
@@ -638,7 +638,8 @@ void acpi_ut_dump_buffer2(u8 *buffer, u32 count, u32 display)* ******************************************************************************/ -void acpi_ut_dump_buffer (u8 *buffer,u32 count,u32 display,u32 component_id) +void +acpi_ut_debug_dump_buffer (u8 *buffer,u32 count,u32 display,u32 component_id) {/* Only dump the buffer if tracing is enabled */
@@ -648,5 +649,5 @@ void acpi_ut_dump_buffer(u8 *buffer, u32 count, u32 display, u32 component_id)return ; } -acpi_ut_dump_buffer2 (buffer, count, display); +acpi_ut_dump_buffer (buffer, count, display,0 ); }
Signature · 1 line
1.7.10
PATCH 23/26 ACPICA: Fix externalize name to complete migration to ACPI_MOVE_NAME
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
Fix for name segment copy and validation.
--- drivers/acpi/acpica/nsutils.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-)
drivers/acpi/acpica/nsutils.c +5 -2
@@ -557,8 +557,11 @@ acpi_ns_externalize_name(u32 internal_name_length, (*converted_name)[j++] ='.' ; } -ACPI_MOVE_NAME (*converted_name, internal_name); -acpi_ut_repair_name (*converted_name); +/* Copy and validate the 4-char name segment */ + +ACPI_MOVE_NAME (&(*converted_name)[j], + &internal_name[names_index]); +acpi_ut_repair_name (&(*converted_name)[j]); j +=ACPI_NAME_SIZE ; names_index +=ACPI_NAME_SIZE ;
Signature · 1 line
1.7.10
PATCH 24/26 ACPICA: Update for 64-bit generation of recent error message changes
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
Fix for errors on printf changes on 64-bit platforms and gcc.
--- drivers/acpi/acpica/psloop.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
drivers/acpi/acpica/psloop.c +4 -4
@@ -139,8 +139,8 @@ static acpi_status acpi_ps_get_aml_opcode(struct acpi_walk_state *walk_state)ACPI_ERROR ((AE_INFO ,"Unknown opcode 0x%.2X at table offset 0x%.4X, ignoring" , walk_state->opcode, - walk_state->aml_offset + -sizeof (struct acpi_table_header ))); + (u32)(walk_state->aml_offset + +sizeof (struct acpi_table_header ))));ACPI_DUMP_BUFFER (walk_state->parser_state.aml -16 ,48 );
@@ -152,8 +152,8 @@ static acpi_status acpi_ps_get_aml_opcode(struct acpi_walk_state *walk_state)acpi_os_printf ("/*\nError: Unknown opcode 0x%.2X at table offset 0x%.4X, context:\n" , walk_state->opcode, - walk_state->aml_offset + -sizeof (struct acpi_table_header )); + (u32)(walk_state->aml_offset + +sizeof (struct acpi_table_header )));/* Dump the context surrounding the invalid opcode */
Signature · 1 line
1.7.10
PATCH 25/26 ACPICA: AcpiGetObjectInfo: Add support for ACPI 5 _SUB method
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
Now calls _SUB in addition to the other ID methods: _HID, _CID, and _UID.
--- drivers/acpi/acpica/acutils.h | 4 +++ drivers/acpi/acpica/nsxfname.c | 29 ++++++++++++++--- drivers/acpi/acpica/utids.c | 67 ++++++++++++++++++++++++++++++++++++++++ include/acpi/acnames.h | 1 + include/acpi/actypes.h | 12 ++++--- 5 files changed, 103 insertions(+), 10 deletions(-)
drivers/acpi/acpica/acutils.h +4 -0
@@ -363,6 +363,10 @@ acpi_ut_execute_UID(struct acpi_namespace_node *device_node, struct acpi_pnp_device_id ** return_id);acpi_status +acpi_ut_execute_SUB (struct acpi_namespace_node *device_node, +struct acpi_pnp_device_id **return_id); + +acpi_status acpi_ut_execute_CID (struct acpi_namespace_node *device_node,struct acpi_pnp_device_id_list ** return_cid_list);
drivers/acpi/acpica/nsxfname.c +24 -5
@@ -231,6 +231,7 @@ static char *acpi_ns_copy_device_id(struct acpi_pnp_device_id *dest,struct acpi_pnp_device_id *source, char *string_area) { +/* Create the destination PNP_DEVICE_ID */ dest->string = string_area;
@@ -255,8 +256,8 @@ static char *acpi_ns_copy_device_id(struct acpi_pnp_device_id *dest,* namespace node and possibly by running several standard * control methods (Such as in the case of a device.) * -* For Device and Processor objects, run the Device _HID, _UID, _CID, _STA, -* _ADR, _sx_w, and _sx_d methods. +* For Device and Processor objects, run the Device _HID, _UID, _CID, _SUB, +* _STA, _ADR, _sx_w, and _sx_d methods. * * Note: Allocates the return buffer, must be freed by the caller. *
@@ -271,6 +272,7 @@ acpi_get_object_info(acpi_handle handle,struct acpi_pnp_device_id_list *cid_list =NULL ;struct acpi_pnp_device_id *hid =NULL ;struct acpi_pnp_device_id *uid =NULL ; +struct acpi_pnp_device_id *sub =NULL ;char *next_id_string;acpi_object_type type;acpi_name name;
@@ -315,7 +317,7 @@ acpi_get_object_info(acpi_handle handle,if ((type ==ACPI_TYPE_DEVICE ) || (type ==ACPI_TYPE_PROCESSOR )) {/* * Get extra info for ACPI Device/Processor objects only: -* Run the Device _HID, _UID, and _CID methods. +* Run the Device _HID, _UID, _SUB, and _CID methods. * * Note: none of these methods are required, so they may or may * not be present for this device. The Info->Valid bitfield is used
@@ -338,6 +340,14 @@ acpi_get_object_info(acpi_handle handle, valid |=ACPI_VALID_UID ; } +/* Execute the Device._SUB method */ + + status =acpi_ut_execute_SUB (node, &sub); +if (ACPI_SUCCESS (status)) { + info_size += sub->length; + valid |=ACPI_VALID_SUB ; + } +/* Execute the Device._CID method */ status =acpi_ut_execute_CID (node, &cid_list);
@@ -425,8 +435,9 @@ acpi_get_object_info(acpi_handle handle, }/* -* Copy the HID, UID, and CIDs to the return buffer. The variable-length -* strings are copied to the reserved area at the end of the buffer. +* Copy the HID, UID, SUB, and CIDs to the return buffer. +* The variable-length strings are copied to the reserved area +* at the end of the buffer. * * For HID and CID, check if the ID is a PCI Root Bridge. */
@@ -444,6 +455,11 @@ acpi_get_object_info(acpi_handle handle, uid, next_id_string); } +if (sub) { + next_id_string =acpi_ns_copy_device_id (&info->subsystem_id, + sub, next_id_string); + } +if (cid_list) { info->compatible_id_list.count = cid_list->count; info->compatible_id_list.list_size = cid_list->list_size;
@@ -480,6 +496,9 @@ acpi_get_object_info(acpi_handle handle,if (uid) {ACPI_FREE (uid); } +if (sub) { +ACPI_FREE (sub); + }if (cid_list) {ACPI_FREE (cid_list); }
drivers/acpi/acpica/utids.c +67 -0
73 lines not on this page · raw message
include/acpi/acnames.h +1 -0
@@ -50,6 +50,7 @@#define METHOD_NAME__HID "_HID"#define METHOD_NAME__CID "_CID"#define METHOD_NAME__UID "_UID" +#define METHOD_NAME__SUB "_SUB"#define METHOD_NAME__ADR "_ADR"#define METHOD_NAME__INI "_INI"#define METHOD_NAME__STA "_STA"
include/acpi/actypes.h +7 -5
@@ -1024,7 +1024,7 @@ u32 (*acpi_interface_handler) (acpi_string interface_name, u32 supported);#define ACPI_UUID_LENGTH 16 -/* Structures used for device/processor HID, UID, CID */ +/* Structures used for device/processor HID, UID, CID, and SUB */ struct acpi_pnp_device_id {u32 length;/* Length of string + null */
@@ -1054,6 +1054,7 @@ struct acpi_device_info {u64 address;/* _ADR value */ struct acpi_pnp_device_id hardware_id;/* _HID value */ struct acpi_pnp_device_id unique_id;/* _UID value */ +struct acpi_pnp_device_id subsystem_id;/* _SUB value */ struct acpi_pnp_device_id_list compatible_id_list;/* _CID list <must be last> */ };
@@ -1067,11 +1068,12 @@ struct acpi_device_info {#define ACPI_VALID_ADR 0x02#define ACPI_VALID_HID 0x04#define ACPI_VALID_UID 0x08 -#define ACPI_VALID_CID 0x10 -#define ACPI_VALID_SXDS 0x20 -#define ACPI_VALID_SXWS 0x40 +#define ACPI_VALID_SUB 0x10 +#define ACPI_VALID_CID 0x20 +#define ACPI_VALID_SXDS 0x40 +#define ACPI_VALID_SXWS 0x80 -/* Flags for _STA method */ +/* Flags for _STA return value (current_status above) */ #define ACPI_STA_DEVICE_PRESENT 0x01#define ACPI_STA_DEVICE_ENABLED 0x02
Signature · 1 line
1.7.10
PATCH 26/26 ACPICA: Update version to 20121018
In reply to Lv ZhengFrom: Bob Moore <[email protected]>
Version 20121018.
--- include/acpi/acpixf.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
include/acpi/acpixf.h +1 -1
@@ -46,7 +46,7 @@/* Current ACPICA subsystem version in YYYYMMDD format */ -#define ACPI_CA_VERSION 0x20120913 +#define ACPI_CA_VERSION 0x20121018#include <acpi/acconfig.h> #include <acpi/actypes.h>
Signature · 1 line
1.7.10
Quoted 2 lines · Lv Zheng
Here are the 20121018 ACPICA linuxized patches based on the acpi/next branch, please review.
Do I think correctly that the patches are targeted at the 3.8 kernel?
Rafael
Quoted 146 lines
Bob Moore (18):
Cleanup of invalid ACPI name handling and repair
ACPICA: Audit/update for ACPICA return macros and debug depth counter
ACPICA: ACPICA core: Cleanup empty lines at file start and end
ACPICA: Fix some typos in comments
ACPICA: Update local C library module comments for ASCII table
ACPICA: Remove extra spaces after periods within comments
ACPICA: Remove extra spaces after periods in the Intel license
ACPICA: Add debug print message for mutex objects that are
force-released
ACPICA: AcpiExec: Improve algorithm for tracking memory leaks
ACPICA: AddACPI_MOVE_NAMEmacro to optimize 4-byteACPI_NAMEcopies
ACPICA: Enhance error reporting for invalid opcodes and bad
ACPI_NAMEs
ACPICA: Update support for ACPI 5 MPST table
ACPICA: DeployACPI_MOVE_NAMEacross ACPICA source base
ACPICA: Add starting offset parameter to common dump buffer routine
ACPICA: Fix externalize name to complete migration toACPI_MOVE_NAME
ACPICA: Update for 64-bit generation of recent error message changes
ACPICA: AcpiGetObjectInfo: Add support for ACPI 5_SUBmethod
ACPICA: Update version to 20121018Feng Tang (1):
ACPICA: Resource Mgr: Small fix for buffer size calculationLv Zheng (6):
ACPICA: Fix unmerged utility divergences.
ACPICA: Fix unmerged debugger divergences.
ACPICA: Fix divergences of definition conflicts.
ACPICA: Fix AcpiSrc caused divergences.
ACPICA: Fix indent caused divergences.
ACPICA: Fix unmergedacmacros.hdivergences.Robert Moore (1):
ACPICA: Fix for predefined name loop during ACPICA initializationdrivers/acpi/acpica/Makefile | 3 + drivers/acpi/acpica/acdebug.h | 94 +++-- drivers/acpi/acpica/acdispat.h | 11 +- drivers/acpi/acpica/acevents.h | 6 +- drivers/acpi/acpica/acglobal.h | 73 ++-- drivers/acpi/acpica/aclocal.h | 16 +- drivers/acpi/acpica/acmacros.h | 163 +++------ drivers/acpi/acpica/acobject.h | 7 +- drivers/acpi/acpica/acopcode.h | 6 +- drivers/acpi/acpica/acparser.h | 3 +- drivers/acpi/acpica/acpredef.h | 11 +- drivers/acpi/acpica/acstruct.h | 2 +- drivers/acpi/acpica/acutils.h | 58 ++- drivers/acpi/acpica/amlresrc.h | 1 - drivers/acpi/acpica/dscontrol.c | 2 +- drivers/acpi/acpica/dsfield.c | 2 +- drivers/acpi/acpica/dsmethod.c | 6 +- drivers/acpi/acpica/dsmthdat.c | 14 +- drivers/acpi/acpica/dsobject.c | 6 +- drivers/acpi/acpica/dsopcode.c | 3 +- drivers/acpi/acpica/dsutils.c | 33 +- drivers/acpi/acpica/dswexec.c | 10 +- drivers/acpi/acpica/dswload2.c | 4 +- drivers/acpi/acpica/dswstate.c | 26 +- drivers/acpi/acpica/evgpe.c | 20 +- drivers/acpi/acpica/evgpeblk.c | 3 +- drivers/acpi/acpica/evgpeutil.c | 3 +- drivers/acpi/acpica/evrgnini.c | 7 +- drivers/acpi/acpica/evxface.c | 2 +- drivers/acpi/acpica/evxfgpe.c | 13 +- drivers/acpi/acpica/exconvrt.c | 4 +- drivers/acpi/acpica/excreate.c | 9 +- drivers/acpi/acpica/exdebug.c | 10 +- drivers/acpi/acpica/exdump.c | 20 +- drivers/acpi/acpica/exfield.c | 4 +- drivers/acpi/acpica/exfldio.c | 15 +- drivers/acpi/acpica/exmisc.c | 5 +- drivers/acpi/acpica/exmutex.c | 9 +- drivers/acpi/acpica/exnames.c | 9 +- drivers/acpi/acpica/exoparg1.c | 11 +- drivers/acpi/acpica/exoparg2.c | 2 +- drivers/acpi/acpica/exoparg3.c | 3 +- drivers/acpi/acpica/exoparg6.c | 5 +- drivers/acpi/acpica/exprep.c | 13 +- drivers/acpi/acpica/exregion.c | 3 +- drivers/acpi/acpica/exresnte.c | 9 +- drivers/acpi/acpica/exresolv.c | 3 +- drivers/acpi/acpica/exresop.c | 8 +- drivers/acpi/acpica/exstore.c | 4 +- drivers/acpi/acpica/exstoren.c | 11 +- drivers/acpi/acpica/exstorob.c | 5 +- drivers/acpi/acpica/exsystem.c | 9 +- drivers/acpi/acpica/exutils.c | 5 +- drivers/acpi/acpica/hwacpi.c | 3 +- drivers/acpi/acpica/hwgpe.c | 4 +- drivers/acpi/acpica/hwpci.c | 4 +- drivers/acpi/acpica/hwregs.c | 1 - drivers/acpi/acpica/hwtimer.c | 6 +- drivers/acpi/acpica/hwvalid.c | 1 - drivers/acpi/acpica/hwxface.c | 1 - drivers/acpi/acpica/hwxfsleep.c | 12 +- drivers/acpi/acpica/nsaccess.c | 7 +- drivers/acpi/acpica/nsalloc.c | 4 +- drivers/acpi/acpica/nsdump.c | 10 +- drivers/acpi/acpica/nsinit.c | 4 +- drivers/acpi/acpica/nsload.c | 10 +- drivers/acpi/acpica/nsnames.c | 2 +- drivers/acpi/acpica/nsobject.c | 8 +- drivers/acpi/acpica/nsparse.c | 8 +- drivers/acpi/acpica/nssearch.c | 17 +- drivers/acpi/acpica/nsutils.c | 18 +- drivers/acpi/acpica/nswalk.c | 10 +- drivers/acpi/acpica/nsxfeval.c | 20 +- drivers/acpi/acpica/nsxfname.c | 66 ++-- drivers/acpi/acpica/nsxfobj.c | 4 +- drivers/acpi/acpica/psargs.c | 8 +- drivers/acpi/acpica/psloop.c | 59 ++- drivers/acpi/acpica/psopcode.c | 29 +- drivers/acpi/acpica/psparse.c | 13 +- drivers/acpi/acpica/psutils.c | 4 +- drivers/acpi/acpica/rscalc.c | 14 +- drivers/acpi/acpica/rslist.c | 4 +- drivers/acpi/acpica/tbfind.c | 2 +- drivers/acpi/acpica/tbinstal.c | 2 + drivers/acpi/acpica/tbutils.c | 2 +- drivers/acpi/acpica/tbxface.c | 4 +- drivers/acpi/acpica/tbxfload.c | 2 +- drivers/acpi/acpica/tbxfroot.c | 3 +- drivers/acpi/acpica/utcache.c | 323 +++++++++++++++++ drivers/acpi/acpica/utclib.c | 749 +++++++++++++++++++++++++++++++++++++++ drivers/acpi/acpica/utdebug.c | 37 +- drivers/acpi/acpica/utids.c | 104 +++++- drivers/acpi/acpica/utmath.c | 2 +- drivers/acpi/acpica/utmisc.c | 150 ++++++-- drivers/acpi/acpica/utmutex.c | 14 +- drivers/acpi/acpica/utobject.c | 8 +- drivers/acpi/acpica/utstate.c | 2 +- drivers/acpi/acpica/uttrack.c | 692 ++++++++++++++++++++++++++++++++++++ drivers/acpi/acpica/utxface.c | 5 +- drivers/acpi/acpica/utxferror.c | 2 +- drivers/acpi/scan.c | 2 +- drivers/acpi/sysfs.c | 4 +- include/acpi/acconfig.h | 1 + include/acpi/acexcep.h | 2 +- include/acpi/acnames.h | 1 + include/acpi/acpiosxf.h | 3 +- include/acpi/acpixf.h | 18 +- include/acpi/actbl3.h | 22 +- include/acpi/actypes.h | 42 ++- 109 files changed, 2669 insertions(+), 664 deletions(-)
Quoted 4 lines
create mode 100644
drivers/acpi/acpica/utcache.ccreate mode 100644drivers/acpi/acpica/utclib.ccreate mode 100644drivers/acpi/acpica/uttrack.c
---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
z siedziba w Gdansku
ul. Slowackiego 173
80-298 Gdansk
Sad Rejonowy Gdansk Polnoc w Gdansku, VII Wydzial Gospodarczy Krajowego Rejestru Sadowego, numer KRS 101882
NIP 957-07-52-316
Kapital zakladowy 200.000 zl
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies.