Re: [PATCH v3 1/7] drm/atomic: Add new iterators over all state, v3.

2017-02-12 Thread Laurent Pinchart
Hi Maarten, Thank you for the patch. On Monday 16 Jan 2017 10:37:38 Maarten Lankhorst wrote: > Add for_each_(old)(new)_(plane,connector,crtc)_in_state iterators to > replace the old for_each_xxx_in_state ones. This is useful for >1 flip > depth and getting rid of all xxx->state dereferences. >

Re: [PATCH v3 1/7] drm/atomic: Add new iterators over all state, v3.

2017-02-12 Thread Laurent Pinchart
Hi Daniel, On Monday 23 Jan 2017 09:48:54 Daniel Vetter wrote: > On Thu, Jan 19, 2017 at 12:56:29AM +0200, Laurent Pinchart wrote: > > On Tuesday 17 Jan 2017 08:41:03 Maarten Lankhorst wrote: > >> Op 17-01-17 om 00:11 schreef Laurent Pinchart: > >>> On Monday 16 Jan 2017 10:37:38 Maarten

Re: [PATCH v3 1/7] drm/atomic: Add new iterators over all state, v3.

2017-01-23 Thread Daniel Vetter
On Thu, Jan 19, 2017 at 12:56:29AM +0200, Laurent Pinchart wrote: > On Tuesday 17 Jan 2017 08:41:03 Maarten Lankhorst wrote: > > Op 17-01-17 om 00:11 schreef Laurent Pinchart: > > > On Monday 16 Jan 2017 10:37:38 Maarten Lankhorst wrote: > > >> @@ -2512,6 +2518,47 @@ struct drm_atomic_state > > >>

Re: [PATCH v3 1/7] drm/atomic: Add new iterators over all state, v3.

2017-01-18 Thread Laurent Pinchart
Hi Maarten, On Tuesday 17 Jan 2017 08:41:03 Maarten Lankhorst wrote: > Op 17-01-17 om 00:11 schreef Laurent Pinchart: > > On Monday 16 Jan 2017 10:37:38 Maarten Lankhorst wrote: > >> Add for_each_(old)(new)_(plane,connector,crtc)_in_state iterators to > >> replace the old for_each_xxx_in_state

Re: [PATCH v3 1/7] drm/atomic: Add new iterators over all state, v3.

2017-01-16 Thread Maarten Lankhorst
Op 17-01-17 om 00:11 schreef Laurent Pinchart: > Hi Maarten, > > Thank you for the patch. > > On Monday 16 Jan 2017 10:37:38 Maarten Lankhorst wrote: >> Add for_each_(old)(new)_(plane,connector,crtc)_in_state iterators to >> replace the old for_each_xxx_in_state ones. This is useful for >1 flip >>

Re: [PATCH v3 1/7] drm/atomic: Add new iterators over all state, v3.

2017-01-16 Thread Laurent Pinchart
Hi Maarten, Thank you for the patch. On Monday 16 Jan 2017 10:37:38 Maarten Lankhorst wrote: > Add for_each_(old)(new)_(plane,connector,crtc)_in_state iterators to > replace the old for_each_xxx_in_state ones. This is useful for >1 flip > depth and getting rid of all xxx->state dereferences. >

[PATCH v3 1/7] drm/atomic: Add new iterators over all state, v3.

2017-01-16 Thread Maarten Lankhorst
Add for_each_(old)(new)_(plane,connector,crtc)_in_state iterators to replace the old for_each_xxx_in_state ones. This is useful for >1 flip depth and getting rid of all xxx->state dereferences. This requires extra fixups done when committing a state after duplicating, which in general isn't valid