Skip to contentSkip to conversation

MAINTAINERS: Add an ARM SMMU section

PATCH Applied Eric Auger 3 participants 5 messages Nov 22, 2018 – Nov 26, 2018

lore
Share & tools
lore

Eric Auger

author
#0 lore raw

PATCH 1/1 MAINTAINERS: Add an ARM SMMU section

Add a new ARM SMMU section and set Eric Auger as the maintainer for ARM SMMU emulation sources.

---
 MAINTAINERS | 7 +++++++
 1 file changed, 7 insertions(+)
MAINTAINERS +7 -0
index 1032406c56..3cac9f0a0c 100644
@@ -151,6 +151,13 @@ F: disas/arm.c
 F: disas/arm-a64.cc
 F: disas/libvixl/
 
+ARM SMMU
+M: Eric Auger <[email protected]>
+L: [email protected]
+S: Maintained
+F: hw/arm/smmu*
+F: include/hw/arm/smmu*
+
 CRIS
 M: Edgar E. Iglesias <[email protected]>
 S: Maintained
Signature · 1 line
2.17.2
#1 lore raw

Hi Eric,

Quoted 14 lines · Eric Auger

Add a new ARM SMMU section and set Eric Auger as the maintainer for ARM SMMU emulation sources.

---
 MAINTAINERS | 7 +++++++
 1 file changed, 7 insertions(+)
MAINTAINERS +0 -0
index 1032406c56..3cac9f0a0c 100644

MAINTAINERS F: disas/arm.c · line 157

Quoted 7 lines
MAINTAINERS +4 -0
@@ -151,6 +151,13 @@ F: disas/arm.c
 F: disas/arm-a64.cc
 F: disas/libvixl/
 
+ARM SMMU
+M: Eric Auger <[email protected]>
+L: [email protected]
+S: Maintained

Shouldn't you use 'Supported': "Someone is actually paid to look after this."?

Quoted 7 lines
MAINTAINERS +3 -0
+F: hw/arm/smmu*
+F: include/hw/arm/smmu*
+
 CRIS
 M: Edgar E. Iglesias <[email protected]>
 S: Maintained
author
#2 lore raw

Hi Philippe,

Quoted 20 lines · Philippe Mathieu-Daudé

Hi Eric,

Quoted 17 lines · Eric Auger

Add a new ARM SMMU section and set Eric Auger as the maintainer for ARM SMMU emulation sources.

---
 MAINTAINERS | 7 +++++++
 1 file changed, 7 insertions(+)
MAINTAINERS +0 -0
index 1032406c56..3cac9f0a0c 100644
@@ -151,6 +151,13 @@ F: disas/arm.c
 F: disas/arm-a64.cc
 F: disas/libvixl/
 
Quoted 7 lines

MAINTAINERS F: disas/arm.c · line 157

Quoted 4 lines
MAINTAINERS +4 -0
+ARM SMMU
+M: Eric Auger <[email protected]>
+L: [email protected]
+S: Maintained

Shouldn't you use 'Supported': "Someone is actually paid to look after this."?

Not sure this is really important as long as get_maintainer.pl returns the right contacts. Many ARM sections do not feature the "Supported" tag. I will follow Peter's guidance though.

Thanks

Eric

Quoted 8 lines
Quoted 8 lines
MAINTAINERS +3 -0
+F: hw/arm/smmu*
+F: include/hw/arm/smmu*
+
 CRIS
 M: Edgar E. Iglesias <[email protected]>
 S: Maintained
#3 lore raw
Quoted 9 lines · Auger Eric
Quoted 8 lines · Philippe Mathieu-Daudé

MAINTAINERS F: disas/arm.c · line 157

Quoted 4 lines · Eric Auger
MAINTAINERS +4 -0
+ARM SMMU
+M: Eric Auger <[email protected]>
+L: [email protected]
+S: Maintained

Shouldn't you use 'Supported': "Someone is actually paid to look after this."?

Quoted 3 lines

Not sure this is really important as long as get_maintainer.pl returns the right contacts. Many ARM sections do not feature the "Supported" tag. I will follow Peter's guidance though.

I don't really understand the purpose of the Supported vs Maintained distinction. Mostly I list stuff as just Maintained when I am dealing with it.

I think in particular that if a part of QEMU is just one of a whole set of things that you are doing as part of your job then Supported is a bit overstating things.

thanks
-- PMM

#4 lore raw
In reply to Eric Auger
Quoted 18 lines · Eric Auger

Add a new ARM SMMU section and set Eric Auger as the maintainer for ARM SMMU emulation sources.

---
 MAINTAINERS | 7 +++++++
 1 file changed, 7 insertions(+)
MAINTAINERS +0 -0
index 1032406c56..3cac9f0a0c 100644
@@ -151,6 +151,13 @@ F: disas/arm.c
 F: disas/arm-a64.cc
 F: disas/libvixl/
 

MAINTAINERS F: disas/arm.c · line 163

Quoted 10 lines
MAINTAINERS +7 -0
+ARM SMMU
+M: Eric Auger <[email protected]>
+L: [email protected]
+S: Maintained
+F: hw/arm/smmu*
+F: include/hw/arm/smmu*
+
 CRIS
 M: Edgar E. Iglesias <[email protected]>
 S: Maintained

Applied to target-arm.next, thanks.

-- PMM