On Tue, May 31, 2016 at 03:05:32PM +0200, Maarten Lankhorst wrote: > Op 30-03-16 om 11:51 schreef Daniel Vetter: > > Code stolen from gma500. > It would be useful to mention why this is done. :) > > But even with this patch it seems the legacy gamma in gamma_get is not very > reliable. > A failed call to gamma_set could leave wrong values in crtc->gamma_store > either by calling the ioctl with lut.blue set to NULL, or call gamma_set and > interrupt with a signal. > For atomic color management get_gamma won't be accurate at all if mixed with > new color management.
Hm, it's just some safety code I found laying around, and figured that looks like a good idea. You think it's worth it with some improved commit message added? -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch