diff mbox

[v2] MAINTAINERS: Add some ARM related files to the corresponding sections

Message ID 1476266275-11155-1-git-send-email-thuth@redhat.com
State New
Headers show

Commit Message

Thomas Huth Oct. 12, 2016, 9:57 a.m. UTC
The files w/cpu/a*mpcore.c are already assigned to the ARM CPU
section, but the corresponding headers include/hw/cpu/a*mpcore.h
are still missing.

The file hw/*/imx* are already assigned to the i.MX31 machine, but
the corresponding header files include/hw/*/imx* are still missing.

The file hw/misc/arm_integrator_debug.c seems to belong to Integrator
CP, hw/cpu/realview_mpcore.c seems to belong to Real View, and
hw/misc/mst_fpga.c seems to belong to PXA2XX.

And the files hw/misc/zynq* and include/hw/misc/zynq* seem to belong
to the Xilinx Zynq machine.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 v2:
 - Added some more file
 - Dropped hw/misc/arm_sysctl.c since it's unclear right now how to
   represent a file that belong to multiple machines

 MAINTAINERS | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

Comments

Alistair Francis Oct. 12, 2016, 1:34 p.m. UTC | #1
On Wed, Oct 12, 2016 at 11:57 AM, Thomas Huth <thuth@redhat.com> wrote:
> The files w/cpu/a*mpcore.c are already assigned to the ARM CPU
> section, but the corresponding headers include/hw/cpu/a*mpcore.h
> are still missing.
>
> The file hw/*/imx* are already assigned to the i.MX31 machine, but
> the corresponding header files include/hw/*/imx* are still missing.
>
> The file hw/misc/arm_integrator_debug.c seems to belong to Integrator
> CP, hw/cpu/realview_mpcore.c seems to belong to Real View, and
> hw/misc/mst_fpga.c seems to belong to PXA2XX.
>
> And the files hw/misc/zynq* and include/hw/misc/zynq* seem to belong
> to the Xilinx Zynq machine.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  v2:
>  - Added some more file
>  - Dropped hw/misc/arm_sysctl.c since it's unclear right now how to
>    represent a file that belong to multiple machines
>
>  MAINTAINERS | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 8c1c0a1..7fc5c9d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -107,6 +107,7 @@ S: Maintained
>  F: target-arm/
>  F: hw/arm/
>  F: hw/cpu/a*mpcore.c
> +F: include/hw/cpu/a*mpcore.h
>  F: disas/arm.c
>  F: disas/arm-a64.cc
>  F: disas/libvixl/
> @@ -408,6 +409,7 @@ M: Peter Chubb <peter.chubb@nicta.com.au>
>  L: qemu-arm@nongnu.org
>  S: Odd fixes
>  F: hw/*/imx*
> +F: include/hw/*/imx*
>  F: hw/arm/kzm.c
>  F: include/hw/arm/fsl-imx31.h
>
> @@ -416,6 +418,7 @@ M: Peter Maydell <peter.maydell@linaro.org>
>  L: qemu-arm@nongnu.org
>  S: Maintained
>  F: hw/arm/integratorcp.c
> +F: hw/misc/arm_integrator_debug.c
>
>  Musicpal
>  M: Jan Kiszka <jan.kiszka@web.de>
> @@ -440,6 +443,7 @@ M: Peter Maydell <peter.maydell@linaro.org>
>  L: qemu-arm@nongnu.org
>  S: Maintained
>  F: hw/arm/realview*
> +F: hw/cpu/realview_mpcore.c
>  F: hw/intc/realview_gic.c
>  F: include/hw/intc/realview_gic.h
>
> @@ -452,6 +456,7 @@ F: hw/arm/spitz.c
>  F: hw/arm/tosa.c
>  F: hw/arm/z2.c
>  F: hw/*/pxa2xx*
> +F: hw/misc/mst_fpga.c
>  F: include/hw/arm/pxa.h
>
>  Stellaris
> @@ -473,7 +478,8 @@ L: qemu-arm@nongnu.org
>  S: Maintained
>  F: hw/*/xilinx_*
>  F: hw/*/cadence_*
> -F: hw/misc/zynq_slcr.c
> +F: hw/misc/zynq*
> +F: include/hw/misc/zynq*
>  X: hw/ssi/xilinx_*

For the Zynq/Xilinx part:

Reviewed-by: Alistair Francis <alistair.francis@xilinx.com>

Thanks,

Alistair

Thanks,

Alistair


>
>  Xilinx ZynqMP
> --
> 1.8.3.1
>
>
Thomas Huth Nov. 4, 2016, 11:18 a.m. UTC | #2
On 12.10.2016 15:34, Alistair Francis wrote:
> On Wed, Oct 12, 2016 at 11:57 AM, Thomas Huth <thuth@redhat.com> wrote:
>> The files w/cpu/a*mpcore.c are already assigned to the ARM CPU
>> section, but the corresponding headers include/hw/cpu/a*mpcore.h
>> are still missing.
>>
>> The file hw/*/imx* are already assigned to the i.MX31 machine, but
>> the corresponding header files include/hw/*/imx* are still missing.
>>
>> The file hw/misc/arm_integrator_debug.c seems to belong to Integrator
>> CP, hw/cpu/realview_mpcore.c seems to belong to Real View, and
>> hw/misc/mst_fpga.c seems to belong to PXA2XX.
>>
>> And the files hw/misc/zynq* and include/hw/misc/zynq* seem to belong
>> to the Xilinx Zynq machine.
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>>  v2:
>>  - Added some more file
>>  - Dropped hw/misc/arm_sysctl.c since it's unclear right now how to
>>    represent a file that belong to multiple machines
>>
>>  MAINTAINERS | 8 +++++++-
>>  1 file changed, 7 insertions(+), 1 deletion(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 8c1c0a1..7fc5c9d 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -107,6 +107,7 @@ S: Maintained
>>  F: target-arm/
>>  F: hw/arm/
>>  F: hw/cpu/a*mpcore.c
>> +F: include/hw/cpu/a*mpcore.h
>>  F: disas/arm.c
>>  F: disas/arm-a64.cc
>>  F: disas/libvixl/
>> @@ -408,6 +409,7 @@ M: Peter Chubb <peter.chubb@nicta.com.au>
>>  L: qemu-arm@nongnu.org
>>  S: Odd fixes
>>  F: hw/*/imx*
>> +F: include/hw/*/imx*
>>  F: hw/arm/kzm.c
>>  F: include/hw/arm/fsl-imx31.h
>>
>> @@ -416,6 +418,7 @@ M: Peter Maydell <peter.maydell@linaro.org>
>>  L: qemu-arm@nongnu.org
>>  S: Maintained
>>  F: hw/arm/integratorcp.c
>> +F: hw/misc/arm_integrator_debug.c
>>
>>  Musicpal
>>  M: Jan Kiszka <jan.kiszka@web.de>
>> @@ -440,6 +443,7 @@ M: Peter Maydell <peter.maydell@linaro.org>
>>  L: qemu-arm@nongnu.org
>>  S: Maintained
>>  F: hw/arm/realview*
>> +F: hw/cpu/realview_mpcore.c
>>  F: hw/intc/realview_gic.c
>>  F: include/hw/intc/realview_gic.h
>>
>> @@ -452,6 +456,7 @@ F: hw/arm/spitz.c
>>  F: hw/arm/tosa.c
>>  F: hw/arm/z2.c
>>  F: hw/*/pxa2xx*
>> +F: hw/misc/mst_fpga.c
>>  F: include/hw/arm/pxa.h
>>
>>  Stellaris
>> @@ -473,7 +478,8 @@ L: qemu-arm@nongnu.org
>>  S: Maintained
>>  F: hw/*/xilinx_*
>>  F: hw/*/cadence_*
>> -F: hw/misc/zynq_slcr.c
>> +F: hw/misc/zynq*
>> +F: include/hw/misc/zynq*
>>  X: hw/ssi/xilinx_*
> 
> For the Zynq/Xilinx part:
> 
> Reviewed-by: Alistair Francis <alistair.francis@xilinx.com>

