[PATCH] android: Support creating sync fence from drm fences

Alexandre Courbot acourbot at nvidia.com
Thu Apr 30 09:19:02 UTC 2015


On 04/30/2015 06:11 PM, Dan Carpenter wrote:
> On Tue, Apr 28, 2015 at 04:58:34PM +0900, Alexandre Courbot wrote:
>> Gentle ping for feedback - some of our future work depends on this
>> and we would like to know whether this has a chance of being
>> accepted?
>
> Arve wasn't CC'd on the first/usable version of this patch.  He or Colin
> need to Ack android patches.  Please resend.

Sorry about that, looks like we neglected running get_maintainers on the 
patch...

>
> Send other future patches along as well that depend on this one.  Those
> are likely more interesting anyway.

Indeed.

>
>>> -/* TODO: implement a create which takes more that one sync_pt */
>>> -struct sync_fence *sync_fence_create(const char *name, struct sync_pt *pt)
>>> +/* TODO: implement a create which takes more that one pt */
>>> +struct sync_fence *sync_fence_create(const char *name, struct fence *pt)
>
> "pt" stands for "point" as in "sync point", but now the variable is not
> a sync_pt struct so the name doesn't make sense.

Will fix that in the respin.

Thanks!
Alex.



More information about the devel mailing list