<html><body><div style="font-family: times new roman, new york, times, serif; font-size: 12pt; color: #000000"><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 7, 2016 at 4:57 PM, Christophe Fergeau <span dir="ltr"><<a href="mailto:cfergeau@redhat.com" target="_blank">cfergeau@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="">On Tue, Sep 06, 2016 at 05:29:22PM +0300, Sameeh Jubran wrote:<br>
> It is the recommended minimum, however Windows can handle low resolution<br>
> just fine. We have tested that,<br>
> even the old driver behaves like this. I see no reason for making any<br>
> restriction at all.<br>
<br>
</span>Can you please add this kind of reasoning/explanation to the very terse<br>
commit log which is there currently ?<br><br></blockquote><div>Sure </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks,<br><br>
Christophe<br></blockquote></div></div></div></blockquote><div>This patch is just changing the "Add arbitrary resolution and monitors_config Escape" patch<br></div><div>which is not in master, so would be better to squash there.</div><div>Also the "Add arbitrary resolution and monitors_config Escape" already has a rationale for<br></div><div>this patch that is:<br></div><div><br></div><div>"Windows can handle low resolution just fine, the old (XPDM)<br> driver behaves like this too. There is no reason for making<br> any restriction at all."</div><div><br></div><div>(actually this rationale apply the the wrong patch!)<br></div><div><br></div><div>Frediano</div><div><br></div></div></body></html>