[Intel-gfx] [PATCH 08/26] drm: Consolidate connector arrays in drm_atomic_state

2016-05-30 Thread Ville Syrjälä
On Mon, May 30, 2016 at 05:33:56PM +0200, Daniel Vetter wrote: > On Mon, May 30, 2016 at 06:25:50PM +0300, Ville Syrjälä wrote: > > On Mon, May 30, 2016 at 05:13:56PM +0200, Daniel Vetter wrote: > > > On Mon, May 30, 2016 at 05:59:24PM +0300, Ville Syrjälä wrote: > > > > On Sun, May 29, 2016 at

[Intel-gfx] [PATCH 08/26] drm: Consolidate connector arrays in drm_atomic_state

2016-05-30 Thread Ville Syrjälä
On Mon, May 30, 2016 at 05:13:56PM +0200, Daniel Vetter wrote: > On Mon, May 30, 2016 at 05:59:24PM +0300, Ville Syrjälä wrote: > > On Sun, May 29, 2016 at 08:35:05PM +0200, Daniel Vetter wrote: > > > It's kinda pointless to have 2 separate mallocs for these. And when we > > > add more per-connec

[Intel-gfx] [PATCH 08/26] drm: Consolidate connector arrays in drm_atomic_state

2016-05-30 Thread Ville Syrjälä
On Sun, May 29, 2016 at 08:35:05PM +0200, Daniel Vetter wrote: > It's kinda pointless to have 2 separate mallocs for these. And when we > add more per-connector state in the future it's even more pointless. > > Right now there's no such thing planned, but both Gustavo's per-crtc > fence patches, a

[Intel-gfx] [PATCH 08/26] drm: Consolidate connector arrays in drm_atomic_state

2016-05-30 Thread Daniel Vetter
On Mon, May 30, 2016 at 5:45 PM, Ville Syrjälä wrote: >> state->connectors[i].connector is really long, and makes a lot of code >> look ugly. "obj" might be a bit better than "ptr" at least. Something >> else? > > How often are you expecting to have to type this anyway? Using any > kind of gener

[Intel-gfx] [PATCH 08/26] drm: Consolidate connector arrays in drm_atomic_state

2016-05-30 Thread Daniel Vetter
On Mon, May 30, 2016 at 06:25:50PM +0300, Ville Syrjälä wrote: > On Mon, May 30, 2016 at 05:13:56PM +0200, Daniel Vetter wrote: > > On Mon, May 30, 2016 at 05:59:24PM +0300, Ville Syrjälä wrote: > > > On Sun, May 29, 2016 at 08:35:05PM +0200, Daniel Vetter wrote: > > > > It's kinda pointless to

[Intel-gfx] [PATCH 08/26] drm: Consolidate connector arrays in drm_atomic_state

2016-05-30 Thread Daniel Vetter
On Mon, May 30, 2016 at 05:59:24PM +0300, Ville Syrjälä wrote: > On Sun, May 29, 2016 at 08:35:05PM +0200, Daniel Vetter wrote: > > It's kinda pointless to have 2 separate mallocs for these. And when we > > add more per-connector state in the future it's even more pointless. > > > > Right now th

[PATCH 08/26] drm: Consolidate connector arrays in drm_atomic_state

2016-05-29 Thread Daniel Vetter
It's kinda pointless to have 2 separate mallocs for these. And when we add more per-connector state in the future it's even more pointless. Right now there's no such thing planned, but both Gustavo's per-crtc fence patches, and some nonblocking commit helpers I'm playing around with will add more