Re: [PATCH 0/3] drm/exynos: modify usage of wait for vblank

2012-12-05 Thread Prathyush K
On Wed, Dec 5, 2012 at 5:33 PM, Eunchul Kim wrote: > Dear Prathyush > > Thank's your clarification. > > I have one opinion about your [PATCH 1/3] > How about to add atomic_read() condition in each irq_handler ? > like this. > > e.g) > Hi Eunchul, Yes, I think this is better. I'll modify this in

Re: [PATCH 0/3] drm/exynos: modify usage of wait for vblank

2012-12-05 Thread Eunchul Kim
Dear Prathyush Thank's your clarification. I have one opinion about your [PATCH 1/3] How about to add atomic_read() condition in each irq_handler ? like this. e.g) + if (atomic_read(&ctx->wait_vsync_event)) { + /* set wait vsync event to zero and wake up queue. */ +

Re: [PATCH 0/3] drm/exynos: modify usage of wait for vblank

2012-12-04 Thread Inki Dae
2012/12/5 Inki Dae > > > 2012/12/5 Inki Dae > >> >> >> 2012/12/5 Prathyush K >> >>> >>> >>> >>> On Wed, Dec 5, 2012 at 11:40 AM, Inki Dae wrote: >>> 2012/12/5 Inki Dae > > > 2012/12/5 Prathyush K > >> This patchset fixes a few issues with use of wait f

Re: [PATCH 0/3] drm/exynos: modify usage of wait for vblank

2012-12-04 Thread Inki Dae
2012/12/5 Inki Dae > > > 2012/12/5 Prathyush K > >> >> >> >> On Wed, Dec 5, 2012 at 11:40 AM, Inki Dae wrote: >> >>> >>> >>> 2012/12/5 Inki Dae >>> 2012/12/5 Prathyush K > This patchset fixes a few issues with use of wait for vblank in > exynos drm. > > Ple

Re: [PATCH 0/3] drm/exynos: modify usage of wait for vblank

2012-12-04 Thread Inki Dae
2012/12/5 Prathyush K > > > > On Wed, Dec 5, 2012 at 11:40 AM, Inki Dae wrote: > >> >> >> 2012/12/5 Inki Dae >> >>> >>> >>> 2012/12/5 Prathyush K >>> This patchset fixes a few issues with use of wait for vblank in exynos drm. Please apply these three patches without "drm/ex

Re: [PATCH 0/3] drm/exynos: modify usage of wait for vblank

2012-12-04 Thread Prathyush K
On Wed, Dec 5, 2012 at 11:40 AM, Inki Dae wrote: > > > 2012/12/5 Inki Dae > >> >> >> 2012/12/5 Prathyush K >> >>> This patchset fixes a few issues with use of wait for vblank in >>> exynos drm. >>> >>> Please apply these three patches without "drm/exynos: release fb pended >>> by page flip" >>>

Re: [PATCH 0/3] drm/exynos: modify usage of wait for vblank

2012-12-04 Thread Inki Dae
2012/12/5 Inki Dae > > > 2012/12/5 Prathyush K > >> This patchset fixes a few issues with use of wait for vblank in >> exynos drm. >> >> Please apply these three patches without "drm/exynos: release fb pended >> by page flip" >> patch. >> >> Patch 1: modify wait for vsync functions to use wait q

Re: [PATCH 0/3] drm/exynos: modify usage of wait for vblank

2012-12-04 Thread Inki Dae
2012/12/5 Prathyush K > This patchset fixes a few issues with use of wait for vblank in > exynos drm. > > Please apply these three patches without "drm/exynos: release fb pended by > page flip" > patch. > > Patch 1: modify wait for vsync functions to use wait queues > This modifies the wait_for_v