qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH v2] MAINTAINERS: Add some ARM related files to the corresponding sections
@ 2016-10-12  9:57 Thomas Huth
  2016-10-12 13:34 ` Alistair Francis
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Huth @ 2016-10-12  9:57 UTC (permalink / raw)
  To: Peter Maydell, qemu-devel
  Cc: qemu-arm, Peter Chubb, Andrzej Zaborowski, Edgar E Iglesias,
	Alistair Francis

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_*
 
 Xilinx ZynqMP
-- 
1.8.3.1

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [PATCH v2] MAINTAINERS: Add some ARM related files to the corresponding sections
  2016-10-12  9:57 [Qemu-devel] [PATCH v2] MAINTAINERS: Add some ARM related files to the corresponding sections Thomas Huth
@ 2016-10-12 13:34 ` Alistair Francis
  2016-11-04 11:18   ` Thomas Huth
  0 siblings, 1 reply; 4+ messages in thread
From: Alistair Francis @ 2016-10-12 13:34 UTC (permalink / raw)
  To: Thomas Huth
  Cc: Peter Maydell, qemu-devel@nongnu.org Developers, Alistair Francis,
	qemu-arm, Peter Chubb, Edgar E Iglesias

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
>
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [PATCH v2] MAINTAINERS: Add some ARM related files to the corresponding sections
  2016-10-12 13:34 ` Alistair Francis
@ 2016-11-04 11:18   ` Thomas Huth
  2016-11-04 11:46     ` Peter Maydell
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Huth @ 2016-11-04 11:18 UTC (permalink / raw)
  To: Peter Maydell, qemu-arm
  Cc: Alistair Francis, qemu-devel@nongnu.org Developers, Peter Chubb,
	Edgar E Iglesias

[-- Attachment #1: Type: text/plain, Size: 2656 bytes --]

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



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [PATCH v2] MAINTAINERS: Add some ARM related files to the corresponding sections
  2016-11-04 11:18   ` Thomas Huth
@ 2016-11-04 11:46     ` Peter Maydell
  0 siblings, 0 replies; 4+ messages in thread
From: Peter Maydell @ 2016-11-04 11:46 UTC (permalink / raw)
  To: Thomas Huth
  Cc: qemu-arm, Alistair Francis, qemu-devel@nongnu.org Developers,
	Peter Chubb, Edgar E Iglesias

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-11-04 11:46 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-12  9:57 [Qemu-devel] [PATCH v2] MAINTAINERS: Add some ARM related files to the corresponding sections Thomas Huth
2016-10-12 13:34 ` Alistair Francis
2016-11-04 11:18   ` Thomas Huth
2016-11-04 11:46     ` Peter Maydell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).