On Thu, Apr 09, 2015 at 01:51:24PM +0800, liu,lei wrote: > From: "liu,lei" <lei.a....@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@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
