[PATCH] Documentation: add fence-array to kernel DocBook

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Include fence-array files in the DocBook. Signed-off-by: Gustavo Padovan --- This should go to drm-misc, as fence-array are only present there at the moment. --- Documentation/DocBook/device-drivers.tmpl | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH 11/14] drm/nouveau: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/nouveau/nouveau_display.c | 8 1 file changed, 4 inse

[PATCH 11/14] drm/nouveau: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/nouveau/nouveau_display.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_display.c b

[PATCH RESEND 05/14] drm/udl: use drm_crtc_send_vblank_event()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/udl/udl_modeset.c | 2 +- 1 file changed, 1 insertion(+), 1 delet

[PATCH 07/14] drm: remove legacy drm_send_vblank_event()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> We don't have any user of this function anymore, let's remove it. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/drm_irq.c | 31 ++- include/drm/drmP.h

[PATCH 01/14] drm/nouveau: use drm_crtc_send_vblank_event() v2

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. v2: add crtc to nouveau_page_flip_state (comment from Mario Kleiner) Cc: Mario Kleiner <mario.kleiner...@gmail.com> Signed-off-by: Gustavo Padovan &

[PATCH 13/14] drm/radeon: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/radeon/radeon_display.c | 6 +++--- drivers/gpu/drm/radeon/radeon_pm

[PATCH 09/14] drm/armada: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/armada/armada_crtc.c | 12 ++-- 1 file changed, 6 inse

[PATCH RESEND 05/14] drm/udl: use drm_crtc_send_vblank_event()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/udl/udl_modeset.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/udl/udl_modeset.c b/drivers/gpu/drm/udl

[PATCH 07/14] drm: remove legacy drm_send_vblank_event()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan We don't have any user of this function anymore, let's remove it. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/drm_irq.c | 31 ++- include/drm/drmP.h| 2 -- 2 files changed, 6 insertions(+), 27 deletions(-) diff --git a/drivers

[PATCH 01/14] drm/nouveau: use drm_crtc_send_vblank_event() v2

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. v2: add crtc to nouveau_page_flip_state (comment from Mario Kleiner) Cc: Mario Kleiner Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/nouveau/nouveau_display.c | 19

[PATCH 13/14] drm/radeon: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/radeon/radeon_display.c | 6 +++--- drivers/gpu/drm/radeon/radeon_pm.c | 13 + 2 files changed, 12 insertions(+), 7 deletions

[PATCH 09/14] drm/armada: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/armada/armada_crtc.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/armada/armada_crtc.c b

[PATCH 12/14] drm/qxl: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/qxl/qxl_display.c | 5 ++--- 1 file changed, 2 insertions(+),

[PATCH 12/14] drm/qxl: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/qxl/qxl_display.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/qxl/qxl_display.c b/drivers/gpu/drm/qxl

[PATCH 14/14] drm/shmobile: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/shmobile/shmob_drm_crtc.c | 4 ++-- 1 file changed, 2 insertions(+),

[PATCH 14/14] drm/shmobile: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/shmobile/shmob_drm_crtc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/shmobile/shmob_drm_crtc.c b

[PATCH RESEND 04/14] drm/qxl: use drm_crtc_send_vblank_event()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/qxl/qxl_display.c | 2 +- 1 file changed, 1 insertion(+), 1 delet

[PATCH 08/14] drm/amdgpu: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 4 ++-- drivers/gpu/drm/amd/amdgpu

[PATCH 10/14] drm/atmel: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 2 +- 1 file changed, 1 insertion(+)

[PATCH RESEND 04/14] drm/qxl: use drm_crtc_send_vblank_event()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/qxl/qxl_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/qxl/qxl_display.c b/drivers/gpu/drm/qxl

[PATCH 08/14] drm/amdgpu: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 4 ++-- drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 2 +- drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 2

[PATCH 10/14] drm/atmel: use drm_crtc_vblank_{get,put}()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Replace the legacy drm_vblank_{get,put}() with the new helper functions. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c

[PATCH RESEND 02/14] drm/armada: use drm_crtc_send_vblank_event()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/armada/armada_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 delet

[PATCH RESEND 03/14] drm/atmel: use drm_crtc_send_vblank_event()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 2 +- 1 file changed, 1 insertion(+)

[PATCH RESEND 06/14] drm/virtio: use drm_crtc_send_vblank_event()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/virtio/virtgpu_display.c | 2 +- 1 file changed, 1 insertion(+)

[PATCH RESEND 02/14] drm/armada: use drm_crtc_send_vblank_event()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/armada/armada_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/armada/armada_crtc.c b/drivers/gpu/drm

[PATCH RESEND 03/14] drm/atmel: use drm_crtc_send_vblank_event()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c

[PATCH RESEND 06/14] drm/virtio: use drm_crtc_send_vblank_event()

2016-06-06 Thread Gustavo Padovan
From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/virtio/virtgpu_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/virtio/virtgpu_display.c b/drivers

Re: [PATCH 03/11] dma-buf/fence: add signal_on_any to the fence array v2

2016-06-01 Thread Gustavo Padovan
signaled test and add comment suggested by Chris Wilson. > > Signed-off-by: Christian König <christian.koe...@amd.com> > --- > drivers/dma-buf/fence-array.c | 33 + > include/linux/fence-array.h | 3 ++- > 2 files changed, 27 inser

Re: [PATCH 03/11] dma-buf/fence: add signal_on_any to the fence array v2

2016-06-01 Thread Gustavo Padovan
n. > > Signed-off-by: Christian König > --- > drivers/dma-buf/fence-array.c | 33 + > include/linux/fence-array.h | 3 ++- > 2 files changed, 27 insertions(+), 9 deletions(-) Reviewed-by: Gustavo Padovan Gustavo

Re: [PATCH 02/11] dma-buf/fence: add fence_array fences v6

2016-06-01 Thread Gustavo Padovan
2016-06-01 Christian König <deathsim...@vodafone.de>: > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > struct fence_collection inherits from struct fence and carries a > collection of fences that needs to be waited together. > > It is useful to tr

Re: [PATCH 02/11] dma-buf/fence: add fence_array fences v6

2016-06-01 Thread Gustavo Padovan
2016-06-01 Christian König : > From: Gustavo Padovan > > struct fence_collection inherits from struct fence and carries a > collection of fences that needs to be waited together. > > It is useful to translate a sync_file to a fence to remove the complexity > of dealing w

Re: [PATCH 01/11] dma-buf/fence: make fence context 64 bit v2

2016-06-01 Thread Gustavo Padovan
| 2 +- > drivers/staging/android/sync.h | 3 ++- > include/linux/fence.h | 13 +++-- > 10 files changed, 21 insertions(+), 18 deletions(-) Reviewed-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Gustavo

Re: [PATCH 01/11] dma-buf/fence: make fence context 64 bit v2

2016-06-01 Thread Gustavo Padovan
| 13 +++-- > 10 files changed, 21 insertions(+), 18 deletions(-) Reviewed-by: Gustavo Padovan Gustavo

Re: [PATCH 02/11] dma-buf/fence: add fence_array fences v6

2016-06-01 Thread Gustavo Padovan
Hi Christian, 2016-06-01 Christian König <deathsim...@vodafone.de>: > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > struct fence_collection inherits from struct fence and carries a > collection of fences that needs to be waited together. > > It is u

Re: [PATCH 02/11] dma-buf/fence: add fence_array fences v6

2016-06-01 Thread Gustavo Padovan
Hi Christian, 2016-06-01 Christian König : > From: Gustavo Padovan > > struct fence_collection inherits from struct fence and carries a > collection of fences that needs to be waited together. > > It is useful to translate a sync_file to a fence to remove the compl

Re: [PATCH] Documentation: Fix some grammar mistakes in sync_file.txt

2016-05-31 Thread Gustavo Padovan
s, so > -the driver an create out-fence to be able to notify, through fence_signal(), > +the driver creates an out-fence to be able to notify, through fence_signal(), > when it has finished using (or processing) that buffer. Out-fences are fences > that the driver creates. Thanks, Javier! Reviewed-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Gustavo

Re: [PATCH] Documentation: Fix some grammar mistakes in sync_file.txt

2016-05-31 Thread Gustavo Padovan
, through fence_signal(), > +the driver creates an out-fence to be able to notify, through fence_signal(), > when it has finished using (or processing) that buffer. Out-fences are fences > that the driver creates. Thanks, Javier! Reviewed-by: Gustavo Padovan Gustavo

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

2016-05-31 Thread Gustavo Padovan
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 13/18] staging/android: make sw_ioctl info internal to sw_sync.c

2016-05-31 Thread Gustavo Padovan
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 18/18] staging/android: add DEBUG_FS dependence on Kconfig

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> SW_SYNC only works with DEBUG_FS so state it in the Kconfig file. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/Kconfig | 1 + drivers/staging/android/sync_debug.c | 4 -

