Re: [PATCH] docs: submitting-patches: document the format for affiliation

[Jonathan Corbet <corbet@lwn.net>]

Jakub Kicinski <kuba@kernel.org> writes:

> Adding company name in round brackets to From/SoB lines
> is fairly common, but I don't see it documented anywhere.
> Every now and then people try to add the sponsorship lines
> to the commit message, fun example from this merge window:
>
>   Sponsored by:   The FreeBSD Foundation
>
> from commit 2ce67f8bf1ce ("wifi: iwlwifi: mvm: fix iwl_ssid_exist()
> check"). Better format would be:
>
>   Author: Miri Korenblit (FreeBSD Foundation) <...
>
> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
> ---
> CC: corbet@lwn.net
> CC: workflows@vger.kernel.org
> CC: linux-doc@vger.kernel.org
> ---
>  Documentation/process/submitting-patches.rst | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/Documentation/process/submitting-patches.rst b/Documentation/process/submitting-patches.rst
> index 8fdc0ef3e604..12ed28b3d113 100644
> --- a/Documentation/process/submitting-patches.rst
> +++ b/Documentation/process/submitting-patches.rst
> @@ -717,6 +717,12 @@ 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.
>  
> +The author may indicate their affiliation or the sponsor of the work
> +by adding the name of an organization to the ``from`` and ``SoB`` lines,
> +e.g.:
> +
> +	From: Patch Author (Company) <author@example.com>
> +
>  Explanation Body
>  ^^^^^^^^^^^^^^^^

So, despite all the conversation, I didn't see any real opposition to
adding this change, so I've applied it.

Amusingly, b4 helpfully picked up these tags from the conversation:

 + Signed-off-by: John Doe <jdoe@example.com> (✓DKIM/infradead.org)
 + Signed-off-by: Mr Maintainer <jdoe@example.com> (✓DKIM/kernel.org)
 + Signed-off-by: Jane Maintainer <jdoe@example.com>

Tempting as it was to leave them in, I took them out :)

Thanks,

jon