Re: [PATCH 01/12] staging/android: remove redundant comments on sync_merge_data

2016-05-02 Pavel Machek : > On Wed 2016-04-27 13:27:08, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > struct sync_merge_data already have documentation on top of the > > struct definition. No need to duplicate it. > > > > Signed-off-by: Gu

[PATCH 04/13] staging/android: remove sw_sync_timeline and sw_sync_pt

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> As we moved value storage to sync_timeline and fence those two structs became useless and can be removed now. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/sw_sy

[PATCH 04/13] staging/android: remove sw_sync_timeline and sw_sync_pt

From: Gustavo Padovan As we moved value storage to sync_timeline and fence those two structs became useless and can be removed now. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sw_sync.c| 24 +++- drivers/staging/android/sw_sync.h| 24

[PATCH 03/13] staging/android: remove struct sync_timeline_ops

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Move drv_name, the last field of sync_timeline_ops, to sync_timeline and remove sync_timeline_ops. struct sync_timeline_ops was just an extra abstraction on top of fence_ops, and in the last few commits we removed all it ops in

[PATCH 03/13] staging/android: remove struct sync_timeline_ops

From: Gustavo Padovan Move drv_name, the last field of sync_timeline_ops, to sync_timeline and remove sync_timeline_ops. struct sync_timeline_ops was just an extra abstraction on top of fence_ops, and in the last few commits we removed all it ops in favor of cleaner fence_ops. Signed-off

[PATCH 06/13] staging/android: rename android_fence to timeline_fence

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> We are moving out of staging/android so rename it to a name that is not related to android anymore. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android

[PATCH 06/13] staging/android: rename android_fence to timeline_fence

From: Gustavo Padovan We are moving out of staging/android so rename it to a name that is not related to android anymore. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sync.c | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git

[PATCH 02/13] staging/android: remove .{fence,timeline}_value_str() from timeline_ops

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Now that the value of fence and the timeline are not stored by sw_sync anymore we can remove this extra abstraction to retrieve this data. This patch changes both fence_ops (.fence_value_str and .timeline_value_str) to return t

[PATCH 02/13] staging/android: remove .{fence,timeline}_value_str() from timeline_ops

From: Gustavo Padovan Now that the value of fence and the timeline are not stored by sw_sync anymore we can remove this extra abstraction to retrieve this data. This patch changes both fence_ops (.fence_value_str and .timeline_value_str) to return the str directly. It also clean up struct

[PATCH 12/13] staging/android: make sync_timeline internal to sw_sync

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> The only use sync_timeline will have in upstream kernel is for debugging through the SW_SYNC interface. So make it internal to SW_SYNC to avoid people use it in the future. Signed-off-by: Gustavo Padovan <gustavo.pado...@collab

[PATCH 12/13] staging/android: make sync_timeline internal to sw_sync

From: Gustavo Padovan The only use sync_timeline will have in upstream kernel is for debugging through the SW_SYNC interface. So make it internal to SW_SYNC to avoid people use it in the future. Signed-off-by: Gustavo Padovan --- drivers/staging/android/Kconfig | 16 +-- drivers/staging

[PATCH 10/13] staging/android: move sw_sync related code to sw_sync.c

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Split sync_debug and sw_sync in two different files. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/Makefile | 1 + drivers/staging/android/sw_syn

[PATCH 10/13] staging/android: move sw_sync related code to sw_sync.c

From: Gustavo Padovan Split sync_debug and sw_sync in two different files. Signed-off-by: Gustavo Padovan --- drivers/staging/android/Makefile | 1 + drivers/staging/android/sw_sync.c| 136 +++ drivers/staging/android/sync.h | 2 + drivers

[PATCH 05/13] staging/android: remove sw_sync.[ch] files

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> We can glue the sw_sync file operations directly on the sync framework without the need to pass through sw_sync wrappers. It only builds sw_sync debugfs file support if CONFIG_SW_SYNC is enabled. Signed-off-by: Gustavo P

[PATCH 05/13] staging/android: remove sw_sync.[ch] files

From: Gustavo Padovan We can glue the sw_sync file operations directly on the sync framework without the need to pass through sw_sync wrappers. It only builds sw_sync debugfs file support if CONFIG_SW_SYNC is enabled. Signed-off-by: Gustavo Padovan --- drivers/staging/android/Makefile

[PATCH 01/13] staging/android: store last signaled value on sync timeline

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Now fence timeline is aware of the last signaled fence, as it receives the increment to the current value in sync_timeline_signal(). That allow us to remove .has_signaled() from timeline_ops as we can directly compare using timeline-

[PATCH 01/13] staging/android: store last signaled value on sync timeline

From: Gustavo Padovan Now fence timeline is aware of the last signaled fence, as it receives the increment to the current value in sync_timeline_signal(). That allow us to remove .has_signaled() from timeline_ops as we can directly compare using timeline->value and fence->seqno in

[PATCH 08/13] staging/android: remove size arg of sync_timeline_create()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> After we removed sw_sync_timeline this arg has not been really used by anyone, all its users pass the size of struct sync_timeline there. So simplify this function but not requiring the size anymore. Signed-off-by: Gustavo P

[PATCH 00/13] staging/android: clean up SW_SYNC

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Hi, The following patches do a clean up on the sw_sync inteface. It starts by removing struct sync_timeline_ops, which was creating unecessary wrappers in the code and the start to organize the sync_timeline and sw_sync code

[PATCH 07/13] staging/android: remove unnecessary check for fence

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> When we call sync_print_fence() fence is always valid. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/sync_debug.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)

[PATCH 13/13] staging/android: make sw_ioctl info internal to sw_sync.c

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> We don't want to export this from the kernel. This is interface is only for testing and debug. So testers shall copy the ioctl info in their own projects. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- dri

[PATCH 08/13] staging/android: remove size arg of sync_timeline_create()

From: Gustavo Padovan After we removed sw_sync_timeline this arg has not been really used by anyone, all its users pass the size of struct sync_timeline there. So simplify this function but not requiring the size anymore. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sync.c

[PATCH 00/13] staging/android: clean up SW_SYNC

From: Gustavo Padovan Hi, The following patches do a clean up on the sw_sync inteface. It starts by removing struct sync_timeline_ops, which was creating unecessary wrappers in the code and the start to organize the sync_timeline and sw_sync code better. sw_sync interface was moved

[PATCH 07/13] staging/android: remove unnecessary check for fence

From: Gustavo Padovan When we call sync_print_fence() fence is always valid. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sync_debug.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/android/sync_debug.c b/drivers/staging/android

[PATCH 13/13] staging/android: make sw_ioctl info internal to sw_sync.c

From: Gustavo Padovan We don't want to export this from the kernel. This is interface is only for testing and debug. So testers shall copy the ioctl info in their own projects. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sw_sync.c | 13 - drivers/staging

[PATCH 09/13] staging/android: bring struct sync_pt back

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Move the list_head members from sync_pt to struct fence was a mistake, they will not be used by struct fence as planned before, so here we create sync_pt again to bring the list heads back. Signed-off-by: Gustavo Padovan <gus

[PATCH 09/13] staging/android: bring struct sync_pt back

From: Gustavo Padovan Move the list_head members from sync_pt to struct fence was a mistake, they will not be used by struct fence as planned before, so here we create sync_pt again to bring the list heads back. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sync.c | 40

[PATCH 11/13] staging/android: clean up #includes in the sync framework

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Most of the includes there are not necessary anymore. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/sync.c | 6 -- drivers/staging/android/sync.h | 3 --- dri

[PATCH 11/13] staging/android: clean up #includes in the sync framework

From: Gustavo Padovan Most of the includes there are not necessary anymore. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sync.c | 6 -- drivers/staging/android/sync.h | 3 --- drivers/staging/android/sync_debug.c | 16 3 files changed, 25

Re: [PATCH v2 06/13] staging/android: remove name arg from sync_file_create()

Hi Hillf, (please keep the whole cc on the reply, so others can see it too) 2016-04-29 Hillf Danton <hillf...@alibaba-inc.com>: > > > > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > > > Simplifies the API to only receive the fence it nee

