Re: [PATCH 3/5] drm/armada: Construct a temporary crtc state for plane checks

On Tue, Jan 23, 2018 at 07:08:55PM +0200, Ville Syrjala wrote:

From: Ville Syrjälä ville.syrjala@linux.intel.com

As armada isn't an atomic driver trying to pass a non-populated crtc->state to drm_atomic_helper_check_plane_state() will end in tears. Construct a temporary crtc state a la drm_plane_helper_check_update() and pass that instead. For now we just really need crtc_state->enable to be there.

Would it be possible to solve this by having the atomic state setup for non-atomic drivers instead, so we're not unwinding some of the work that's already been done to try and convert drivers /to/ atomic modeset?

Thanks.