[v2] MAINTAINERS: Assign some more files in the hw/arm/ directory

[Qemu-devel] [PATCH v2] MAINTAINERS: Assign some more files in the hw/arm/ directory

Posted by Thomas Huth 5 years, 5 months ago

I apparently missed some more files and even a complete machine (the
"imx25-pdk") in my previous patch... but now we should hopefully have
a completely coverage for all available ARM boards.

Fixes: 95a5db3ae5698b49c63144610ad02913e780c828
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 v2: Dropped smmu* files from this patch

 MAINTAINERS | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 1032406..36ffc01 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -533,6 +533,18 @@ L: qemu-arm@nongnu.org
 S: Odd Fixes
 F: hw/arm/gumstix.c
 
+i.MX25 PDK
+M: Peter Maydell <peter.maydell@linaro.org>
+R: Jean-Christophe Dubois <jcd@tribudubois.net>
+L: qemu-arm@nongnu.org
+S: Odd Fixes
+F: hw/arm/fsl-imx25.c
+F: hw/arm/imx25_pdk.c
+F: hw/misc/imx25_ccm.c
+F: include/hw/arm/fsl-imx25.h
+F: include/hw/misc/imx25_ccm.h
+F: tests/ds1338-test.c
+
 i.MX31 (kzm)
 M: Peter Chubb <peter.chubb@nicta.com.au>
 M: Peter Maydell <peter.maydell@linaro.org>
@@ -615,6 +627,7 @@ R: Andrew Baumann <Andrew.Baumann@microsoft.com>
 R: Philippe Mathieu-Daudé <f4bug@amsat.org>
 L: qemu-arm@nongnu.org
 S: Odd Fixes
+F: hw/arm/raspi.c
 F: hw/arm/raspi_platform.h
 F: hw/*/bcm283*
 F: include/hw/arm/raspi*
@@ -641,6 +654,7 @@ F: hw/arm/z2.c
 F: hw/*/pxa2xx*
 F: hw/misc/mst_fpga.c
 F: include/hw/arm/pxa.h
+F: include/hw/arm/sharpsl.h
 
 SABRELITE / i.MX6
 M: Peter Maydell <peter.maydell@linaro.org>
-- 
1.8.3.1

Re: [Qemu-devel] [PATCH v2] MAINTAINERS: Assign some more files in the hw/arm/ directory

Posted by Philippe Mathieu-Daudé 5 years, 5 months ago

On 20/11/18 18:42, Thomas Huth wrote:
> I apparently missed some more files and even a complete machine (the
> "imx25-pdk") in my previous patch... but now we should hopefully have
> a completely coverage for all available ARM boards.
> 
> Fixes: 95a5db3ae5698b49c63144610ad02913e780c828
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   v2: Dropped smmu* files from this patch
> 
>   MAINTAINERS | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 1032406..36ffc01 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -533,6 +533,18 @@ L: qemu-arm@nongnu.org
>   S: Odd Fixes
>   F: hw/arm/gumstix.c
>   
> +i.MX25 PDK
> +M: Peter Maydell <peter.maydell@linaro.org>
> +R: Jean-Christophe Dubois <jcd@tribudubois.net>
> +L: qemu-arm@nongnu.org
> +S: Odd Fixes
> +F: hw/arm/fsl-imx25.c
> +F: hw/arm/imx25_pdk.c
> +F: hw/misc/imx25_ccm.c
> +F: include/hw/arm/fsl-imx25.h
> +F: include/hw/misc/imx25_ccm.h

> +F: tests/ds1338-test.c

This last entry is weird alone (at least without the hw/timer/ds1338.c 
device listed).

This RTC is also used by the aspeed/realview/versatilepb boards.

> +
>   i.MX31 (kzm)
>   M: Peter Chubb <peter.chubb@nicta.com.au>
>   M: Peter Maydell <peter.maydell@linaro.org>
> @@ -615,6 +627,7 @@ R: Andrew Baumann <Andrew.Baumann@microsoft.com>
>   R: Philippe Mathieu-Daudé <f4bug@amsat.org>
>   L: qemu-arm@nongnu.org
>   S: Odd Fixes
> +F: hw/arm/raspi.c
>   F: hw/arm/raspi_platform.h
>   F: hw/*/bcm283*
>   F: include/hw/arm/raspi*
> @@ -641,6 +654,7 @@ F: hw/arm/z2.c
>   F: hw/*/pxa2xx*
>   F: hw/misc/mst_fpga.c
>   F: include/hw/arm/pxa.h
> +F: include/hw/arm/sharpsl.h

I think hw/gpio/zaurus.c goes there too.

>   
>   SABRELITE / i.MX6
>   M: Peter Maydell <peter.maydell@linaro.org>
> 

Regards,

Phil.

Re: [Qemu-devel] [PATCH v2] MAINTAINERS: Assign some more files in the hw/arm/ directory

Posted by Thomas Huth 5 years, 5 months ago

On 2018-11-20 21:23, Philippe Mathieu-Daudé wrote:
> On 20/11/18 18:42, Thomas Huth wrote:
>> I apparently missed some more files and even a complete machine (the
>> "imx25-pdk") in my previous patch... but now we should hopefully have
>> a completely coverage for all available ARM boards.
>>
>> Fixes: 95a5db3ae5698b49c63144610ad02913e780c828
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>>   v2: Dropped smmu* files from this patch
>>
>>   MAINTAINERS | 14 ++++++++++++++
>>   1 file changed, 14 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 1032406..36ffc01 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -533,6 +533,18 @@ L: qemu-arm@nongnu.org
>>   S: Odd Fixes
>>   F: hw/arm/gumstix.c
>>   +i.MX25 PDK
>> +M: Peter Maydell <peter.maydell@linaro.org>
>> +R: Jean-Christophe Dubois <jcd@tribudubois.net>
>> +L: qemu-arm@nongnu.org
>> +S: Odd Fixes
>> +F: hw/arm/fsl-imx25.c
>> +F: hw/arm/imx25_pdk.c
>> +F: hw/misc/imx25_ccm.c
>> +F: include/hw/arm/fsl-imx25.h
>> +F: include/hw/misc/imx25_ccm.h
> 
>> +F: tests/ds1338-test.c
> 
> This last entry is weird alone (at least without the hw/timer/ds1338.c
> device listed).
> 
> This RTC is also used by the aspeed/realview/versatilepb boards.

You're right. I've added it since the test uses "-machine imx25-pdk",
but the correct way is likely to add a separate section for the timer
device instead. I'll remove this line in v3.

>> +
>>   i.MX31 (kzm)
>>   M: Peter Chubb <peter.chubb@nicta.com.au>
>>   M: Peter Maydell <peter.maydell@linaro.org>
>> @@ -615,6 +627,7 @@ R: Andrew Baumann <Andrew.Baumann@microsoft.com>
>>   R: Philippe Mathieu-Daudé <f4bug@amsat.org>
>>   L: qemu-arm@nongnu.org
>>   S: Odd Fixes
>> +F: hw/arm/raspi.c
>>   F: hw/arm/raspi_platform.h
>>   F: hw/*/bcm283*
>>   F: include/hw/arm/raspi*
>> @@ -641,6 +654,7 @@ F: hw/arm/z2.c
>>   F: hw/*/pxa2xx*
>>   F: hw/misc/mst_fpga.c
>>   F: include/hw/arm/pxa.h
>> +F: include/hw/arm/sharpsl.h
> 
> I think hw/gpio/zaurus.c goes there too.

Well spotted. I'll add it in v3.

And I think we should add hw/display/tc6393xb.c here as well.

 Thomas