<div dir="ltr">Depending on the order of tests, this also can cause some tests to fail, if they verify (explicitly or implicitly) presence of 1024x768.<div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 13, 2017 at 11:17 AM, Frediano Ziglio <span dir="ltr"><<a href="mailto:fziglio@redhat.com" target="_blank">fziglio@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">><br>
> When the video mode is changed and then the driver disabled and<br>
> enabled, it did not enumerate available video modes with lower<br>
> resolution than current one. All modes starting from 1024x768<br>
> should be available regardless what is current resolution<br>
> on driver startup<br>
><br>
> Signed-off-by: Yuri Benditovich <<a href="mailto:yuri.benditovich@daynix.com">yuri.benditovich@daynix.com</a>><br>
<br>
</span>Looks like this patch is a bit OT in this series<br>
and can be applied even independently without<br>
waiting for other patches.<br>
<br>
Acked-by: Frediano Ziglio <<a href="mailto:fziglio@redhat.com">fziglio@redhat.com</a>><br>
<div class="HOEnZb"><div class="h5"><br>
> ---<br>
>  qxldod/QxlDod.cpp | 8 ++++----<br>
>  1 file changed, 4 insertions(+), 4 deletions(-)<br>
><br>
> diff --git a/qxldod/QxlDod.cpp b/qxldod/QxlDod.cpp<br>
> index 78d4340..b3e0938 100755<br>
> --- a/qxldod/QxlDod.cpp<br>
> +++ b/qxldod/QxlDod.cpp<br>
> @@ -2512,8 +2512,8 @@ NTSTATUS<br>
> VgaDevice::GetModeList(DXGK_<wbr>DISPLAY_INFORMATION* pDispInfo)<br>
><br>
>          DbgPrint(TRACE_LEVEL_<wbr>INFORMATION, ("ModeTemp = 0x%X %dx%d@%d\n",<br>
>          ModeTemp, tmpModeInfo.XResolution, tmpModeInfo.YResolution,<br>
>          tmpModeInfo.BitsPerPixel));<br>
><br>
> -        if (tmpModeInfo.XResolution >= Width &&<br>
> -            tmpModeInfo.YResolution >= Height &&<br>
> +        if (tmpModeInfo.XResolution >= MIN_WIDTH_SIZE &&<br>
> +            tmpModeInfo.YResolution >= MIN_HEIGHT_SIZE &&<br>
>              tmpModeInfo.BitsPerPixel == BitsPerPixel &&<br>
>              tmpModeInfo.PhysBasePtr != 0)<br>
>          {<br>
> @@ -3138,8 +3138,8 @@ NTSTATUS<br>
> QxlDevice::GetModeList(DXGK_<wbr>DISPLAY_INFORMATION* pDispInfo)<br>
><br>
>          DbgPrint(TRACE_LEVEL_<wbr>INFORMATION, ("%s: modes[%d] x_res = %d, y_res<br>
>          = %d, bits = %d BitsPerPixel = %d\n", __FUNCTION__, CurrentMode,<br>
>          tmpModeInfo->x_res, tmpModeInfo->y_res, tmpModeInfo->bits,<br>
>          BitsPerPixel));<br>
><br>
> -        if (tmpModeInfo->x_res >= Width &&<br>
> -            tmpModeInfo->y_res >= Height &&<br>
> +        if (tmpModeInfo->x_res >= MIN_WIDTH_SIZE &&<br>
> +            tmpModeInfo->y_res >= MIN_HEIGHT_SIZE &&<br>
>              tmpModeInfo->bits == QXL_BPP)<br>
>          {<br>
>              m_ModeNumbers[<wbr>SuitableModeCount] = SuitableModeCount;<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">Frediano<br>
</font></span></blockquote></div><br></div>