[Spice-devel] [PATCH v2] Add support for resolution 800x600

[Frediano Ziglio]

> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1477492
> https://docs.microsoft.com/en-us/windows-hardware/design/minimum/minimum-hardware-requirements-overview
> requires 800x600 to be supported.
> 
> Signed-off-by: Yuri Benditovich <yuri.benditovich at daynix.com>

Was getting lost in the ML,

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

> ---
>  qxldod/QxlDod.cpp | 16 ++++++++--------
>  qxldod/QxlDod.h   |  6 ++++--
>  2 files changed, 12 insertions(+), 10 deletions(-)
> 
> diff --git a/qxldod/QxlDod.cpp b/qxldod/QxlDod.cpp
> index 4f508bd..f74f54e 100755
> --- a/qxldod/QxlDod.cpp
> +++ b/qxldod/QxlDod.cpp
> @@ -2579,8 +2579,8 @@ NTSTATUS
> VgaDevice::GetModeList(DXGK_DISPLAY_INFORMATION* pDispInfo)
>          {
>              m_ModeNumbers[SuitableModeCount] = ModeTemp;
>              SetVideoModeInfo(SuitableModeCount, &tmpModeInfo);
> -            if (tmpModeInfo.XResolution == MIN_WIDTH_SIZE &&
> -                tmpModeInfo.YResolution == MIN_HEIGHT_SIZE)
> +            if (tmpModeInfo.XResolution == INITIAL_WIDTH &&
> +                tmpModeInfo.YResolution == INITIAL_HEIGHT)
>              {
>                  m_CurrentMode = (USHORT)SuitableModeCount;
>              }
> @@ -3186,8 +3186,8 @@ NTSTATUS
> QxlDevice::GetModeList(DXGK_DISPLAY_INFORMATION* pDispInfo)
>      UINT BitsPerPixel = BPPFromPixelFormat(pDispInfo->ColorFormat);
>      if (Width == 0 || Height == 0 || BitsPerPixel != QXL_BPP)
>      {
> -        Width = MIN_WIDTH_SIZE;
> -        Height = MIN_HEIGHT_SIZE;
> +        Width = INITIAL_WIDTH;
> +        Height = INITIAL_HEIGHT;
>          BitsPerPixel = QXL_BPP;
>      }
>  
> @@ -3206,8 +3206,8 @@ NTSTATUS
> QxlDevice::GetModeList(DXGK_DISPLAY_INFORMATION* pDispInfo)
>          {
>              m_ModeNumbers[SuitableModeCount] = SuitableModeCount;
>              SetVideoModeInfo(SuitableModeCount, tmpModeInfo);
> -            if (tmpModeInfo->x_res == MIN_WIDTH_SIZE &&
> -                tmpModeInfo->y_res == MIN_HEIGHT_SIZE)
> +            if (tmpModeInfo->x_res == INITIAL_WIDTH &&
> +                tmpModeInfo->y_res == INITIAL_HEIGHT)
>              {
>                  m_CurrentMode = SuitableModeCount;
>              }
> @@ -5147,8 +5147,8 @@ NTSTATUS
> HwDeviceInterface::AcquireDisplayInfo(DXGK_DISPLAY_INFORMATION& DispInf
>      if (DispInfo.Width == 0)
>      {
>          DispInfo.ColorFormat = D3DDDIFMT_A8R8G8B8;
> -        DispInfo.Width = MIN_WIDTH_SIZE;
> -        DispInfo.Height = MIN_HEIGHT_SIZE;
> +        DispInfo.Width = INITIAL_WIDTH;
> +        DispInfo.Height = INITIAL_HEIGHT;
>          DispInfo.Pitch = DispInfo.Width *
>          BPPFromPixelFormat(DispInfo.ColorFormat) / BITS_PER_BYTE;
>          DispInfo.TargetId = 0;
>      }
> diff --git a/qxldod/QxlDod.h b/qxldod/QxlDod.h
> index 695b83a..eb6b78d 100755
> --- a/qxldod/QxlDod.h
> +++ b/qxldod/QxlDod.h
> @@ -19,8 +19,10 @@
>  #define BITS_PER_BYTE              8
>  
>  #define POINTER_SIZE               64
> -#define MIN_WIDTH_SIZE             1024
> -#define MIN_HEIGHT_SIZE            768
> +#define MIN_WIDTH_SIZE             800
> +#define MIN_HEIGHT_SIZE            600
> +#define INITIAL_WIDTH              1024
> +#define INITIAL_HEIGHT             768
>  #define QXL_BPP                    32
>  #define VGA_BPP                    24
>  

Frediano