Demi Marie Obenour <demiobenour@gmail.com> writes:

> - Place SPDX-FileCopyrightText before SPDX-License-Identifier, except in
>   patches that already have Reviewed-by tags.

For the future, Reviewed-by does not mean you can't change the patch at
all:

> Both Tested-by and Reviewed-by tags, once received on mailing list
> from tester or reviewer, should be added by author to the applicable
> patches when sending next versions.  However if the patch has changed
> substantially in following version, these tags might not be applicable
> anymore and thus should be removed.  Usually removal of someone's
> Acked-by, Tested-by or Reviewed-by tags should be mentioned in the
> patch changelog with an explanation (after the '---' separator).

https://www.kernel.org/doc/html/latest/process/submitting-patches.html#using-reported-by-tested-by-reviewed-by-suggested-by-and-fixes

I suppose this is somewhere we probably ought to link to the kernel
docs as Valentin recently suggested.