All of lore.kernel.org
 help / color / mirror / Atom feed
* [RESEND] fence: add missing descriptions for fence
@ 2016-04-11 11:48 ` Luis de Bethencourt
  0 siblings, 0 replies; 8+ messages in thread
From: Luis de Bethencourt @ 2016-04-11 11:48 UTC (permalink / raw)
  To: linux-kernel
  Cc: linaro-mm-sig, dri-devel, linux-media, sumit.semwal, Luis de Bethencourt

The members child_list and active_list were added to the fence struct
without descriptions for the Documentation. Adding these.

Fixes: b55b54b5db33 ("staging/android: remove struct sync_pt")
Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
---
Hi,

Just resending this patch since it hasn't had any reviews in since
March 21st.

Thanks,
Luis

 include/linux/fence.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/include/linux/fence.h b/include/linux/fence.h
index 2b17698..2056e9f 100644
--- a/include/linux/fence.h
+++ b/include/linux/fence.h
@@ -49,6 +49,8 @@ struct fence_cb;
  * @timestamp: Timestamp when the fence was signaled.
  * @status: Optional, only valid if < 0, must be set before calling
  * fence_signal, indicates that the fence has completed with an error.
+ * @child_list: list of children fences
+ * @active_list: list of active fences
  *
  * the flags member must be manipulated and read using the appropriate
  * atomic ops (bit_*), so taking the spinlock will not be needed most
-- 
2.6.4

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [RESEND] fence: add missing descriptions for fence
@ 2016-04-11 11:48 ` Luis de Bethencourt
  0 siblings, 0 replies; 8+ messages in thread
From: Luis de Bethencourt @ 2016-04-11 11:48 UTC (permalink / raw)
  To: linux-kernel; +Cc: linaro-mm-sig, Luis de Bethencourt, dri-devel, linux-media

The members child_list and active_list were added to the fence struct
without descriptions for the Documentation. Adding these.

Fixes: b55b54b5db33 ("staging/android: remove struct sync_pt")
Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
---
Hi,

Just resending this patch since it hasn't had any reviews in since
March 21st.

Thanks,
Luis

 include/linux/fence.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/include/linux/fence.h b/include/linux/fence.h
index 2b17698..2056e9f 100644
--- a/include/linux/fence.h
+++ b/include/linux/fence.h
@@ -49,6 +49,8 @@ struct fence_cb;
  * @timestamp: Timestamp when the fence was signaled.
  * @status: Optional, only valid if < 0, must be set before calling
  * fence_signal, indicates that the fence has completed with an error.
+ * @child_list: list of children fences
+ * @active_list: list of active fences
  *
  * the flags member must be manipulated and read using the appropriate
  * atomic ops (bit_*), so taking the spinlock will not be needed most
-- 
2.6.4

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [RESEND] fence: add missing descriptions for fence
  2016-04-11 11:48 ` Luis de Bethencourt
@ 2016-04-11 20:09   ` Gustavo Padovan
  -1 siblings, 0 replies; 8+ messages in thread
From: Gustavo Padovan @ 2016-04-11 20:09 UTC (permalink / raw)
  To: Luis de Bethencourt; +Cc: linux-kernel, linaro-mm-sig, dri-devel, linux-media

Hi Luis,

2016-04-11 Luis de Bethencourt <luisbg@osg.samsung.com>:

> The members child_list and active_list were added to the fence struct
> without descriptions for the Documentation. Adding these.
> 
> Fixes: b55b54b5db33 ("staging/android: remove struct sync_pt")
> Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
> Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
> ---
> Hi,
> 
> Just resending this patch since it hasn't had any reviews in since
> March 21st.
> 
> Thanks,
> Luis
> 
>  include/linux/fence.h | 2 ++
>  1 file changed, 2 insertions(+)

Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>

	Gustavo

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [RESEND] fence: add missing descriptions for fence
@ 2016-04-11 20:09   ` Gustavo Padovan
  0 siblings, 0 replies; 8+ messages in thread
From: Gustavo Padovan @ 2016-04-11 20:09 UTC (permalink / raw)
  To: Luis de Bethencourt; +Cc: linaro-mm-sig, linux-kernel, dri-devel, linux-media

Hi Luis,

2016-04-11 Luis de Bethencourt <luisbg@osg.samsung.com>:

> The members child_list and active_list were added to the fence struct
> without descriptions for the Documentation. Adding these.
> 
> Fixes: b55b54b5db33 ("staging/android: remove struct sync_pt")
> Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
> Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
> ---
> Hi,
> 
> Just resending this patch since it hasn't had any reviews in since
> March 21st.
> 
> Thanks,
> Luis
> 
>  include/linux/fence.h | 2 ++
>  1 file changed, 2 insertions(+)

Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>

	Gustavo
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [RESEND] fence: add missing descriptions for fence
  2016-04-11 20:09   ` Gustavo Padovan
