Re: [PATCH net 5/7] MAINTAINERS: Bluetooth: retire Johan (for now?)

From: Jakub Kicinski <kuba@kernel.org>
To: Simon Horman <horms@kernel.org>
Cc: davem@davemloft.net, netdev@vger.kernel.org, edumazet@google.com,
	pabeni@redhat.com, Marcel Holtmann <marcel@holtmann.org>,
	Johan Hedberg <johan.hedberg@gmail.com>,
	Luiz Augusto von Dentz <luiz.dentz@gmail.com>,
	linux-bluetooth@vger.kernel.org,
	Johan Hedberg <johan.hedberg@intel.com>
Subject: Re: [PATCH net 5/7] MAINTAINERS: Bluetooth: retire Johan (for now?)
Date: Thu, 11 Jan 2024 16:21:14 -0800	[thread overview]
Message-ID: <20240111162114.658a752b@kernel.org> (raw)
In-Reply-To: <20240110151309.GD9296@kernel.org>

On Wed, 10 Jan 2024 15:13:09 +0000 Simon Horman wrote:
> + Johan Hedberg <johan.hedberg@intel.com>
> 
> On Tue, Jan 09, 2024 at 08:45:15AM -0800, Jakub Kicinski wrote:
> > Johan moved to maintaining the Zephyr Bluetooth stack,
> > and we haven't heard from him on the ML in 3 years
> > (according to lore), and seen any tags in git in 4 years.
> > Trade the MAINTAINER entry for CREDITS, we can revert
> > whenever Johan comes back to Linux hacking :)
> > 
> > Subsystem BLUETOOTH SUBSYSTEM
> >   Changes 173 / 986 (17%)
> >   Last activity: 2023-12-22
> >   Marcel Holtmann <marcel@holtmann.org>:
> >     Author 91cb4c19118a 2022-01-27 00:00:00 52
> >     Committer edcb185fa9c4 2022-05-23 00:00:00 446
> >     Tags 000c2fa2c144 2023-04-23 00:00:00 523
> >   Johan Hedberg <johan.hedberg@gmail.com>:  
> 
> I'm not arguing that this change isn't appropriate.
> But, nit picking the description above,
> I do think there has been some git activity within
> the last 4 years, albeit from a different email address.
> 
> The most recent Bluetooth Drivers patch I found committed by Johan is:
> 
> commit 0671c0662383eefc272e107364cba7fe229dee44
> Author:     Hans de Goede <hdegoede@redhat.com>
> AuthorDate: Sat Dec 5 16:02:01 2020 +0100
> Commit:     Johan Hedberg <johan.hedberg@intel.com>
> CommitDate: Mon Dec 7 17:01:54 2020 +0200
> 
> For reference, the most recent patched that I could
> find authored by Johan for Bluetooth Drivers is:
> 
> commit 6c3711ec64fd23a9abc8aaf59a9429569a6282df
> Author:     Johan Hedberg <johan.hedberg@intel.com>
> AuthorDate: Sat Aug 4 23:40:26 2018 +0300
> Commit:     Marcel Holtmann <marcel@holtmann.org>
> CommitDate: Sat Aug 4 23:23:32 2018 +0200

Indeed, my grepping must have failed me :(
If someone pops up in the top three reviewers at the end of the report
it's fairly easy to catch that they use a different address, but that
wasn't the case here. In any case, here is the report after having
added the correct email alias -- just to confirm that the script itself
is not buggy :)

Subsystem BLUETOOTH SUBSYSTEM
  Changes 173 / 986 (17%)
  Last activity: 2023-12-22
  Marcel Holtmann <marcel@holtmann.org>:
    Author 91cb4c19118a 2022-01-27 00:00:00 52
    Committer edcb185fa9c4 2022-05-23 00:00:00 446
    Tags 000c2fa2c144 2023-04-23 00:00:00 523
  Johan Hedberg <johan.hedberg@gmail.com>:
    Committer e6ed8b78eae5 2020-12-07 00:00:00 60
    Tags e6ed8b78eae5 2020-12-07 00:00:00 69
  Luiz Augusto von Dentz <luiz.dentz@gmail.com>:
    Author d03376c18592 2023-12-22 00:00:00 241
    Committer da9065caa594 2023-12-22 00:00:00 341
    Tags da9065caa594 2023-12-22 00:00:00 493
  Top reviewers:
    [33]: alainm@chromium.org
    [31]: mcchou@chromium.org
    [27]: abhishekpandit@chromium.org