[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20170215213150.4aa1e10d@grimm.local.home>
Date: Wed, 15 Feb 2017 21:31:50 -0500
From: Steven Rostedt <rostedt@...dmis.org>
To: Byungchul Park <byungchul.park@....com>
Cc: Juri Lelli <juri.lelli@....com>, peterz@...radead.org,
mingo@...nel.org, linux-kernel@...r.kernel.org,
juri.lelli@...il.com, kernel-team@....com,
Luca Abeni <luca.abeni@...tannapisa.it>
Subject: Re: [PATCH] sched/deadline: Remove unnecessary condition in
push_dl_task()
On Thu, 16 Feb 2017 11:17:58 +0900
Byungchul Park <byungchul.park@....com> wrote:
> Juri and steven, thank you very much for reviewing it.
>
> I'm not sure and familiar with... Should I add your 'reviewed by' into
> my patches by myself?
>
No, it's the maintainer's job to add these tags when they take your
patch. If you were to resend the patch because of a minor change
(something a reviewer told you to make but still gave you their
reviewed-by tag), then you could add it. Or if you have a patch series
and resend a new series, you can add the reviewed-by tags to the
patches that were not changed. But if you were to change a patch, then
you need to ask the reviewer to give their tag again, because they need
to review the changes made before their tag should go on it.
Make sense?
-- Steve
Powered by blists - more mailing lists