qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [EXTERNAL]Re: [PATCH v2 2/5] MAINTAINERS: Adjust maintainership for


From: Philippe Mathieu-Daudé
Subject: Re: [EXTERNAL]Re: [PATCH v2 2/5] MAINTAINERS: Adjust maintainership for Fulong 2E board
Date: Mon, 25 Nov 2019 23:58:48 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.1

Hi Zoltan,

On 11/14/19 2:50 PM, Philippe Mathieu-Daudé wrote:
On 11/14/19 2:08 PM, chen huacai wrote:
Hi, all,

On Thu, Nov 14, 2019 at 8:34 PM Aleksandar Markovic
<address@hidden> wrote:

Hi, Philippe,

From: Philippe Mathieu-Daudé <address@hidden>

Hi Aleksandar,

On 11/13/19 2:47 PM, Aleksandar Markovic wrote:
From: Aleksandar Markovic <address@hidden>

Change the maintainership for Fulong 2E board to improve its quality.

IIRC you told me once this board is named Fuloong, and its CPU is a
‎> Loongson, both with 2x 'o' :) I have a patch renaming the various
occurrences.


I still think that the oficial name is "Fuloong 2E", however, it is
shortened to "Fulong 2E" quite often in communication, and, it seems,
sometimes even in various docs/app notes etc.

Can perhaps Huacai Chen enlighten us regarding the right spelling?
The right spelling is Fuloong.

[...]>>> The original author is active on the Linux kernel, let ask him if he'd
be OK to keep an eye on his work.

Huacai, would you agree to be listed as a reviewer on the Fuloong
related QEMU files? You don't have to worry about the generic QEMU code
plate (like keeping API up to date) I'll manage that, but I'd like to
have you listed to assert the hardware is properly modeled.

You would appear as:
R: Huacai Chen <address@hidden>


That is a great idea!

Please, Chen, get involved, you would be very welcomed, there is a place
for you in QEMU community!
I'm sorry that I'm busy now, but I think I will do something in QEMU
in the next year.

No problem, we'll keep in touch.

+R: Hervé Poussineau <address@hidden>

I don't think Hervé is interested by this board, he has not modified the
code.

+R: Aleksandar Markovic <address@hidden>
+S: Maintained

Let keep it as "Odd Fixes" :)


OK.

    Odd Fixes:   It has a maintainer but they don't have
                 time to do much other than throw the odd
                 patch in.

   F: hw/mips/mips_fulong2e.c
   F: hw/isa/vt82c686.c
   F: hw/pci-host/bonito.c


So the patch would be:

-- 8< --
   Fulong 2E
-M: Aleksandar Markovic <address@hidden>
-R: Aleksandar Rikalo <address@hidden>
+M: Philippe Mathieu-Daudé <address@hidden>
+R: Aleksandar Markovic <address@hidden>
+R: Huacai Chen <address@hidden>
   S: Odd Fixes
   F: hw/mips/mips_fulong2e.c
   F: hw/isa/vt82c686.c

Plus possible s/Fulong 2E/Fuloong 2E/

I'd prefer to keep this change for later, with the patch cleaning its use in the codebase (not that trivial because we need to alias the machine name to keep backward compatibility).


---

But let's wait to see what Huacai Chen thinks of it, before posting it.

Aleksandar, can you stay as co-maintainer?

The patch would be:

-- 8< --
    Fulong 2E
+M: Philippe Mathieu-Daudé <address@hidden>
  M: Aleksandar Markovic <address@hidden>
-R: Aleksandar Rikalo <address@hidden>

You are a prolific contributor of the Fuloong, are you OK to be listed as reviewer in this section?

    S: Odd Fixes
    F: hw/mips/mips_fulong2e.c
    F: hw/isa/vt82c686.c
---

Either this way, or if you prefer to be listed with a R-tag:
Acked-by: Philippe Mathieu-Daudé <address@hidden>




reply via email to

[Prev in Thread] Current Thread [Next in Thread]