From 110d3a9ac5fc482d15f286ca1588ca87be5000ab Mon Sep 17 00:00:00 2001 From: jljusten Date: Thu, 9 Jul 2009 00:04:41 +0000 Subject: [PATCH] Use PixelFormat PixelBitMask in CirrusLogic5430Dxe driver. Modify CirrusLogic5430Dxe to return mode info with PixelFormat set to PixelBitMask. This enables the Linux framebuffer code to function with the UEFI GOP framebuffer. Previously the code would return PixelBltOnly for the PixelFormat, and thus would not provide any framebuffer information. git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@8825 6f19259b-4bc3-4df7-8a09-765794883524 --- .../CirrusLogic5430Dxe/CirrusLogic5430.h | 18 +++++ .../CirrusLogic5430GraphicsOutput.c | 76 +++++++++++++++---- 2 files changed, 79 insertions(+), 15 deletions(-) diff --git a/OptionRomPkg/CirrusLogic5430Dxe/CirrusLogic5430.h b/OptionRomPkg/CirrusLogic5430Dxe/CirrusLogic5430.h index 93abd523d2..c426cb2ab7 100644 --- a/OptionRomPkg/CirrusLogic5430Dxe/CirrusLogic5430.h +++ b/OptionRomPkg/CirrusLogic5430Dxe/CirrusLogic5430.h @@ -62,6 +62,24 @@ typedef struct { UINT32 RefreshRate; } CIRRUS_LOGIC_5430_MODE_DATA; +#define PIXEL_RED_SHIFT 0 +#define PIXEL_GREEN_SHIFT 3 +#define PIXEL_BLUE_SHIFT 6 + +#define PIXEL_RED_MASK (BIT7 | BIT6 | BIT5) +#define PIXEL_GREEN_MASK (BIT4 | BIT3 | BIT2) +#define PIXEL_BLUE_MASK (BIT1 | BIT0) + +#define PIXEL_TO_COLOR_BYTE(pixel, mask, shift) ((UINT8) ((pixel & mask) << shift)) +#define PIXEL_TO_RED_BYTE(pixel) PIXEL_TO_COLOR_BYTE(pixel, PIXEL_RED_MASK, PIXEL_RED_SHIFT) +#define PIXEL_TO_GREEN_BYTE(pixel) PIXEL_TO_COLOR_BYTE(pixel, PIXEL_GREEN_MASK, PIXEL_GREEN_SHIFT) +#define PIXEL_TO_BLUE_BYTE(pixel) PIXEL_TO_COLOR_BYTE(pixel, PIXEL_BLUE_MASK, PIXEL_BLUE_SHIFT) + +#define RGB_BYTES_TO_PIXEL(Red, Green, Blue) \ + (UINT8) ( (((Red) >> PIXEL_RED_SHIFT) & PIXEL_RED_MASK) | \ + (((Green) >> PIXEL_GREEN_SHIFT) & PIXEL_GREEN_MASK) | \ + (((Blue) >> PIXEL_BLUE_SHIFT) & PIXEL_BLUE_MASK) ) + #define GRAPHICS_OUTPUT_INVALIDE_MODE_NUMBER 0xffff // diff --git a/OptionRomPkg/CirrusLogic5430Dxe/CirrusLogic5430GraphicsOutput.c b/OptionRomPkg/CirrusLogic5430Dxe/CirrusLogic5430GraphicsOutput.c index fceb419198..86535b7964 100644 --- a/OptionRomPkg/CirrusLogic5430Dxe/CirrusLogic5430GraphicsOutput.c +++ b/OptionRomPkg/CirrusLogic5430Dxe/CirrusLogic5430GraphicsOutput.c @@ -20,6 +20,51 @@ Abstract: **/ #include "CirrusLogic5430.h" +#include + + +STATIC +VOID +CirrusLogic5430CompleteModeInfo ( + OUT EFI_GRAPHICS_OUTPUT_MODE_INFORMATION *Info + ) +{ + Info->Version = 0; + Info->PixelFormat = PixelBitMask; + Info->PixelInformation.RedMask = PIXEL_RED_MASK; + Info->PixelInformation.GreenMask = PIXEL_GREEN_MASK; + Info->PixelInformation.BlueMask = PIXEL_BLUE_MASK; + Info->PixelInformation.ReservedMask = 0; + Info->PixelsPerScanLine = Info->HorizontalResolution; +} + + +STATIC +EFI_STATUS +CirrusLogic5430CompleteModeData ( + IN CIRRUS_LOGIC_5430_PRIVATE_DATA *Private, + OUT EFI_GRAPHICS_OUTPUT_PROTOCOL_MODE *Mode + ) +{ + EFI_GRAPHICS_OUTPUT_MODE_INFORMATION *Info; + EFI_ACPI_ADDRESS_SPACE_DESCRIPTOR *FrameBufDesc; + + Info = Mode->Info; + CirrusLogic5430CompleteModeInfo (Info); + + Private->PciIo->GetBarAttributes ( + Private->PciIo, + 0, + NULL, + (VOID**) &FrameBufDesc + ); + + Mode->FrameBufferBase = FrameBufDesc->AddrRangeMin; + Mode->FrameBufferSize = Info->HorizontalResolution * Info->VerticalResolution; + + return EFI_SUCCESS; +} + // // Graphics Output Protocol Member Functions @@ -72,11 +117,9 @@ Routine Description: *SizeOfInfo = sizeof (EFI_GRAPHICS_OUTPUT_MODE_INFORMATION); - (*Info)->Version = 0; (*Info)->HorizontalResolution = Private->ModeData[ModeNumber].HorizontalResolution; (*Info)->VerticalResolution = Private->ModeData[ModeNumber].VerticalResolution; - (*Info)->PixelFormat = PixelBltOnly; - (*Info)->PixelsPerScanLine = (*Info)->HorizontalResolution; + CirrusLogic5430CompleteModeInfo (*Info); return EFI_SUCCESS; } @@ -104,8 +147,8 @@ Routine Description: --*/ { - CIRRUS_LOGIC_5430_PRIVATE_DATA *Private; - CIRRUS_LOGIC_5430_MODE_DATA *ModeData; + CIRRUS_LOGIC_5430_PRIVATE_DATA *Private; + CIRRUS_LOGIC_5430_MODE_DATA *ModeData; Private = CIRRUS_LOGIC_5430_PRIVATE_DATA_FROM_GRAPHICS_OUTPUT_THIS (This); @@ -130,12 +173,9 @@ Routine Description: This->Mode->Mode = ModeNumber; This->Mode->Info->HorizontalResolution = ModeData->HorizontalResolution; This->Mode->Info->VerticalResolution = ModeData->VerticalResolution; - This->Mode->Info->PixelFormat = PixelBltOnly; - This->Mode->Info->PixelsPerScanLine = ModeData->HorizontalResolution; This->Mode->SizeOfInfo = sizeof(EFI_GRAPHICS_OUTPUT_MODE_INFORMATION); - This->Mode->FrameBufferBase = (EFI_PHYSICAL_ADDRESS)(UINTN)NULL; - This->Mode->FrameBufferSize = 0; + CirrusLogic5430CompleteModeData (Private, This->Mode); Private->HardwareNeedsStarting = FALSE; @@ -289,9 +329,9 @@ Returns: for (X = 0; X < Width; X++) { Blt = (EFI_GRAPHICS_OUTPUT_BLT_PIXEL *) ((UINT8 *) BltBuffer + (DstY * Delta) + (DestinationX + X) * sizeof (EFI_GRAPHICS_OUTPUT_BLT_PIXEL)); - Blt->Red = (UINT8) (Private->LineBuffer[X] & 0xe0); - Blt->Green = (UINT8) ((Private->LineBuffer[X] & 0x1c) << 3); - Blt->Blue = (UINT8) ((Private->LineBuffer[X] & 0x03) << 6); + Blt->Red = PIXEL_TO_RED_BYTE (Private->LineBuffer[X]); + Blt->Green = PIXEL_TO_GREEN_BYTE (Private->LineBuffer[X]); + Blt->Blue = PIXEL_TO_BLUE_BYTE (Private->LineBuffer[X]); } } break; @@ -344,7 +384,7 @@ Returns: case EfiBltVideoFill: Blt = BltBuffer; - Pixel = (UINT8) ((Blt->Red & 0xe0) | ((Blt->Green >> 3) & 0x1c) | ((Blt->Blue >> 6) & 0x03)); + Pixel = RGB_BYTES_TO_PIXEL (Blt->Red, Blt->Green, Blt->Blue); WidePixel = (Pixel << 8) | Pixel; WidePixel = (WidePixel << 16) | WidePixel; @@ -399,8 +439,14 @@ Returns: for (SrcY = SourceY, DstY = DestinationY; SrcY < (Height + SourceY); SrcY++, DstY++) { for (X = 0; X < Width; X++) { - Blt = (EFI_GRAPHICS_OUTPUT_BLT_PIXEL *) ((UINT8 *) BltBuffer + (SrcY * Delta) + (SourceX + X) * sizeof (EFI_GRAPHICS_OUTPUT_BLT_PIXEL)); - Private->LineBuffer[X] = (UINT8) ((Blt->Red & 0xe0) | ((Blt->Green >> 3) & 0x1c) | ((Blt->Blue >> 6) & 0x03)); + Blt = + (EFI_GRAPHICS_OUTPUT_BLT_PIXEL *) ( + (UINT8 *) BltBuffer + + (SrcY * Delta) + + ((SourceX + X) * sizeof (EFI_GRAPHICS_OUTPUT_BLT_PIXEL)) + ); + Private->LineBuffer[X] = + RGB_BYTES_TO_PIXEL (Blt->Red, Blt->Green, Blt->Blue); } Offset = (DstY * Private->ModeData[CurrentMode].HorizontalResolution) + DestinationX; -- 2.39.2