aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBob Moore <robert.moore@intel.com>2016-02-19 01:17:36 -0500
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2016-03-08 16:58:38 -0500
commit0dda8851a88d6c9d6389f1caf69627154c39aceb (patch)
tree076b0b9db3836892f4f2bb15b2765849af037947
parentf6cede5b49e822ebc41a099fe41ab4989f64e2cb (diff)
ACPICA: Revert "Parser: Fix for SuperName method invocation"
ACPICA commit eade8f78f2aa21e8eabc3380a5728db47273bcf1 Revert commit ae90fbf562d7 (ACPICA: Parser: Fix for SuperName method invocation). Support for method invocations as part of super_name will be removed from the ACPI specification, since no AML interpreter supports it. Fixes: ae90fbf562d7 (ACPICA: Parser: Fix for SuperName method invocation) Link: https://github.com/acpica/acpica/commit/eade8f78 Signed-off-by: Bob Moore <robert.moore@intel.com> Signed-off-by: Lv Zheng <lv.zheng@intel.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r--drivers/acpi/acpica/psargs.c9
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/acpi/acpica/psargs.c b/drivers/acpi/acpica/psargs.c
index 305218539df2..d48cbed342c1 100644
--- a/drivers/acpi/acpica/psargs.c
+++ b/drivers/acpi/acpica/psargs.c
@@ -269,8 +269,7 @@ acpi_ps_get_next_namepath(struct acpi_walk_state *walk_state,
269 */ 269 */
270 if (ACPI_SUCCESS(status) && 270 if (ACPI_SUCCESS(status) &&
271 possible_method_call && (node->type == ACPI_TYPE_METHOD)) { 271 possible_method_call && (node->type == ACPI_TYPE_METHOD)) {
272 if (GET_CURRENT_ARG_TYPE(walk_state->arg_types) == 272 if (walk_state->opcode == AML_UNLOAD_OP) {
273 ARGP_SUPERNAME) {
274 /* 273 /*
275 * acpi_ps_get_next_namestring has increased the AML pointer, 274 * acpi_ps_get_next_namestring has increased the AML pointer,
276 * so we need to restore the saved AML pointer for method call. 275 * so we need to restore the saved AML pointer for method call.
@@ -697,7 +696,7 @@ static union acpi_parse_object *acpi_ps_get_next_field(struct acpi_parse_state
697 * 696 *
698 * PARAMETERS: walk_state - Current state 697 * PARAMETERS: walk_state - Current state
699 * parser_state - Current parser state object 698 * parser_state - Current parser state object
700 * arg_type - The parser argument type (ARGP_*) 699 * arg_type - The argument type (AML_*_ARG)
701 * return_arg - Where the next arg is returned 700 * return_arg - Where the next arg is returned
702 * 701 *
703 * RETURN: Status, and an op object containing the next argument. 702 * RETURN: Status, and an op object containing the next argument.
@@ -817,9 +816,9 @@ acpi_ps_get_next_arg(struct acpi_walk_state *walk_state,
817 return_ACPI_STATUS(AE_NO_MEMORY); 816 return_ACPI_STATUS(AE_NO_MEMORY);
818 } 817 }
819 818
820 /* super_name allows argument to be a method call */ 819 /* To support super_name arg of Unload */
821 820
822 if (arg_type == ARGP_SUPERNAME) { 821 if (walk_state->opcode == AML_UNLOAD_OP) {
823 status = 822 status =
824 acpi_ps_get_next_namepath(walk_state, 823 acpi_ps_get_next_namepath(walk_state,
825 parser_state, arg, 824 parser_state, arg,