On Mon, Oct 23, 2017 at 12:15:34PM +0200, Maarten Lankhorst wrote: > Op 23-10-17 om 11:35 schreef Petri Latvala: > > > > > > On Fri, Oct 20, 2017 at 03:24:15PM +0200, Maarten Lankhorst wrote: > > > > Subject: [PATCH i-g-t] tests/kms_atomic_transition: Make tests pass > > > > > > You have an excellent explanation of what this patch does in the long > > description, and yet this short description says nothing of value. > > > > > > > Is this patch good with subject: 'tests/kms_atomic_transition: Do not update unbound planes'? > LGTM. Explain the change of making fd_completed() call conditional too in the commit message and Reviewed-by: Petri Latvala <petri.latvala@xxxxxxxxx> _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
