From fe6e53b5ba692463aa5c3daac1788e8375ceef80 Mon Sep 17 00:00:00 2001 From: Gary Lin Date: Fri, 29 Jul 2016 11:25:27 +0800 Subject: [PATCH] Vlv2TbltDevicePkg/AcpiPlatform: Remove unused variables Fix the following errors from gcc: Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c: In function 'PlatformUpdateTables': Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c:236:63: warning: variable 'LocalApicCounter' set but not used [-Wunused-but-set-variable] Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c:214:64: warning: variable 'ProcessorLocalApicEntry' set but not used [-Wunused-but-set-variable] Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c:213:63: warning: variable 'BufferSize' set but not used [-Wunused-but-set-variable] Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c: In function 'AcpiPlatformEntryPoint': Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c:797:33: warning: variable 'Data32' set but not used [-Wunused-but-set-variable] Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatformHooks.c: In function ?PatchGv3SsdtTable?: Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatformHooks.c:243:31: error: variable ?Status? set but not used [-Werror=unused-but-set-variable] Cc: David Wei Cc: "Wu, Hao A" Cc: "Lu, ShifeiX A" Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Gary Lin Reviewed-by: David Wei --- Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c | 8 -------- Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatformHooks.c | 3 +-- 2 files changed, 1 insertion(+), 10 deletions(-) diff --git a/Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c b/Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c index 8a883f2a09..f533d742a4 100644 --- a/Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c +++ b/Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c @@ -211,8 +211,6 @@ PlatformUpdateTables ( EFI_MP_SERVICES_PROTOCOL *MpService; UINTN MaximumNumberOfCPUs; UINTN NumberOfEnabledCPUs; - UINTN BufferSize; - ACPI_APIC_STRUCTURE_PTR *ProcessorLocalApicEntry; UINTN BspIndex; EFI_ACPI_1_0_ASF_DESCRIPTION_TABLE *AsfEntry; EFI_ACPI_HIGH_PRECISION_EVENT_TIMER_TABLE_HEADER *HpetTbl; @@ -234,7 +232,6 @@ PlatformUpdateTables ( UINT16 NumberOfHpets; UINT16 HpetCapIdValue; UINT32 HpetBlockID; - UINTN LocalApicCounter; EFI_PROCESSOR_INFORMATION ProcessorInfoBuffer; UINT8 TempVal; EFI_ACPI_3_0_IO_APIC_STRUCTURE *IOApicType; @@ -244,9 +241,7 @@ PlatformUpdateTables ( CurrPtr = NULL; EndPtr = NULL; ApicPtr = NULL; - LocalApicCounter = 0; CurrProcessor = 0; - ProcessorLocalApicEntry = NULL; if (Table->Signature != EFI_ACPI_1_0_FIRMWARE_ACPI_CONTROL_STRUCTURE_SIGNATURE) { @@ -374,7 +369,6 @@ PlatformUpdateTables ( ApicPtr->AcpiLocalApic.AcpiProcessorId = (UINT8)MaximumNumberOfCPUs; } - BufferSize = 0; ApicPtr->AcpiLocalApic.Flags = 0; for (CurrProcessor = 0; CurrProcessor < MaximumNumberOfCPUs; CurrProcessor++) { @@ -807,7 +801,6 @@ AcpiPlatformEntryPoint ( EFI_MP_SERVICES_PROTOCOL *MpService; UINTN MaximumNumberOfCPUs; UINTN NumberOfEnabledCPUs; - UINT32 Data32; PCH_STEPPING pchStepping; mFirstNotify = FALSE; @@ -816,7 +809,6 @@ AcpiPlatformEntryPoint ( Instance = 0; CurrentTable = NULL; TableHandle = 0; - Data32 = 0; // // Update HOB variable for PCI resource information. diff --git a/Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatformHooks.c b/Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatformHooks.c index 3489650808..c89c07f073 100644 --- a/Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatformHooks.c +++ b/Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatformHooks.c @@ -232,7 +232,6 @@ PatchGv3SsdtTable ( IN OUT EFI_ACPI_DESCRIPTION_HEADER *TableHeader ) { - EFI_STATUS Status; UINT8 *CurrPtr; UINT8 *SsdtPointer; UINT32 Signature; @@ -328,7 +327,7 @@ PatchGv3SsdtTable ( continue; } - Status = EistProt->GetEistTable (EistProt, &EistInfo, (VOID **) &PssState); + EistProt->GetEistTable (EistProt, &EistInfo, (VOID **) &PssState); AdjustSize = PssTable->NumEntries * sizeof (EFI_PSS_PACKAGE); AdjustSize -= EistInfo->NumStates * sizeof (EFI_PSS_PACKAGE); -- 2.39.2