[Spice-devel,v2,4/4] qxl-wddm-dod: Use 32 bpp for fallback mode

Submitted by Yuri Benditovich on Feb. 4, 2017, 4 p.m.

Details

Message ID 1486224023-155996-5-git-send-email-yuri.benditovich@daynix.com
State New
Headers show
Series "qxl-wddm-dod: fix for black screen after VGA driver enable" ( rev: 2 ) in Spice

Not browsing as part of any series.

Commit Message

Yuri Benditovich Feb. 4, 2017, 4 p.m.
The driver uses fallback mode when it does not receive
startup display information from the OS. This change
allows solution for https://bugzilla.redhat.com/show_bug.cgi?id=1417448
without fix in qemu for video mode switch from 32 bpp to 24 bpp.

Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
---
 qxldod/QxlDod.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch hide | download patch | download mbox

diff --git a/qxldod/QxlDod.cpp b/qxldod/QxlDod.cpp
index 37a4b34..978a6fc 100755
--- a/qxldod/QxlDod.cpp
+++ b/qxldod/QxlDod.cpp
@@ -4830,7 +4830,7 @@  NTSTATUS HwDeviceInterface::AcquireDisplayInfo(DXGK_DISPLAY_INFORMATION& DispInf
 
     if (DispInfo.Width == 0)
     {
-        DispInfo.ColorFormat = D3DDDIFMT_R8G8B8;
+        DispInfo.ColorFormat = D3DDDIFMT_A8R8G8B8;
         DispInfo.Width = MIN_WIDTH_SIZE;
         DispInfo.Height = MIN_HEIGHT_SIZE;
         DispInfo.Pitch = DispInfo.Width * BPPFromPixelFormat(DispInfo.ColorFormat) / BITS_PER_BYTE;

Comments

> 
> The driver uses fallback mode when it does not receive
> startup display information from the OS. This change
> allows solution for https://bugzilla.redhat.com/show_bug.cgi?id=1417448
> without fix in qemu for video mode switch from 32 bpp to 24 bpp.
> 
> Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
> ---
>  qxldod/QxlDod.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/qxldod/QxlDod.cpp b/qxldod/QxlDod.cpp
> index 37a4b34..978a6fc 100755
> --- a/qxldod/QxlDod.cpp
> +++ b/qxldod/QxlDod.cpp
> @@ -4830,7 +4830,7 @@ NTSTATUS
> HwDeviceInterface::AcquireDisplayInfo(DXGK_DISPLAY_INFORMATION& DispInf
>  
>      if (DispInfo.Width == 0)
>      {
> -        DispInfo.ColorFormat = D3DDDIFMT_R8G8B8;
> +        DispInfo.ColorFormat = D3DDDIFMT_A8R8G8B8;
>          DispInfo.Width = MIN_WIDTH_SIZE;
>          DispInfo.Height = MIN_HEIGHT_SIZE;
>          DispInfo.Pitch = DispInfo.Width *
>          BPPFromPixelFormat(DispInfo.ColorFormat) / BITS_PER_BYTE;

Acked-by: Frediano Ziglio <fziglio@redhat.com>

Frediano