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@xxxxxxxxxxxxxxx> > > > > 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@xxxxxxxxxx> > > Cc: Russell King <rmk+kernel@xxxxxxxxxxxxxxx> > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > For patches 3&4: Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx> > > 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@xxxxxxxxxx> > > for the same two patches. Cool. Thank you both. Remainder of the series pushed to drm-misc-next. -- Ville Syrjälä Intel OTC _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