[PATCH 14/18] staging/android: remove 'destroyed' member from struct sync_timeline

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> 'destroyed' was set but not used ny anyone. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/sw_sync.c | 5 - drivers/staging/android/sync.h| 5 + 2 files changed, 1 ins

[PATCH 17/18] staging/android: rename sync.h to sync_debug.h

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> This header file only contains information for debugging and SW_SYNC, so rename it to sync_debug.h instead of having a more generic name. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/sta

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

2016-05-31 Thread Gustavo Padovan
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 13/18] staging/android: make sw_ioctl info internal to sw_sync.c

2016-05-31 Thread Gustavo Padovan
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 18/18] staging/android: add DEBUG_FS dependence on Kconfig

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan SW_SYNC only works with DEBUG_FS so state it in the Kconfig file. Signed-off-by: Gustavo Padovan --- drivers/staging/android/Kconfig | 1 + drivers/staging/android/sync_debug.c | 4 drivers/staging/android/sync_debug.h | 4 +--- 3 files changed, 2 insertions

[PATCH 14/18] staging/android: remove 'destroyed' member from struct sync_timeline

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan 'destroyed' was set but not used ny anyone. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sw_sync.c | 5 - drivers/staging/android/sync.h| 5 + 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/drivers/staging/android/sw_sync.c b

[PATCH 17/18] staging/android: rename sync.h to sync_debug.h

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan This header file only contains information for debugging and SW_SYNC, so rename it to sync_debug.h instead of having a more generic name. Signed-off-by: Gustavo Padovan --- drivers/staging/android/sw_sync.c| 2 +- drivers/staging/android/sync_debug.c

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

2016-05-31 Thread Gustavo Padovan
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 12/18] staging/android: make sync_timeline internal to sw_sync

2016-05-31 Thread Gustavo Padovan
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 11/18] staging/android: clean up #includes in the sync framework

2016-05-31 Thread Gustavo Padovan
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

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

2016-05-31 Thread Gustavo Padovan
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 15/18] staging/android: remove sync_timeline_destroy()

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> This function was just used by the file release function, so we just fold its content there and remove sync_timeline_destroy(). Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/sw

[PATCH 16/18] staging/android: remove drv_name from sync_timeline

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> As it is internal to sw_sync now this value will always be "sw_sync". Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/staging/android/sw_sync.c| 11 +++ drivers/staging/and

[PATCH 15/18] staging/android: remove sync_timeline_destroy()

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan This function was just used by the file release function, so we just fold its content there and remove sync_timeline_destroy(). Signed-off-by: Gustavo Padovan --- drivers/staging/android/sw_sync.c | 19 +++ 1 file changed, 3 insertions(+), 16 deletions

[PATCH 16/18] staging/android: remove drv_name from sync_timeline

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan As it is internal to sw_sync now this value will always be "sw_sync". Signed-off-by: Gustavo Padovan --- drivers/staging/android/sw_sync.c| 11 +++ drivers/staging/android/sync.h | 2 -- drivers/staging/android/sync_debug.c | 2 +- 3 files

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

2016-05-31 Thread Gustavo Padovan
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 09/18] staging/android: bring struct sync_pt back

2016-05-31 Thread Gustavo Padovan
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 04/18] staging/android: remove sw_sync_timeline and sw_sync_pt

2016-05-31 Thread Gustavo Padovan
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 03/18] staging/android: remove struct sync_timeline_ops

2016-05-31 Thread Gustavo Padovan
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 09/18] staging/android: bring struct sync_pt back

2016-05-31 Thread Gustavo Padovan
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 04/18] staging/android: remove sw_sync_timeline and sw_sync_pt

2016-05-31 Thread Gustavo Padovan
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 08/18] staging/android: remove size arg of sync_timeline_create()

2016-05-31 Thread Gustavo Padovan
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 05/18] staging/android: remove sw_sync.[ch] files

2016-05-31 Thread Gustavo Padovan
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 07/18] staging/android: remove unnecessary check for fence

2016-05-31 Thread Gustavo Padovan
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 08/18] staging/android: remove size arg of sync_timeline_create()

2016-05-31 Thread Gustavo Padovan
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 05/18] staging/android: remove sw_sync.[ch] files

2016-05-31 Thread Gustavo Padovan
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 07/18] staging/android: remove unnecessary check for fence

2016-05-31 Thread Gustavo Padovan
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 06/18] staging/android: rename android_fence to timeline_fence

