[PULL,3/6] MAINTAINERS: Add Loongson-3 maintainer and reviewer

Commit Message

Aleksandar Markovic June 27, 2020, 7:51 p.m. UTC

From: Huacai Chen <[e-mail]>

Add myself as the maintainer for Loongson-3 virtual platforms, and
also add Jiaxun Yang as the reviewer.

Signed-off-by: Huacai Chen <[e-mail]>
Co-developed-by: Jiaxun Yang <[e-mail]>
Reviewed-by: Aleksandar Markovic <[e-mail]>
Signed-off-by: Aleksandar Markovic <[e-mail]>
Message-Id: <[e-mail]>
---
 MAINTAINERS | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Philippe Mathieu-Daudé June 29, 2020, 8:09 a.m. UTC | #1

On 6/27/20 9:51 PM, Aleksandar Markovic wrote:
> From: Huacai Chen <[e-mail]>
> 
> Add myself as the maintainer for Loongson-3 virtual platforms, and
> also add Jiaxun Yang as the reviewer.
> 
> Signed-off-by: Huacai Chen <[e-mail]>
> Co-developed-by: Jiaxun Yang <[e-mail]>
> Reviewed-by: Aleksandar Markovic <[e-mail]>
> Signed-off-by: Aleksandar Markovic <[e-mail]>
> Message-Id: <[e-mail]>
> ---
>  MAINTAINERS | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 1b40446..fe925ea 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1096,6 +1096,13 @@ F: hw/isa/vt82c686.c
>  F: hw/pci-host/bonito.c
>  F: include/hw/isa/vt82c686.h
>  
> +Loongson-3 Virtual Platform
> +M: Huacai Chen <[e-mail]>
> +R: Jiaxun Yang <[e-mail]>
> +S: Maintained
> +F: hw/mips/loongson3_virt.c

This file has not been commited, is this pull request incomplete?

> +F: hw/intc/loongson_liointc.c

How can we test this device?

> +
>  Boston
>  M: Paul Burton <[e-mail]>
>  R: Aleksandar Rikalo <[e-mail]>
>

Aleksandar Markovic June 29, 2020, 9:56 a.m. UTC | #2

понедељак, 29. јун 2020., Philippe Mathieu-Daudé <[e-mail]> је
написао/ла:

> On 6/27/20 9:51 PM, Aleksandar Markovic wrote:
> > From: Huacai Chen <[e-mail]>
> >
> > Add myself as the maintainer for Loongson-3 virtual platforms, and
> > also add Jiaxun Yang as the reviewer.
> >
> > Signed-off-by: Huacai Chen <[e-mail]>
> > Co-developed-by: Jiaxun Yang <[e-mail]>
> > Reviewed-by: Aleksandar Markovic <[e-mail]>
> > Signed-off-by: Aleksandar Markovic <[e-mail]>
> > Message-Id: <[e-mail]>
> > ---
> >  MAINTAINERS | 7 +++++++
> >  1 file changed, 7 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 1b40446..fe925ea 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -1096,6 +1096,13 @@ F: hw/isa/vt82c686.c
> >  F: hw/pci-host/bonito.c
> >  F: include/hw/isa/vt82c686.h
> >
> > +Loongson-3 Virtual Platform
> > +M: Huacai Chen <[e-mail]>
> > +R: Jiaxun Yang <[e-mail]>
> > +S: Maintained
> > +F: hw/mips/loongson3_virt.c
>
> This file has not been commited, is this pull request incomplete?
>
> > +F: hw/intc/loongson_liointc.c
>
>
This line, in this patch, was my mistake. I did delete the line, but forgot
to do "git add MAINTAINERS". So, the deletion happened in the next patch
dealing with MAINTAINERS.

So, my bad. An honest mistake. I apoligize.

However, the final outcome of the pull request is 100% as desired, and as I
informed other in my responses to patches.  No need to do any follow ups.

How can we test this device?
>
>
Please read the commit message of patch 3/4 of Huacai's recent series v6.


> > +
> >  Boston
> >  M: Paul Burton <[e-mail]>
> >  R: Aleksandar Rikalo <[e-mail]>
> >
>
>

Philippe Mathieu-Daudé July 1, 2020, 10:19 a.m. UTC | #3

On 6/29/20 11:56 AM, Aleksandar Markovic wrote:
> понедељак, 29. јун 2020., Philippe Mathieu-Daudé <[e-mail]
> <mailto:[e-mail]>> је написао/ла:
> 
>     On 6/27/20 9:51 PM, Aleksandar Markovic wrote:
>     > From: Huacai Chen <[e-mail]
>     <mailto:[e-mail]>>
>     >
>     > Add myself as the maintainer for Loongson-3 virtual platforms, and
>     > also add Jiaxun Yang as the reviewer.
>     >
>     > Signed-off-by: Huacai Chen <[e-mail]
>     <mailto:[e-mail]>>
>     > Co-developed-by: Jiaxun Yang <[e-mail]
>     <mailto:[e-mail]>>
>     > Reviewed-by: Aleksandar Markovic <[e-mail]
>     <mailto:[e-mail]>>
>     > Signed-off-by: Aleksandar Markovic
>     <[e-mail]
>     <mailto:[e-mail]>>
>     > Message-Id: <[e-mail]
>     <mailto:[e-mail]>>
>     > ---
>     >  MAINTAINERS | 7 +++++++
>     >  1 file changed, 7 insertions(+)
>     >
>     > diff --git a/MAINTAINERS b/MAINTAINERS
>     > index 1b40446..fe925ea 100644
>     > --- a/MAINTAINERS
>     > +++ b/MAINTAINERS
>     > @@ -1096,6 +1096,13 @@ F: hw/isa/vt82c686.c
>     >  F: hw/pci-host/bonito.c
>     >  F: include/hw/isa/vt82c686.h
>     > 
>     > +Loongson-3 Virtual Platform
>     > +M: Huacai Chen <[e-mail] <mailto:[e-mail]>>
>     > +R: Jiaxun Yang <[e-mail]
>     <mailto:[e-mail]>>
>     > +S: Maintained
>     > +F: hw/mips/loongson3_virt.c
> 
>     This file has not been commited, is this pull request incomplete?
> 
>     > +F: hw/intc/loongson_liointc.c
> 
> 
> This line, in this patch, was my mistake. I did delete the line, but
> forgot to do "git add MAINTAINERS". So, the deletion happened in the
> next patch dealing with MAINTAINERS.
> 
> So, my bad. An honest mistake. I apoligize.
> 
> However, the final outcome of the pull request is 100% as desired, and
> as I informed other in my responses to patches.  No need to do any
> follow ups.
> 
>     How can we test this device?
> 
> 
> Please read the commit message of patch 3/4 of Huacai's recent series v6.

I suppose you are referring to this post:
https://www.mail-archive.com/[e-mail]/msg716861.html

I prepare the kernel as suggested to add an acceptance test, but then
when testing I get:

$ qemu-system-mips64el -M loongson3-virt,accel=tcg
qemu-system-mips64el: -M loongson3-virt,accel=tcg: unsupported machine type

Are you deliberately merging code into mainstream that is not build
neither used? This might be an acceptable exception, but please
explain why this is an exceptional case, and set a deadline for the
machine to be merged, it has to be in for the next 5.1 release,
else we'll have to revert commit c012e0b1f9 ("hw/intc: Add Loongson
LIOINTC support").

In the past the policy has been to reject contributions from finshed
devices but "unfinished machines". The policy is not strict regarding
that.

>  
> 
>     > +
>     >  Boston
>     >  M: Paul Burton <[e-mail] <mailto:[e-mail]>>
>     >  R: Aleksandar Rikalo <[e-mail]
>     <mailto:[e-mail]>>
>     >
>

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 1b40446..fe925ea 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1096,6 +1096,13 @@  F: hw/isa/vt82c686.c
 F: hw/pci-host/bonito.c
 F: include/hw/isa/vt82c686.h
 
+Loongson-3 Virtual Platform
+M: Huacai Chen <[e-mail]>
+R: Jiaxun Yang <[e-mail]>
+S: Maintained
+F: hw/mips/loongson3_virt.c
+F: hw/intc/loongson_liointc.c
+
 Boston
 M: Paul Burton <[e-mail]>
 R: Aleksandar Rikalo <[e-mail]>