| Message ID | 1479117566-17709-5-git-send-email-gustavo@padovan.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Mon, Nov 14, 2016 at 06:59:16PM +0900, Gustavo Padovan wrote: > From: Gustavo Padovan <gustavo.padovan@collabora.co.uk> > > Signed-off-by: Gustavo Padovan <gustavo.padovan@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? -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@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
On 15 November 2016 at 09:01, Daniel Vetter <daniel@ffwll.ch> wrote: > On Mon, Nov 14, 2016 at 06:59:16PM +0900, Gustavo Padovan wrote: >> From: Gustavo Padovan <gustavo.padovan@collabora.co.uk> >> >> Signed-off-by: Gustavo Padovan <gustavo.padovan@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. Regards, Tomeu > -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@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@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel
On Tue, 2016-11-15 at 14:25 +0100, Tomeu Vizoso wrote: > On 15 November 2016 at 09:01, Daniel Vetter <daniel@ffwll.ch> wrote: > > > > On Mon, Nov 14, 2016 at 06:59:16PM +0900, Gustavo Padovan wrote: > > > > > > From: Gustavo Padovan <gustavo.padovan@collabora.co.uk> > > > > > > Signed-off-by: Gustavo Padovan <gustavo.padovan@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@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@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/dri-devel > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel
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; igt_pipe_crc_t *pipe_crcs[I915_MAX_PIPES]; igt_output_t *output; unsigned width = 0, height = 0;