Re: [Intel-gfx] [PATCH] drm: atomic: fix legacy gamma set helper

2016-04-18 Thread Daniel Vetter
On Mon, Apr 18, 2016 at 11:07:43AM +0100, Lionel Landwerlin wrote: > On 12/04/16 17:07, Daniel Vetter wrote: > >On Tue, Apr 12, 2016 at 01:51:02PM +0100, Lionel Landwerlin wrote: > >>On 12/04/16 12:57, Daniel Vetter wrote: > >>>On Mon, Apr 11, 2016 at 02:43:39PM +0100, Lionel Landwerlin wrote: > >>

Re: [Intel-gfx] [PATCH] drm: atomic: fix legacy gamma set helper

2016-04-18 Thread Lionel Landwerlin
On 12/04/16 17:07, Daniel Vetter wrote: On Tue, Apr 12, 2016 at 01:51:02PM +0100, Lionel Landwerlin wrote: On 12/04/16 12:57, Daniel Vetter wrote: On Mon, Apr 11, 2016 at 02:43:39PM +0100, Lionel Landwerlin wrote: Color management properties are a bit of an odd use case because they're not mar

Re: [Intel-gfx] [PATCH] drm: atomic: fix legacy gamma set helper

2016-04-12 Thread Daniel Vetter
On Tue, Apr 12, 2016 at 01:51:02PM +0100, Lionel Landwerlin wrote: > On 12/04/16 12:57, Daniel Vetter wrote: > >On Mon, Apr 11, 2016 at 02:43:39PM +0100, Lionel Landwerlin wrote: > >>Color management properties are a bit of an odd use case because > >>they're not marked as atomic properties. Curren

Re: [Intel-gfx] [PATCH] drm: atomic: fix legacy gamma set helper

2016-04-12 Thread Lionel Landwerlin
On 12/04/16 12:57, Daniel Vetter wrote: On Mon, Apr 11, 2016 at 02:43:39PM +0100, Lionel Landwerlin wrote: Color management properties are a bit of an odd use case because they're not marked as atomic properties. Currently we're not updating the non atomic values so the drm_crtc_state is out of

Re: [Intel-gfx] [PATCH] drm: atomic: fix legacy gamma set helper

2016-04-12 Thread Daniel Vetter
On Mon, Apr 11, 2016 at 02:43:39PM +0100, Lionel Landwerlin wrote: > Color management properties are a bit of an odd use case because > they're not marked as atomic properties. Currently we're not updating > the non atomic values so the drm_crtc_state is out of sync with the > values stored in the

Re: [Intel-gfx] [PATCH] drm: atomic: fix legacy gamma set helper

2016-04-11 Thread Bob Paauwe
On Mon, 11 Apr 2016 14:43:39 +0100 Lionel Landwerlin wrote: > Color management properties are a bit of an odd use case because > they're not marked as atomic properties. Currently we're not updating > the non atomic values so the drm_crtc_state is out of sync with the > values stored in the crtc

[Intel-gfx] [PATCH] drm: atomic: fix legacy gamma set helper

2016-04-11 Thread Lionel Landwerlin
Color management properties are a bit of an odd use case because they're not marked as atomic properties. Currently we're not updating the non atomic values so the drm_crtc_state is out of sync with the values stored in the crtc object. v2: Update non atomic values only if commit succeeds (Bob Paa

Re: [Intel-gfx] [PATCH] drm: atomic: fix legacy gamma set helper

2016-04-11 Thread Maarten Lankhorst
Op 11-04-16 om 12:37 schreef Lionel Landwerlin: > Color management properties are a bit of an odd use case because > they're not marked as atomic properties. Currently we're not updating > the non atomic values so the drm_crtc_state is out of sync with the > values stored in the crtc object. > > v2

[Intel-gfx] [PATCH] drm: atomic: fix legacy gamma set helper

2016-04-11 Thread Lionel Landwerlin
Color management properties are a bit of an odd use case because they're not marked as atomic properties. Currently we're not updating the non atomic values so the drm_crtc_state is out of sync with the values stored in the crtc object. v2: Update non atomic values only if commit succeeds (Bob Paa

Re: [Intel-gfx] [PATCH] drm: atomic: fix legacy gamma set helper

2016-04-08 Thread Bob Paauwe
On Fri, 8 Apr 2016 18:17:51 +0100 Lionel Landwerlin wrote: > Color management properties are a bit of an odd use case because > they're not marked as atomic properties. Currently we're not updating > the non atomic values so the drm_crtc_state is out of sync with the > values stored in the crtc o

[Intel-gfx] [PATCH] drm: atomic: fix legacy gamma set helper

2016-04-08 Thread Lionel Landwerlin
Color management properties are a bit of an odd use case because they're not marked as atomic properties. Currently we're not updating the non atomic values so the drm_crtc_state is out of sync with the values stored in the crtc object. Cc: Maarten Lankhorst Cc: Bob Paauwe Cc: Signed-off-by: Li