[Intel-gfx] [PATCH] Correct kms_fbc_crc case 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.

[Daniel Vetter]

On Thu, Apr 09, 2015 at 01:51:24PM +0800, liu,lei wrote:
> From: "liu,lei" <lei.a.liu at intel.com>

In igt we follow the kernel conventions for patch commit messages:
- First line (subject) summarizes the patch in at most 70 chars
- After an empty line comes the extended commit message (you have that in
  the first line so it ends up in the subject).
- signed-off-by line at the bottom.

Can you please fix that up and resubmit?

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 at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch