Re: [PATCH 1/1] MAINTAINERS: Adjust sh4 maintainership

[Philippe Mathieu-Daudé]

On 6/11/20 12:17 AM, Aleksandar Markovic wrote:
> This patch transfers sh4 sections to Yoshinori Sato, who is best
> positioned in the community to assume their maintainership.

https://www.mail-archive.com/qemu-devel@nongnu.org/msg708235.html

"BTW, you also dictate there Yoshimoto that to do and
 what not to do. Why?"

I suppose you meant Yoshinori? Again I'm didn't want to appear
as dictating, I spend hours trying to raise the quality of the
RX hardware model to be sure it'd be merged. You can not say
otherwise, see the latest series posted:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg708317.html

We can not say likewise with the AVR port you almost killed,
asking changes then disappearing, letting other contributors
stuck.

> He is
> the maintainer of the related target rx.
> 
> Further adjustments, reorganizations, and improvements of sh4
> sections are left to the future maintainer to be devised and
> executed, as he deems suitable.
> 
> Aurealien and Magnus are deleted from some sections from the

Typo 'Aurelien'.

> MAINTAINERS file with this patch. However, they will not be
> deleted from QEMU Hall of Fame, where their names will always
> be carved in stone as QEMU pioneers and granddadies.

Typo 'granddaddies'.

> 
> Acked-by: Aurelien Jarno <aurelien@aurel32.net>
> Acked-by: Magnus Damm <magnus.damm@gmail.com>
> Acked-by: Yoshinori Sato <ysato@users.sourceforge.jp>
> Signed-off-by: Aleksandar Markovic <aleksandar.qemu.devel@gmail.com>
> ---
>  MAINTAINERS | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 6e7890ce82..7ddb6db38c 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -296,7 +296,7 @@ F: tests/tcg/s390x/
>  L: qemu-s390x@nongnu.org
>  
>  SH4 TCG CPUs
> -M: Aurelien Jarno <aurelien@aurel32.net>
> +M: Yoshinori Sato <ysato@users.sourceforge.jp>
>  S: Odd Fixes

'Maintained'? See:

https://www.mail-archive.com/qemu-devel@nongnu.org/msg710928.html

>  F: target/sh4/
>  F: hw/sh4/
> @@ -1251,14 +1251,14 @@ F: include/hw/riscv/opentitan.h
>  SH4 Machines
>  ------------
>  R2D
> -M: Magnus Damm <magnus.damm@gmail.com>
> +M: Yoshinori Sato <ysato@users.sourceforge.jp>
>  S: Maintained
>  F: hw/sh4/r2d.c
>  F: hw/intc/sh_intc.c
>  F: hw/timer/sh_timer.c
>  
>  Shix
> -M: Magnus Damm <magnus.damm@gmail.com>
> +M: Yoshinori Sato <ysato@users.sourceforge.jp>
>  S: Odd Fixes

Ditto 'Maintained'?

Whichever status Yoshinori like:
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

>  F: hw/sh4/shix.c
>  
>