kernel development process question, patch review
Greg KH
greg at kroah.com
Tue Feb 21 02:02:47 EST 2017
On Tue, Feb 21, 2017 at 12:35:33PM +1100, Tobin C. Harding wrote:
> If a reviewer makes a suggestion and one intends on making the change
> as suggested is it required (normal protocol) to reply stating that
> the you understand their suggestion and intend on implementing it or
> is this just noise. Should one simply re send the next version of the
> patch?
If you were a reviewer, which would you like to see happen if you took
the time to review someone else's work?
> Continuing on this theme, should one add a Suggested-by: tag in the
> previous situation. Some where Documentation/process it states that no
> tag except the CC tag should be added without the authority of the
> person being named?
Suggested-by: is usually for the whole idea of the patch itself, not for
revisions within a patch.
good luck!
greg k-h
More information about the Kernelnewbies
mailing list