Re: [PATCH xf86-video-amdgpu] Add checking color management properties

2018-09-11 Thread Michel Dänzer
On 2018-09-10 5:58 p.m., Michel Dänzer wrote: > On 2018-09-10 1:14 p.m., Aaron Liu wrote: >> Add gamma_lut/degamma_lut/ctm checking before pushing >> staged color management properties on the CRTC. >> If above object is NULL, return directly. > > (How) were you able to actually trigger this

Re: [PATCH xf86-video-amdgpu] Add checking color management properties

2018-09-10 Thread Michel Dänzer
Hi Aaron, thanks for the patch. Moving to the amd-gfx mailing list, where xf86-video-amdgpu patches are reviewed. Comments inline below. On 2018-09-10 1:14 p.m., Aaron Liu wrote: > Add gamma_lut/degamma_lut/ctm checking before pushing > staged color management properties on the CRTC. > If