Re: [PATCH v2 06/13] staging/android: remove name arg from sync_file_create()

Hi Hillf, (please keep the whole cc on the reply, so others can see it too) 2016-04-29 Hillf Danton : > > > > From: Gustavo Padovan > > > > Simplifies the API to only receive the fence it needs to add to the > > sync and create a name for the sync_f

Re: [PATCH v2 02/13] staging/android: drop sync_file_install() and sync_file_put()

2016-04-29 Hillf Danton <hillf...@alibaba-inc.com>: > > > > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > > > These two functions are just wrappers for one line functions, they > > call fd_install() and fput() respectively, so just get rid

Re: [PATCH v2 02/13] staging/android: drop sync_file_install() and sync_file_put()

2016-04-29 Hillf Danton : > > > > From: Gustavo Padovan > > > > These two functions are just wrappers for one line functions, they > > call fd_install() and fput() respectively, so just get rid of them > > and use fd_install() and fput() directly for mor

Re: [PATCH v2 01/13] staging/android: remove redundant comments on sync_merge_data

Hi Hillf, 2016-04-29 Hillf Danton <hillf...@alibaba-inc.com>: > > > > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > > > struct sync_merge_data already have documentation on top of the > > struct definition. No need to duplicate i

Re: [PATCH v2 01/13] staging/android: remove redundant comments on sync_merge_data

Hi Hillf, 2016-04-29 Hillf Danton : > > > > From: Gustavo Padovan > > > > struct sync_merge_data already have documentation on top of the > > struct definition. No need to duplicate it. > > > > Signed-off-by: Gustavo Padovan > > Reviewed-by: M

Re: [RFC v2 5/8] drm/fence: add in-fences support

2016-04-28 Ville Syrjälä : > On Thu, Apr 28, 2016 at 07:43:16PM +0200, Daniel Vetter wrote: > > On Thu, Apr 28, 2016 at 6:56 PM, Ville Syrjälä > > wrote: > > >> - better for tracing, can identify the buffer/fence promptly > > > > > >

Re: [RFC v2 5/8] drm/fence: add in-fences support

2016-04-28 Ville Syrjälä : > On Thu, Apr 28, 2016 at 07:43:16PM +0200, Daniel Vetter wrote: > > On Thu, Apr 28, 2016 at 6:56 PM, Ville Syrjälä > > wrote: > > >> - better for tracing, can identify the buffer/fence promptly > > > > > > Can fences be reused somehow while still attached to a plane,

Re: [RFC v2 8/8] drm/fence: add out-fences support

2016-04-26 Daniel Vetter <dan...@ffwll.ch>: > On Mon, Apr 25, 2016 at 07:33:28PM -0300, Gustavo Padovan wrote: > > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > > > Support DRM out-fences creating a sync_file with a fence for each crtc > > up

Re: [RFC v2 8/8] drm/fence: add out-fences support

2016-04-26 Daniel Vetter : > On Mon, Apr 25, 2016 at 07:33:28PM -0300, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > Support DRM out-fences creating a sync_file with a fence for each crtc > > update with the DRM_MODE_ATOMIC_OUT_FENCE flag. >

Re: [RFC v2 1/8] dma-buf/fence: add fence_collection fences

