diff options
author | Bob Moore <robert.moore@intel.com> | 2016-02-19 01:17:36 -0500 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2016-03-08 16:58:38 -0500 |
commit | 0dda8851a88d6c9d6389f1caf69627154c39aceb (patch) | |
tree | 076b0b9db3836892f4f2bb15b2765849af037947 | |
parent | f6cede5b49e822ebc41a099fe41ab4989f64e2cb (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.c | 9 |
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, |