*ping* - Peter, do you think this update to MAINTAINERS is OK now?

 Thomas
Peter Maydell Nov. 4, 2016, 11:46 a.m. UTC | #3
On 4 November 2016 at 11:18, Thomas Huth <thuth@redhat.com> wrote:
> On 12.10.2016 15:34, Alistair Francis wrote:
>> On Wed, Oct 12, 2016 at 11:57 AM, Thomas Huth <thuth@redhat.com> wrote:
>>> The files w/cpu/a*mpcore.c are already assigned to the ARM CPU
>>> section, but the corresponding headers include/hw/cpu/a*mpcore.h
>>> are still missing.
>>>
>>> The file hw/*/imx* are already assigned to the i.MX31 machine, but
>>> the corresponding header files include/hw/*/imx* are still missing.
>>>
>>> The file hw/misc/arm_integrator_debug.c seems to belong to Integrator
>>> CP, hw/cpu/realview_mpcore.c seems to belong to Real View, and
>>> hw/misc/mst_fpga.c seems to belong to PXA2XX.
>>>
>>> And the files hw/misc/zynq* and include/hw/misc/zynq* seem to belong
>>> to the Xilinx Zynq machine.
>>>
>>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>>> ---

> *ping* - Peter, do you think this update to MAINTAINERS is OK now?

Yeah, looks OK.

Acked-by: Peter Maydell <peter.maydell@linaro.org>

-- PMM
diff mbox

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 8c1c0a1..7fc5c9d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -107,6 +107,7 @@  S: Maintained
 F: target-arm/
 F: hw/arm/
 F: hw/cpu/a*mpcore.c
+F: include/hw/cpu/a*mpcore.h
 F: disas/arm.c
 F: disas/arm-a64.cc
 F: disas/libvixl/
@@ -408,6 +409,7 @@  M: Peter Chubb <peter.chubb@nicta.com.au>
 L: qemu-arm@nongnu.org
 S: Odd fixes
 F: hw/*/imx*
+F: include/hw/*/imx*
 F: hw/arm/kzm.c
 F: include/hw/arm/fsl-imx31.h
 
@@ -416,6 +418,7 @@  M: Peter Maydell <peter.maydell@linaro.org>
 L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/arm/integratorcp.c
+F: hw/misc/arm_integrator_debug.c
 
 Musicpal
 M: Jan Kiszka <jan.kiszka@web.de>
@@ -440,6 +443,7 @@  M: Peter Maydell <peter.maydell@linaro.org>
 L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/arm/realview*
+F: hw/cpu/realview_mpcore.c
 F: hw/intc/realview_gic.c
 F: include/hw/intc/realview_gic.h
 
@@ -452,6 +456,7 @@  F: hw/arm/spitz.c
 F: hw/arm/tosa.c
 F: hw/arm/z2.c
 F: hw/*/pxa2xx*
+F: hw/misc/mst_fpga.c
 F: include/hw/arm/pxa.h
 
 Stellaris
@@ -473,7 +478,8 @@  L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/*/xilinx_*
 F: hw/*/cadence_*
-F: hw/misc/zynq_slcr.c
+F: hw/misc/zynq*
+F: include/hw/misc/zynq*
 X: hw/ssi/xilinx_*
 
 Xilinx ZynqMP