Re: [PATCH qxl-wddm-dod v2 16/25] Remove minimum size restrict for custom resolution.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Wed, Sep 7, 2016 at 4:57 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
On Tue, Sep 06, 2016 at 05:29:22PM +0300, Sameeh Jubran wrote:
> It is the recommended minimum, however Windows can handle low resolution
> just fine. We have tested that,
> even the old driver behaves like this. I see no reason for making any
> restriction at all.

Can you please add this kind of reasoning/explanation to the very terse
commit log which is there currently ?

Sure 
Thanks,

Christophe
This patch is just changing the "Add arbitrary resolution and monitors_config Escape" patch
which is not in master, so would be better to squash there.
Also the "Add arbitrary resolution and monitors_config Escape" already has a rationale for
this patch that is:

"Windows can handle low resolution just fine, the old (XPDM)
  driver behaves like this too. There is no reason for making
  any restriction at all."

(actually this rationale apply the the wrong patch!)

Frediano

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]