2016-05-31 Thread Gustavo Padovan
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/18] staging/android: rename android_fence to timeline_fence

2016-05-31 Thread Gustavo Padovan
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/18] staging/android: remove .{fence,timeline}_value_str() from timeline_ops

2016-05-31 Thread Gustavo Padovan
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/18] staging/android: remove .{fence,timeline}_value_str() from timeline_ops

2016-05-31 Thread Gustavo Padovan
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 00/18] staging/android: clean up SW_SYNC

2016-05-31 Thread Gustavo Padovan
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 00/18] staging/android: clean up SW_SYNC

2016-05-31 Thread Gustavo Padovan
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 01/18] staging/android: store last signaled value on sync timeline

2016-05-31 Thread Gustavo Padovan
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/18] staging/android: store last signaled value on sync timeline

2016-05-31 Thread Gustavo Padovan
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 2/2] dma-buf/sync_file: improve Kconfig description for Sync Files

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> We've got a complaint saying that the description was quite obtuse and indeed it was. This patch tries to improve it. Cc: Dave Jones <da...@codemonkey.org.uk> Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk>

[PATCH 2/2] dma-buf/sync_file: improve Kconfig description for Sync Files

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan We've got a complaint saying that the description was quite obtuse and indeed it was. This patch tries to improve it. Cc: Dave Jones Signed-off-by: Gustavo Padovan --- drivers/dma-buf/Kconfig | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff

[PATCH 1/2] MAINTAINERS: add entry for the Sync File Framework

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Add Gustavo as maintainer for the Sync File Framework. Sumit is co-maintainer as he maintains drivers/dma-buf/. It also uses Sumit's tree as base. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Acked-by:

[PATCH 1/2] MAINTAINERS: add entry for the Sync File Framework

2016-05-31 Thread Gustavo Padovan
From: Gustavo Padovan Add Gustavo as maintainer for the Sync File Framework. Sumit is co-maintainer as he maintains drivers/dma-buf/. It also uses Sumit's tree as base. Signed-off-by: Gustavo Padovan Acked-by: Sumit Semwal Acked-by: Maarten Lankhorst --- MAINTAINERS | 11 +++ 1 file

Re: dma-buf/sync_file: de-stage sync_file

2016-05-27 Thread Gustavo Padovan
t; > used by userspace to track fences between drivers that share DMA bufs. > > > > Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> > > Signed-off-by: Greg Kroa

Re: dma-buf/sync_file: de-stage sync_file

2016-05-27 Thread Gustavo Padovan
e to track fences between drivers that share DMA bufs. > > > > Signed-off-by: Gustavo Padovan > > Reviewed-by: Daniel Vetter > > Signed-off-by: Greg Kroah-Hartman > > ... > > > +config SYNC_FILE > > + bool "sync_f

Re: [PATCH 2/2] dma-buf/fence: add fence_array fences v4

2016-05-20 Thread Gustavo Padovan
2016-05-20 Christian König <deathsim...@vodafone.de>: > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > struct fence_collection inherits from struct fence and carries a > collection of fences that needs to be waited together. > > It is useful to tr

Re: [PATCH 2/2] dma-buf/fence: add fence_array fences v4

2016-05-20 Thread Gustavo Padovan
2016-05-20 Christian König : > From: Gustavo Padovan > > struct fence_collection inherits from struct fence and carries a > collection of fences that needs to be waited together. > > It is useful to translate a sync_file to a fence to remove the complexity > of dealing w

Re: [PATCH 1/2] dma-buf/fence: add fence_collection fences