@ 2016-04-11 22:33     ` Luis de Bethencourt
  -1 siblings, 0 replies; 8+ messages in thread
From: Luis de Bethencourt @ 2016-04-11 22:33 UTC (permalink / raw)
  To: Gustavo Padovan, linux-kernel, linaro-mm-sig, dri-devel, linux-media

On 11/04/16 21:09, Gustavo Padovan wrote:
> Hi Luis,
> 
> 2016-04-11 Luis de Bethencourt <luisbg@osg.samsung.com>:
> 
>> The members child_list and active_list were added to the fence struct
>> without descriptions for the Documentation. Adding these.
>>
>> Fixes: b55b54b5db33 ("staging/android: remove struct sync_pt")
>> Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
>> Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
>> ---
>> Hi,
>>
>> Just resending this patch since it hasn't had any reviews in since
>> March 21st.
>>
>> Thanks,
>> Luis
>>
>>  include/linux/fence.h | 2 ++
>>  1 file changed, 2 insertions(+)
> 
> Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
> 
> 	Gustavo
> 

Thank you Gustavo.

Nice seeing you around here :)

Luis

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [RESEND] fence: add missing descriptions for fence
@ 2016-04-11 22:33     ` Luis de Bethencourt
  0 siblings, 0 replies; 8+ messages in thread
From: Luis de Bethencourt @ 2016-04-11 22:33 UTC (permalink / raw)
  To: Gustavo Padovan, linux-kernel, linaro-mm-sig, dri-devel, linux-media

On 11/04/16 21:09, Gustavo Padovan wrote:
> Hi Luis,
> 
> 2016-04-11 Luis de Bethencourt <luisbg@osg.samsung.com>:
> 
>> The members child_list and active_list were added to the fence struct
>> without descriptions for the Documentation. Adding these.
>>
>> Fixes: b55b54b5db33 ("staging/android: remove struct sync_pt")
>> Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
>> Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
>> ---
>> Hi,
>>
>> Just resending this patch since it hasn't had any reviews in since
>> March 21st.
>>
>> Thanks,
>> Luis
>>
>>  include/linux/fence.h | 2 ++
>>  1 file changed, 2 insertions(+)
> 
> Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
> 
> 	Gustavo
> 

Thank you Gustavo.

Nice seeing you around here :)

Luis
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [RESEND] fence: add missing descriptions for fence
  2016-04-11 22:33     ` Luis de Bethencourt
@ 2016-04-12  5:15       ` Sumit Semwal
  -1 siblings, 0 replies; 8+ messages in thread
From: Sumit Semwal @ 2016-04-12  5:15 UTC (permalink / raw)
  To: Luis de Bethencourt
  Cc: Gustavo Padovan, LKML, Linaro MM SIG Mailman List,
	DRI mailing list, linux-media

Hi Luis,


On 12 April 2016 at 04:03, Luis de Bethencourt <luisbg@osg.samsung.com> wrote:
> On 11/04/16 21:09, Gustavo Padovan wrote:
>> Hi Luis,
>>
>> 2016-04-11 Luis de Bethencourt <luisbg@osg.samsung.com>:
>>
>>> The members child_list and active_list were added to the fence struct
>>> without descriptions for the Documentation. Adding these.
>>>
Thanks for the patch; will get it queued for for-next.

>>> Fixes: b55b54b5db33 ("staging/android: remove struct sync_pt")
>>> Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
>>> Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
>>> ---
>>> Hi,
>>>
>>> Just resending this patch since it hasn't had any reviews in since
>>> March 21st.
>>>
>>> Thanks,
>>> Luis
>>>
>>>  include/linux/fence.h | 2 ++
>>>  1 file changed, 2 insertions(+)
>>
>> Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
>>
>>       Gustavo
>>
>
> Thank you Gustavo.
>
> Nice seeing you around here :)
>
> Luis

BR,
Sumit.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [RESEND] fence: add missing descriptions for fence
@ 2016-04-12  5:15       ` Sumit Semwal
  0 siblings, 0 replies; 8+ messages in thread
From: Sumit Semwal @ 2016-04-12  5:15 UTC (permalink / raw)
  To: Luis de Bethencourt
  Cc: Linaro MM SIG Mailman List, LKML, DRI mailing list, linux-media

Hi Luis,


On 12 April 2016 at 04:03, Luis de Bethencourt <luisbg@osg.samsung.com> wrote:
> On 11/04/16 21:09, Gustavo Padovan wrote:
>> Hi Luis,
>>
>> 2016-04-11 Luis de Bethencourt <luisbg@osg.samsung.com>:
>>
>>> The members child_list and active_list were added to the fence struct
>>> without descriptions for the Documentation. Adding these.
>>>
Thanks for the patch; will get it queued for for-next.

>>> Fixes: b55b54b5db33 ("staging/android: remove struct sync_pt")
>>> Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
>>> Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
>>> ---
>>> Hi,
>>>
>>> Just resending this patch since it hasn't had any reviews in since
>>> March 21st.
>>>
>>> Thanks,
>>> Luis
>>>
>>>  include/linux/fence.h | 2 ++
>>>  1 file changed, 2 insertions(+)
>>
>> Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
>>
>>       Gustavo
>>
>
> Thank you Gustavo.
>
> Nice seeing you around here :)
>
> Luis

BR,
Sumit.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2016-04-12  5:16 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-11 11:48 [RESEND] fence: add missing descriptions for fence Luis de Bethencourt
2016-04-11 11:48 ` Luis de Bethencourt
2016-04-11 20:09 ` Gustavo Padovan
2016-04-11 20:09   ` Gustavo Padovan
2016-04-11 22:33   ` Luis de Bethencourt
2016-04-11 22:33     ` Luis de Bethencourt
2016-04-12  5:15     ` Sumit Semwal
2016-04-12  5:15       ` Sumit Semwal

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.