maintainers: update for interrupt subsystem

Message ID 1591197738-16866-1-git-send-email-hkalra@marvell.com (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series maintainers: update for interrupt subsystem |

Checks

Context Check Description
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-nxp-Performance success Performance Testing PASS
ci/travis-robot success Travis build: passed
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-testing warning Testing issues
ci/Intel-compilation success Compilation OK
ci/checkpatch success coding style OK

Commit Message

Harman Kalra June 3, 2020, 3:22 p.m. UTC
  updating MAINTAINERS file for interrupt subsytem.

Signed-off-by: Harman Kalra <hkalra@marvell.com>
---
 MAINTAINERS | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)
  

Comments

Thomas Monjalon June 16, 2020, 10:10 p.m. UTC | #1
03/06/2020 17:22, Harman Kalra:
> updating MAINTAINERS file for interrupt subsytem.
> 
> Signed-off-by: Harman Kalra <hkalra@marvell.com>
> ---
> +Interrupt Subsystem
> +M: Harman Kalra <hkalra@marvell.com>
> +F: lib/librte_eal/include/rte_interrupts.h
> +F: lib/librte_eal/include/rte_eal_interrupts.h
> +F: lib/librte_eal/linux/eal_interrupts.c
> +F: lib/librte_eal/freebsd/eal_interrupts.c

Would you agree simplifying those lines to the following?
F: lib/librte_eal/*/*interrupts.*
  
Harman Kalra June 17, 2020, 12:13 p.m. UTC | #2
On Wed, Jun 17, 2020 at 12:10:40AM +0200, Thomas Monjalon wrote:
> External Email
> 
> ----------------------------------------------------------------------
> 03/06/2020 17:22, Harman Kalra:
> > updating MAINTAINERS file for interrupt subsytem.
> > 
> > Signed-off-by: Harman Kalra <hkalra@marvell.com>
> > ---
> > +Interrupt Subsystem
> > +M: Harman Kalra <hkalra@marvell.com>
> > +F: lib/librte_eal/include/rte_interrupts.h
> > +F: lib/librte_eal/include/rte_eal_interrupts.h
> > +F: lib/librte_eal/linux/eal_interrupts.c
> > +F: lib/librte_eal/freebsd/eal_interrupts.c
> 
> Would you agree simplifying those lines to the following?
> F: lib/librte_eal/*/*interrupts.*

Sure, will send the updated patch

Thanks
Harman
		       
> 
> 
>
  

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index d2b286701..2f93e4aec 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -181,7 +181,6 @@  F: app/test/test_cycles.c
 F: app/test/test_debug.c
 F: app/test/test_eal*
 F: app/test/test_errno.c
-F: app/test/test_interrupts.c
 F: app/test/test_logs.c
 F: app/test/test_memcpy*
 F: app/test/test_per_lcore.c
@@ -221,6 +220,14 @@  F: app/test/test_malloc.c
 F: app/test/test_memory.c
 F: app/test/test_memzone.c
 
+Interrupt Subsystem
+M: Harman Kalra <hkalra@marvell.com>
+F: lib/librte_eal/include/rte_interrupts.h
+F: lib/librte_eal/include/rte_eal_interrupts.h
+F: lib/librte_eal/linux/eal_interrupts.c
+F: lib/librte_eal/freebsd/eal_interrupts.c
+F: app/test/test_interrupts.c
+
 Keep alive
 F: lib/librte_eal/include/rte_keepalive.h
 F: lib/librte_eal/common/rte_keepalive.c