On 6/30/20 7:19 PM, Philippe Mathieu-Daudé wrote: > On 6/30/20 6:46 PM, Aleksandar Markovic wrote: >> Paul Burton and Aurelien Jarno removed for not being present. >> >> Huacai Chen and Jiaxun Yang step in as new energy. >> >> CC: Paul Burton <[email protected]> >> CC: Aurelien Jarno <[email protected]> >> Signed-off-by: Aleksandar Markovic <[email protected]> >> --- >> MAINTAINERS | 15 ++++++++++----- >> 1 file changed, 10 insertions(+), 5 deletions(-) >> >> diff --git a/MAINTAINERS b/MAINTAINERS >> index 5d8acf8d31..7fc16e21c9 100644 >> --- a/MAINTAINERS >> +++ b/MAINTAINERS >> @@ -213,7 +213,8 @@ F: disas/microblaze.c >> >> MIPS TCG CPUs >> M: Aleksandar Markovic <[email protected]> >> -R: Aurelien Jarno <[email protected]> >> +M: Huacai Chen <[email protected]> >> +R: Jiaxun Yang <[email protected]> >> R: Aleksandar Rikalo <[email protected]> >> S: Maintained >> F: target/mips/ >> @@ -377,6 +378,7 @@ F: target/arm/kvm.c >> >> MIPS KVM CPUs >> M: Aleksandar Markovic <[email protected]> >> +M: Huacai Chen <[email protected]> >> S: Odd Fixes >> F: target/mips/kvm.c >> >> @@ -1052,6 +1054,7 @@ MIPS Machines >> ------------- >> Jazz >> M: Hervé Poussineau <[email protected]> >> +M: Huacai Chen <[email protected]>
Hmm this is the Jazz hobbyist machine, Huacai are you sure this the correct section you want to be? Anyway Hervé Poussineau has to ack that, Aleksandar, it would be easier if you split this as a separate patch. >> R: Aleksandar Rikalo <[email protected]> >> S: Maintained >> F: hw/mips/jazz.c >> @@ -1060,8 +1063,8 @@ F: hw/dma/rc4030.c >> >> Malta >> M: Aleksandar Markovic <[email protected]> >> +M: Huacai Chen <[email protected]> >> M: Philippe Mathieu-Daudé <[email protected]> >> -R: Aurelien Jarno <[email protected]> Last time Aurelien commented on the Malta machine, was on March 23, 3 months ago, then there hasn't been Malta patches. As a reviewer he is present and provides valuable feedback, why are you kicking him out? See: https://www.mail-archive.com/[email protected]/msg691406.html Huacai, has Jiangsu Lemote Tech Co., Ltd real interest to paid employees to maintain the Malta board? >> S: Maintained >> F: hw/isa/piix4.c >> F: hw/acpi/piix4.c >> @@ -1073,6 +1076,7 @@ F: tests/acceptance/machine_mips_malta.py >> >> Mipssim >> M: Aleksandar Markovic <[email protected]> >> +M: Huacai Chen <[email protected]> >> R: Aleksandar Rikalo <[email protected]> >> S: Odd Fixes Now that you have 2 maintainers, you can raise the status to Maintained. >> F: hw/mips/mipssim.c >> @@ -1080,7 +1084,6 @@ F: hw/net/mipsnet.c >> >> R4000 >> M: Aleksandar Markovic <[email protected]> >> -R: Aurelien Jarno <[email protected]> I'm sure Aurelien will agree with this part. But let's wait for his feedback. >> R: Aleksandar Rikalo <[email protected]> >> S: Obsolete >> F: hw/mips/r4k.c >> @@ -1103,7 +1106,8 @@ S: Maintained >> F: hw/intc/loongson_liointc.c >> >> Boston >> -M: Paul Burton <[email protected]> >> +M: Aleksandar Markovic <[email protected]> > > Missing in the patch description that you are taking this > machine over. After some rest I remember Paul said he'd come back on QEMU, so it doesn't look right to kick him out that way. Also in this thread you said you never saw a Boston board: https://www.mail-archive.com/[email protected]/msg691778.html Paul might welcome you as a co-maintainer, but I rather keep him for his expertise. > >> +M: Huacai Chen <[email protected]> Huacai, similarly does Jiangsu Lemote Tech Co., Ltd have real interest in time to provide developers to maintain the Boston machine? >> R: Aleksandar Rikalo <[email protected]> >> S: Maintained >> F: hw/core/loader-fit.c >> @@ -2677,7 +2681,8 @@ F: disas/i386.c >> >> MIPS TCG target >> M: Aleksandar Markovic <[email protected]> >> -R: Aurelien Jarno <[email protected]> >> +M: Huacai Chen <[email protected]> $ git log --author='Huacai Chen <[email protected]>' tcg/mips/ | wc -l 0 $ git log --grep='Reviewed-by: Huacai Chen <[email protected]>' tcg/mips/ | wc -l 0 git log --grep='Acked-by: Huacai Chen <[email protected]>' tcg/mips/ | wc -l 0 >> +R: Jiaxun Yang <[email protected]> $ git log --grep='Reviewed-by: Jiaxun Yang <[email protected]>' tcg/mips/ | wc -l 0 $ git log --grep='Acked-by: Jiaxun Yang <[email protected]>' tcg/mips/ | wc -l 0 I think you are confusing sections, Since Huacai and Jiaxun never showed interest in the TCG target code, how can they become maintainers? >> R: Aleksandar Rikalo <[email protected]> $ git log --grep='Reviewed-by: Aleksandar Rikalo' tcg/mips/ | wc -l 0 $ git log --grep='Acked-by: Aleksandar Rikalo' tcg/mips/ | wc -l 0 Apparently Aleksandar Rikalo can be "removed for not being present." OTOH FWIW: $ git log --author='Philippe Mathieu-Daudé' tcg/mips/ | wc -l 25 $ git log --grep='Reviewed-by: Philippe Mathieu-Daudé' tcg/mips/ | wc -l 99 So for this section changes: NAcked-by: Philippe Mathieu-Daudé <[email protected]> The rest is conditional of Paul Burton and Aurelien Jarno Ack-by. >> S: Maintained >> F: tcg/mips/ >> > >
