<div dir="ltr"><div><div>I pushed this one.  Let's get a follow-up that lets weston actually use the bigger cursors.  It would also be good to hack together a little client that attaches a big cursor so we can verify that it's working.  I don't think we need to put it in the repo, I'd just like proof that we're actually taking advantage of our new-found big cursors.<br>
</div>Thanks,<br></div>--Jason Ekstrand<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jul 29, 2014 at 12:40 AM, Michel Dänzer <span dir="ltr"><<a href="mailto:michel@daenzer.net" target="_blank">michel@daenzer.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 29.07.2014 16:36, Jason Ekstrand wrote:<br>
> On Tue, Jul 29, 2014 at 12:17 AM, Michel Dänzer <<a href="mailto:michel@daenzer.net">michel@daenzer.net</a><br>
</div><div class="">> <mailto:<a href="mailto:michel@daenzer.net">michel@daenzer.net</a>>> wrote:<br>
><br>
>     On 29.07.2014 16:01, Jason Ekstrand wrote:<br>
>     > Couple thoughs.  First, we need to also update<br>
>     > drm_output_prepare_cursor_view to check against the size coming<br>
>     from GBM<br>
>     > instead of against the hard-coded 64x64 it's currently checking<br>
>     > against.  Without changing that, we are still restricted to 64x64<br>
>     > regardless of the GBM checking.<br>
><br>
>     You mean weston will still refuse to use the hardware cursor for images<br>
>     larger than 64x64 without that change? That sounds like something that<br>
>     should indeed be fixed, though it's not really critical compared to the<br>
>     problem fixed by this patch, which is that the hardware cursor appears<br>
>     corrupted beyond usability on hardware which only supports hardware<br>
>     cursors of sizes other than 64x64.<br>
><br>
><br>
> Yup.  That is exactly what it means.  It should be a fairly easy fix.<br>
> If you'd rather I push this and fix in a follow-up patch, that's<br>
> probably ok,<br>
<br>
</div>That would be great.<br>
<div class=""><br>
> but let's make sure one is coming.<br>
<br>
</div>Alvaro, do you want to send a follow-up patch addressing the issue Jason<br>
described above?<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
--<br>
Earthling Michel Dänzer            |                  <a href="http://www.amd.com" target="_blank">http://www.amd.com</a><br>
Libre software enthusiast          |                Mesa and X developer<br>
</div></div></blockquote></div><br></div>