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 <[email protected]> > > Cc: Eric Anholt <[email protected]> > > Cc: Arkadiusz Hiler <[email protected]> > > Cc: Daniel Vetter <[email protected]> > > > > Signed-off-by: Vinay Belgaumkar <[email protected]> > > LGTM > Reviewed-by: Radoslaw Szwichtenberg <[email protected]> > 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 [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