2016-04-26 Chris Wilson <ch...@chris-wilson.co.uk>: > On Mon, Apr 25, 2016 at 07:33:21PM -0300, Gustavo Padovan wrote: > > +static const char *fence_collection_get_timeline_name(struct fence *fence) > > +{ > > + return "no context"; > > "un

Re: [RFC v2 1/8] dma-buf/fence: add fence_collection fences

2016-04-26 Chris Wilson : > On Mon, Apr 25, 2016 at 07:33:21PM -0300, Gustavo Padovan wrote: > > +static const char *fence_collection_get_timeline_name(struct fence *fence) > > +{ > > + return "no context"; > > "unbound" to distinguish from fence

Re: [RFC v2 5/8] drm/fence: add in-fences support

2016-04-27 Daniel Stone : > Hi, > > On 26 April 2016 at 21:48, Greg Hackmann wrote: > > On 04/26/2016 01:05 PM, Daniel Vetter wrote: > >> On Tue, Apr 26, 2016 at 09:55:06PM +0300, Ville Syrjälä wrote: > >>> What are they doing that can't stuff the

Re: [RFC v2 5/8] drm/fence: add in-fences support

2016-04-27 Daniel Stone : > Hi, > > On 26 April 2016 at 21:48, Greg Hackmann wrote: > > On 04/26/2016 01:05 PM, Daniel Vetter wrote: > >> On Tue, Apr 26, 2016 at 09:55:06PM +0300, Ville Syrjälä wrote: > >>> What are they doing that can't stuff the fences into an array > >>> instead of props? >

[PATCH v2 03/13] staging/android: move sync_file functions comments to sync.c

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> To keep comments in line with drivers/dma-buf/ move all sync_file comments to sync.c. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> --- drivers/stag

[PATCH v2 03/13] staging/android: move sync_file functions comments to sync.c

From: Gustavo Padovan To keep comments in line with drivers/dma-buf/ move all sync_file comments to sync.c. Signed-off-by: Gustavo Padovan Reviewed-by: Daniel Vetter --- drivers/staging/android/sync.c | 26 +- drivers/staging/android/sync.h | 31

[PATCH v2 08/13] staging/android: improve documentation for sync_file

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> num_fences was missing a colon mark and sync_file_create() now have better description. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> --- drive

[PATCH v2 08/13] staging/android: improve documentation for sync_file

From: Gustavo Padovan num_fences was missing a colon mark and sync_file_create() now have better description. Signed-off-by: Gustavo Padovan Reviewed-by: Daniel Vetter --- drivers/staging/android/sync_file.c | 5 +++-- drivers/staging/android/sync_file.h | 2 +- 2 files changed, 4 insertions

[PATCH v2 05/13] staging/android: make sync_file_fdget() static

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> There is no plan in the near future to use this function outside of this file so keep it as static. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> --

[PATCH v2 05/13] staging/android: make sync_file_fdget() static

From: Gustavo Padovan There is no plan in the near future to use this function outside of this file so keep it as static. Signed-off-by: Gustavo Padovan Reviewed-by: Daniel Vetter --- drivers/staging/android/sync.c | 3 +-- drivers/staging/android/sync.h | 1 - 2 files changed, 1 insertion

[PATCH v2 10/13] dma-buf/sync_file: de-stage sync_file headers

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Move sync_file headers file to include/ dir. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> --- drivers/staging/android/sync.h

[PATCH v2 10/13] dma-buf/sync_file: de-stage sync_file headers

From: Gustavo Padovan Move sync_file headers file to include/ dir. Signed-off-by: Gustavo Padovan Reviewed-by: Daniel Vetter --- drivers/staging/android/sync.h | 4 ++-- drivers/staging/android/sync_debug.c | 2 +- drivers/staging

[PATCH v2 09/13] staging/android: style fix: alignment to match the open parenthesis

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Fix checks reported by checkpatch.pl. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> --- drivers/staging/android/sync_file.c | 8 1 file cha

[PATCH v2 13/13] Documentation: add Sync File doc

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Add sync_file documentation on dma-buf-sync_file.txt Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> --- Documentation/sync_file.txt | 69 + 1 file changed, 69 insertions(+)

[PATCH v2 11/13] dma-buf/sync_file: de-stage sync_file

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> sync_file is useful to connect one or more fences to the file. The file is used by userspace to track fences between drivers that share DMA bufs. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: D

[PATCH v2 12/13] Documentation: include sync_file into DocBook

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Add entry in device-drivers.tmpl for sync_file documentation. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> --- Documentation/DocBook/device-drivers.t

[PATCH v2 11/13] dma-buf/sync_file: de-stage sync_file

From: Gustavo Padovan sync_file is useful to connect one or more fences to the file. The file is used by userspace to track fences between drivers that share DMA bufs. Signed-off-by: Gustavo Padovan Reviewed-by: Daniel Vetter --- drivers/Kconfig | 2

[PATCH v2 12/13] Documentation: include sync_file into DocBook

From: Gustavo Padovan Add entry in device-drivers.tmpl for sync_file documentation. Signed-off-by: Gustavo Padovan Reviewed-by: Daniel Vetter --- Documentation/DocBook/device-drivers.tmpl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/DocBook/device-drivers.tmpl b

[PATCH v2 13/13] Documentation: add Sync File doc

From: Gustavo Padovan Add sync_file documentation on dma-buf-sync_file.txt Reviewed-by: Daniel Vetter --- Documentation/sync_file.txt | 69 + 1 file changed, 69 insertions(+) create mode 100644 Documentation/sync_file.txt diff --git a/Documentation

[PATCH v2 09/13] staging/android: style fix: alignment to match the open parenthesis

From: Gustavo Padovan Fix checks reported by checkpatch.pl. Signed-off-by: Gustavo Padovan Reviewed-by: Daniel Vetter --- drivers/staging/android/sync_file.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/android/sync_file.c b/drivers/staging

[PATCH v2 07/13] staging/android: prepare sync_file for de-staging

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Move its functions and structs to their own file. Also moves function's docs to the .c file. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> --- drive

[PATCH v2 07/13] staging/android: prepare sync_file for de-staging

From: Gustavo Padovan Move its functions and structs to their own file. Also moves function's docs to the .c file. Signed-off-by: Gustavo Padovan Reviewed-by: Daniel Vetter --- drivers/staging/android/Makefile | 2 +- drivers/staging/android/sync.c

[PATCH v2 00/13] De-stage Sync File Framework

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Hi, This patchset sits on top of Sync ABI Rework v13: https://www.spinics.net/lists/dri-devel/msg105667.html The first eight clean up and prepare sync_file for de-staging. The last four patches do the de-staging, moving files to d

[PATCH v2 00/13] De-stage Sync File Framework

From: Gustavo Padovan Hi, This patchset sits on top of Sync ABI Rework v13: https://www.spinics.net/lists/dri-devel/msg105667.html The first eight clean up and prepare sync_file for de-staging. The last four patches do the de-staging, moving files to drivers/dma-buf/ and include/linux/ plus

[PATCH v2 04/13] staging/android: make sync_file_merge() static

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> There is no plan in the near future to use this function outside of this file so keep it as static. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> --

[PATCH v2 06/13] staging/android: remove name arg from sync_file_create()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Simplifies the API to only receive the fence it needs to add to the sync and create a name for the sync_file based on the fence context and seqno. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: D

[PATCH v2 01/13] staging/android: remove redundant comments on sync_merge_data

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> struct sync_merge_data already have documentation on top of the struct definition. No need to duplicate it. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Maarten Lankhorst <maarten.lankho...@

[PATCH v2 04/13] staging/android: make sync_file_merge() static

From: Gustavo Padovan There is no plan in the near future to use this function outside of this file so keep it as static. Signed-off-by: Gustavo Padovan Reviewed-by: Daniel Vetter --- drivers/staging/android/sync.c | 5 ++--- drivers/staging/android/sync.h | 2 -- 2 files changed, 2

[PATCH v2 06/13] staging/android: remove name arg from sync_file_create()

From: Gustavo Padovan Simplifies the API to only receive the fence it needs to add to the sync and create a name for the sync_file based on the fence context and seqno. Signed-off-by: Gustavo Padovan Reviewed-by: Daniel Vetter --- drivers/staging/android/sync.c | 16

[PATCH v2 01/13] staging/android: remove redundant comments on sync_merge_data

From: Gustavo Padovan struct sync_merge_data already have documentation on top of the struct definition. No need to duplicate it. Signed-off-by: Gustavo Padovan Reviewed-by: Maarten Lankhorst Reviewed-by: Daniel Vetter --- drivers/staging/android/uapi/sync.h | 10 +- 1 file changed

[PATCH v2 02/13] staging/android: drop sync_file_install() and sync_file_put()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> These two functions are just wrappers for one line functions, they call fd_install() and fput() respectively, so just get rid of them and use fd_install() and fput() directly for more simplicity. Signed-off-by: Gustavo Padovan <gus

[PATCH v2 02/13] staging/android: drop sync_file_install() and sync_file_put()

From: Gustavo Padovan These two functions are just wrappers for one line functions, they call fd_install() and fput() respectively, so just get rid of them and use fd_install() and fput() directly for more simplicity. Signed-off-by: Gustavo Padovan Reviewed-by: Daniel Vetter --- drivers

Re: [PATCH 00/12] De-stage Sync File Framework

2016-04-27 Daniel Vetter <dan...@ffwll.ch>: > On Wed, Apr 27, 2016 at 01:27:07PM -0300, Gustavo Padovan wrote: > > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > > > Hi, > > > > This patchset sits on top of Sync ABI Rework v13: >

Re: [PATCH 00/12] De-stage Sync File Framework

2016-04-27 Daniel Vetter : > On Wed, Apr 27, 2016 at 01:27:07PM -0300, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > Hi, > > > > This patchset sits on top of Sync ABI Rework v13: > > > > https://www.spinics.net/lists/dri-devel/msg

[PATCH 03/12] staging/android: move sync_file functions comments to sync.c

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> To keep comments in line with drivers/dma-buf/ move all sync_file comments to sync.c. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/sync.c | 26 +- dri

[PATCH 12/12] Documentation: add Sync File doc

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Add sync_file documentation on dma-buf-sync_file.txt --- Documentation/dma-buf-sync_file.txt | 65 + 1 file changed, 65 insertions(+) create mode 100644 Documentation/dma-buf-sync_file.txt diff

[PATCH 10/12] dma-buf/sync_file: de-stage sync_file

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> sync_file is useful to connect one or more fences to the file. The file is used by userspace to track fences between drivers that share DMA bufs. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- dri

[PATCH 04/12] staging/android: make sync_file_merge() static

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> There is no plan in the near future to use this function outside of this file so keep it as static. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/sync.c | 4 ++-- drivers/sta

[PATCH 03/12] staging/android: move sync_file functions comments to sync.c

From: Gustavo Padovan To keep comments in line with drivers/dma-buf/ move all sync_file comments to sync.c. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sync.c | 26 +- drivers/staging/android/sync.h | 31 --- 2 files changed

[PATCH 12/12] Documentation: add Sync File doc

From: Gustavo Padovan Add sync_file documentation on dma-buf-sync_file.txt --- Documentation/dma-buf-sync_file.txt | 65 + 1 file changed, 65 insertions(+) create mode 100644 Documentation/dma-buf-sync_file.txt diff --git a/Documentation/dma-buf

[PATCH 10/12] dma-buf/sync_file: de-stage sync_file

From: Gustavo Padovan sync_file is useful to connect one or more fences to the file. The file is used by userspace to track fences between drivers that share DMA bufs. Signed-off-by: Gustavo Padovan --- drivers/Kconfig | 2 ++ drivers/dma-buf/Kconfig

[PATCH 04/12] staging/android: make sync_file_merge() static

From: Gustavo Padovan There is no plan in the near future to use this function outside of this file so keep it as static. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sync.c | 4 ++-- drivers/staging/android/sync.h | 2 -- 2 files changed, 2 insertions(+), 4 deletions(-) diff

[PATCH 08/12] staging/android: style fix: alignment to match the open parenthesis

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Fix checks reported by checkpatch.pl. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/sync_file.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/dri

[PATCH 11/12] Documentation: include sync_file into DocBook

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Add entry in device-drivers.tmpl for sync_file documentation. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- Documentation/DocBook/device-drivers.tmpl | 2 ++ 1 file changed, 2 insertions(+)

[PATCH 07/12] staging/android: improve documentation for sync_file

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> num_fences was missing a colon mark and sync_file_create() now have better description. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/sync_file.c | 3 ++- drivers/staging/android

[PATCH 08/12] staging/android: style fix: alignment to match the open parenthesis

From: Gustavo Padovan Fix checks reported by checkpatch.pl. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sync_file.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/android/sync_file.c b/drivers/staging/android/sync_file.c index

[PATCH 11/12] Documentation: include sync_file into DocBook

From: Gustavo Padovan Add entry in device-drivers.tmpl for sync_file documentation. Signed-off-by: Gustavo Padovan --- Documentation/DocBook/device-drivers.tmpl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/DocBook/device-drivers.tmpl b/Documentation/DocBook/device

[PATCH 07/12] staging/android: improve documentation for sync_file

From: Gustavo Padovan num_fences was missing a colon mark and sync_file_create() now have better description. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sync_file.c | 3 ++- drivers/staging/android/sync_file.h | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff

[PATCH 05/12] staging/android: make sync_file_fdget() static

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> There is no plan in the near future to use this function outside of this file so keep it as static. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/sync.c | 3 +-- drivers/staging/an

[PATCH 05/12] staging/android: make sync_file_fdget() static

From: Gustavo Padovan There is no plan in the near future to use this function outside of this file so keep it as static. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sync.c | 3 +-- drivers/staging/android/sync.h | 1 - 2 files changed, 1 insertion(+), 3 deletions(-) diff

[PATCH 06/12] staging/android: prepare sync_file for de-staging

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Move its functions and structs to their own file. Also moves function's docs to the .c file. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/Makefile | 2 +- dri

[PATCH 06/12] staging/android: prepare sync_file for de-staging

From: Gustavo Padovan Move its functions and structs to their own file. Also moves function's docs to the .c file. Signed-off-by: Gustavo Padovan --- drivers/staging/android/Makefile | 2 +- drivers/staging/android/sync.c | 373

[PATCH 01/12] staging/android: remove redundant comments on sync_merge_data

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> struct sync_merge_data already have documentation on top of the struct definition. No need to duplicate it. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Maarten Lankhorst <maarten.lankho...@

[PATCH 01/12] staging/android: remove redundant comments on sync_merge_data

From: Gustavo Padovan struct sync_merge_data already have documentation on top of the struct definition. No need to duplicate it. Signed-off-by: Gustavo Padovan Reviewed-by: Maarten Lankhorst --- drivers/staging/android/uapi/sync.h | 10 +- 1 file changed, 5 insertions(+), 5

[PATCH 09/12] dma-buf/sync_file: de-stage sync_file headers

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Move sync_file headers file to include/ dir. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/sync.h | 4 ++-- drivers/staging/android/

[PATCH 09/12] dma-buf/sync_file: de-stage sync_file headers

From: Gustavo Padovan Move sync_file headers file to include/ dir. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sync.h | 4 ++-- drivers/staging/android/sync_debug.c | 2 +- drivers/staging/android/sync_file.c

[PATCH 02/12] staging/android: drop sync_file_install() and sync_file_put()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> These two functions are just wrappers for one line functions, they call fd_install() and fput() respectively, so just get rid of them and use fd_install() and fput() directly for more simplicity. Signed-off-by: Gustavo Padovan <gus

[PATCH 00/12] De-stage Sync File Framework

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Hi, This patchset sits on top of Sync ABI Rework v13: https://www.spinics.net/lists/dri-devel/msg105667.html The first eight clean up and prepare sync_file for de-staging. The last four patches do the de-staging, moving files to d

[PATCH 02/12] staging/android: drop sync_file_install() and sync_file_put()

From: Gustavo Padovan These two functions are just wrappers for one line functions, they call fd_install() and fput() respectively, so just get rid of them and use fd_install() and fput() directly for more simplicity. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sync.c

[PATCH 00/12] De-stage Sync File Framework

From: Gustavo Padovan Hi, This patchset sits on top of Sync ABI Rework v13: https://www.spinics.net/lists/dri-devel/msg105667.html The first eight clean up and prepare sync_file for de-staging. The last four patches do the de-staging, moving files to drivers/dma-buf/ and include/linux/ plus

[PATCH v13 0/2] staging/android: Sync ABI rework

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Hi Greg, This patchset clean up the Sync ABI and then improve in to a more optimized version. Also it is now less likely to need changes in the future. This is not breaking any upstream user of the sync framework, as no driver

<    4   5   6   7   8   9   10   11   12   13   >