diff options
| author | Jung-uk Kim <jkim@FreeBSD.org> | 2013-05-17 23:13:40 +0000 | 
|---|---|---|
| committer | Jung-uk Kim <jkim@FreeBSD.org> | 2013-05-17 23:13:40 +0000 | 
| commit | b7f987c19dad2c6d33c64e7f96a9b4deca9e2650 (patch) | |
| tree | 740dae2325e162bb086ea6e7e5d481c4b669e232 /source/components/executer | |
| parent | b4a951799e313e9ec15d955b72dd3097e4880724 (diff) | |
Notes
Diffstat (limited to 'source/components/executer')
| -rw-r--r-- | source/components/executer/exconfig.c | 3 | ||||
| -rw-r--r-- | source/components/executer/exconvrt.c | 25 | ||||
| -rw-r--r-- | source/components/executer/excreate.c | 3 | ||||
| -rw-r--r-- | source/components/executer/exdebug.c | 2 | ||||
| -rw-r--r-- | source/components/executer/exdump.c | 30 | ||||
| -rw-r--r-- | source/components/executer/exfield.c | 4 | ||||
| -rw-r--r-- | source/components/executer/exfldio.c | 7 | ||||
| -rw-r--r-- | source/components/executer/exmisc.c | 23 | ||||
| -rw-r--r-- | source/components/executer/exnames.c | 5 | ||||
| -rw-r--r-- | source/components/executer/exoparg1.c | 43 | ||||
| -rw-r--r-- | source/components/executer/exoparg2.c | 9 | ||||
| -rw-r--r-- | source/components/executer/exoparg3.c | 3 | ||||
| -rw-r--r-- | source/components/executer/exoparg6.c | 7 | ||||
| -rw-r--r-- | source/components/executer/exprep.c | 9 | ||||
| -rw-r--r-- | source/components/executer/exregion.c | 19 | ||||
| -rw-r--r-- | source/components/executer/exresnte.c | 7 | ||||
| -rw-r--r-- | source/components/executer/exresolv.c | 18 | ||||
| -rw-r--r-- | source/components/executer/exresop.c | 21 | ||||
| -rw-r--r-- | source/components/executer/exstore.c | 12 | ||||
| -rw-r--r-- | source/components/executer/exstoren.c | 6 | 
20 files changed, 76 insertions, 180 deletions
diff --git a/source/components/executer/exconfig.c b/source/components/executer/exconfig.c index 7d72a5f884aa..3e9f12b37fc0 100644 --- a/source/components/executer/exconfig.c +++ b/source/components/executer/exconfig.c @@ -483,7 +483,6 @@ AcpiExLoadOp (          TableDesc.Address = ObjDesc->Region.Address;          break; -      case ACPI_TYPE_BUFFER: /* Buffer or resolved RegionField */          ACPI_DEBUG_PRINT ((ACPI_DB_EXEC, @@ -526,8 +525,8 @@ AcpiExLoadOp (          TableDesc.Address = ACPI_TO_INTEGER (TableDesc.Pointer);          break; -      default: +          return_ACPI_STATUS (AE_AML_OPERAND_TYPE);      } diff --git a/source/components/executer/exconvrt.c b/source/components/executer/exconvrt.c index 2ba2e3e076d2..6f803c088d48 100644 --- a/source/components/executer/exconvrt.c +++ b/source/components/executer/exconvrt.c @@ -114,6 +114,7 @@ AcpiExConvertToInteger (          break;      default: +          return_ACPI_STATUS (AE_TYPE);      } @@ -133,7 +134,6 @@ AcpiExConvertToInteger (      switch (ObjDesc->Common.Type)      {      case ACPI_TYPE_STRING: -          /*           * Convert string to an integer - for most cases, the string must be           * hexadecimal as per the ACPI specification. The only exception (as @@ -147,7 +147,6 @@ AcpiExConvertToInteger (          }          break; -      case ACPI_TYPE_BUFFER:          /* Check for zero-length buffer */ @@ -179,10 +178,10 @@ AcpiExConvertToInteger (          }          break; -      default:          /* No other types can get here */ +          break;      } @@ -242,7 +241,6 @@ AcpiExConvertToBuffer (      case ACPI_TYPE_INTEGER: -          /*           * Create a new Buffer object.           * Need enough space for one integer @@ -261,9 +259,7 @@ AcpiExConvertToBuffer (                          AcpiGbl_IntegerByteWidth);          break; -      case ACPI_TYPE_STRING: -          /*           * Create a new Buffer object           * Size will be the string length @@ -287,8 +283,8 @@ AcpiExConvertToBuffer (              ObjDesc->String.Length);          break; -      default: +          return_ACPI_STATUS (AE_TYPE);      } @@ -344,15 +340,18 @@ AcpiExConvertToAscii (          switch (DataWidth)          {          case 1: +              DecimalLength = ACPI_MAX8_DECIMAL_DIGITS;              break;          case 4: +              DecimalLength = ACPI_MAX32_DECIMAL_DIGITS;              break;          case 8:          default: +              DecimalLength = ACPI_MAX64_DECIMAL_DIGITS;              break;          } @@ -461,7 +460,6 @@ AcpiExConvertToString (          *ResultDesc = ObjDesc;          return_ACPI_STATUS (AE_OK); -      case ACPI_TYPE_INTEGER:          switch (Type) @@ -505,7 +503,6 @@ AcpiExConvertToString (          NewBuf [StringLength] = 0;          break; -      case ACPI_TYPE_BUFFER:          /* Setup string length, base, and separator */ @@ -604,6 +601,7 @@ AcpiExConvertToString (          break;      default: +          return_ACPI_STATUS (AE_TYPE);      } @@ -663,6 +661,7 @@ AcpiExConvertToTargetType (              break;          default: +              /* No conversion allowed for these types */              if (DestinationType != SourceDesc->Common.Type) @@ -676,7 +675,6 @@ AcpiExConvertToTargetType (          }          break; -      case ARGI_TARGETREF:          switch (DestinationType) @@ -693,7 +691,6 @@ AcpiExConvertToTargetType (                          16);              break; -          case ACPI_TYPE_STRING:              /*               * The operand must be a String. We can convert an @@ -703,7 +700,6 @@ AcpiExConvertToTargetType (                          ACPI_IMPLICIT_CONVERT_HEX);              break; -          case ACPI_TYPE_BUFFER:              /*               * The operand must be a Buffer. We can convert an @@ -712,8 +708,8 @@ AcpiExConvertToTargetType (              Status = AcpiExConvertToBuffer (SourceDesc, ResultDesc);              break; -          default: +              ACPI_ERROR ((AE_INFO, "Bad destination type during conversion: 0x%X",                  DestinationType));              Status = AE_AML_INTERNAL; @@ -721,15 +717,14 @@ AcpiExConvertToTargetType (          }          break; -      case ARGI_REFERENCE:          /*           * CreateXxxxField cases - we are storing the field object into the name           */          break; -      default: +          ACPI_ERROR ((AE_INFO,              "Unknown Target type ID 0x%X AmlOpcode 0x%X DestType %s",              GET_CURRENT_ARG_TYPE (WalkState->OpInfo->RuntimeArgs), diff --git a/source/components/executer/excreate.c b/source/components/executer/excreate.c index 822f2a50cecf..376e1fd77562 100644 --- a/source/components/executer/excreate.c +++ b/source/components/executer/excreate.c @@ -113,7 +113,6 @@ AcpiExCreateAlias (      case ACPI_TYPE_BUFFER:      case ACPI_TYPE_PACKAGE:      case ACPI_TYPE_BUFFER_FIELD: -      /*       * These types open a new scope, so we need the NS node in order to access       * any children. @@ -123,7 +122,6 @@ AcpiExCreateAlias (      case ACPI_TYPE_PROCESSOR:      case ACPI_TYPE_THERMAL:      case ACPI_TYPE_LOCAL_SCOPE: -          /*           * The new alias has the type ALIAS and points to the original           * NS node, not the object itself. @@ -133,7 +131,6 @@ AcpiExCreateAlias (          break;      case ACPI_TYPE_METHOD: -          /*           * Control method aliases need to be differentiated           */ diff --git a/source/components/executer/exdebug.c b/source/components/executer/exdebug.c index 923d2ce929de..9db779fb3e2d 100644 --- a/source/components/executer/exdebug.c +++ b/source/components/executer/exdebug.c @@ -208,6 +208,7 @@ AcpiExDoDebugObject (              return_VOID;          default: +              break;          } @@ -241,6 +242,7 @@ AcpiExDoDebugObject (                      break;                  default: +                      AcpiExDoDebugObject ((SourceDesc->Reference.Node)->Object,                          Level+4, 0);                      break; diff --git a/source/components/executer/exdump.c b/source/components/executer/exdump.c index 48ff6e34c37f..333168ec48a8 100644 --- a/source/components/executer/exdump.c +++ b/source/components/executer/exdump.c @@ -380,6 +380,7 @@ AcpiExDumpObject (          switch (Info->Opcode)          {          case ACPI_EXD_INIT: +              break;          case ACPI_EXD_TYPE: @@ -541,19 +542,16 @@ AcpiExDumpOperand (              AcpiOsPrintf ("\n");              break; -          case ACPI_REFCLASS_INDEX:              AcpiOsPrintf ("%p\n", ObjDesc->Reference.Object);              break; -          case ACPI_REFCLASS_TABLE:              AcpiOsPrintf ("Table Index %X\n", ObjDesc->Reference.Value);              break; -          case ACPI_REFCLASS_REFOF:              AcpiOsPrintf ("%p [%s]\n", ObjDesc->Reference.Object, @@ -561,20 +559,17 @@ AcpiExDumpOperand (                      ObjDesc->Reference.Object)->Common.Type));              break; -          case ACPI_REFCLASS_NAME:              AcpiOsPrintf ("- [%4.4s]\n", ObjDesc->Reference.Node->Name.Ascii);              break; -          case ACPI_REFCLASS_ARG:          case ACPI_REFCLASS_LOCAL:              AcpiOsPrintf ("%X\n", ObjDesc->Reference.Value);              break; -          default:    /* Unknown reference class */              AcpiOsPrintf ("%2.2X\n", ObjDesc->Reference.Class); @@ -582,7 +577,6 @@ AcpiExDumpOperand (          }          break; -      case ACPI_TYPE_BUFFER:          AcpiOsPrintf ("Buffer length %.2X @ %p\n", @@ -604,14 +598,12 @@ AcpiExDumpOperand (          }          break; -      case ACPI_TYPE_INTEGER:          AcpiOsPrintf ("Integer %8.8X%8.8X\n",              ACPI_FORMAT_UINT64 (ObjDesc->Integer.Value));          break; -      case ACPI_TYPE_PACKAGE:          AcpiOsPrintf ("Package [Len %X] ElementArray %p\n", @@ -632,7 +624,6 @@ AcpiExDumpOperand (          }          break; -      case ACPI_TYPE_REGION:          AcpiOsPrintf ("Region %s (%X)", @@ -655,7 +646,6 @@ AcpiExDumpOperand (          }          break; -      case ACPI_TYPE_STRING:          AcpiOsPrintf ("String length %X @ %p ", @@ -666,13 +656,11 @@ AcpiExDumpOperand (          AcpiOsPrintf ("\n");          break; -      case ACPI_TYPE_LOCAL_BANK_FIELD:          AcpiOsPrintf ("BankField\n");          break; -      case ACPI_TYPE_LOCAL_REGION_FIELD:          AcpiOsPrintf ("RegionField: Bits=%X AccWidth=%X Lock=%X Update=%X at " @@ -687,13 +675,11 @@ AcpiExDumpOperand (          AcpiExDumpOperand (ObjDesc->Field.RegionObj, Depth+1);          break; -      case ACPI_TYPE_LOCAL_INDEX_FIELD:          AcpiOsPrintf ("IndexField\n");          break; -      case ACPI_TYPE_BUFFER_FIELD:          AcpiOsPrintf ("BufferField: %X bits at byte %X bit %X of\n", @@ -716,13 +702,11 @@ AcpiExDumpOperand (          }          break; -      case ACPI_TYPE_EVENT:          AcpiOsPrintf ("Event\n");          break; -      case ACPI_TYPE_METHOD:          AcpiOsPrintf ("Method(%X) @ %p:%X\n", @@ -731,38 +715,33 @@ AcpiExDumpOperand (              ObjDesc->Method.AmlLength);          break; -      case ACPI_TYPE_MUTEX:          AcpiOsPrintf ("Mutex\n");          break; -      case ACPI_TYPE_DEVICE:          AcpiOsPrintf ("Device\n");          break; -      case ACPI_TYPE_POWER:          AcpiOsPrintf ("Power\n");          break; -      case ACPI_TYPE_PROCESSOR:          AcpiOsPrintf ("Processor\n");          break; -      case ACPI_TYPE_THERMAL:          AcpiOsPrintf ("Thermal\n");          break; -      default: +          /* Unknown Type */          AcpiOsPrintf ("Unknown Type %X\n", ObjDesc->Common.Type); @@ -1005,7 +984,6 @@ AcpiExDumpPackageObj (              ACPI_FORMAT_UINT64 (ObjDesc->Integer.Value));          break; -      case ACPI_TYPE_STRING:          AcpiOsPrintf ("[String]  Value: "); @@ -1013,7 +991,6 @@ AcpiExDumpPackageObj (          AcpiOsPrintf ("\n");          break; -      case ACPI_TYPE_BUFFER:          AcpiOsPrintf ("[Buffer] Length %.2X = ", ObjDesc->Buffer.Length); @@ -1028,7 +1005,6 @@ AcpiExDumpPackageObj (          }          break; -      case ACPI_TYPE_PACKAGE:          AcpiOsPrintf ("[Package] Contains %u Elements:\n", @@ -1040,7 +1016,6 @@ AcpiExDumpPackageObj (          }          break; -      case ACPI_TYPE_LOCAL_REFERENCE:          AcpiOsPrintf ("[Object Reference] Type [%s] %2.2X", @@ -1049,7 +1024,6 @@ AcpiExDumpPackageObj (          AcpiExDumpReferenceObj (ObjDesc);          break; -      default:          AcpiOsPrintf ("[Unknown Type] %X\n", ObjDesc->Common.Type); diff --git a/source/components/executer/exfield.c b/source/components/executer/exfield.c index 18021c5daaf9..2b4f5f7cd1e3 100644 --- a/source/components/executer/exfield.c +++ b/source/components/executer/exfield.c @@ -360,21 +360,25 @@ AcpiExWriteDataToField (      switch (SourceDesc->Common.Type)      {      case ACPI_TYPE_INTEGER: +          Buffer = &SourceDesc->Integer.Value;          Length = sizeof (SourceDesc->Integer.Value);          break;      case ACPI_TYPE_BUFFER: +          Buffer = SourceDesc->Buffer.Pointer;          Length = SourceDesc->Buffer.Length;          break;      case ACPI_TYPE_STRING: +          Buffer = SourceDesc->String.Pointer;          Length = SourceDesc->String.Length;          break;      default: +          return_ACPI_STATUS (AE_AML_OPERAND_TYPE);      } diff --git a/source/components/executer/exfldio.c b/source/components/executer/exfldio.c index 85da123903da..cd3c40f6367c 100644 --- a/source/components/executer/exfldio.c +++ b/source/components/executer/exfldio.c @@ -480,9 +480,7 @@ AcpiExFieldDatumIo (          Status = AE_OK;          break; -      case ACPI_TYPE_LOCAL_BANK_FIELD: -          /*           * Ensure that the BankValue is not beyond the capacity of           * the register @@ -512,7 +510,6 @@ AcpiExFieldDatumIo (          /*lint -fallthrough */ -      case ACPI_TYPE_LOCAL_REGION_FIELD:          /*           * For simple RegionFields, we just directly access the owning @@ -522,10 +519,7 @@ AcpiExFieldDatumIo (                      ReadWrite);          break; -      case ACPI_TYPE_LOCAL_INDEX_FIELD: - -          /*           * Ensure that the IndexValue is not beyond the capacity of           * the register @@ -575,7 +569,6 @@ AcpiExFieldDatumIo (          }          break; -      default:          ACPI_ERROR ((AE_INFO, "Wrong object type in field I/O %u", diff --git a/source/components/executer/exmisc.c b/source/components/executer/exmisc.c index e527514e4cfb..7520c9b89c60 100644 --- a/source/components/executer/exmisc.c +++ b/source/components/executer/exmisc.c @@ -115,16 +115,13 @@ AcpiExGetObjectReference (          }          break; -      case ACPI_DESC_TYPE_NAMED: -          /*           * A named reference that has already been resolved to a Node           */          ReferencedObj = ObjDesc;          break; -      default:          ACPI_ERROR ((AE_INFO, "Invalid descriptor type 0x%X", @@ -286,19 +283,23 @@ AcpiExDoConcatenate (      switch (Operand0->Common.Type)      {      case ACPI_TYPE_INTEGER: +          Status = AcpiExConvertToInteger (Operand1, &LocalOperand1, 16);          break;      case ACPI_TYPE_STRING: +          Status = AcpiExConvertToString (Operand1, &LocalOperand1,                      ACPI_IMPLICIT_CONVERT_HEX);          break;      case ACPI_TYPE_BUFFER: +          Status = AcpiExConvertToBuffer (Operand1, &LocalOperand1);          break;      default: +          ACPI_ERROR ((AE_INFO, "Invalid object type: 0x%X",              Operand0->Common.Type));          Status = AE_AML_INTERNAL; @@ -450,37 +451,30 @@ AcpiExDoMathOp (          return (Integer0 + Integer1); -      case AML_BIT_AND_OP:            /* And (Integer0, Integer1, Result) */          return (Integer0 & Integer1); -      case AML_BIT_NAND_OP:           /* NAnd (Integer0, Integer1, Result) */          return (~(Integer0 & Integer1)); -      case AML_BIT_OR_OP:             /* Or (Integer0, Integer1, Result) */          return (Integer0 | Integer1); -      case AML_BIT_NOR_OP:            /* NOr (Integer0, Integer1, Result) */          return (~(Integer0 | Integer1)); -      case AML_BIT_XOR_OP:            /* XOr (Integer0, Integer1, Result) */          return (Integer0 ^ Integer1); -      case AML_MULTIPLY_OP:           /* Multiply (Integer0, Integer1, Result) */          return (Integer0 * Integer1); -      case AML_SHIFT_LEFT_OP:         /* ShiftLeft (Operand, ShiftCount, Result)*/          /* @@ -493,7 +487,6 @@ AcpiExDoMathOp (          }          return (Integer0 << Integer1); -      case AML_SHIFT_RIGHT_OP:        /* ShiftRight (Operand, ShiftCount, Result) */          /* @@ -506,7 +499,6 @@ AcpiExDoMathOp (          }          return (Integer0 >> Integer1); -      case AML_SUBTRACT_OP:           /* Subtract (Integer0, Integer1, Result) */          return (Integer0 - Integer1); @@ -571,6 +563,7 @@ AcpiExDoLogicalNumericOp (          break;      default: +          Status = AE_AML_INTERNAL;          break;      } @@ -638,19 +631,23 @@ AcpiExDoLogicalOp (      switch (Operand0->Common.Type)      {      case ACPI_TYPE_INTEGER: +          Status = AcpiExConvertToInteger (Operand1, &LocalOperand1, 16);          break;      case ACPI_TYPE_STRING: +          Status = AcpiExConvertToString (Operand1, &LocalOperand1,                      ACPI_IMPLICIT_CONVERT_HEX);          break;      case ACPI_TYPE_BUFFER: +          Status = AcpiExConvertToBuffer (Operand1, &LocalOperand1);          break;      default: +          Status = AE_AML_INTERNAL;          break;      } @@ -699,6 +696,7 @@ AcpiExDoLogicalOp (              break;          default: +              Status = AE_AML_INTERNAL;              break;          } @@ -776,6 +774,7 @@ AcpiExDoLogicalOp (              break;          default: +              Status = AE_AML_INTERNAL;              break;          } diff --git a/source/components/executer/exnames.c b/source/components/executer/exnames.c index fee4e5c74f07..ac12e1ac56e6 100644 --- a/source/components/executer/exnames.c +++ b/source/components/executer/exnames.c @@ -334,7 +334,6 @@ AcpiExGetNameString (              HasPrefix = TRUE;              break; -          case AML_PARENT_PREFIX:              /* Increment past possibly multiple parent prefixes */ @@ -352,7 +351,6 @@ AcpiExGetNameString (              HasPrefix = TRUE;              break; -          default:              /* Not a prefix character */ @@ -388,7 +386,6 @@ AcpiExGetNameString (              }              break; -          case AML_MULTI_NAME_PREFIX_OP:              ACPI_DEBUG_PRINT ((ACPI_DB_LOAD, "MultiNamePrefix at %p\n", @@ -420,7 +417,6 @@ AcpiExGetNameString (              break; -          case 0:              /* NullName valid as of 8-12-98 ASL/AML Grammar Update */ @@ -443,7 +439,6 @@ AcpiExGetNameString (              break; -          default:              /* Name segment string */ diff --git a/source/components/executer/exoparg1.c b/source/components/executer/exoparg1.c index 73fc0268f479..ff8dee2487aa 100644 --- a/source/components/executer/exoparg1.c +++ b/source/components/executer/exoparg1.c @@ -180,37 +180,31 @@ AcpiExOpcode_1A_0T_0R (          Status = AcpiExReleaseMutex (Operand[0], WalkState);          break; -      case AML_RESET_OP:      /*  Reset (EventObject) */          Status = AcpiExSystemResetEvent (Operand[0]);          break; -      case AML_SIGNAL_OP:     /*  Signal (EventObject) */          Status = AcpiExSystemSignalEvent (Operand[0]);          break; -      case AML_SLEEP_OP:      /*  Sleep (MsecTime) */          Status = AcpiExSystemDoSleep (Operand[0]->Integer.Value);          break; -      case AML_STALL_OP:      /*  Stall (UsecTime) */          Status = AcpiExSystemDoStall ((UINT32) Operand[0]->Integer.Value);          break; -      case AML_UNLOAD_OP:     /*  Unload (Handle) */          Status = AcpiExUnloadTable (Operand[0]);          break; -      default:                /*  Unknown opcode  */          ACPI_ERROR ((AE_INFO, "Unknown AML opcode 0x%X", @@ -330,7 +324,6 @@ AcpiExOpcode_1A_1T_1R (              ReturnDesc->Integer.Value = ~Operand[0]->Integer.Value;              break; -          case AML_FIND_SET_LEFT_BIT_OP:  /* FindSetLeftBit (Operand, Result) */              ReturnDesc->Integer.Value = Operand[0]->Integer.Value; @@ -348,7 +341,6 @@ AcpiExOpcode_1A_1T_1R (              ReturnDesc->Integer.Value = Temp32;              break; -          case AML_FIND_SET_RIGHT_BIT_OP: /* FindSetRightBit (Operand, Result) */              ReturnDesc->Integer.Value = Operand[0]->Integer.Value; @@ -369,9 +361,7 @@ AcpiExOpcode_1A_1T_1R (                  Temp32 == 0 ? 0 : (ACPI_INTEGER_BIT_SIZE + 1) - Temp32;              break; -          case AML_FROM_BCD_OP:           /* FromBcd (BCDValue, Result)  */ -              /*               * The 64-bit ACPI integer can hold 16 4-bit BCD characters               * (if table is 32-bit, integer can hold 8 BCD characters) @@ -416,7 +406,6 @@ AcpiExOpcode_1A_1T_1R (              }              break; -          case AML_TO_BCD_OP:             /* ToBcd (Operand, Result)  */              ReturnDesc->Integer.Value = 0; @@ -448,9 +437,7 @@ AcpiExOpcode_1A_1T_1R (              }              break; -          case AML_COND_REF_OF_OP:        /* CondRefOf (SourceObject, Result)  */ -              /*               * This op is a little strange because the internal return value is               * different than the return value stored in the result descriptor @@ -485,14 +472,14 @@ AcpiExOpcode_1A_1T_1R (          default: +              /* No other opcodes get here */ +              break;          }          break; -      case AML_STORE_OP:              /* Store (Source, Target) */ -          /*           * A store operand is typically a number, string, buffer or lvalue           * Be careful about deleting the source object, @@ -519,7 +506,6 @@ AcpiExOpcode_1A_1T_1R (          }          return_ACPI_STATUS (Status); -      /*       * ACPI 2.0 Opcodes       */ @@ -529,7 +515,6 @@ AcpiExOpcode_1A_1T_1R (                      WalkState);          break; -      case AML_TO_DECSTRING_OP:       /* ToDecimalString (Data, Result) */          Status = AcpiExConvertToString (Operand[0], &ReturnDesc, @@ -541,7 +526,6 @@ AcpiExOpcode_1A_1T_1R (          }          break; -      case AML_TO_HEXSTRING_OP:       /* ToHexString (Data, Result) */          Status = AcpiExConvertToString (Operand[0], &ReturnDesc, @@ -553,7 +537,6 @@ AcpiExOpcode_1A_1T_1R (          }          break; -      case AML_TO_BUFFER_OP:          /* ToBuffer (Data, Result) */          Status = AcpiExConvertToBuffer (Operand[0], &ReturnDesc); @@ -564,7 +547,6 @@ AcpiExOpcode_1A_1T_1R (          }          break; -      case AML_TO_INTEGER_OP:         /* ToInteger (Data, Result) */          Status = AcpiExConvertToInteger (Operand[0], &ReturnDesc, @@ -576,7 +558,6 @@ AcpiExOpcode_1A_1T_1R (          }          break; -      case AML_SHIFT_LEFT_BIT_OP:     /* ShiftLeftBit (Source, BitNum)  */      case AML_SHIFT_RIGHT_BIT_OP:    /* ShiftRightBit (Source, BitNum) */ @@ -588,7 +569,6 @@ AcpiExOpcode_1A_1T_1R (          Status = AE_SUPPORT;          goto Cleanup; -      default:                        /* Unknown opcode */          ACPI_ERROR ((AE_INFO, "Unknown AML opcode 0x%X", @@ -676,10 +656,8 @@ AcpiExOpcode_1A_0T_1R (          }          break; -      case AML_DECREMENT_OP:          /* Decrement (Operand)  */      case AML_INCREMENT_OP:          /* Increment (Operand)  */ -          /*           * Create a new integer. Can't just get the base integer and           * increment it because it may be an Arg or Field. @@ -744,9 +722,7 @@ AcpiExOpcode_1A_0T_1R (          Status = AcpiExStore (ReturnDesc, Operand[0], WalkState);          break; -      case AML_TYPE_OP:               /* ObjectType (SourceObject) */ -          /*           * 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 @@ -772,9 +748,7 @@ AcpiExOpcode_1A_0T_1R (          }          break; -      case AML_SIZE_OF_OP:            /* SizeOf (SourceObject)  */ -          /*           * Note: The operand is not resolved at this point because we want to           * get the associated object, not its value. @@ -801,10 +775,12 @@ AcpiExOpcode_1A_0T_1R (          switch (Type)          {          case ACPI_TYPE_INTEGER: +              Value = AcpiGbl_IntegerByteWidth;              break;          case ACPI_TYPE_STRING: +              Value = TempDesc->String.Length;              break; @@ -825,6 +801,7 @@ AcpiExOpcode_1A_0T_1R (              break;          default: +              ACPI_ERROR ((AE_INFO,                  "Operand must be Buffer/Integer/String/Package - found type %s",                  AcpiUtGetTypeName (Type))); @@ -932,9 +909,11 @@ AcpiExOpcode_1A_0T_1R (                  break;              case ACPI_TYPE_STRING: +                  break;              default: +                  Status = AE_AML_OPERAND_TYPE;                  goto Cleanup;              } @@ -993,7 +972,6 @@ AcpiExOpcode_1A_0T_1R (              switch (Operand[0]->Reference.Class)              {              case ACPI_REFCLASS_INDEX: -                  /*                   * The target type for the Index operator must be                   * either a Buffer or a Package @@ -1025,9 +1003,7 @@ AcpiExOpcode_1A_0T_1R (                      }                      break; -                  case ACPI_TYPE_PACKAGE: -                      /*                       * Return the referenced element of the package. We must                       * add another reference to the referenced object, however. @@ -1039,7 +1015,6 @@ AcpiExOpcode_1A_0T_1R (                      }                      break; -                  default:                      ACPI_ERROR ((AE_INFO, @@ -1050,7 +1025,6 @@ AcpiExOpcode_1A_0T_1R (                  }                  break; -              case ACPI_REFCLASS_REFOF:                  ReturnDesc = Operand[0]->Reference.Object; @@ -1067,8 +1041,8 @@ AcpiExOpcode_1A_0T_1R (                  AcpiUtAddReference (ReturnDesc);                  break; -              default: +                  ACPI_ERROR ((AE_INFO,                      "Unknown class in reference(%p) - 0x%2.2X",                      Operand[0], Operand[0]->Reference.Class)); @@ -1079,7 +1053,6 @@ AcpiExOpcode_1A_0T_1R (          }          break; -      default:          ACPI_ERROR ((AE_INFO, "Unknown AML opcode 0x%X", diff --git a/source/components/executer/exoparg2.c b/source/components/executer/exoparg2.c index 6151fa8ddf8f..ff1d8bbf7d76 100644 --- a/source/components/executer/exoparg2.c +++ b/source/components/executer/exoparg2.c @@ -144,7 +144,6 @@ AcpiExOpcode_2A_0T_0R (          Status = AcpiEvQueueNotifyRequest (Node, Value);          break; -      default:          ACPI_ERROR ((AE_INFO, "Unknown AML opcode 0x%X", @@ -217,7 +216,6 @@ AcpiExOpcode_2A_2T_1R (          }          break; -      default:          ACPI_ERROR ((AE_INFO, "Unknown AML opcode 0x%X", @@ -331,16 +329,13 @@ AcpiExOpcode_2A_1T_1R (                                 &ReturnDesc->Integer.Value);          break; -      case AML_CONCAT_OP: /* Concatenate (Data1, Data2, Result) */          Status = AcpiExDoConcatenate (Operand[0], Operand[1],                      &ReturnDesc, WalkState);          break; -      case AML_TO_STRING_OP: /* ToString (Buffer, Length, Result) (ACPI 2.0) */ -          /*           * Input object is guaranteed to be a buffer at this point (it may have           * been converted.)  Copy the raw buffer data to a new object of @@ -380,7 +375,6 @@ AcpiExOpcode_2A_1T_1R (              Operand[0]->Buffer.Pointer, Length);          break; -      case AML_CONCAT_RES_OP:          /* ConcatenateResTemplate (Buffer, Buffer, Result) (ACPI 2.0) */ @@ -389,7 +383,6 @@ AcpiExOpcode_2A_1T_1R (                      &ReturnDesc, WalkState);          break; -      case AML_INDEX_OP:              /* Index (Source Index Result) */          /* Create the internal return object */ @@ -479,7 +472,6 @@ AcpiExOpcode_2A_1T_1R (          WalkState->ResultObj = ReturnDesc;          goto Cleanup; -      default:          ACPI_ERROR ((AE_INFO, "Unknown AML opcode 0x%X", @@ -602,7 +594,6 @@ AcpiExOpcode_2A_0T_1R (          }          break; -      default:          ACPI_ERROR ((AE_INFO, "Unknown AML opcode 0x%X", diff --git a/source/components/executer/exoparg3.c b/source/components/executer/exoparg3.c index a7d44311ccae..b850485faa28 100644 --- a/source/components/executer/exoparg3.c +++ b/source/components/executer/exoparg3.c @@ -129,7 +129,6 @@ AcpiExOpcode_3A_0T_0R (          ACPI_FREE (Fatal);          break; -      default:          ACPI_ERROR ((AE_INFO, "Unknown AML opcode 0x%X", @@ -176,7 +175,6 @@ AcpiExOpcode_3A_1T_1R (      switch (WalkState->Opcode)      {      case AML_MID_OP:    /* Mid (Source[0], Index[1], Length[2], Result[3]) */ -          /*           * Create the return object. The Source operand is guaranteed to be           * either a String or a Buffer, so just use its type. @@ -268,7 +266,6 @@ AcpiExOpcode_3A_1T_1R (          ReturnDesc->Buffer.Flags |= AOPOBJ_DATA_VALID;          break; -      default:          ACPI_ERROR ((AE_INFO, "Unknown AML opcode 0x%X", diff --git a/source/components/executer/exoparg6.c b/source/components/executer/exoparg6.c index 6b04784bef48..974d4c86f91f 100644 --- a/source/components/executer/exoparg6.c +++ b/source/components/executer/exoparg6.c @@ -130,7 +130,6 @@ AcpiExDoMatch (          break;      case MATCH_MEQ: -          /*           * True if equal: (P[i] == M)           * Change to:     (M == P[i]) @@ -144,7 +143,6 @@ AcpiExDoMatch (          break;      case MATCH_MLE: -          /*           * True if less than or equal: (P[i] <= M) (P[i] NotGreater than M)           * Change to:                  (M >= P[i]) (M NotLess than P[i]) @@ -159,7 +157,6 @@ AcpiExDoMatch (          break;      case MATCH_MLT: -          /*           * True if less than: (P[i] < M)           * Change to:         (M > P[i]) @@ -173,7 +170,6 @@ AcpiExDoMatch (          break;      case MATCH_MGE: -          /*           * True if greater than or equal: (P[i] >= M) (P[i] NotLess than M)           * Change to:                     (M <= P[i]) (M NotGreater than P[i]) @@ -188,7 +184,6 @@ AcpiExDoMatch (          break;      case MATCH_MGT: -          /*           * True if greater than: (P[i] > M)           * Change to:            (M < P[i]) @@ -328,13 +323,11 @@ AcpiExOpcode_6A_0T_1R (          }          break; -      case AML_LOAD_TABLE_OP:          Status = AcpiExLoadTableOp (WalkState, &ReturnDesc);          break; -      default:          ACPI_ERROR ((AE_INFO, "Unknown AML opcode 0x%X", diff --git a/source/components/executer/exprep.c b/source/components/executer/exprep.c index 90687a5839a8..c6292241db19 100644 --- a/source/components/executer/exprep.c +++ b/source/components/executer/exprep.c @@ -267,26 +267,31 @@ AcpiExDecodeFieldAccess (      case AML_FIELD_ACCESS_BYTE:      case AML_FIELD_ACCESS_BUFFER:   /* ACPI 2.0 (SMBus Buffer) */ +          ByteAlignment = 1;          BitLength     = 8;          break;      case AML_FIELD_ACCESS_WORD: +          ByteAlignment = 2;          BitLength     = 16;          break;      case AML_FIELD_ACCESS_DWORD: +          ByteAlignment = 4;          BitLength     = 32;          break;      case AML_FIELD_ACCESS_QWORD:    /* ACPI 2.0 */ +          ByteAlignment = 8;          BitLength     = 64;          break;      default: +          /* Invalid field access type */          ACPI_ERROR ((AE_INFO, @@ -536,7 +541,6 @@ AcpiExPrepFieldValue (              ObjDesc->Field.AccessByteWidth, ObjDesc->Field.RegionObj));          break; -      case ACPI_TYPE_LOCAL_BANK_FIELD:          ObjDesc->BankField.Value = Info->BankValue; @@ -571,7 +575,6 @@ AcpiExPrepFieldValue (          break; -      case ACPI_TYPE_LOCAL_INDEX_FIELD:          /* Get the Index and Data registers */ @@ -625,7 +628,9 @@ AcpiExPrepFieldValue (          break;      default: +          /* No other types should get here */ +          break;      } diff --git a/source/components/executer/exregion.c b/source/components/executer/exregion.c index 1092a42b6cf4..304cfe6262f6 100644 --- a/source/components/executer/exregion.c +++ b/source/components/executer/exregion.c @@ -99,22 +99,27 @@ AcpiExSystemMemorySpaceHandler (      switch (BitWidth)      {      case 8: +          Length = 1;          break;      case 16: +          Length = 2;          break;      case 32: +          Length = 4;          break;      case 64: +          Length = 8;          break;      default: +          ACPI_ERROR ((AE_INFO, "Invalid SystemMemory width %u",              BitWidth));          return_ACPI_STATUS (AE_AML_OPERAND_VALUE); @@ -231,23 +236,29 @@ AcpiExSystemMemorySpaceHandler (          switch (BitWidth)          {          case 8: +              *Value = (UINT64) ACPI_GET8 (LogicalAddrPtr);              break;          case 16: +              *Value = (UINT64) ACPI_GET16 (LogicalAddrPtr);              break;          case 32: +              *Value = (UINT64) ACPI_GET32 (LogicalAddrPtr);              break;          case 64: +              *Value = (UINT64) ACPI_GET64 (LogicalAddrPtr);              break;          default: +              /* BitWidth was already validated */ +              break;          }          break; @@ -257,28 +268,35 @@ AcpiExSystemMemorySpaceHandler (          switch (BitWidth)          {          case 8: +              ACPI_SET8 (LogicalAddrPtr, *Value);              break;          case 16: +              ACPI_SET16 (LogicalAddrPtr, *Value);              break;          case 32: +              ACPI_SET32 (LogicalAddrPtr, *Value);              break;          case 64: +              ACPI_SET64 (LogicalAddrPtr, *Value);              break;          default: +              /* BitWidth was already validated */ +              break;          }          break;      default: +          Status = AE_BAD_PARAMETER;          break;      } @@ -343,6 +361,7 @@ AcpiExSystemIoSpaceHandler (          break;      default: +          Status = AE_BAD_PARAMETER;          break;      } diff --git a/source/components/executer/exresnte.c b/source/components/executer/exresnte.c index fae97db10e98..c26cbb9b3e82 100644 --- a/source/components/executer/exresnte.c +++ b/source/components/executer/exresnte.c @@ -164,7 +164,6 @@ AcpiExResolveNodeToValue (          }          break; -      case ACPI_TYPE_BUFFER:          if (SourceDesc->Common.Type != ACPI_TYPE_BUFFER) @@ -184,7 +183,6 @@ AcpiExResolveNodeToValue (          }          break; -      case ACPI_TYPE_STRING:          if (SourceDesc->Common.Type != ACPI_TYPE_STRING) @@ -200,7 +198,6 @@ AcpiExResolveNodeToValue (          AcpiUtAddReference (ObjDesc);          break; -      case ACPI_TYPE_INTEGER:          if (SourceDesc->Common.Type != ACPI_TYPE_INTEGER) @@ -216,7 +213,6 @@ AcpiExResolveNodeToValue (          AcpiUtAddReference (ObjDesc);          break; -      case ACPI_TYPE_BUFFER_FIELD:      case ACPI_TYPE_LOCAL_REGION_FIELD:      case ACPI_TYPE_LOCAL_BANK_FIELD: @@ -252,7 +248,6 @@ AcpiExResolveNodeToValue (          return_ACPI_STATUS (AE_AML_OPERAND_TYPE);  /* Cannot be AE_TYPE */ -      case ACPI_TYPE_LOCAL_REFERENCE:          switch (SourceDesc->Reference.Class) @@ -268,6 +263,7 @@ AcpiExResolveNodeToValue (              break;          default: +              /* No named references are allowed here */              ACPI_ERROR ((AE_INFO, @@ -278,7 +274,6 @@ AcpiExResolveNodeToValue (          }          break; -      default:          /* Default case is for unknown types */ diff --git a/source/components/executer/exresolv.c b/source/components/executer/exresolv.c index e68ea05107a2..e99d46c5b5b0 100644 --- a/source/components/executer/exresolv.c +++ b/source/components/executer/exresolv.c @@ -176,7 +176,6 @@ AcpiExResolveObjectToValue (          {          case ACPI_REFCLASS_LOCAL:          case ACPI_REFCLASS_ARG: -              /*               * Get the local from the method's state info               * Note: this increments the local's object reference count @@ -199,7 +198,6 @@ AcpiExResolveObjectToValue (              *StackPtr = ObjDesc;              break; -          case ACPI_REFCLASS_INDEX:              switch (StackDesc->Reference.TargetType) @@ -209,7 +207,6 @@ AcpiExResolveObjectToValue (                  /* Just return - do not dereference */                  break; -              case ACPI_TYPE_PACKAGE:                  /* If method call or CopyObject - do not dereference */ @@ -247,7 +244,6 @@ AcpiExResolveObjectToValue (                  }                  break; -              default:                  /* Invalid reference object */ @@ -260,7 +256,6 @@ AcpiExResolveObjectToValue (              }              break; -          case ACPI_REFCLASS_REFOF:          case ACPI_REFCLASS_DEBUG:          case ACPI_REFCLASS_TABLE: @@ -300,19 +295,16 @@ AcpiExResolveObjectToValue (          }          break; -      case ACPI_TYPE_BUFFER:          Status = AcpiDsGetBufferArguments (StackDesc);          break; -      case ACPI_TYPE_PACKAGE:          Status = AcpiDsGetPackageArguments (StackDesc);          break; -      case ACPI_TYPE_BUFFER_FIELD:      case ACPI_TYPE_LOCAL_REGION_FIELD:      case ACPI_TYPE_LOCAL_BANK_FIELD: @@ -330,6 +322,7 @@ AcpiExResolveObjectToValue (          break;      default: +          break;      } @@ -374,10 +367,12 @@ AcpiExResolveMultiple (      switch (ACPI_GET_DESCRIPTOR_TYPE (ObjDesc))      {      case ACPI_DESC_TYPE_OPERAND: +          Type = ObjDesc->Common.Type;          break;      case ACPI_DESC_TYPE_NAMED: +          Type = ((ACPI_NAMESPACE_NODE *) ObjDesc)->Type;          ObjDesc = AcpiNsGetAttachedObject ((ACPI_NAMESPACE_NODE *) ObjDesc); @@ -454,7 +449,6 @@ AcpiExResolveMultiple (              }              break; -          case ACPI_REFCLASS_INDEX:              /* Get the type of this reference (index into another object) */ @@ -482,13 +476,11 @@ AcpiExResolveMultiple (              }              break; -          case ACPI_REFCLASS_TABLE:              Type = ACPI_TYPE_DDB_HANDLE;              goto Exit; -          case ACPI_REFCLASS_LOCAL:          case ACPI_REFCLASS_ARG: @@ -520,7 +512,6 @@ AcpiExResolveMultiple (              }              break; -          case ACPI_REFCLASS_DEBUG:              /* The Debug Object is of type "DebugObject" */ @@ -528,7 +519,6 @@ AcpiExResolveMultiple (              Type = ACPI_TYPE_DEBUG_OBJECT;              goto Exit; -          default:              ACPI_ERROR ((AE_INFO, @@ -564,7 +554,9 @@ Exit:          break;      default: +          /* No change to Type required */ +          break;      } diff --git a/source/components/executer/exresop.c b/source/components/executer/exresop.c index 96410b2a52ef..59602ba9d47b 100644 --- a/source/components/executer/exresop.c +++ b/source/components/executer/exresop.c @@ -224,7 +224,6 @@ AcpiExResolveOperands (              }              break; -          case ACPI_DESC_TYPE_OPERAND:              /* ACPI internal object */ @@ -277,7 +276,6 @@ AcpiExResolveOperands (              }              break; -          default:              /* Invalid descriptor */ @@ -324,7 +322,6 @@ AcpiExResolveOperands (          case ARGI_TARGETREF:     /* Allows implicit conversion rules before store */          case ARGI_FIXED_TARGET:  /* No implicit conversion before store to target */          case ARGI_SIMPLE_TARGET: /* Name, Local, or Arg - no implicit conversion  */ -              /*               * Need an operand of type ACPI_TYPE_LOCAL_REFERENCE               * A Namespace Node is OK as-is @@ -342,9 +339,7 @@ AcpiExResolveOperands (              }              goto NextOperand; -          case ARGI_DATAREFOBJ:  /* Store operator only */ -              /*               * We don't want to resolve IndexOp reference objects during               * a store because this would be an implicit DeRefOf operation. @@ -360,7 +355,9 @@ AcpiExResolveOperands (              break;          default: +              /* All cases covered above */ +              break;          } @@ -453,9 +450,7 @@ AcpiExResolveOperands (              }              goto NextOperand; -          case ARGI_BUFFER: -              /*               * Need an operand of type ACPI_TYPE_BUFFER,               * But we can implicitly convert from a STRING or INTEGER @@ -482,9 +477,7 @@ AcpiExResolveOperands (              }              goto NextOperand; -          case ARGI_STRING: -              /*               * Need an operand of type ACPI_TYPE_STRING,               * But we can implicitly convert from a BUFFER or INTEGER @@ -512,7 +505,6 @@ AcpiExResolveOperands (              }              goto NextOperand; -          case ARGI_COMPUTEDATA:              /* Need an operand of type INTEGER, STRING or BUFFER */ @@ -535,7 +527,6 @@ AcpiExResolveOperands (              }              goto NextOperand; -          case ARGI_BUFFER_OR_STRING:              /* Need an operand of type STRING or BUFFER */ @@ -573,7 +564,6 @@ AcpiExResolveOperands (              }              goto NextOperand; -          case ARGI_DATAOBJECT:              /*               * ARGI_DATAOBJECT is only used by the SizeOf operator. @@ -593,6 +583,7 @@ AcpiExResolveOperands (                  break;              default: +                  ACPI_ERROR ((AE_INFO,                      "Needed [Buffer/String/Package/Reference], found [%s] %p",                      AcpiUtGetObjectTypeName (ObjDesc), ObjDesc)); @@ -601,7 +592,6 @@ AcpiExResolveOperands (              }              goto NextOperand; -          case ARGI_COMPLEXOBJ:              /* Need a buffer or package or (ACPI 2.0) String */ @@ -616,6 +606,7 @@ AcpiExResolveOperands (                  break;              default: +                  ACPI_ERROR ((AE_INFO,                      "Needed [Buffer/String/Package], found [%s] %p",                      AcpiUtGetObjectTypeName (ObjDesc), ObjDesc)); @@ -624,7 +615,6 @@ AcpiExResolveOperands (              }              goto NextOperand; -          case ARGI_REGION_OR_BUFFER: /* Used by Load() only */              /* Need an operand of type REGION or a BUFFER (which could be a resolved region field) */ @@ -638,6 +628,7 @@ AcpiExResolveOperands (                  break;              default: +                  ACPI_ERROR ((AE_INFO,                      "Needed [Region/Buffer], found [%s] %p",                      AcpiUtGetObjectTypeName (ObjDesc), ObjDesc)); @@ -646,7 +637,6 @@ AcpiExResolveOperands (              }              goto NextOperand; -          case ARGI_DATAREFOBJ:              /* Used by the Store() operator only */ @@ -694,7 +684,6 @@ AcpiExResolveOperands (              }              goto NextOperand; -          default:              /* Unknown type */ diff --git a/source/components/executer/exstore.c b/source/components/executer/exstore.c index 98c3eca3b628..2ec392e90e60 100644 --- a/source/components/executer/exstore.c +++ b/source/components/executer/exstore.c @@ -124,6 +124,7 @@ AcpiExStore (      switch (DestDesc->Common.Type)      {      case ACPI_TYPE_LOCAL_REFERENCE: +          break;      case ACPI_TYPE_INTEGER: @@ -167,7 +168,6 @@ AcpiExStore (                      WalkState, ACPI_IMPLICIT_CONVERSION);          break; -      case ACPI_REFCLASS_INDEX:          /* Storing to an Index (pointer into a packager or buffer) */ @@ -175,7 +175,6 @@ AcpiExStore (          Status = AcpiExStoreObjectToIndex (SourceDesc, RefDesc, WalkState);          break; -      case ACPI_REFCLASS_LOCAL:      case ACPI_REFCLASS_ARG: @@ -185,9 +184,7 @@ AcpiExStore (                      RefDesc->Reference.Value, SourceDesc, WalkState);          break; -      case ACPI_REFCLASS_DEBUG: -          /*           * Storing to the Debug object causes the value stored to be           * displayed and otherwise has no effect -- see ACPI Specification @@ -199,7 +196,6 @@ AcpiExStore (          ACPI_DEBUG_OBJECT (SourceDesc, 0, 0);          break; -      default:          ACPI_ERROR ((AE_INFO, "Unknown Reference Class 0x%2.2X", @@ -308,9 +304,7 @@ AcpiExStoreObjectToIndex (          break; -      case ACPI_TYPE_BUFFER_FIELD: -          /*           * Store into a Buffer or String (not actually a real BufferField)           * at a location defined by an Index. @@ -368,7 +362,6 @@ AcpiExStoreObjectToIndex (          ObjDesc->Buffer.Pointer[IndexDesc->Reference.Value] = Value;          break; -      default:          ACPI_ERROR ((AE_INFO,              "Target is not a Package or BufferField")); @@ -473,11 +466,9 @@ AcpiExStoreObjectToNode (                      &WalkState->ResultObj);          break; -      case ACPI_TYPE_INTEGER:      case ACPI_TYPE_STRING:      case ACPI_TYPE_BUFFER: -          /*           * These target types are all of type Integer/String/Buffer, and           * therefore support implicit conversion before the store. @@ -511,7 +502,6 @@ AcpiExStoreObjectToNode (          }          break; -      default:          ACPI_DEBUG_PRINT ((ACPI_DB_EXEC, diff --git a/source/components/executer/exstoren.c b/source/components/executer/exstoren.c index ae3629636e01..a21d8b9e7536 100644 --- a/source/components/executer/exstoren.c +++ b/source/components/executer/exstoren.c @@ -94,11 +94,9 @@ AcpiExResolveObject (           * These cases all require only Integers or values that           * can be converted to Integers (Strings or Buffers)           */ -      case ACPI_TYPE_INTEGER:      case ACPI_TYPE_STRING:      case ACPI_TYPE_BUFFER: -          /*           * Stores into a Field/Region or into a Integer/Buffer/String           * are all essentially the same. This case handles the @@ -140,10 +138,8 @@ AcpiExResolveObject (          }          break; -      case ACPI_TYPE_LOCAL_ALIAS:      case ACPI_TYPE_LOCAL_METHOD_ALIAS: -          /*           * All aliases should have been resolved earlier, during the           * operand resolution phase. @@ -152,10 +148,8 @@ AcpiExResolveObject (          Status = AE_AML_INTERNAL;          break; -      case ACPI_TYPE_PACKAGE:      default: -          /*           * All other types than Alias and the various Fields come here,           * including the untyped case - ACPI_TYPE_ANY.  | 
