Branch: refs/heads/master
  Home:   https://github.com/tianocore/edk2
  Commit: 69e16a2870cb546249c3c5271ecef36ee9fb7131
      
https://github.com/tianocore/edk2/commit/69e16a2870cb546249c3c5271ecef36ee9fb7131
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/bld_vlv.sh

  Log Message:
  -----------
  Vlv2TbltDevicePkg: Amend the build script

Take the gcc version detection from OvmfPkg/build.sh instead of the
hardcoded GCC46.

Cc: David Wei <david....@intel.com>
Cc: "Wu, Hao A" <hao.a...@intel.com>
Cc: "Lu, ShifeiX A" <shifeix.a...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>
Reviewed-by: David Wei <david....@intel.com>


  Commit: 9e9d99231033ce2eca150a3fd6f8c8d6217234bd
      
https://github.com/tianocore/edk2/commit/9e9d99231033ce2eca150a3fd6f8c8d6217234bd
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc

  Log Message:
  -----------
  Vlv2TbltDevicePkg: Only define MDEPKG_NDEBUG for gcc release build

We only enable "-Wno-unused-but-set-variable" for the release build
and gcc would complain that the varible passed to ASSERT_EFI_ERROR
wasn't used in the debug build. Just don't define MDEPKG_NDEBUG for
the debug build to make gcc happy with ASSERT_EFI_ERROR.

Suggested-by: Laszlo Ersek <ler...@redhat.com>
Cc: David Wei <david....@intel.com>
CC: Laszlo Ersek <ler...@redhat.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>


  Commit: b3a4a852edc1ff131b219047b5342b899f695e08
      
https://github.com/tianocore/edk2/commit/b3a4a852edc1ff131b219047b5342b899f695e08
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/Library/MultiPlatformLib/BoardClkGens/BoardClkGens.c
    M Vlv2TbltDevicePkg/Library/MultiPlatformLib/MultiPlatformLib.c

  Log Message:
  -----------
  Vlv2TbltDevicePkg/MultiPlatformLib: Remove the unused variables

Fix the following errors from gcc:

Vlv2TbltDevicePkg/Library/MultiPlatformLib/MultiPlatformLib.c: In function 
?MultiPlatformInfoInit?:
Vlv2TbltDevicePkg/Library/MultiPlatformLib/MultiPlatformLib.c:47:14: error: 
variable ?Status? set but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/Library/MultiPlatformLib/BoardClkGens/BoardClkGens.c: In 
function ?ReadClockGeneratorID?:
Vlv2TbltDevicePkg/Library/MultiPlatformLib/BoardClkGens/BoardClkGens.c:222:33: 
error: variable ?Status? set but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/Library/MultiPlatformLib/BoardClkGens/BoardClkGens.c: In 
function ?ConfigurePlatformClocks?:
Vlv2TbltDevicePkg/Library/MultiPlatformLib/BoardClkGens/BoardClkGens.c:283:33: 
error: variable ?ClockGenID? set but not used [-Werror=unused-but-set-variable]

Also fix the coding style of MultiPlatformInfoInit()

Cc: David Wei <david....@intel.com>
Cc: "Wu, Hao A" <hao.a...@intel.com>
Cc: "Lu, ShifeiX A" <shifeix.a...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>
Reviewed-by: David Wei <david....@intel.com>


  Commit: 824dfb5f8b1d7322e9163134c68bbc51ee2c04c0
      
https://github.com/tianocore/edk2/commit/824dfb5f8b1d7322e9163134c68bbc51ee2c04c0
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/PlatformPei/MemoryCallback.c

  Log Message:
  -----------
  Vlv2TbltDevicePkg/PlatformPei: Remove the unused variable

Fix the following error from gcc:

Vlv2TbltDevicePkg/PlatformPei/MemoryCallback.c: In function 
?MemoryDiscoveredPpiNotifyCallback?:
Vlv2TbltDevicePkg/PlatformPei/MemoryCallback.c:115:20: error: variable ?Status? 
set but not used [-Werror=unused-but-set-variable]

