1 files changed, 10 insertions, 20 deletions

diff --git a/drivers/acpi/parser/psopcode.c b/drivers/acpi/parser/psopcode.c
index 5744673568c0..6f7594a516d2 100644
--- a/drivers/acpi/parser/psopcode.c
+++ b/drivers/acpi/parser/psopcode.c
@@ -311,7 +311,7 @@ const struct acpi_opcode_info     acpi_gbl_aml_op_info[AML_NUM_OPCODES] =
 /* ACPI 2.0 opcodes */
 
 /* 6E */ ACPI_OP ("QwordConst",         ARGP_QWORD_OP,             ARGI_QWORD_OP,              ACPI_TYPE_INTEGER,           AML_CLASS_ARGUMENT,        AML_TYPE_LITERAL,         AML_CONSTANT),
-/* 6F */ ACPI_OP ("Package /*Var*/",    ARGP_VAR_PACKAGE_OP,       ARGI_VAR_PACKAGE_OP,        ACPI_TYPE_PACKAGE,           AML_CLASS_CREATE,          AML_TYPE_CREATE_OBJECT,   AML_HAS_ARGS | AML_DEFER),
+/* 6F */ ACPI_OP ("Package", /* Var */  ARGP_VAR_PACKAGE_OP,       ARGI_VAR_PACKAGE_OP,        ACPI_TYPE_PACKAGE,           AML_CLASS_CREATE,          AML_TYPE_CREATE_OBJECT,   AML_HAS_ARGS | AML_DEFER),
 /* 70 */ ACPI_OP ("ConcatenateResTemplate", ARGP_CONCAT_RES_OP,    ARGI_CONCAT_RES_OP,         ACPI_TYPE_ANY,               AML_CLASS_EXECUTE,         AML_TYPE_EXEC_2A_1T_1R,   AML_FLAGS_EXEC_2A_1T_1R | AML_CONSTANT),
 /* 71 */ ACPI_OP ("Mod",                ARGP_MOD_OP,               ARGI_MOD_OP,                ACPI_TYPE_ANY,               AML_CLASS_EXECUTE,         AML_TYPE_EXEC_2A_1T_1R,   AML_FLAGS_EXEC_2A_1T_1R | AML_CONSTANT),
 /* 72 */ ACPI_OP ("CreateQWordField",   ARGP_CREATE_QWORD_FIELD_OP,ARGI_CREATE_QWORD_FIELD_OP, ACPI_TYPE_BUFFER_FIELD,      AML_CLASS_CREATE,          AML_TYPE_CREATE_FIELD,    AML_HAS_ARGS | AML_NSOBJECT | AML_NSNODE | AML_DEFER | AML_CREATE),
@@ -428,33 +428,23 @@ acpi_ps_get_opcode_info (
         /*
          * Detect normal 8-bit opcode or extended 16-bit opcode
          */
-        switch ((u8) (opcode >> 8)) {
-        case 0:
-
+        if (!(opcode & 0xFF00)) {
                 /* Simple (8-bit) opcode: 0-255, can't index beyond table  */
 
                 return (&acpi_gbl_aml_op_info [acpi_gbl_short_op_index [(u8) opcode]]);
+        }
 
-        case AML_EXTOP:
-
-                /* Extended (16-bit, prefix+opcode) opcode */
-
-                if (((u8) opcode) <= MAX_EXTENDED_OPCODE) {
-                        return (&acpi_gbl_aml_op_info [acpi_gbl_long_op_index [(u8) opcode]]);
-                }
-
-                /* Else fall through to error case below */
-                /*lint -fallthrough */
-
-        default:
+        if (((opcode & 0xFF00) == AML_EXTENDED_OPCODE) &&
+                (((u8) opcode) <= MAX_EXTENDED_OPCODE)) {
+                /* Valid extended (16-bit) opcode */
 
-                ACPI_DEBUG_PRINT ((ACPI_DB_ERROR,
-                        "Unknown AML opcode [%4.4X]\n", opcode));
-                break;
+                return (&acpi_gbl_aml_op_info [acpi_gbl_long_op_index [(u8) opcode]]);
         }
 
+        /* Unknown AML opcode */
 
-        /* Default is "unknown opcode" */
+        ACPI_DEBUG_PRINT ((ACPI_DB_ERROR,
+                "Unknown AML opcode [%4.4X]\n", opcode));
 
         return (&acpi_gbl_aml_op_info [_UNK]);
 }