On Fri, 2017-09-15 at 07:34 +0000, Szwichtenberg, Radoslaw wrote: > On Thu, 2017-09-14 at 11:09 -0700, Vinay Belgaumkar wrote: > > Added the missing IGT_TEST_DESCRIPTION and some subtest > > descriptions. > > > > v2: Removed duplication, addressed comments, cc'd test author > > > > v3: Only comment abstract code, change some igt_info to igt_debug. > > Changed description to reflect this is a patch, not an RFC. > > > > Cc: Michał Winiarski <michal.winiarski@xxxxxxxxx> > > Cc: Eric Anholt <eric@xxxxxxxxxx> > > Cc: Arkadiusz Hiler <arkadiusz.hiler@xxxxxxxxx> > > Cc: Daniel Vetter <daniel.vetter@xxxxxxxxx> > > > > Signed-off-by: Vinay Belgaumkar <vinay.belgaumkar@xxxxxxxxx> > > LGTM > Reviewed-by: Radoslaw Szwichtenberg <radoslaw.szwichtenberg@xxxxxxxxx> > Maybe just one minor with the comment style - it would be good to make it consistent across whole file. Let's start comment with a capital letter and finish it with a dot. Still - my r-b is yours :) -Radek _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx