On Mon, Mar 05, 2018 at 09:41:43AM +0100, Daniel Vetter wrote:
On Tue, Jan 23, 2018 at 07:08:56PM +0200, Ville Syrjala wrote:
From: Ville Syrjälä ville.syrjala@linux.intel.com
Use drm_mode_get_hv_timing() to fill out the plane clip rectangle.
Since armada isn't atomic we'll use crtc->enabled and crtc->mode instead of the stuff under crtc_state.
Once everyone agrees on how the clip rectangle gets set up we can move the code into drm_atomic_helper_check_plane_state().
Cc: Dave Airlie airlied@redhat.com Cc: Russell King rmk+kernel@armlinux.org.uk Signed-off-by: Ville Syrjälä ville.syrjala@linux.intel.com
For patches 3&4: Reviewed-by: Daniel Vetter daniel.vetter@ffwll.ch
In case they do unexpectly break armada we can always just copypaste the previous version of the non-atomic helper into armada code to fix any regressions cropping up, so I don't see any big risks here. And I'd like to get this cleanup merged, just stumbled over it again.
I pinged Dave Airlie on irc, and he's fine with that approach too.
Acked-by: Dave Airlie airlied@redhat.com
for the same two patches.
Cool. Thank you both.
Remainder of the series pushed to drm-misc-next.