On Wed, Feb 27, 2019 at 04:24:21PM +0000, Claudiu.Beznea@xxxxxxxxxxxxx wrote: > From: Claudiu Beznea <claudiu.beznea@xxxxxxxxxxxxx> > > 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.beznea@xxxxxxxxxxxxx> Looks good, good to have simpler code. Reviewed-by: Sam Ravnborg <sam@xxxxxxxxxxxx>