Le 27/11/2022 à 13:25, Krzysztof Kozlowski a écrit :
On 26/11/2022 17:03, Neil Armstrong wrote:
Reviewed-by: Neil Armstrong <neil.armstrong@xxxxxxxxxx>
Aren't you maintainer of the platform? This means I should take care of
this patch?
Ah, this will be picked up by Kevin or Jerome? So this tag means only
Reviewer's statement of oversight, right?
Of course I'll pick it.
So that's confusing to see Review which does not mean review, but
maintainer pick...
Reviewed-by means reviewed, there's no confusion here, the significance
is clearly documented on https://docs.kernel.org/process/submitting-patches.html
Reviewed-by means, I should take your tag and add to v2 or the
maintainer will add it when picking up.
Reviewed-by means reviewed, fully reviewed patches are ready to
be picked unless a change is asked by a reviewer.
Decision to pick or not a patch does not depend on any tags, but
is the sole decision of the maintainer adding it's Signed-off-by.
While I use Acked-by to signify acceptance to other maintainers
on files I authored or files I maintain but cannot pick, here I don't
need to signify acceptance to myself.
Neil
Best regards,
Krzysztof
