On Tue, Nov 10, 2020 at 10:43:54AM +0100, Christoph Hellwig wrote: > > - if (id->nsattr & NVME_NS_ATTR_RO) > > + if (id->nsattr & NVME_NS_ATTR_RO || > > + test_bit(NVME_NS_FORCE_RO, &ns->flags)) Indentation for the test_bit() looks off. I assume that Christoph can fixup that when applying. $ ./scripts/checkpatch.pl --strict ~/javier.patch CHECK: Alignment should match open parenthesis #280: FILE: drivers/nvme/host/core.c:2062: + if (id->nsattr & NVME_NS_ATTR_RO || + test_bit(NVME_NS_FORCE_RO, &ns->flags)) For the record: WARNING: From:/Signed-off-by: email address mismatch: 'From: "Javier González" <javier@xxxxxxxxxxx>' != 'Signed-off-by: Javier González <javier.gonz@xxxxxxxxxxx>' If you want to use a SoB that is different from the email address which you are sending from, then according to the The canonical patch format: https://www.kernel.org/doc/html/latest/process/submitting-patches.html#the-canonical-patch-format """ The from line must be the very first line in the message body, and has the form: From: Patch Author <author@xxxxxxxxxxx> The from line specifies who will be credited as the author of the patch in the permanent changelog. If the from line is missing, then the From: line from the email header will be used to determine the patch author in the changelog. Note, the From: tag is optional when the From: author is also the person (and email) listed in the From: line of the email header. """ That way, when the maintainers use git am, it will pick the author from the "From:" in the message body, rather than from the email header. Otherwise the Author: field in the git log will be different from your SoB. There are several ways you can fix this, either by using the correct email when you do the commit in the first place, then git format-patch will add From: automatically, or by using the git config sendemail.from, or --from option to git-send-email. Kind regards, Niklas
