[v2,2/2] MAINTAINERS: Move the OMAP section inside the ARM Machines one
diff mbox series

Message ID 20200118085509.30182-3-philmd@redhat.com
State New
Headers show
Series
  • MAINTAINERS: ARM & SD fixes
Related show

Commit Message

Philippe Mathieu-Daudé Jan. 18, 2020, 8:55 a.m. UTC
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(-)

Comments

Markus Armbruster Jan. 18, 2020, 1:30 p.m. UTC | #1
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?
Philippe Mathieu-Daudé Jan. 18, 2020, 2:08 p.m. UTC | #2
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.
Peter Maydell Jan. 20, 2020, 1:24 p.m. UTC | #3
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
Philippe Mathieu-Daudé Jan. 20, 2020, 2:11 p.m. UTC | #4
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.

Patch
diff mbox series

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