Cc: David Wei <david....@intel.com>
Cc: "Wu, Hao A" <hao.a...@intel.com>
Cc: "Lu, ShifeiX A" <shifeix.a...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>
Reviewed-by: David Wei <david....@intel.com>


  Commit: 5fc1373db620bd17b3e3efe7d79f4e8bd5aa8413
      
https://github.com/tianocore/edk2/commit/5fc1373db620bd17b3e3efe7d79f4e8bd5aa8413
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/Library/PlatformBdsLib/BdsPlatform.c

  Log Message:
  -----------
  Vlv2TbltDevicePkg/PlatformBdsLib: Remove the unused variable

Fix the following error from gcc:

Vlv2TbltDevicePkg/Library/PlatformBdsLib/BdsPlatform.c: In function ?BdsLockFv?:
Vlv2TbltDevicePkg/Library/PlatformBdsLib/BdsPlatform.c:2170:31: error: variable 
?Data? set but not used [-Werror=unused-but-set-variable]

Cc: David Wei <david....@intel.com>
Cc: "Wu, Hao A" <hao.a...@intel.com>
Cc: "Lu, ShifeiX A" <shifeix.a...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>
Reviewed-by: David Wei <david....@intel.com>


  Commit: ec9b12b9c62f654d6238053b42f3194d85355116
      
https://github.com/tianocore/edk2/commit/ec9b12b9c62f654d6238053b42f3194d85355116
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/PlatformInitPei/MemoryCallback.c
    M Vlv2TbltDevicePkg/PlatformInitPei/PchInitPeim.c

  Log Message:
  -----------
  Vlv2TbltDevicePkg/PlatformInitPei: Remove the unused variables

Fix the following errors from gcc:

Vlv2TbltDevicePkg/PlatformInitPei/PchInitPeim.c: In function 'IchRcrbInit':
Vlv2TbltDevicePkg/PlatformInitPei/PchInitPeim.c:513:36: error: variable 
'PlatformCpuInfo' set but not used [-Werror=unused-but-set-variable]
Vlv2TbltDevicePkg/PlatformInitPei/PchInitPeim.c:512:35: error: variable 
'LpcRevisionID' set but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/PlatformInitPei/MemoryCallback.c: In function 
'EndOfPeiPpiNotifyCallback':
Vlv2TbltDevicePkg/PlatformInitPei/MemoryCallback.c:64:31: error: variable 
'MemoryTop' set but not used [-Werror=unused-but-set-variable]

v2:
  Also remove the variable, Hob, in PchInitPeim.c:IchRcrbInit() since
  no one really uses it.

Cc: David Wei <david....@intel.com>
Cc: "Lu, ShifeiX A" <shifeix.a...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>


  Commit: c513680445da705e71526b27251f63e4f8b47c2a
      
https://github.com/tianocore/edk2/commit/c513680445da705e71526b27251f63e4f8b47c2a
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/PlatformInitPei/PlatformEarlyInit.c

  Log Message:
  -----------
  Vlv2TbltDevicePkg/PlatformInitPei: Suppress the gcc error message

Fix the following error from gcc:

Vlv2TbltDevicePkg/PlatformInitPei/PlatformEarlyInit.c: In function 
'GetWakeupEventAndSaveToHob':
Vlv2TbltDevicePkg/PlatformInitPei/PlatformEarlyInit.c:147:11: error: variable 
'WakeEventData' set but not used [-Werror=unused-but-set-variable]

Although the function name is "GetWakeupEventAndSaveToHob",
WakeEventData was never really used or passed to any other function.
Given the fact that the function is served as an example, it'd better to
keep the code related to WakeEventData. Just add a debug statement to
suppress the error message.

Cc: David Wei <david....@intel.com>
Cc: "Wu, Hao A" <hao.a...@intel.com>
Cc: "Lu, ShifeiX A" <shifeix.a...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>
Reviewed-by: David Wei <david....@intel.com>


  Commit: fe6e53b5ba692463aa5c3daac1788e8375ceef80
      
