On 2017å¹´09æ??29æ?¥ 20:37, Marek Olšák wrote:
> On Fri, Sep 29, 2017 at 4:12 AM, Chunming Zhou <zhoucm1 at amd.com> wrote:
>>
>> On 2017å¹´09æ??29æ?¥ 06:10, Marek Olšák wrote:
>>> From: Marek Olšák <marek.olsak at amd.com>
>>>
>>> ---
>>> include/drm/drm.h | 24 ++++++++++++++++++++++++
>>> xf86drm.c | 22 ++++++++++++++++++++++
>>> xf86drm.h | 3 +++
>>> 3 files changed, 49 insertions(+)
>>>
>>> diff --git a/include/drm/drm.h b/include/drm/drm.h
>>> index bf3674a..4da1667 100644
>>> --- a/include/drm/drm.h
>>> +++ b/include/drm/drm.h
>>> @@ -687,38 +687,57 @@ struct drm_prime_handle {
>>> /** Flags.. only applicable for handle->fd */
>>> __u32 flags;
>>> /** Returned dmabuf file descriptor */
>>> __s32 fd;
>>> };
>>> struct drm_syncobj_create {
>>> __u32 handle;
>>> +#define DRM_SYNCOBJ_CREATE_SIGNALED (1 << 0)
>>> __u32 flags;
>>> };
>>> struct drm_syncobj_destroy {
>>> __u32 handle;
>>> __u32 pad;
>>> };
>>> #define DRM_SYNCOBJ_FD_TO_HANDLE_FLAGS_IMPORT_SYNC_FILE (1 << 0)
>>> #define DRM_SYNCOBJ_HANDLE_TO_FD_FLAGS_EXPORT_SYNC_FILE (1 << 0)
>> Typo for '(1 << 1)' ?
> No, this was copied from kernel headers.
Yes, I double checked just now, that's two separate ioctls use them, so
they are safe.
Sorry for noise of this.
David Zhou
>
>> With that fixes, the set is Reviewed-by: Chunming Zhou <david1.zhou at amd.com>
> Thanks.
>
> Marek
