Re: [PATCH v2] docs: submitting-patches: Clarify half-baked review tags

[Krzysztof Kozlowski <krzk@kernel.org>]

On 05/11/2026 16:59, Jürgen Groß wrote:
> On 05.11.26 16:53, Krzysztof Kozlowski wrote:
>> Reviewing pieces of the patch while ignoring the main part of it, is
>> much closer to Acknowledgment, thus Ack, and "Reviewed-by" credit should
>> not be applicable.
> 
> I disagree with:
> 
>  >> Reviewing pieces of the patch while ignoring the main part of it, is
>  >> much closer to Acknowledgment, thus Ack, and "Reviewed-by" credit should
>  >> not be applicable.
> 
> And I agree with Andy's statement:
> 
>  >> TL;DR: As a reviewer for the piece in question I have done my part fully. It's
>  >> not my issue that the piece is bundled into a single patch. We have options:
>  >> 1) ask to split (and review as standalone piece);
>  >> 2) add (partial) Rb with the pointers of what exactly has been reviewed;
>  >> 3) ignore (silent agreement with the respective subsystem maintainer's opinion);
>  >> 4) ...something I forgot?..
>  >>
>  >> (even partial) Rb != Ack.

So the patch contents is fine, but commit msg is not? OK, thank you.

Best regards,
Krzysztof