[Intel-gfx] [PATCH 02/12] tests/kms_atomic_transition: don't assume max pipes

Robert Foss robert.foss at collabora.com
Tue Nov 15 15:30:35 UTC 2016


On Tue, 2016-11-15 at 14:25 +0100, Tomeu Vizoso wrote:
> On 15 November 2016 at 09:01, Daniel Vetter <daniel at ffwll.ch> wrote:
> > 
> > On Mon, Nov 14, 2016 at 06:59:16PM +0900, Gustavo Padovan wrote:
> > > 
> > > From: Gustavo Padovan <gustavo.padovan at collabora.co.uk>
> > > 
> > > Signed-off-by: Gustavo Padovan <gustavo.padovan at collabora.co.uk>
> > > ---
> > >  tests/kms_atomic_transition.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/tests/kms_atomic_transition.c
> > > b/tests/kms_atomic_transition.c
> > > index e693c88..8b26b53 100644
> > > --- a/tests/kms_atomic_transition.c
> > > +++ b/tests/kms_atomic_transition.c
> > > @@ -404,7 +404,7 @@ static void run_modeset_tests(igt_display_t
> > > *display, int howmany, bool nonblock
> > >  {
> > >       struct igt_fb fbs[2];
> > >       int i, j;
> > > -     unsigned iter_max = 1 << I915_MAX_PIPES;
> > > +     unsigned iter_max = 1 << display->n_pipes;
> > Didn't Tomeu have some patch series to fix these all up?
> Don't remember, and couldn't find any within my local branches. Maybe
> Robert? But I'm adding it to my backlog anyway.
> 

I don't recognize it, but thanks tomeu!

Rob.

> 
> > 
> > -Daniel
> > 
> > > 
> > >       igt_pipe_crc_t *pipe_crcs[I915_MAX_PIPES];
> > >       igt_output_t *output;
> > >       unsigned width = 0, height = 0;
> > > --
> > > 2.5.5
> > > 
> > > _______________________________________________
> > > Intel-gfx mailing list
> > > Intel-gfx at lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> > --
> > Daniel Vetter
> > Software Engineer, Intel Corporation
> > http://blog.ffwll.ch
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel


More information about the dri-devel mailing list