https://github.com/tianocore/edk2/commit/fe6e53b5ba692463aa5c3daac1788e8375ceef80
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c
    M Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatformHooks.c

  Log Message:
  -----------
  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 <david....@intel.com>
Cc: "Wu, Hao A" <hao.a...@intel.com>
Cc: "Lu, ShifeiX A" <shifeix.a...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>
Reviewed-by: David Wei <david....@intel.com>


  Commit: e7643ce70cabf09d89ba069b5460aab5901fa89d
      
https://github.com/tianocore/edk2/commit/e7643ce70cabf09d89ba069b5460aab5901fa89d
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.c
    M Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatform.h
    M Vlv2TbltDevicePkg/AcpiPlatform/AcpiPlatformHooks.c

  Log Message:
  -----------
  Vlv2TbltDevicePkg/AcpiPlatform: Amend the declaration of mGlobalNvsArea

mGlobalNvsArea wasn't declared as extern in AcpiPlatform.h and the header
was included in AcpiPlatform.c and AcpiPlatformHooks.c. Although it's
declared as extern in AcpiPlatformHooks.c, gcc still created an instance
of mGlobalNvsArea in AcpiPlatformHooks.c since the header was expanded
first and then the linker complained "multiple definition". This commit
rearrange the delaration of mGlobalNvsArea to satisfy gcc.

Cc: David Wei <david....@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>
Reviewed-by: David Wei <david....@intel.com>


  Commit: c50ff97c96ca5f0e90cb344e1cc438ebb1e51f0a
      
https://github.com/tianocore/edk2/commit/c50ff97c96ca5f0e90cb344e1cc438ebb1e51f0a
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/PlatformGopPolicy/PlatformGopPolicy.c

  Log Message:
  -----------
  Vlv2TbltDevicePkg/PlatformGopPolicy: include UefiBootServicesTableLib.h

gBS is already defined in
MdePkg/Library/UefiBootServicesTableLib/UefiBootServicesTableLib.c

Remove the declaration of gBS and include the proper header file to
access the variable.

Suggested-by: David Wei <david....@intel.com>
Cc: David Wei <david....@intel.com>
Cc: "Wu, Hao A" <hao.a...@intel.com>
Cc: "Lu, ShifeiX A" <shifeix.a...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>


  Commit: 35672950bea580e51b37fc7b3019fc713ede578e
      
https://github.com/tianocore/edk2/commit/35672950bea580e51b37fc7b3019fc713ede578e
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2DeviceRefCodePkg/ValleyView2Soc/SouthCluster/Include/PchAccess.h

  Log Message:
  -----------
  Vlv2DeviceRefCodePkg/ValleyView2Soc: Remove the unused code

The code in _S3SUPPORT_ in PchAccess.h wasn't used in anywhere. Also,
The declaration of MCRX and MCR made gcc generate the error of
"multiple definition". Just remove the unused code to avoid any
potential issue.

Cc: David Wei <david....@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>


  Commit: 530006647a8ab6d833c0ee839f999fb0e60f58b4
      
https://github.com/tianocore/edk2/commit/530006647a8ab6d833c0ee839f999fb0e60f58b4
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c

  Log Message:
  -----------
  Vlv2TbltDevicePkg/PlatformSetupDxe: Remove the unused variables

Fix the following errors from gcc:

Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c: In function 
?PrepareSetupInformation?:
Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c:540:31: error: variable 
?EeState? set but not used [-Werror=unused-but-set-variable]
Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c:532:31: error: variable 
?SrcDataSize? set but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c: In function 
?UpdateAdditionalInformation?:
Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c:818:35: error: variable 
?Size? set but not used [-Werror=unused-but-set-variable]
Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c:817:36: error: variable 
?SystemManufacturer? set but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c: In function 
?JudgeHandleIsPCIDevice?:
Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c:1094:22: error: variable 
?DevicePath? set but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c: In function 
?GetChipsetSataPortSpeed?:
Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c:1470:31: error: variable 
?DwordReg? set but not used [-Werror=unused-but-set-variable]
Vlv2TbltDevicePkg/PlatformSetupDxe/SetupInfoRecords.c:1469:31: error: variable 
?Status? set but not used [-Werror=unused-but-set-variable]

Cc: David Wei <david....@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>


  Commit: 5c3bd3500f8a8503bde2620421a9054a9a41a3cb
      
https://github.com/tianocore/edk2/commit/5c3bd3500f8a8503bde2620421a9054a9a41a3cb
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/VlvPlatformInitDxe/IgdOpRegion.c

  Log Message:
  -----------
  Vlv2TbltDevicePkg/VlvPlatformInitDxe: Remove unused variables

Fix the following errors from gcc:

Vlv2TbltDevicePkg/VlvPlatformInitDxe/IgdOpRegion.c: In function 
?GetIntegratedIntelVbtPtr?:
Vlv2TbltDevicePkg/VlvPlatformInitDxe/IgdOpRegion.c:145:33: error: variable 
?VbtSize? set but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/VlvPlatformInitDxe/IgdOpRegion.c: In function 
?JudgeHandleIsPCIDevice?:
Vlv2TbltDevicePkg/VlvPlatformInitDxe/IgdOpRegion.c:418:22: error: variable 
?DevicePath? set but not used [-Werror=unused-but-set-variable]

Cc: David Wei <david....@intel.com>
Cc: "Wu, Hao A" <hao.a...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>
Reviewed-by: David Wei <david....@intel.com>


  Commit: 076d0d64aaec3fcb06e95f83f0041e4529927985
      
https://github.com/tianocore/edk2/commit/076d0d64aaec3fcb06e95f83f0041e4529927985
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/PlatformDxe/IchRegTable.c
    M Vlv2TbltDevicePkg/PlatformDxe/PciDevice.c
    M Vlv2TbltDevicePkg/PlatformDxe/Platform.c

  Log Message:
  -----------
  Vlv2TbltDevicePkg/PlatformDxe: Remove the unused variables

Fix the following errors from gcc:

Vlv2TbltDevicePkg/PlatformDxe/Platform.c: In function ?InitPciDevPME?:
Vlv2TbltDevicePkg/PlatformDxe/Platform.c:516:26: error: variable ?Status? set 
but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/PlatformDxe/Platform.c: In function ?InitThermalZone?:
Vlv2TbltDevicePkg/PlatformDxe/Platform.c:575:26: error: variable ?Status? set 
but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/PlatformDxe/IchRegTable.c: In function 
?InitializeSubsystemIds?:
Vlv2TbltDevicePkg/PlatformDxe/IchRegTable.c:111:10: error: variable 
?SubsystemAudioVidDid? set but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/PlatformDxe/PciDevice.c: In function ?InitBadBars?:
Vlv2TbltDevicePkg/PlatformDxe/PciDevice.c:115:40: error: variable ?PciIoDevice? 
set but not used [-Werror=unused-but-set-variable]
Vlv2TbltDevicePkg/PlatformDxe/PciDevice.c:114:39: error: variable ?Status? set 
but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/PlatformDxe/PciDevice.c: In function ?ProgramPciLatency?:
Vlv2TbltDevicePkg/PlatformDxe/PciDevice.c:320:39: error: variable ?Status? set 
but not used [-Werror=unused-but-set-variable]

Cc: David Wei <david....@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>
Reviewed-by: David Wei <david....@intel.com>


  Commit: 4e518f755552ee805e247b34e33fdd5edbd2857b
      
https://github.com/tianocore/edk2/commit/4e518f755552ee805e247b34e33fdd5edbd2857b
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/Wpce791/LpcDriver.c

  Log Message:
  -----------
  Vlv2TbltDevicePkg/Wpce791: Remove the unused variable

Fix the following error from gcc:

Vlv2TbltDevicePkg/Wpce791/LpcDriver.c: In function ?LpcDriverStart?:
Vlv2TbltDevicePkg/Wpce791/LpcDriver.c:239:27: error: variable ?LpcDev? set but 
not used [-Werror=unused-but-set-variable]

Cc: David Wei <david....@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>


  Commit: 135ea32a199b68b0edbdc8699efda5a90d0a41e0
      
https://github.com/tianocore/edk2/commit/135ea32a199b68b0edbdc8699efda5a90d0a41e0
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/SmBiosMiscDxe/MiscOemType0x90Function.c
    M Vlv2TbltDevicePkg/SmBiosMiscDxe/MiscProcessorCacheFunction.c
    M Vlv2TbltDevicePkg/SmBiosMiscDxe/MiscProcessorInformationFunction.c

  Log Message:
  -----------
  Vlv2TbltDevicePkg/SmBiosMiscDxe: Remove unused variables

Fix the following errors from gcc:

Vlv2TbltDevicePkg/SmBiosMiscDxe/MiscOemType0x90Function.c: In function 
?JudgeHandleIsPCIDevice?:
Vlv2TbltDevicePkg/SmBiosMiscDxe/MiscOemType0x90Function.c:164:22: error: 
variable ?DevicePath? set but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/SmBiosMiscDxe/MiscOemType0x90Function.c: In function 
?AddSmbiosT0x90Callback?:
Vlv2TbltDevicePkg/SmBiosMiscDxe/MiscOemType0x90Function.c:292:28: error: 
variable ?ForType90InputData? set but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/SmBiosMiscDxe/MiscProcessorInformationFunction.c: In function 
?MiscProcessorInformationFunction?:
Vlv2TbltDevicePkg/SmBiosMiscDxe/MiscProcessorInformationFunction.c:191:37: 
error: variable ?SrcDataSize? set but not used [-Werror=unused-but-set-variable]

Vlv2TbltDevicePkg/SmBiosMiscDxe/MiscProcessorCacheFunction.c: In function 
?MiscProcessorCacheFunction?:
Vlv2TbltDevicePkg/SmBiosMiscDxe/MiscProcessorCacheFunction.c:78:34: error: 
variable ?SrcDataSize? set but not used [-Werror=unused-but-set-variable]

Cc: David Wei <david....@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>
Reviewed-by: David Wei <david....@intel.com>


  Commit: 55ef35d21eea88a0785cfec47df5059945fb0f0c
      
https://github.com/tianocore/edk2/commit/55ef35d21eea88a0785cfec47df5059945fb0f0c
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c
    M Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.h

  Log Message:
  -----------
  Vlv2TbltDevicePkg/PpmPolicy: Remove the usage of global variables

gcc issued the error of "multiple deifintion" since gBS was also defined
in MdePkg/Library/UefiBootServicesTableLib. Actually those global variables,
gBS, pBS, and pRS, in PpmPolicy.h were only used in one function. Besides,
gBS and pRS were not really used. Remove gBS and pRS and declare pBS in
PpmPolicyEntry() to satisfy gcc.

Cc: David Wei <david....@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>
Reviewed-by: David Wei <david....@intel.com>


  Commit: 4884e81b5d0da1bcc05361dd4c5fc06b3722f144
      
https://github.com/tianocore/edk2/commit/4884e81b5d0da1bcc05361dd4c5fc06b3722f144
  Author: Gary Lin <g...@suse.com>
  Date:   2016-08-03 (Wed, 03 Aug 2016)

  Changed paths:
    M Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c

  Log Message:
  -----------
  Vlv2TbltDevicePkg/PpmPolicy: Remove the unused variable

Fix the following error from gcc:

Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c: In function ?PpmPolicyEntry?:
Vlv2TbltDevicePkg/PpmPolicy/PpmPolicy.c:50:29: error: variable ?MaxRatio? set 
but not used [-Werror=unused-but-set-variable]

Cc: David Wei <david....@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Gary Lin <g...@suse.com>
Reviewed-by: David Wei <david....@intel.com>


Compare: https://github.com/tianocore/edk2/compare/06eb35402e45...4884e81b5d0d
------------------------------------------------------------------------------
_______________________________________________
edk2-commits mailing list
edk2-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to