Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1375748 > unrolled thread
| Started by | Luis de Bethencourt <luisbg@osg.samsung.com> |
|---|---|
| First post | 2016-04-11 13:50 +0200 |
| Last post | 2016-04-12 07:20 +0200 |
| Articles | 4 — 3 participants |
Back to article view | Back to linux.kernel
[RESEND] fence: add missing descriptions for fence Luis de Bethencourt <luisbg@osg.samsung.com> - 2016-04-11 13:50 +0200
Re: [RESEND] fence: add missing descriptions for fence Gustavo Padovan <gustavo@padovan.org> - 2016-04-11 22:10 +0200
Re: [RESEND] fence: add missing descriptions for fence Luis de Bethencourt <luisbg@osg.samsung.com> - 2016-04-12 00:40 +0200
Re: [RESEND] fence: add missing descriptions for fence Sumit Semwal <sumit.semwal@linaro.org> - 2016-04-12 07:20 +0200
| From | Luis de Bethencourt <luisbg@osg.samsung.com> |
|---|---|
| Date | 2016-04-11 13:50 +0200 |
| Subject | [RESEND] fence: add missing descriptions for fence |
| Message-ID | <rmIvV-5d9-31@gated-at.bofh.it> |
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
[toc] | [next] | [standalone]
| From | Gustavo Padovan <gustavo@padovan.org> |
|---|---|
| Date | 2016-04-11 22:10 +0200 |
| Message-ID | <rmQjM-3dx-9@gated-at.bofh.it> |
| In reply to | #1375748 |
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
[toc] | [prev] | [next] | [standalone]
| From | Luis de Bethencourt <luisbg@osg.samsung.com> |
|---|---|
| Date | 2016-04-12 00:40 +0200 |
| Message-ID | <rmSEW-4S4-5@gated-at.bofh.it> |
| In reply to | #1376262 |
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
[toc] | [prev] | [next] | [standalone]
| From | Sumit Semwal <sumit.semwal@linaro.org> |
|---|---|
| Date | 2016-04-12 07:20 +0200 |
| Message-ID | <rmYU2-1GZ-17@gated-at.bofh.it> |
| In reply to | #1376310 |
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.
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web