[dpdk-dev] [PATCH] maintainers: add self to baseband maintainers

Tom Rix trix at redhat.com
Mon Oct 5 19:22:08 CEST 2020


On 10/5/20 9:26 AM, Chautru, Nicolas wrote:
> Hey Tom, 
> Thanks for your recent reviews.
> My only concern with this is how in reality you would be able to serve the role of maintainer for these drivers/lib/test and engage in meaningful discussions? 
> Do you understand 3GPP and the related processing and what these devices are actually doing? Supporting them in linux and generic coding guide line review is one thing, but maintainer would need broader understanding arguably. 
> I think you reviews are super valuable (dpdk coding guide lines, error handling...) and I can make sure to include you generally for review for all related contributions, I am just unsure whether in reality that corresponds to what would be required to be a maintainer. 
> I am just thinking aloud here, let me know what you think

I share your concern.

I am interested in reviewing baseband changes.

In the kernel this is handled by the R: Reviewer tag which was part of the original patch.

When i submit kernel patches, i use the get_maintainter.pl script so i do not need to remember everyone interested. When i receive patches i look at just the cc,to's to figure out which project needs help.  While you can create rules to look for strings in the subject line, it does not scale at all.

I went this new route based on Thomas' feed back on my original change.

Now with your feedback, is there a strong enough reason to accept my original change ?

If not, well that's ok.

Tom

> Thanks
> Nic
>
>> -----Original Message-----
>> From: trix at redhat.com <trix at redhat.com>
>> Sent: Monday, October 5, 2020 8:00 AM
>> To: thomas at monjalon.net; Chautru, Nicolas <nicolas.chautru at intel.com>
>> Cc: dev at dpdk.org; Tom Rix <trix at redhat.com>
>> Subject: [PATCH] maintainers: add self to baseband maintainers
>>
>> From: Tom Rix <trix at redhat.com>
>>
>> I am a Linux kernel Reviewer for the fpga n3000/vista creek which has
>> several bitstream based baseband devices.  So I want to help out here as
>> well.
>>
>> Signed-off-by: Tom Rix <trix at redhat.com>
>> ---
>>  MAINTAINERS | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index c0abbe0fc..22e80580a 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -18,7 +18,6 @@ Descriptions of section entries:
>>  	K: Keyword regex pattern to match content.
>>  	   One regex pattern per line. Multiple K: lines acceptable.
>>
>> -
>>  General Project Administration
>>  ------------------------------
>>
>> @@ -392,6 +391,7 @@ F: lib/librte_ethdev/rte_mtr*
>>
>>  Baseband API - EXPERIMENTAL
>>  M: Nicolas Chautru <nicolas.chautru at intel.com>
>> +M: Tom Rix <trix at redhat.com>
>>  T: git://dpdk.org/next/dpdk-next-crypto
>>  F: lib/librte_bbdev/
>>  F: doc/guides/prog_guide/bbdev.rst
>> --
>> 2.18.1



More information about the dev mailing list