Re: [Qemu-devel] [PATCH v2] MAINTAINERS: Add some ARM related files to the corresponding sections

[Thomas Huth]

On 12.10.2016 15:34, Alistair Francis wrote:
> On Wed, Oct 12, 2016 at 11:57 AM, Thomas Huth <address@hidden> 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 <address@hidden>
>> ---
>>  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 <address@hidden>
>>  L: address@hidden
>>  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 <address@hidden>
>>  L: address@hidden
>>  S: Maintained
>>  F: hw/arm/integratorcp.c
>> +F: hw/misc/arm_integrator_debug.c
>>
>>  Musicpal
>>  M: Jan Kiszka <address@hidden>
>> @@ -440,6 +443,7 @@ M: Peter Maydell <address@hidden>
>>  L: address@hidden
>>  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: address@hidden
>>  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 <address@hidden>

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

 Thomas

signature.asc
Description: OpenPGP digital signature