Re: [PATCH] kms_atomic: Fix crtc_id comparison failing.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Apr 22, 2016 at 12:54:59PM +0300, Marius Vlad wrote:
> You've already sent this yesterday. Its pushed.

When you push patches (especially from other people), please send out a
quick confirmation mail to the mailing list. Avoids confusion like this
one here.
-Daniel

> 
> On Thu, Apr 21, 2016 at 10:24:28PM -0400, robert.foss@xxxxxxxxxxxxx wrote:
> > From: Robert Foss <robert.foss@xxxxxxxxxxxxx>
> > 
> > Fixed ctrc_id comparison failing due to bad initialization of
> > crtc variable.
> > 
> > Signed-off-by: Robert Foss <robert.foss@xxxxxxxxxxxxx>
> > ---
> >  tests/kms_atomic.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/tests/kms_atomic.c b/tests/kms_atomic.c
> > index 13a5db1..f27ee46 100644
> > --- a/tests/kms_atomic.c
> > +++ b/tests/kms_atomic.c
> > @@ -1361,7 +1361,7 @@ igt_main
> >  
> >  	igt_subtest("atomic_invalid_params") {
> >  		struct kms_atomic_state *scratch = atomic_state_dup(current);
> > -		struct kms_atomic_crtc_state *crtc = &scratch->crtcs[0];
> > +		struct kms_atomic_crtc_state *crtc = find_crtc(scratch, true);
> >  		struct kms_atomic_plane_state *plane =
> >  			find_plane(scratch, NUM_PLANE_TYPE_PROPS, crtc);
> >  		struct kms_atomic_connector_state *conn =
> > -- 
> > 2.5.0
> > 
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx



> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx


-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux