Re: [PATCH] drm: Fix crtc color management when doing suspend/resume

2018-08-24 Thread Maarten Lankhorst
Op 24-08-18 om 10:18 schreef Alexandru-Cosmin Gheorghe: > Hi, > > On Fri, Aug 24, 2018 at 09:51:03AM +0200, Maarten Lankhorst wrote: >> Op 23-08-18 om 17:11 schreef Alexandru Gheorghe: >>> When doing suspend/resume drivers usually use the >>> drm_atomic_helper_suspend/drm_atomic_helper_resume pair

Re: [PATCH] drm: Fix crtc color management when doing suspend/resume

2018-08-24 Thread Alexandru-Cosmin Gheorghe
Hi, On Fri, Aug 24, 2018 at 09:51:03AM +0200, Maarten Lankhorst wrote: > Op 23-08-18 om 17:11 schreef Alexandru Gheorghe: > > When doing suspend/resume drivers usually use the > > drm_atomic_helper_suspend/drm_atomic_helper_resume pair for saving the > > state and then re-comitting it. > > > >

Re: [PATCH] drm: Fix crtc color management when doing suspend/resume

2018-08-24 Thread Maarten Lankhorst
Op 23-08-18 om 17:11 schreef Alexandru Gheorghe: > When doing suspend/resume drivers usually use the > drm_atomic_helper_suspend/drm_atomic_helper_resume pair for saving the > state and then re-comitting it. > > The problems is that drm_crtc_state has a bool field called > color_mgmt_changed,

Re: [PATCH] drm: Fix crtc color management when doing suspend/resume

2018-08-23 Thread Brian Starkey
Hi Alex, On Thu, Aug 23, 2018 at 04:11:17PM +0100, Alexandru Gheorghe wrote: When doing suspend/resume drivers usually use the drm_atomic_helper_suspend/drm_atomic_helper_resume pair for saving the state and then re-comitting it. The problems is that drm_crtc_state has a bool field called

[PATCH] drm: Fix crtc color management when doing suspend/resume

2018-08-23 Thread Alexandru Gheorghe
When doing suspend/resume drivers usually use the drm_atomic_helper_suspend/drm_atomic_helper_resume pair for saving the state and then re-comitting it. The problems is that drm_crtc_state has a bool field called color_mgmt_changed, which mali-dp and other drivers uses it to detect if