* [PATCH i-g-t] kms_universal_plane: Don't assert outside of fixtures/subtests
@ 2014-07-04 11:01 Damien Lespiau
2014-07-07 21:18 ` Daniel Vetter
0 siblings, 1 reply; 3+ messages in thread
From: Damien Lespiau @ 2014-07-04 11:01 UTC (permalink / raw)
To: intel-gfx
Doing otherwise breaks listing the subtests. The test was throwing an
error out when universal planes were disabled as well because of that.
Cc: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
---
tests/kms_universal_plane.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/tests/kms_universal_plane.c b/tests/kms_universal_plane.c
index fe0fde3..c64cdbb 100644
--- a/tests/kms_universal_plane.c
+++ b/tests/kms_universal_plane.c
@@ -133,6 +133,7 @@ functional_test_pipe(data_t *data, enum pipe pipe, igt_output_t *output)
int num_primary = 0, num_cursor = 0;
int i;
+ igt_assert(data->display.has_universal_planes);
igt_skip_on(pipe >= display->n_pipes);
fprintf(stdout, "Testing connector %s using pipe %c\n",
@@ -535,8 +536,6 @@ run_tests_for_pipe(data_t *data, enum pipe pipe)
{
igt_output_t *output;
- igt_assert(data->display.has_universal_planes);
-
igt_subtest_f("universal-plane-pipe-%c-functional", pipe_name(pipe))
for_each_connected_output(&data->display, output)
functional_test_pipe(data, pipe, output);
--
1.8.3.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH i-g-t] kms_universal_plane: Don't assert outside of fixtures/subtests
2014-07-04 11:01 [PATCH i-g-t] kms_universal_plane: Don't assert outside of fixtures/subtests Damien Lespiau
@ 2014-07-07 21:18 ` Daniel Vetter
2014-07-07 23:01 ` Damien Lespiau
0 siblings, 1 reply; 3+ messages in thread
From: Daniel Vetter @ 2014-07-07 21:18 UTC (permalink / raw)
To: Damien Lespiau; +Cc: intel-gfx
On Fri, Jul 04, 2014 at 12:01:24PM +0100, Damien Lespiau wrote:
> Doing otherwise breaks listing the subtests. The test was throwing an
> error out when universal planes were disabled as well because of that.
>
> Cc: Matt Roper <matthew.d.roper@intel.com>
> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
> ---
> tests/kms_universal_plane.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/tests/kms_universal_plane.c b/tests/kms_universal_plane.c
> index fe0fde3..c64cdbb 100644
> --- a/tests/kms_universal_plane.c
> +++ b/tests/kms_universal_plane.c
> @@ -133,6 +133,7 @@ functional_test_pipe(data_t *data, enum pipe pipe, igt_output_t *output)
> int num_primary = 0, num_cursor = 0;
> int i;
>
> + igt_assert(data->display.has_universal_planes);
Shouldn't this be an igt_require instead? In any case moving this into
subtests is correct.
-Daniel
> igt_skip_on(pipe >= display->n_pipes);
>
> fprintf(stdout, "Testing connector %s using pipe %c\n",
> @@ -535,8 +536,6 @@ run_tests_for_pipe(data_t *data, enum pipe pipe)
> {
> igt_output_t *output;
>
> - igt_assert(data->display.has_universal_planes);
> -
> igt_subtest_f("universal-plane-pipe-%c-functional", pipe_name(pipe))
> for_each_connected_output(&data->display, output)
> functional_test_pipe(data, pipe, output);
> --
> 1.8.3.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH i-g-t] kms_universal_plane: Don't assert outside of fixtures/subtests
2014-07-07 21:18 ` Daniel Vetter
@ 2014-07-07 23:01 ` Damien Lespiau
0 siblings, 0 replies; 3+ messages in thread
From: Damien Lespiau @ 2014-07-07 23:01 UTC (permalink / raw)
To: Daniel Vetter; +Cc: intel-gfx
On Mon, Jul 07, 2014 at 11:18:49PM +0200, Daniel Vetter wrote:
> On Fri, Jul 04, 2014 at 12:01:24PM +0100, Damien Lespiau wrote:
> > Doing otherwise breaks listing the subtests. The test was throwing an
> > error out when universal planes were disabled as well because of that.
> >
> > Cc: Matt Roper <matthew.d.roper@intel.com>
> > Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
> > ---
> > tests/kms_universal_plane.c | 3 +--
> > 1 file changed, 1 insertion(+), 2 deletions(-)
> >
> > diff --git a/tests/kms_universal_plane.c b/tests/kms_universal_plane.c
> > index fe0fde3..c64cdbb 100644
> > --- a/tests/kms_universal_plane.c
> > +++ b/tests/kms_universal_plane.c
> > @@ -133,6 +133,7 @@ functional_test_pipe(data_t *data, enum pipe pipe, igt_output_t *output)
> > int num_primary = 0, num_cursor = 0;
> > int i;
> >
> > + igt_assert(data->display.has_universal_planes);
>
> Shouldn't this be an igt_require instead? In any case moving this into
> subtests is correct.
There's already a igt_require in the fixture block, I guess Matt wanted
to be extra extra sure. Could have removed it as well.
--
Damien
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-07-07 23:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-04 11:01 [PATCH i-g-t] kms_universal_plane: Don't assert outside of fixtures/subtests Damien Lespiau
2014-07-07 21:18 ` Daniel Vetter
2014-07-07 23:01 ` Damien Lespiau
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.