On Wed, Feb 27, 2019 at 04:24:21PM +0000, claudiu.bez...@microchip.com wrote: > From: Claudiu Beznea <claudiu.bez...@microchip.com> > > Remove cfg initialization with zero and read state with > drm_crtc_state_to_atmel_hlcdc_crtc_state() so that cfg to be initialized > with state's output_mode. > > Signed-off-by: Claudiu Beznea <claudiu.bez...@microchip.com> Looks good, good to have simpler code. Reviewed-by: Sam Ravnborg <s...@ravnborg.org>