Re: [PATCH] Avoid Wunused-but-set warning

From: Mark Brown
Date: Mon Jul 11 2011 - 18:42:57 EST


On Mon, Jul 11, 2011 at 10:11:34AM -0700, Randy Dunlap wrote:
> On Mon, 11 Jul 2011 06:09:52 +0100 Mark Brown wrote:

> > Current practice seems to be that Acked-by is used instead of
> > Reviewed-by - the latter is comparatively rare.

> ISTM that more education and encouragement are needed about Reviewed-by:.
> (Patch Review is a possible kernel summit topic.)

> and that SubmittingPatches should be updated since we generally refer people
> to that file and not to Documentation/development-process/

I'm not sure I see any value in that - they're both pretty much
interchangible as far as I can tell. The important thing is that people
are doing review, not the form filling part.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/