* Re: [igt-dev] [PATCH] [i-g-t] tests/i915_pm_lpsp: remove screens-disabled subtest [not found] <20200831115515.16531-1-tejaskumarx.surendrakumar.upadhyay@intel.com> @ 2020-08-31 11:55 ` Anshuman Gupta 2020-08-31 12:10 ` Surendrakumar Upadhyay, TejaskumarX 0 siblings, 1 reply; 3+ messages in thread From: Anshuman Gupta @ 2020-08-31 11:55 UTC (permalink / raw) To: Tejas Upadhyay; +Cc: igt-dev On 2020-08-31 at 17:25:15 +0530, Tejas Upadhyay wrote: > screens-disabled subtest is not required to run when > DMC firmware is loaded. Thus this test code becomes > redundant and increases needless SKIP counts. > > Cc: Anshuman Gupta <Anshuman.Gupta@intel.com> > Signed-off-by: Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay@intel.com> > --- > tests/i915/i915_pm_lpsp.c | 7 ------- > 1 file changed, 7 deletions(-) > > diff --git a/tests/i915/i915_pm_lpsp.c b/tests/i915/i915_pm_lpsp.c > index 4a187810..d1efd9e0 100644 > --- a/tests/i915/i915_pm_lpsp.c > +++ b/tests/i915/i915_pm_lpsp.c > @@ -185,13 +185,6 @@ igt_main > igt_require(igt_pm_dmc_loaded(data.debugfs_fd)); > } > > - igt_describe("This test validates lpsp while all crtc are disabled"); > - igt_subtest("screens-disabled") { > - igt_require_f(!dmc_supported(data.debugfs_fd), > - "DC states supported platform don't have ROI for this subtest\n"); > - screens_disabled_subtest(&data); > - } > - Platfrom like HASWELL still need this test, you can't simply remove the test, this is a valid skip for platform above skl+. Thanks, Anshuman Gupta. > igt_describe("This test validates lpsp on all connected outputs on low power PIPE_A"); > igt_subtest_with_dynamic_f("kms-lpsp") { > igt_display_t *display = &data.display; > -- > 2.25.1 > _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [igt-dev] [PATCH] [i-g-t] tests/i915_pm_lpsp: remove screens-disabled subtest 2020-08-31 11:55 ` [igt-dev] [PATCH] [i-g-t] tests/i915_pm_lpsp: remove screens-disabled subtest Anshuman Gupta @ 2020-08-31 12:10 ` Surendrakumar Upadhyay, TejaskumarX 2020-08-31 12:07 ` Anshuman Gupta 0 siblings, 1 reply; 3+ messages in thread From: Surendrakumar Upadhyay, TejaskumarX @ 2020-08-31 12:10 UTC (permalink / raw) To: Gupta, Anshuman; +Cc: igt-dev@lists.freedesktop.org Inline -----Original Message----- From: Anshuman Gupta <anshuman.gupta@intel.com> Sent: 31 August 2020 17:25 To: Surendrakumar Upadhyay, TejaskumarX <tejaskumarx.surendrakumar.upadhyay@intel.com> Cc: igt-dev@lists.freedesktop.org Subject: Re: [PATCH] [i-g-t] tests/i915_pm_lpsp: remove screens-disabled subtest On 2020-08-31 at 17:25:15 +0530, Tejas Upadhyay wrote: > screens-disabled subtest is not required to run when DMC firmware is > loaded. Thus this test code becomes redundant and increases needless > SKIP counts. > > Cc: Anshuman Gupta <Anshuman.Gupta@intel.com> > Signed-off-by: Tejas Upadhyay > <tejaskumarx.surendrakumar.upadhyay@intel.com> > --- > tests/i915/i915_pm_lpsp.c | 7 ------- > 1 file changed, 7 deletions(-) > > diff --git a/tests/i915/i915_pm_lpsp.c b/tests/i915/i915_pm_lpsp.c > index 4a187810..d1efd9e0 100644 > --- a/tests/i915/i915_pm_lpsp.c > +++ b/tests/i915/i915_pm_lpsp.c > @@ -185,13 +185,6 @@ igt_main > igt_require(igt_pm_dmc_loaded(data.debugfs_fd)); > } > > - igt_describe("This test validates lpsp while all crtc are disabled"); > - igt_subtest("screens-disabled") { > - igt_require_f(!dmc_supported(data.debugfs_fd), > - "DC states supported platform don't have ROI for this subtest\n"); > - screens_disabled_subtest(&data); > - } > - Platfrom like HASWELL still need this test, you can't simply remove the test, this is a valid skip for platform above skl+. Thanks, Anshuman Gupta. Tejas : But those platforms will have DMC loaded? If DMC not loaded test will not enter in. And if DMC loaded this subtest will not run. How it will get executed? > igt_describe("This test validates lpsp on all connected outputs on low power PIPE_A"); > igt_subtest_with_dynamic_f("kms-lpsp") { > igt_display_t *display = &data.display; > -- > 2.25.1 > _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [igt-dev] [PATCH] [i-g-t] tests/i915_pm_lpsp: remove screens-disabled subtest 2020-08-31 12:10 ` Surendrakumar Upadhyay, TejaskumarX @ 2020-08-31 12:07 ` Anshuman Gupta 0 siblings, 0 replies; 3+ messages in thread From: Anshuman Gupta @ 2020-08-31 12:07 UTC (permalink / raw) To: Surendrakumar Upadhyay, TejaskumarX; +Cc: igt-dev@lists.freedesktop.org On 2020-08-31 at 17:40:18 +0530, Surendrakumar Upadhyay, TejaskumarX wrote: > Inline > > -----Original Message----- > From: Anshuman Gupta <anshuman.gupta@intel.com> > Sent: 31 August 2020 17:25 > To: Surendrakumar Upadhyay, TejaskumarX <tejaskumarx.surendrakumar.upadhyay@intel.com> > Cc: igt-dev@lists.freedesktop.org > Subject: Re: [PATCH] [i-g-t] tests/i915_pm_lpsp: remove screens-disabled subtest > > On 2020-08-31 at 17:25:15 +0530, Tejas Upadhyay wrote: > > screens-disabled subtest is not required to run when DMC firmware is > > loaded. Thus this test code becomes redundant and increases needless > > SKIP counts. > > > > Cc: Anshuman Gupta <Anshuman.Gupta@intel.com> > > Signed-off-by: Tejas Upadhyay > > <tejaskumarx.surendrakumar.upadhyay@intel.com> > > --- > > tests/i915/i915_pm_lpsp.c | 7 ------- > > 1 file changed, 7 deletions(-) > > > > diff --git a/tests/i915/i915_pm_lpsp.c b/tests/i915/i915_pm_lpsp.c > > index 4a187810..d1efd9e0 100644 > > --- a/tests/i915/i915_pm_lpsp.c > > +++ b/tests/i915/i915_pm_lpsp.c > > @@ -185,13 +185,6 @@ igt_main > > igt_require(igt_pm_dmc_loaded(data.debugfs_fd)); > > } > > > > -igt_describe("This test validates lpsp while all crtc are disabled"); > > -igt_subtest("screens-disabled") { > > -igt_require_f(!dmc_supported(data.debugfs_fd), > > - "DC states supported platform don't have ROI for this subtest\n"); > > -screens_disabled_subtest(&data); > > -} > > - > Platfrom like HASWELL still need this test, you can't simply remove the test, this is a valid skip for platform above skl+. > Thanks, > Anshuman Gupta. > > Tejas : But those platforms will have DMC loaded? If DMC not loaded test will not enter in. And if DMC loaded this subtest will not run. How it will get executed? DMC is not supported on platfrom prior to SKL. Check igt_pm_dmc_loaded() carefully, it will return true in case DMC is not supported. > > > igt_describe("This test validates lpsp on all connected outputs on low power PIPE_A"); > > igt_subtest_with_dynamic_f("kms-lpsp") { > > igt_display_t *display = &data.display; > > -- > > 2.25.1 > > _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-08-31 12:19 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20200831115515.16531-1-tejaskumarx.surendrakumar.upadhyay@intel.com>
2020-08-31 11:55 ` [igt-dev] [PATCH] [i-g-t] tests/i915_pm_lpsp: remove screens-disabled subtest Anshuman Gupta
2020-08-31 12:10 ` Surendrakumar Upadhyay, TejaskumarX
2020-08-31 12:07 ` Anshuman Gupta
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox