[Qemu-devel] [PATCH] MAINTAINERS: Add ARM-related files for hw/[misc|input|timer]/

Thomas Huth posted 1 patch 6 years, 10 months ago
Test asan passed
Test checkpatch passed
Test docker-mingw@fedora passed
Test docker-quick@centos7 passed
Test docker-clang@ubuntu passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1546433583-18397-1-git-send-email-thuth@redhat.com
MAINTAINERS | 16 ++++++++++++++--
1 file changed, 14 insertions(+), 2 deletions(-)
[Qemu-devel] [PATCH] MAINTAINERS: Add ARM-related files for hw/[misc|input|timer]/
Posted by Thomas Huth 6 years, 10 months ago
Some of the files in hw/input/, hw/misc/ and hw/timer/ are only
used by one of the ARM machines, so we can assign these files to
the corresponding boards.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 180695f..693abdf 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -510,6 +510,7 @@ F: hw/intc/arm*
 F: hw/intc/gic_internal.h
 F: hw/misc/a9scu.c
 F: hw/misc/arm11scu.c
+F: hw/misc/arm_l2x0.c
 F: hw/timer/a9gtimer*
 F: hw/timer/arm*
 F: include/hw/arm/arm*.h
@@ -582,6 +583,7 @@ L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/arm/integratorcp.c
 F: hw/misc/arm_integrator_debug.c
+F: include/hw/misc/arm_integrator_debug.h
 
 MCIMX6UL EVK / i.MX6ul
 M: Peter Maydell <peter.maydell@linaro.org>
@@ -601,7 +603,9 @@ L: qemu-arm@nongnu.org
 S: Odd Fixes
 F: hw/arm/mcimx7d-sabre.c
 F: hw/arm/fsl-imx7.c
+F: hw/misc/imx7_*.c
 F: include/hw/arm/fsl-imx7.h
+F: include/hw/misc/imx7_*.h
 F: hw/pci-host/designware.c
 F: include/hw/pci-host/designware.h
 
@@ -635,6 +639,10 @@ M: Peter Maydell <peter.maydell@linaro.org>
 L: qemu-arm@nongnu.org
 S: Odd Fixes
 F: hw/arm/nseries.c
+F: hw/input/lm832x.c
+F: hw/input/tsc2005.c
+F: hw/misc/cbus.c
+F: hw/timer/twl92230.c
 
 Palm
 M: Andrzej Zaborowski <balrogg@gmail.com>
@@ -642,6 +650,7 @@ M: Peter Maydell <peter.maydell@linaro.org>
 L: qemu-arm@nongnu.org
 S: Odd Fixes
 F: hw/arm/palm.c
+F: hw/input/tsc210x.c
 
 Raspberry Pi
 M: Peter Maydell <peter.maydell@linaro.org>
@@ -678,6 +687,7 @@ F: hw/display/tc6393xb.c
 F: hw/gpio/max7310.c
 F: hw/gpio/zaurus.c
 F: hw/misc/mst_fpga.c
+F: hw/misc/max111x.c
 F: include/hw/arm/pxa.h
 F: include/hw/arm/sharpsl.h
 
@@ -688,10 +698,10 @@ L: qemu-arm@nongnu.org
 S: Odd Fixes
 F: hw/arm/sabrelite.c
 F: hw/arm/fsl-imx6.c
-F: hw/misc/imx6_src.c
+F: hw/misc/imx6_*.c
 F: hw/ssi/imx_spi.c
 F: include/hw/arm/fsl-imx6.h
-F: include/hw/misc/imx6_src.h
+F: include/hw/misc/imx6_*.h
 F: include/hw/ssi/imx_spi.h
 
 Sharp SL-5500 (Collie) PDA
@@ -802,7 +812,9 @@ R: Joel Stanley <joel@jms.id.au>
 L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/*/*aspeed*
+F: hw/misc/pca9552.c
 F: include/hw/*/*aspeed*
+F: include/hw/misc/pca9552*.h
 F: hw/net/ftgmac100.c
 F: include/hw/net/ftgmac100.h
 
-- 
1.8.3.1


Re: [Qemu-devel] [PATCH] MAINTAINERS: Add ARM-related files for hw/[misc|input|timer]/
Posted by Philippe Mathieu-Daudé 6 years, 10 months ago
On 1/2/19 1:53 PM, Thomas Huth wrote:
> Some of the files in hw/input/, hw/misc/ and hw/timer/ are only
> used by one of the ARM machines, so we can assign these files to
> the corresponding boards.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  MAINTAINERS | 16 ++++++++++++++--
>  1 file changed, 14 insertions(+), 2 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 180695f..693abdf 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -510,6 +510,7 @@ F: hw/intc/arm*
>  F: hw/intc/gic_internal.h
>  F: hw/misc/a9scu.c
>  F: hw/misc/arm11scu.c
> +F: hw/misc/arm_l2x0.c
>  F: hw/timer/a9gtimer*
>  F: hw/timer/arm*
>  F: include/hw/arm/arm*.h
> @@ -582,6 +583,7 @@ L: qemu-arm@nongnu.org
>  S: Maintained
>  F: hw/arm/integratorcp.c
>  F: hw/misc/arm_integrator_debug.c
> +F: include/hw/misc/arm_integrator_debug.h

OK

>  
>  MCIMX6UL EVK / i.MX6ul
>  M: Peter Maydell <peter.maydell@linaro.org>
> @@ -601,7 +603,9 @@ L: qemu-arm@nongnu.org
>  S: Odd Fixes
>  F: hw/arm/mcimx7d-sabre.c
>  F: hw/arm/fsl-imx7.c
> +F: hw/misc/imx7_*.c
>  F: include/hw/arm/fsl-imx7.h
> +F: include/hw/misc/imx7_*.h
>  F: hw/pci-host/designware.c
>  F: include/hw/pci-host/designware.h

OK

>  
> @@ -635,6 +639,10 @@ M: Peter Maydell <peter.maydell@linaro.org>
>  L: qemu-arm@nongnu.org
>  S: Odd Fixes
>  F: hw/arm/nseries.c
> +F: hw/input/lm832x.c
> +F: hw/input/tsc2005.c
> +F: hw/misc/cbus.c
> +F: hw/timer/twl92230.c

OK

>  
>  Palm
>  M: Andrzej Zaborowski <balrogg@gmail.com>
> @@ -642,6 +650,7 @@ M: Peter Maydell <peter.maydell@linaro.org>
>  L: qemu-arm@nongnu.org
>  S: Odd Fixes
>  F: hw/arm/palm.c
> +F: hw/input/tsc210x.c

OK

>  
>  Raspberry Pi
>  M: Peter Maydell <peter.maydell@linaro.org>
> @@ -678,6 +687,7 @@ F: hw/display/tc6393xb.c
>  F: hw/gpio/max7310.c
>  F: hw/gpio/zaurus.c
>  F: hw/misc/mst_fpga.c
> +F: hw/misc/max111x.c
>  F: include/hw/arm/pxa.h
>  F: include/hw/arm/sharpsl.h

OK

>  
> @@ -688,10 +698,10 @@ L: qemu-arm@nongnu.org
>  S: Odd Fixes
>  F: hw/arm/sabrelite.c
>  F: hw/arm/fsl-imx6.c
> -F: hw/misc/imx6_src.c
> +F: hw/misc/imx6_*.c
>  F: hw/ssi/imx_spi.c
>  F: include/hw/arm/fsl-imx6.h
> -F: include/hw/misc/imx6_src.h
> +F: include/hw/misc/imx6_*.h
>  F: include/hw/ssi/imx_spi.h

New models for the MCIMX6UL EVK might eventually match here.

>  
>  Sharp SL-5500 (Collie) PDA
> @@ -802,7 +812,9 @@ R: Joel Stanley <joel@jms.id.au>
>  L: qemu-arm@nongnu.org
>  S: Maintained
>  F: hw/*/*aspeed*
> +F: hw/misc/pca9552.c
>  F: include/hw/*/*aspeed*
> +F: include/hw/misc/pca9552*.h
>  F: hw/net/ftgmac100.c
>  F: include/hw/net/ftgmac100.h

OK

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Re: [Qemu-devel] [PATCH] MAINTAINERS: Add ARM-related files for hw/[misc|input|timer]/
Posted by Thomas Huth 6 years, 10 months ago
On 2019-01-02 15:42, Philippe Mathieu-Daudé wrote:
> On 1/2/19 1:53 PM, Thomas Huth wrote:
>> Some of the files in hw/input/, hw/misc/ and hw/timer/ are only
>> used by one of the ARM machines, so we can assign these files to
>> the corresponding boards.
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>>  MAINTAINERS | 16 ++++++++++++++--
>>  1 file changed, 14 insertions(+), 2 deletions(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 180695f..693abdf 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
[...]
>> @@ -688,10 +698,10 @@ L: qemu-arm@nongnu.org
>>  S: Odd Fixes
>>  F: hw/arm/sabrelite.c
>>  F: hw/arm/fsl-imx6.c
>> -F: hw/misc/imx6_src.c
>> +F: hw/misc/imx6_*.c
>>  F: hw/ssi/imx_spi.c
>>  F: include/hw/arm/fsl-imx6.h
>> -F: include/hw/misc/imx6_src.h
>> +F: include/hw/misc/imx6_*.h
>>  F: include/hw/ssi/imx_spi.h
> 
> New models for the MCIMX6UL EVK might eventually match here.

The current files use "*imx6ul*" which do not match here due to the
underscore after the "6". So I hope future files will use this naming
scheme, too, then we should be fine.

>>  
>>  Sharp SL-5500 (Collie) PDA
>> @@ -802,7 +812,9 @@ R: Joel Stanley <joel@jms.id.au>
>>  L: qemu-arm@nongnu.org
>>  S: Maintained
>>  F: hw/*/*aspeed*
>> +F: hw/misc/pca9552.c
>>  F: include/hw/*/*aspeed*
>> +F: include/hw/misc/pca9552*.h
>>  F: hw/net/ftgmac100.c
>>  F: include/hw/net/ftgmac100.h
> 
> OK
> 
> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Thanks!

 Thomas

Re: [Qemu-devel] [PATCH] MAINTAINERS: Add ARM-related files for hw/[misc|input|timer]/
Posted by Peter Maydell 6 years, 10 months ago
On Wed, 2 Jan 2019 at 12:53, Thomas Huth <thuth@redhat.com> wrote:
>
> Some of the files in hw/input/, hw/misc/ and hw/timer/ are only
> used by one of the ARM machines, so we can assign these files to
> the corresponding boards.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  MAINTAINERS | 16 ++++++++++++++--
>  1 file changed, 14 insertions(+), 2 deletions(-)
>


Applied to target-arm.next, thanks.

-- PMM