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