All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
To: Harry Wentland <harry.wentland@amd.com>
Cc: igt-dev@lists.freedesktop.org
Subject: Re: [igt-dev] [PATCH i-g-t 0/3] Enable kms_color tests for amdgpu devices
Date: Mon, 16 Apr 2018 18:51:25 +0300	[thread overview]
Message-ID: <20180416155125.GA31572@ahiler-desk1.ger.corp.intel.com> (raw)
In-Reply-To: <8c581c7a-8522-c2b3-f0e1-2e1c07e95af5@amd.com>

On Mon, Apr 16, 2018 at 09:42:37AM -0400, Harry Wentland wrote:
> On 2018-04-13 06:12 PM, Arkadiusz Hiler wrote:
> > On Thu, Apr 12, 2018 at 05:37:23PM -0400, sunpeng.li@amd.com wrote:
> >> From: "Leo (Sunpeng) Li" <sunpeng.li@amd.com>
> >>
> >> The kms_color tests has some assumptions that are not compatible with
> >> AMD hardware. These patches make it more generic.
> >>
> >> Leo (Sunpeng) Li (3):
> >>   tests/kms_color: Fill entire frame when painting rectangles
> >>   tests/kms_color: Do not mask LUT for non i915 drivers.
> >>   tests/kms_color: Enable color tests for AMDGPU
> >>
> >>  tests/kms_color.c | 28 +++++++++++++++++++++++++---
> >>  1 file changed, 25 insertions(+), 3 deletions(-)
> > 
> > The whole series is
> > Acked-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
> > 
> > It was my assumption that much of the kms tests can be enabled on
> > non-Intel hardware with just a little bit of work. It's good to see that
> > someone is taking an effort.
> > 
> > Are you planning on enabling more of the test? I guess we have quite a
> > lot of superfluous igt_require_gem() calls scattered around, that should
> > be easy to get rid of.
> > 
> 
> Over time, yes. We have quite limited resources to spend on IGT but would love to do more.

I understand. This is awesome nonetheless.

> > Thanks for the patches, I've just pushed them :-)
> > 
> 
> Thanks for pushing. I think I might have commit rights but didn't want
> to rush pushing them without giving someone else a chance to review.

Sorry for not adding your r-b. I think the mails was stuck in the
moderation queue. Are you subscribed to this new-ish mailing list?

- Arek
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

      reply	other threads:[~2018-04-16 15:51 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-12 21:37 [igt-dev] [PATCH i-g-t 0/3] Enable kms_color tests for amdgpu devices sunpeng.li
2018-04-12 21:37 ` [igt-dev] [PATCH i-g-t 1/3] tests/kms_color: Fill entire frame when painting rectangles sunpeng.li
2018-04-12 21:37 ` [igt-dev] [PATCH i-g-t 2/3] tests/kms_color: Do not mask LUT for non i915 drivers sunpeng.li
2018-04-12 21:46   ` [igt-dev] [PATCH i-g-t v2 " sunpeng.li
2018-04-12 21:37 ` [igt-dev] [PATCH i-g-t 3/3] tests/kms_color: Enable color tests for AMDGPU sunpeng.li
2018-04-12 22:24 ` [igt-dev] ✓ Fi.CI.BAT: success for Enable kms_color tests for amdgpu devices (rev2) Patchwork
2018-04-13 19:11 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2018-04-13 19:44 ` [igt-dev] [PATCH i-g-t 0/3] Enable kms_color tests for amdgpu devices Harry Wentland
2018-04-13 22:12 ` Arkadiusz Hiler
2018-04-16 13:42   ` Harry Wentland
2018-04-16 15:51     ` Arkadiusz Hiler [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180416155125.GA31572@ahiler-desk1.ger.corp.intel.com \
    --to=arkadiusz.hiler@intel.com \
    --cc=harry.wentland@amd.com \
    --cc=igt-dev@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.