[PATCH v2 0/2] MAINTAINERS: ARM & SD fixes

[PATCH v2 0/2] MAINTAINERS: ARM & SD fixes

[Philippe Mathieu-Daudé]

Few fixes after looking at Vladimir's commit-per-subsystem.py script:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg651545.html

Since v1 [*]:
- Rebased (dropping patch 1 merged as 1232ed1f27 via qemu-trivial)
- Add qemu-arm@nongnu.org to STM32F405 and Netduino Plus 2
- Added Alistair R-b tag

[*]  https://www.mail-archive.com/qemu-devel@nongnu.org/msg651746.html
Supersedes: <20191012065426.10772-1-philmd@redhat.com>

Philippe Mathieu-Daudé (2):
  MAINTAINERS: Cc the qemu-arm@nongnu.org for the ARM machines
  MAINTAINERS: Move the OMAP section inside the ARM Machines one

 MAINTAINERS | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

-- 
2.21.1

[PATCH v2 1/2] MAINTAINERS: Cc the qemu-arm@nongnu.org for the ARM machines

[Philippe Mathieu-Daudé]

Not all ARM machines sections Cc the qemu-arm@nongnu.org list,
fix this.

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
v2: Also fix STM32F405 (commit 529fc5fd3)
    and Netduino Plus 2 (commit 60d6c4278)
---
 MAINTAINERS | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 55d3642e6c..52fbc18566 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -807,6 +807,7 @@ F: hw/arm/virt-acpi-build.c
 STM32F205
 M: Alistair Francis <alistair@alistair23.me>
 M: Peter Maydell <peter.maydell@linaro.org>
+L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/arm/stm32f205_soc.c
 F: hw/misc/stm32f2xx_syscfg.c
@@ -819,6 +820,7 @@ F: include/hw/*/stm32*.h
 STM32F405
 M: Alistair Francis <alistair@alistair23.me>
 M: Peter Maydell <peter.maydell@linaro.org>
+L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/arm/stm32f405_soc.c
 F: hw/misc/stm32f4xx_syscfg.c
@@ -827,18 +829,21 @@ F: hw/misc/stm32f4xx_exti.c
 Netduino 2
 M: Alistair Francis <alistair@alistair23.me>
 M: Peter Maydell <peter.maydell@linaro.org>
+L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/arm/netduino2.c
 
 Netduino Plus 2
 M: Alistair Francis <alistair@alistair23.me>
 M: Peter Maydell <peter.maydell@linaro.org>
+L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/arm/netduinoplus2.c
 
 SmartFusion2
 M: Subbaraya Sundeep <sundeep.lkml@gmail.com>
 M: Peter Maydell <peter.maydell@linaro.org>
+L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/arm/msf2-soc.c
 F: hw/misc/msf2-sysreg.c
@@ -852,6 +857,7 @@ F: include/hw/ssi/mss-spi.h
 Emcraft M2S-FG484
 M: Subbaraya Sundeep <sundeep.lkml@gmail.com>
 M: Peter Maydell <peter.maydell@linaro.org>
+L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/arm/msf2-som.c
 
@@ -1400,6 +1406,7 @@ T: git https://github.com/jnsnow/qemu.git ide
 
 OMAP
 M: Peter Maydell <peter.maydell@linaro.org>
+L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/*/omap*
 F: include/hw/arm/omap.h
-- 
2.21.1

[PATCH v2 2/2] MAINTAINERS: Move the OMAP section inside the ARM Machines one

[Philippe Mathieu-Daudé]

Add the Siemens SX1 (OMAP310) machines with the other ARM machines.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 MAINTAINERS | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 52fbc18566..e2c148d8d8 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -560,6 +560,13 @@ F: include/hw/arm/digic.h
 F: hw/*/digic*
 F: include/hw/*/digic*
 
+OMAP
+M: Peter Maydell <peter.maydell@linaro.org>
+L: qemu-arm@nongnu.org
+S: Maintained
+F: hw/*/omap*
+F: include/hw/arm/omap.h
+
 Gumstix
 M: Peter Maydell <peter.maydell@linaro.org>
 R: Philippe Mathieu-Daudé <f4bug@amsat.org>
@@ -1404,13 +1411,6 @@ F: include/hw/block/fdc.h
 F: tests/qtest/fdc-test.c
 T: git https://github.com/jnsnow/qemu.git ide
 
-OMAP
-M: Peter Maydell <peter.maydell@linaro.org>
-L: qemu-arm@nongnu.org
-S: Maintained
-F: hw/*/omap*
-F: include/hw/arm/omap.h
-
 IPack
 M: Alberto Garcia <berto@igalia.com>
 S: Odd Fixes
-- 
2.21.1

Re: [PATCH v2 2/2] MAINTAINERS: Move the OMAP section inside the ARM Machines one

[Markus Armbruster]

Philippe Mathieu-Daudé <philmd@redhat.com> writes:

> Add the Siemens SX1 (OMAP310) machines with the other ARM machines.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>  MAINTAINERS | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 52fbc18566..e2c148d8d8 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -560,6 +560,13 @@ F: include/hw/arm/digic.h
>  F: hw/*/digic*
>  F: include/hw/*/digic*
>  
> +OMAP
> +M: Peter Maydell <peter.maydell@linaro.org>
> +L: qemu-arm@nongnu.org
> +S: Maintained
> +F: hw/*/omap*
> +F: include/hw/arm/omap.h
> +
>  Gumstix
>  M: Peter Maydell <peter.maydell@linaro.org>
>  R: Philippe Mathieu-Daudé <f4bug@amsat.org>
> @@ -1404,13 +1411,6 @@ F: include/hw/block/fdc.h
>  F: tests/qtest/fdc-test.c
>  T: git https://github.com/jnsnow/qemu.git ide
>  
> -OMAP
> -M: Peter Maydell <peter.maydell@linaro.org>
> -L: qemu-arm@nongnu.org
> -S: Maintained
> -F: hw/*/omap*
> -F: include/hw/arm/omap.h
> -
>  IPack
>  M: Alberto Garcia <berto@igalia.com>
>  S: Odd Fixes

Commit message says "add", patch seems to merely move an existing
entry.  What am I missing?

Re: [PATCH v2 2/2] MAINTAINERS: Move the OMAP section inside the ARM Machines one

[Philippe Mathieu-Daudé]

On 1/18/20 2:30 PM, Markus Armbruster wrote:
> Philippe Mathieu-Daudé <philmd@redhat.com> writes:
> 
>> Add the Siemens SX1 (OMAP310) machines with the other ARM machines.
>>
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> ---
>>   MAINTAINERS | 14 +++++++-------
>>   1 file changed, 7 insertions(+), 7 deletions(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 52fbc18566..e2c148d8d8 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -560,6 +560,13 @@ F: include/hw/arm/digic.h
>>   F: hw/*/digic*
>>   F: include/hw/*/digic*
>>   
>> +OMAP
>> +M: Peter Maydell <peter.maydell@linaro.org>
>> +L: qemu-arm@nongnu.org
>> +S: Maintained
>> +F: hw/*/omap*
>> +F: include/hw/arm/omap.h
>> +
>>   Gumstix
>>   M: Peter Maydell <peter.maydell@linaro.org>
>>   R: Philippe Mathieu-Daudé <f4bug@amsat.org>
>> @@ -1404,13 +1411,6 @@ F: include/hw/block/fdc.h
>>   F: tests/qtest/fdc-test.c
>>   T: git https://github.com/jnsnow/qemu.git ide
>>   
>> -OMAP
>> -M: Peter Maydell <peter.maydell@linaro.org>
>> -L: qemu-arm@nongnu.org
>> -S: Maintained
>> -F: hw/*/omap*
>> -F: include/hw/arm/omap.h
>> -
>>   IPack
>>   M: Alberto Garcia <berto@igalia.com>
>>   S: Odd Fixes
> 
> Commit message says "add", patch seems to merely move an existing
> entry.  What am I missing?

Ah yes indeed. Peter, can you modify s/Add/Move/ in the description or 
do you prefer a respin?

Thanks,

Phil.

Re: [PATCH v2 2/2] MAINTAINERS: Move the OMAP section inside the ARM Machines one

[Peter Maydell]

On Sat, 18 Jan 2020 at 08:55, Philippe Mathieu-Daudé <philmd@redhat.com> wrote:
>
> Add the Siemens SX1 (OMAP310) machines with the other ARM machines.

> +OMAP
> +M: Peter Maydell <peter.maydell@linaro.org>
> +L: qemu-arm@nongnu.org
> +S: Maintained
> +F: hw/*/omap*
> +F: include/hw/arm/omap.h

This is the block that covers the various devices
and SoC objects for all the OMAP boards. I'm not clear
why the commit message calls out the SX1 in particular ?

thanks
-- PMM

Re: [PATCH v2 2/2] MAINTAINERS: Move the OMAP section inside the ARM Machines one

[Philippe Mathieu-Daudé]

On 1/20/20 2:24 PM, Peter Maydell wrote:
> On Sat, 18 Jan 2020 at 08:55, Philippe Mathieu-Daudé <philmd@redhat.com> wrote:
>>
>> Add the Siemens SX1 (OMAP310) machines with the other ARM machines.
> 
>> +OMAP
>> +M: Peter Maydell <peter.maydell@linaro.org>
>> +L: qemu-arm@nongnu.org
>> +S: Maintained
>> +F: hw/*/omap*
>> +F: include/hw/arm/omap.h
> 
> This is the block that covers the various devices
> and SoC objects for all the OMAP boards. I'm not clear
> why the commit message calls out the SX1 in particular ?

I missed that indeed, thanks.