Re: [Freedreno] [PATCH 2/2] drm/msm: subclass work object for vblank events

2018-11-05 Thread Jeykumar Sankaran
On 2018-11-05 09:24, Sean Paul wrote: On Fri, Nov 02, 2018 at 04:38:48PM -0700, Jeykumar Sankaran wrote: On 2018-11-01 12:18, Sean Paul wrote: > On Wed, Oct 31, 2018 at 05:19:05PM -0700, Jeykumar Sankaran wrote: > > msm maintains a separate structure to define vblank > > work definitions and a

Re: [Freedreno] [PATCH 2/2] drm/msm: subclass work object for vblank events

2018-11-05 Thread Sean Paul
On Fri, Nov 02, 2018 at 04:38:48PM -0700, Jeykumar Sankaran wrote: > On 2018-11-01 12:18, Sean Paul wrote: > > On Wed, Oct 31, 2018 at 05:19:05PM -0700, Jeykumar Sankaran wrote: > > > msm maintains a separate structure to define vblank > > > work definitions and a list to track events submitted >

Re: [Freedreno] [PATCH 2/2] drm/msm: subclass work object for vblank events

2018-11-02 Thread Jeykumar Sankaran
On 2018-11-01 12:18, Sean Paul wrote: On Wed, Oct 31, 2018 at 05:19:05PM -0700, Jeykumar Sankaran wrote: msm maintains a separate structure to define vblank work definitions and a list to track events submitted to the display worker thread. We can avoid these redundant list and its protection

Re: [Freedreno] [PATCH 2/2] drm/msm: subclass work object for vblank events

2018-11-01 Thread Sean Paul
On Wed, Oct 31, 2018 at 05:19:05PM -0700, Jeykumar Sankaran wrote: > msm maintains a separate structure to define vblank > work definitions and a list to track events submitted > to the display worker thread. We can avoid these > redundant list and its protection mechanism, if we > subclass the

[Freedreno] [PATCH 2/2] drm/msm: subclass work object for vblank events

2018-10-31 Thread Jeykumar Sankaran
msm maintains a separate structure to define vblank work definitions and a list to track events submitted to the display worker thread. We can avoid these redundant list and its protection mechanism, if we subclass the work object to encapsulate vblank event parameters. Signed-off-by: Jeykumar