Re: [PATCH] Correct kms_fbc_crc case

From: Daniel Vetter <daniel@ffwll.ch>
To: "liu,lei" <lei.a.liu@intel.com>
Cc: intel-gfx@lists.freedesktop.org, gordon.jin@intel.com,
	yex.tian@intel.com
Subject: Re: [PATCH] Correct kms_fbc_crc case
Date: Thu, 9 Apr 2015 15:47:18 +0200	[thread overview]
Message-ID: <20150409134718.GN12038@phenom.ffwll.local> (raw)
In-Reply-To: <1428585453-27344-1-git-send-email-lei.a.liu@intel.com>

On Thu, Apr 09, 2015 at 09:17:33PM +0800, liu,lei wrote:
> From: "liu,lei" <lei.a.liu@intel.com>
> 
> Debugfs i915_fbc_status shows "FBC unsupported on this chipset"
> not "unsupported by this chipset" if the platform doesn't support
> FBC feature. That typo will cause case fail on some platforms such
> as byt, bsw.
> 
> Signed-off-by: Lei Liu <lei.a.liu@intel.com>

Applied, thanks.
-Daniel

> ---
>  tests/kms_fbc_crc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/kms_fbc_crc.c b/tests/kms_fbc_crc.c
> index ccdec33..1320bad 100644
> --- a/tests/kms_fbc_crc.c
> +++ b/tests/kms_fbc_crc.c
> @@ -513,7 +513,7 @@ igt_main
>  		igt_assert_lt(0, fread(buf, 1, sizeof(buf), status));
>  		fclose(status);
>  		buf[sizeof(buf) - 1] = '\0';
> -		igt_require_f(!strstr(buf, "unsupported by this chipset") &&
> +		igt_require_f(!strstr(buf, "unsupported on this chipset") &&
>  			      !strstr(buf, "disabled per module param") &&
>  			      !strstr(buf, "disabled per chip default"),
>  			      "FBC not supported/enabled\n");
> -- 
> 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
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx