On 06.08.2021 15:07:43, Stefan Mätje wrote:

I was not aware of the .mailmap file and its workings. 

The mailmap file works on the git log, and thus tools using it: e.g.
scripts/get_maintainer.pl.

My question is should then also [off-list ref] be put in this
file

Probably yes, as there are two patches by him in the Linux kernel:

| Thomas Körper (2):
|       can: dev: avoid calling kfree_skb() from interrupt context
|       can: esd_usb2: Fix sparse warnings

I'll update this patch.

because he is the co-author of the patch you're reviewing atm.

For co-authorship, use the Co-developed-by tag, that was recently added:

https://elixir.bootlin.com/linux/v5.12/source/Documentation/process/submitting-patches.rst#L449

Speaking of that patch. Can you please create an entry in the
MAINTAINERS file, too? I suggest to add yourself as M: and the
socketcan@e.u as R:. The change in the MAINTAINERS file will be a second
patch in that series. Just include it the next iteration of that series.

and his email address shows up in MODULE_AUTHOR() macro. He also left
our company already.

In newly added changes, it makes sense that all email addresses are
working. :D

regards,
Marc

-- 
Pengutronix e.K.                 | Marc Kleine-Budde           |
Embedded Linux                   | https://www.pengutronix.de  |
Vertretung West/Dortmund         | Phone: +49-231-2826-924     |
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-5555 |