diff mbox series

[iwl-next,v1] MAINTAINERS: update Intel Ethernet maintainers

Message ID 20240617190759.2378532-1-jesse.brandeburg@intel.com
State Accepted
Delegated to: Anthony Nguyen
Headers show
Series [iwl-next,v1] MAINTAINERS: update Intel Ethernet maintainers | expand

Commit Message

Jesse Brandeburg June 17, 2024, 7:07 p.m. UTC
Since Jesse has moved to a new role, replace him with a new maintainer
to work with Tony on representing Intel networking drivers in the
kernel.

Cc: Przemek Kitszel <przemyslaw.kitszel@intel.com>
Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
---
 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Paul Menzel June 17, 2024, 8:29 p.m. UTC | #1
Dear Jesse,


Am 17.06.24 um 21:07 schrieb Jesse Brandeburg:
> Since Jesse has moved to a new role, replace him with a new maintainer
> to work with Tony on representing Intel networking drivers in the
> kernel.

Maybe add the names to the summary/title:

MAINTAINERS: Intel Ethernet: Replace Jesse Brandeburg by Przemek Kitszel

> Cc: Przemek Kitszel <przemyslaw.kitszel@intel.com>
> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
> ---
>   MAINTAINERS | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index cd3277a98cfe..007f3bda5eeb 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -11050,8 +11050,8 @@ F:	include/drm/xe*
>   F:	include/uapi/drm/xe_drm.h
>   
>   INTEL ETHERNET DRIVERS
> -M:	Jesse Brandeburg <jesse.brandeburg@intel.com>
>   M:	Tony Nguyen <anthony.l.nguyen@intel.com>
> +M:	Przemek Kitszel <przemyslaw.kitszel@intel.com>
>   L:	intel-wired-lan@lists.osuosl.org (moderated for non-subscribers)
>   S:	Supported
>   W:	https://www.intel.com/content/www/us/en/support.html

Acked-by: Paul Menzel <pmenzel@molgen.mpg.de>

Thank you for all your work, and good luck and much fun with your new role.


Kind regards,

Paul
Jesse Brandeburg June 17, 2024, 9:19 p.m. UTC | #2
On 6/17/2024 1:29 PM, Paul Menzel wrote:
> Dear Jesse,
> 
> 
> Am 17.06.24 um 21:07 schrieb Jesse Brandeburg:
>> Since Jesse has moved to a new role, replace him with a new maintainer
>> to work with Tony on representing Intel networking drivers in the
>> kernel.
> 
> Maybe add the names to the summary/title:
> 
> MAINTAINERS: Intel Ethernet: Replace Jesse Brandeburg by Przemek Kitszel

Thanks for the feedback Paul! I am more comfortable with the "summary
subject" style instead of the "detailed subject" style. I also looked at
a bunch of changes to the MAINTAINERS file, and there isn't much
precedent for putting names in the subject (there are some examples but
not many)

Do you have specific guidance you can point me at that I may not be
aware of?

>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -11050,8 +11050,8 @@ F:    include/drm/xe*
>>   F:    include/uapi/drm/xe_drm.h
>>     INTEL ETHERNET DRIVERS
>> -M:    Jesse Brandeburg <jesse.brandeburg@intel.com>
>>   M:    Tony Nguyen <anthony.l.nguyen@intel.com>
>> +M:    Przemek Kitszel <przemyslaw.kitszel@intel.com>
>>   L:    intel-wired-lan@lists.osuosl.org (moderated for non-subscribers)
>>   S:    Supported
>>   W:    https://www.intel.com/content/www/us/en/support.html
> 
> Acked-by: Paul Menzel <pmenzel@molgen.mpg.de>
> 
> Thank you for all your work, and good luck and much fun with your new role.

We appreciate your feedback and reviews, and thanks!
Paul Menzel June 17, 2024, 9:25 p.m. UTC | #3
Dear Jesse,


Am 17.06.24 um 23:19 schrieb Jesse Brandeburg:
> On 6/17/2024 1:29 PM, Paul Menzel wrote:

>> Am 17.06.24 um 21:07 schrieb Jesse Brandeburg:
>>> Since Jesse has moved to a new role, replace him with a new maintainer
>>> to work with Tony on representing Intel networking drivers in the
>>> kernel.
>>
>> Maybe add the names to the summary/title:
>>
>> MAINTAINERS: Intel Ethernet: Replace Jesse Brandeburg by Przemek Kitszel
> 
> Thanks for the feedback Paul! I am more comfortable with the "summary
> subject" style instead of the "detailed subject" style. I also looked at
> a bunch of changes to the MAINTAINERS file, and there isn't much
> precedent for putting names in the subject (there are some examples but
> not many)
> 
> Do you have specific guidance you can point me at that I may not be
> aware of?

There is no guidance, just my frustration looking through `git log 
--oneline` and always having to copy the hash and do `git show …` for 
such a commit to find out what “update” means.

[…]


Kind regards,

Paul
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index cd3277a98cfe..007f3bda5eeb 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11050,8 +11050,8 @@  F:	include/drm/xe*
 F:	include/uapi/drm/xe_drm.h
 
 INTEL ETHERNET DRIVERS
-M:	Jesse Brandeburg <jesse.brandeburg@intel.com>
 M:	Tony Nguyen <anthony.l.nguyen@intel.com>
+M:	Przemek Kitszel <przemyslaw.kitszel@intel.com>
 L:	intel-wired-lan@lists.osuosl.org (moderated for non-subscribers)
 S:	Supported
 W:	https://www.intel.com/content/www/us/en/support.html