Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Mon, 9 Aug 2021 09:24:14 +0200
From:   Marc Kleine-Budde <mkl@pengutronix.de>
To:     Stefan =?utf-8?B?TcOkdGpl?= <Stefan.Maetje@esd.eu>
Cc:     "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-can@vger.kernel.org" <linux-can@vger.kernel.org>
Subject: Re: [PATCH] mailmap: update email address of Matthias Fuchs
Message-ID: <20210809072414.syapcl2v7urytygg@pengutronix.de>
References: <20210806104833.102081-1-mkl@pengutronix.de>
 <10024d316c23d7be2ce6037f7994aa2af033208f.camel@esd.eu>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="risgigwcdpvsdsfu"
Content-Disposition: inline
In-Reply-To: <10024d316c23d7be2ce6037f7994aa2af033208f.camel@esd.eu>
Precedence: bulk


--risgigwcdpvsdsfu
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On 06.08.2021 15:07:43, Stefan M=C3=A4tje wrote:
> I was not aware of the .mailmap file and its workings.=20

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 <thomas.koerper@esd.eu> be put in this
> file

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

| Thomas K=C3=B6rper (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/submitt=
ing-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

--=20
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 |

--risgigwcdpvsdsfu
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEK3kIWJt9yTYMP3ehqclaivrt76kFAmEQ2BsACgkQqclaivrt
76lOqwgAr1IYSf0dfTCapqSbuZ1GmzEx7tywKkkZFHcm7YGuOyEq7k0xoD8rXXdl
8Z/aYTe+XVdhv8tHJD0vpk7kYjn31ac18RBEKgmyHhwoA8ucCc3FZrOWuGxOXxBM
U4IbWzOpZQe9YUraeiznYS+SxvMyKS5jxaqBjKC9qen/5UU9DdzOISbiXFhHr2Cn
n1ejuJwSt0nCKQjKEuHTl1TwWocn8VsjkdQAJIO0J4XAaKQT6tzElvvUQp+/YX4X
wj/uoa/LzXbyA3CHuFiMAc+RLd5Lj+x+NyRVnVomf6Z03j5tBkWMuvcDrqwOebPe
swed81X6voXLlQHZ4j+MMENh7UsTXQ==
=tAUi
-----END PGP SIGNATURE-----

--risgigwcdpvsdsfu--