Re: [PATCH] scripts/checkpatch: Check for Reviewed-by under --strict

2016-10-28 Thread Joe Perches
On Fri, 2016-10-28 at 13:49 +0100, Chris Wilson wrote: > Some subsystem polices have a strict requirement that every patch must > have at least one reviewer before being approved for upstream. Since > encouraging review is good policy (great review is even better policy!) > enforce checking for a

Re: [PATCH] scripts/checkpatch: Check for Reviewed-by under --strict

2016-10-28 Thread Joe Perches
On Fri, 2016-10-28 at 13:49 +0100, Chris Wilson wrote: > Some subsystem polices have a strict requirement that every patch must > have at least one reviewer before being approved for upstream. Since > encouraging review is good policy (great review is even better policy!) > enforce checking for a

Re: [Intel-gfx] [PATCH] scripts/checkpatch: Check for Reviewed-by under --strict

2016-10-28 Thread Chris Wilson
On Fri, Oct 28, 2016 at 04:33:10PM +0300, Jani Nikula wrote: > On Fri, 28 Oct 2016, Chris Wilson wrote: > > Some subsystem polices have a strict requirement that every patch must > > have at least one reviewer before being approved for upstream. Since > > encouraging

Re: [Intel-gfx] [PATCH] scripts/checkpatch: Check for Reviewed-by under --strict

2016-10-28 Thread Chris Wilson
On Fri, Oct 28, 2016 at 04:33:10PM +0300, Jani Nikula wrote: > On Fri, 28 Oct 2016, Chris Wilson wrote: > > Some subsystem polices have a strict requirement that every patch must > > have at least one reviewer before being approved for upstream. Since > > encouraging review is good policy (great

Re: [Intel-gfx] [PATCH] scripts/checkpatch: Check for Reviewed-by under --strict

2016-10-28 Thread Jani Nikula
On Fri, 28 Oct 2016, Chris Wilson wrote: > Some subsystem polices have a strict requirement that every patch must > have at least one reviewer before being approved for upstream. Since > encouraging review is good policy (great review is even better policy!) > enforce

Re: [Intel-gfx] [PATCH] scripts/checkpatch: Check for Reviewed-by under --strict

2016-10-28 Thread Jani Nikula
On Fri, 28 Oct 2016, Chris Wilson wrote: > Some subsystem polices have a strict requirement that every patch must > have at least one reviewer before being approved for upstream. Since > encouraging review is good policy (great review is even better policy!) > enforce checking for a Reviewed-by

[PATCH] scripts/checkpatch: Check for Reviewed-by under --strict

2016-10-28 Thread Chris Wilson
Some subsystem polices have a strict requirement that every patch must have at least one reviewer before being approved for upstream. Since encouraging review is good policy (great review is even better policy!) enforce checking for a Reviewed-by when checkpath is run with --strict (or with

[PATCH] scripts/checkpatch: Check for Reviewed-by under --strict

2016-10-28 Thread Chris Wilson
Some subsystem polices have a strict requirement that every patch must have at least one reviewer before being approved for upstream. Since encouraging review is good policy (great review is even better policy!) enforce checking for a Reviewed-by when checkpath is run with --strict (or with