tyler.durden
/
audk
mirror of https://github.com/acidanthera/audk.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

ArmPlatformPkg: Fix Ecc error 5007 in LcdGraphicsOutputDxe 

This patch fixes the following Ecc reported error:
There should be no initialization of a variable as
part of its declaration

Signed-off-by: Pierre Gondois <Pierre.Gondois@arm.com>
Reviewed-by: Ard Biesheuvel <ard.biesheuvel@arm.com>
This commit is contained in:
Pierre Gondois 2020-10-23 14:35:07 +01:00 committed by mergify[bot]
parent d315bd2286
commit eb97f13839
2 changed files with 12 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/LcdGraphicsOutputBlt.c
View File

@ -1,6 +1,6 @@
/** @file /** @file
Copyright (c) 2011-2013, ARM Ltd. All rights reserved.<BR> Copyright (c) 2011 - 2020, Arm Limited. All rights reserved.<BR>
SPDX-License-Identifier: BSD-2-Clause-Patent SPDX-License-Identifier: BSD-2-Clause-Patent
**/ **/
@ -36,7 +36,7 @@ VideoCopyNoHorizontalOverlap (
IN UINTN Height IN UINTN Height
) )
{ {
EFI_STATUS Status = EFI_SUCCESS; EFI_STATUS Status;
UINTN SourceLine; UINTN SourceLine;
UINTN DestinationLine; UINTN DestinationLine;
UINTN WidthInBytes; UINTN WidthInBytes;
@ -45,6 +45,8 @@ VideoCopyNoHorizontalOverlap (
VOID *SourceAddr; VOID *SourceAddr;
VOID *DestinationAddr; VOID *DestinationAddr;
Status = EFI_SUCCESS;
if( DestinationY <= SourceY ) { if( DestinationY <= SourceY ) {
// scrolling up (or horizontally but without overlap) // scrolling up (or horizontally but without overlap)
SourceLine = SourceY; SourceLine = SourceY;
@ -128,7 +130,7 @@ VideoCopyHorizontalOverlap (
IN UINTN Height IN UINTN Height
) )
{ {
EFI_STATUS Status = EFI_SUCCESS; EFI_STATUS Status;
UINT32 *PixelBuffer32bit; UINT32 *PixelBuffer32bit;
UINT32 *SourcePixel32bit; UINT32 *SourcePixel32bit;
@ -143,6 +145,8 @@ VideoCopyHorizontalOverlap (
UINTN SizeIn32Bits; UINTN SizeIn32Bits;
UINTN SizeIn16Bits; UINTN SizeIn16Bits;
Status = EFI_SUCCESS;
switch (BitsPerPixel) { switch (BitsPerPixel) {
case LCD_BITS_PER_PIXEL_24: case LCD_BITS_PER_PIXEL_24:

10
ArmPlatformPkg/Drivers/LcdGraphicsOutputDxe/LcdGraphicsOutputDxe.c
View File

@ -1,7 +1,7 @@
/** @file /** @file
This file implements the Graphics Output protocol for Arm platforms This file implements the Graphics Output protocol for Arm platforms
Copyright (c) 2011-2018, ARM Ltd. All rights reserved.<BR> Copyright (c) 2011 - 2020, Arm Limited. All rights reserved.<BR>
SPDX-License-Identifier: BSD-2-Clause-Patent SPDX-License-Identifier: BSD-2-Clause-Patent
**/ **/
@ -102,7 +102,7 @@ InitializeDisplay (
IN LCD_INSTANCE* Instance IN LCD_INSTANCE* Instance
) )
{ {
EFI_STATUS Status = EFI_SUCCESS; EFI_STATUS Status;
EFI_PHYSICAL_ADDRESS VramBaseAddress; EFI_PHYSICAL_ADDRESS VramBaseAddress;
UINTN VramSize; UINTN VramSize;
@ -148,7 +148,7 @@ LcdGraphicsOutputDxeInitialize (
IN EFI_SYSTEM_TABLE *SystemTable IN EFI_SYSTEM_TABLE *SystemTable
) )
{ {
EFI_STATUS Status = EFI_SUCCESS; EFI_STATUS Status;
LCD_INSTANCE* Instance; LCD_INSTANCE* Instance;
Status = LcdIdentify (); Status = LcdIdentify ();
@ -246,7 +246,7 @@ LcdGraphicsQueryMode (
OUT EFI_GRAPHICS_OUTPUT_MODE_INFORMATION **Info OUT EFI_GRAPHICS_OUTPUT_MODE_INFORMATION **Info
) )
{ {
EFI_STATUS Status = EFI_SUCCESS; EFI_STATUS Status;
LCD_INSTANCE *Instance; LCD_INSTANCE *Instance;
Instance = LCD_INSTANCE_FROM_GOP_THIS (This); Instance = LCD_INSTANCE_FROM_GOP_THIS (This);
@ -296,7 +296,7 @@ LcdGraphicsSetMode (
IN UINT32 ModeNumber IN UINT32 ModeNumber
) )
{ {
EFI_STATUS Status = EFI_SUCCESS; EFI_STATUS Status;
EFI_GRAPHICS_OUTPUT_BLT_PIXEL FillColour; EFI_GRAPHICS_OUTPUT_BLT_PIXEL FillColour;
LCD_INSTANCE* Instance; LCD_INSTANCE* Instance;
LCD_BPP Bpp; LCD_BPP Bpp;