Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Daniel Vetter
On Tue, May 24, 2016 at 10:41:30AM +0200, Heiko Stuebner wrote: > Am Dienstag, 24. Mai 2016, 10:37:49 schrieb Daniel Vetter: > > On Tue, May 24, 2016 at 10:30:50AM +0200, Daniel Vetter wrote: > > > On Tue, May 24, 2016 at 10:28:42AM +0200, Heiko Stuebner wrote: > > > > Hi Tomeu, > > > > > > > >

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Daniel Vetter
On Tue, May 24, 2016 at 10:41:30AM +0200, Heiko Stuebner wrote: > Am Dienstag, 24. Mai 2016, 10:37:49 schrieb Daniel Vetter: > > On Tue, May 24, 2016 at 10:30:50AM +0200, Daniel Vetter wrote: > > > On Tue, May 24, 2016 at 10:28:42AM +0200, Heiko Stuebner wrote: > > > > Hi Tomeu, > > > > > > > >

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Heiko Stuebner
Am Dienstag, 24. Mai 2016, 10:37:49 schrieb Daniel Vetter: > On Tue, May 24, 2016 at 10:30:50AM +0200, Daniel Vetter wrote: > > On Tue, May 24, 2016 at 10:28:42AM +0200, Heiko Stuebner wrote: > > > Hi Tomeu, > > > > > > Patch subject: please put the version into the brackets, so [PATCH v5] > > >

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Heiko Stuebner
Am Dienstag, 24. Mai 2016, 10:37:49 schrieb Daniel Vetter: > On Tue, May 24, 2016 at 10:30:50AM +0200, Daniel Vetter wrote: > > On Tue, May 24, 2016 at 10:28:42AM +0200, Heiko Stuebner wrote: > > > Hi Tomeu, > > > > > > Patch subject: please put the version into the brackets, so [PATCH v5] > > >

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Daniel Vetter
On Tue, May 24, 2016 at 10:30:50AM +0200, Daniel Vetter wrote: > On Tue, May 24, 2016 at 10:28:42AM +0200, Heiko Stuebner wrote: > > Hi Tomeu, > > > > Patch subject: please put the version into the brackets, so [PATCH v5] as > > it > > shouldn't be part of the commit log. > > > > Am Dienstag,

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Daniel Vetter
On Tue, May 24, 2016 at 10:30:50AM +0200, Daniel Vetter wrote: > On Tue, May 24, 2016 at 10:28:42AM +0200, Heiko Stuebner wrote: > > Hi Tomeu, > > > > Patch subject: please put the version into the brackets, so [PATCH v5] as > > it > > shouldn't be part of the commit log. > > > > Am Dienstag,

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Daniel Vetter
On Tue, May 24, 2016 at 10:28:42AM +0200, Heiko Stuebner wrote: > Hi Tomeu, > > Patch subject: please put the version into the brackets, so [PATCH v5] as it > shouldn't be part of the commit log. > > Am Dienstag, 24. Mai 2016, 09:27:37 schrieb Tomeu Vizoso: > > As per the docs, atomic_commit

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Daniel Vetter
On Tue, May 24, 2016 at 10:28:42AM +0200, Heiko Stuebner wrote: > Hi Tomeu, > > Patch subject: please put the version into the brackets, so [PATCH v5] as it > shouldn't be part of the commit log. > > Am Dienstag, 24. Mai 2016, 09:27:37 schrieb Tomeu Vizoso: > > As per the docs, atomic_commit

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Heiko Stuebner
Hi Tomeu, Patch subject: please put the version into the brackets, so [PATCH v5] as it shouldn't be part of the commit log. Am Dienstag, 24. Mai 2016, 09:27:37 schrieb Tomeu Vizoso: > As per the docs, atomic_commit should return -EBUSY "if an asycnhronous > updated is requested and there is an

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Heiko Stuebner
Hi Tomeu, Patch subject: please put the version into the brackets, so [PATCH v5] as it shouldn't be part of the commit log. Am Dienstag, 24. Mai 2016, 09:27:37 schrieb Tomeu Vizoso: > As per the docs, atomic_commit should return -EBUSY "if an asycnhronous > updated is requested and there is an

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Daniel Stone
On 24 May 2016 at 09:27, Tomeu Vizoso wrote: > As per the docs, atomic_commit should return -EBUSY "if an asycnhronous > updated is requested and there is an earlier updated pending". > > v2: Use the status of the workqueue instead of vop->event, and don't add > a

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Daniel Stone
On 24 May 2016 at 09:27, Tomeu Vizoso wrote: > As per the docs, atomic_commit should return -EBUSY "if an asycnhronous > updated is requested and there is an earlier updated pending". > > v2: Use the status of the workqueue instead of vop->event, and don't add > a superfluous wait on the

[PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Tomeu Vizoso
As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". v2: Use the status of the workqueue instead of vop->event, and don't add a superfluous wait on the workqueue. v3: Drop work_busy, as there's a sizeable delay

[PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v5

2016-05-24 Thread Tomeu Vizoso
As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". v2: Use the status of the workqueue instead of vop->event, and don't add a superfluous wait on the workqueue. v3: Drop work_busy, as there's a sizeable delay

[PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v4

2016-05-24 Thread Tomeu Vizoso
As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". v2: Use the status of the workqueue instead of vop->event, and don't add a superfluous wait on the workqueue. v3: Drop work_busy, as there's a sizeable delay

[PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v4

2016-05-24 Thread Tomeu Vizoso
As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". v2: Use the status of the workqueue instead of vop->event, and don't add a superfluous wait on the workqueue. v3: Drop work_busy, as there's a sizeable delay

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v3

2016-05-24 Thread Daniel Stone
Hi Tomeu, On 5 April 2016 at 16:07, Tomeu Vizoso wrote: > On 4 April 2016 at 17:44, Daniel Stone wrote: >> On 4 April 2016 at 14:55, Tomeu Vizoso wrote: >>> + if (async) { >>> +

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v3

2016-05-24 Thread Daniel Stone
Hi Tomeu, On 5 April 2016 at 16:07, Tomeu Vizoso wrote: > On 4 April 2016 at 17:44, Daniel Stone wrote: >> On 4 April 2016 at 14:55, Tomeu Vizoso wrote: >>> + if (async) { >>> + for_each_crtc_in_state(state, crtc, crtc_state, i) { >>> + if

[PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v4

2016-04-20 Thread Tomeu Vizoso
As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". v2: Use the status of the workqueue instead of vop->event, and don't add a superfluous wait on the workqueue. v3: Drop work_busy, as there's a sizeable delay

[PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v4

2016-04-20 Thread Tomeu Vizoso
As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". v2: Use the status of the workqueue instead of vop->event, and don't add a superfluous wait on the workqueue. v3: Drop work_busy, as there's a sizeable delay

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v3

2016-04-05 Thread Tomeu Vizoso
On 4 April 2016 at 17:44, Daniel Stone wrote: > Hi Tomeu, > > On 4 April 2016 at 14:55, Tomeu Vizoso wrote: >> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_fb.c >> b/drivers/gpu/drm/rockchip/rockchip_drm_fb.c >> index

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v3

2016-04-05 Thread Tomeu Vizoso
On 4 April 2016 at 17:44, Daniel Stone wrote: > Hi Tomeu, > > On 4 April 2016 at 14:55, Tomeu Vizoso wrote: >> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_fb.c >> b/drivers/gpu/drm/rockchip/rockchip_drm_fb.c >> index 3b8f652698f8..8305bbd2a4d7 100644 >> ---

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v3

2016-04-04 Thread Daniel Stone
Hi Tomeu, On 4 April 2016 at 14:55, Tomeu Vizoso wrote: > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_fb.c > b/drivers/gpu/drm/rockchip/rockchip_drm_fb.c > index 3b8f652698f8..8305bbd2a4d7 100644 > --- a/drivers/gpu/drm/rockchip/rockchip_drm_fb.c > +++

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v3

2016-04-04 Thread Daniel Stone
Hi Tomeu, On 4 April 2016 at 14:55, Tomeu Vizoso wrote: > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_fb.c > b/drivers/gpu/drm/rockchip/rockchip_drm_fb.c > index 3b8f652698f8..8305bbd2a4d7 100644 > --- a/drivers/gpu/drm/rockchip/rockchip_drm_fb.c > +++

[PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v3

2016-04-04 Thread Tomeu Vizoso
As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". v2: Use the status of the workqueue instead of vop->event, and don't add a superfluous wait on the workqueue. v3: Drop work_busy, as there's a sizeable delay

[PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v3

2016-04-04 Thread Tomeu Vizoso
As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". v2: Use the status of the workqueue instead of vop->event, and don't add a superfluous wait on the workqueue. v3: Drop work_busy, as there's a sizeable delay

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v2

2016-04-01 Thread Mark yao
On 2016年04月01日 19:47, Tomeu Vizoso wrote: On 04/01/2016 01:26 PM, Mark yao wrote: On 2016年03月31日 16:08, Tomeu Vizoso wrote: As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". v2: Use the status of the

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v2

2016-04-01 Thread Mark yao
On 2016年04月01日 19:47, Tomeu Vizoso wrote: On 04/01/2016 01:26 PM, Mark yao wrote: On 2016年03月31日 16:08, Tomeu Vizoso wrote: As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". v2: Use the status of the

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v2

2016-04-01 Thread Tomeu Vizoso
On 04/01/2016 01:26 PM, Mark yao wrote: > On 2016年03月31日 16:08, Tomeu Vizoso wrote: >> As per the docs, atomic_commit should return -EBUSY "if an asycnhronous >> updated is requested and there is an earlier updated pending". >> >> v2: Use the status of the workqueue instead of vop->event, and

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v2

2016-04-01 Thread Tomeu Vizoso
On 04/01/2016 01:26 PM, Mark yao wrote: > On 2016年03月31日 16:08, Tomeu Vizoso wrote: >> As per the docs, atomic_commit should return -EBUSY "if an asycnhronous >> updated is requested and there is an earlier updated pending". >> >> v2: Use the status of the workqueue instead of vop->event, and

[PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v2

2016-03-31 Thread Tomeu Vizoso
As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". v2: Use the status of the workqueue instead of vop->event, and don't add a superfluous wait on the workqueue. Signed-off-by: Tomeu Vizoso

[PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event v2

2016-03-31 Thread Tomeu Vizoso
As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". v2: Use the status of the workqueue instead of vop->event, and don't add a superfluous wait on the workqueue. Signed-off-by: Tomeu Vizoso ---

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event

2016-03-31 Thread Tomeu Vizoso
On 31 March 2016 at 03:25, Mark yao wrote: > On 2016年03月30日 21:48, Tomeu Vizoso wrote: >> >> As per the docs, atomic_commit should return -EBUSY "if an asycnhronous >> updated is requested and there is an earlier updated pending". >> >> Also wait for the pending event to

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event

2016-03-31 Thread Tomeu Vizoso
On 31 March 2016 at 03:25, Mark yao wrote: > On 2016年03月30日 21:48, Tomeu Vizoso wrote: >> >> As per the docs, atomic_commit should return -EBUSY "if an asycnhronous >> updated is requested and there is an earlier updated pending". >> >> Also wait for the pending event to complete when a sync

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event

2016-03-30 Thread Mark yao
On 2016年03月30日 21:48, Tomeu Vizoso wrote: As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". Also wait for the pending event to complete when a sync update is requested. Signed-off-by: Tomeu Vizoso

Re: [PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event

2016-03-30 Thread Mark yao
On 2016年03月30日 21:48, Tomeu Vizoso wrote: As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". Also wait for the pending event to complete when a sync update is requested. Signed-off-by: Tomeu Vizoso ---

[PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event

2016-03-30 Thread Tomeu Vizoso
As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". Also wait for the pending event to complete when a sync update is requested. Signed-off-by: Tomeu Vizoso ---

[PATCH] drm/rockchip: Return -EBUSY if there's already a pending flip event

2016-03-30 Thread Tomeu Vizoso
As per the docs, atomic_commit should return -EBUSY "if an asycnhronous updated is requested and there is an earlier updated pending". Also wait for the pending event to complete when a sync update is requested. Signed-off-by: Tomeu Vizoso --- drivers/gpu/drm/rockchip/rockchip_drm_drv.h | 1 +