2016-05-19 Thread Gustavo Padovan
Hi Christian, 2016-05-19 Christian König <christian.koe...@amd.com>: > Am 19.05.2016 um 00:57 schrieb Chris Wilson: > > On Wed, May 18, 2016 at 05:59:52PM -0300, Gustavo Padovan wrote: > > > +static void collection_check_cb_func(struct fence *fence, st

Re: [PATCH 1/2] dma-buf/fence: add fence_collection fences

2016-05-19 Thread Gustavo Padovan
Hi Christian, 2016-05-19 Christian König : > Am 19.05.2016 um 00:57 schrieb Chris Wilson: > > On Wed, May 18, 2016 at 05:59:52PM -0300, Gustavo Padovan wrote: > > > +static void collection_check_cb_func(struct fence *fence, struct > > > fence_cb *cb) > > > +

[PATCH 1/2] dma-buf/fence: add fence_collection fences

2016-05-18 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> struct fence_collection inherits from struct fence and carries a collection of fences that needs to be waited together. It is useful to translate a sync_file to a fence to remove the complexity of dealing with sync_files on DRM drive

[PATCH 1/2] dma-buf/fence: add fence_collection fences

2016-05-18 Thread Gustavo Padovan
From: Gustavo Padovan struct fence_collection inherits from struct fence and carries a collection of fences that needs to be waited together. It is useful to translate a sync_file to a fence to remove the complexity of dealing with sync_files on DRM drivers. So even if there are many fences

[PATCH 2/2] Documentation: add fence-collection to kernel DocBook

2016-05-18 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Include fence-collection files in the DocBook. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- Documentation/DocBook/device-drivers.tmpl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documenta

[PATCH 2/2] Documentation: add fence-collection to kernel DocBook

2016-05-18 Thread Gustavo Padovan
From: Gustavo Padovan Include fence-collection files in the DocBook. 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-drivers.tmpl index

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

2016-05-18 Thread Gustavo Padovan
Hi Christian, 2016-05-18 Christian König <christian.koe...@amd.com>: > Am 15.04.2016 um 21:25 schrieb Daniel Vetter: > > On Fri, Apr 15, 2016 at 11:27:50AM -0700, Gustavo Padovan wrote: > > > 2016-04-15 Christian König <christian.koe...@amd.com>: > > &

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

2016-05-18 Thread Gustavo Padovan
Hi Christian, 2016-05-18 Christian König : > Am 15.04.2016 um 21:25 schrieb Daniel Vetter: > > On Fri, Apr 15, 2016 at 11:27:50AM -0700, Gustavo Padovan wrote: > > > 2016-04-15 Christian König : > > > > Amdgpu also has an implementation for a fence collection wh

[PATCH v2] MAINTAINERS: add entry for the Sync File Framework

2016-05-13 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Add Gustavo as maintainer for the Sync File Framework. Sumit is co-maintainer as he maintains drivers/dma-buf/. It also uses Sumit's tree as base. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Acked-by:

[PATCH v2] MAINTAINERS: add entry for the Sync File Framework

2016-05-13 Thread Gustavo Padovan
From: Gustavo Padovan Add Gustavo as maintainer for the Sync File Framework. Sumit is co-maintainer as he maintains drivers/dma-buf/. It also uses Sumit's tree as base. Signed-off-by: Gustavo Padovan Acked-by: Sumit Semwal Acked-by: Maarten Lankhorst --- MAINTAINERS | 11 +++ 1 file

[PATCH] MAINTAINERS: add entry for the Sync File Framework

2016-05-11 Thread Gustavo Padovan
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Add Gustavo as maintainer for the Sync File Framework. Sumit is co-maintainer as he maintains drivers/dma-buf/. It also uses Sumit's tree as base. Cc: Sumit Semwal <sumit.sem...@linaro.org> Signed-off-by: Gustavo Padovan &

[PATCH] MAINTAINERS: add entry for the Sync File Framework

2016-05-11 Thread Gustavo Padovan
From: Gustavo Padovan Add Gustavo as maintainer for the Sync File Framework. Sumit is co-maintainer as he maintains drivers/dma-buf/. It also uses Sumit's tree as base. Cc: Sumit Semwal Signed-off-by: Gustavo Padovan --- MAINTAINERS | 10 ++ 1 file changed, 10 insertions(+) diff

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

2016-05-02 Thread Gustavo Padovan
Hi Pavel, 2016-05-02 Pavel Machek : > Hi! > > > > -} > > -EXPORT_SYMBOL(sync_file_merge); > > - > > static const char *android_fence_get_driver_name(struct fence *fence) > > { > > struct sync_timeline *parent = fence_parent(fence); > > if this is meant to be used outside

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

2016-05-02 Thread Gustavo Padovan
Hi Pavel, 2016-05-02 Pavel Machek : > Hi! > > > > -} > > -EXPORT_SYMBOL(sync_file_merge); > > - > > static const char *android_fence_get_driver_name(struct fence *fence) > > { > > struct sync_timeline *parent = fence_parent(fence); > > if this is meant to be used outside android, should

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

2016-05-02 Thread Gustavo Padovan
2016-05-02 Pavel Machek <pa...@ucw.cz>: > On Wed 2016-04-27 13:27:08, Gustavo Padovan wrote: > > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > > > struct sync_merge_data already have documentation on top of the > > struct definition. No ne

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