On Fri, Dec 20, 2013 at 1:29 PM, Jani Nikula <[email protected]> wrote: >> Signed-off-by: Pradeep Bhat <[email protected]> >> Signed-off-by: Vandana Kannan <[email protected]> >> Reviewed-by: Daniel Vetter <[email protected]> >> Reviewed-by: Chris Wilson <[email protected]> >> Reviewed-by: Jani Nikula <[email protected]> > > I'm sorry, this is not true. Only I get to say when you can add my > Reviewed-by.
Yeah, you can't just add review-by comments. In the linux kernel this has a very specific meaning and can't just get handed out for "has looked at the patches". For details of what a reviewed-by tag means please see the "Reviewer's statement of oversight" in Documentation/SubmittingPatches. I've noticed that you've also done this with Ville's r-b tag for the picture ratio patch, I'll send out a note that this r-b isn't legit. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
