[PATCH 0/4] MAINTAINERS: Update arm machines with missing files

[PATCH 0/4] MAINTAINERS: Update arm machines with missing files

[Thomas Huth]

Some currently unvalued files can be associated with certain arm machines.
Let's list them in the corresponding sections in the MAINTAINERS file
so that the get_maintainers.pl script can suggest a maintainer for these
files.

Thomas Huth (4):
  MAINTAINERS: Add include/hw/input/pl050.h to the PrimeCell/CMSDK
    section
  MAINTAINERS: Add hw/input/ads7846.c to the PXA2XX section
  MAINTAINERS: Add hw/display/sii9022.c to the Versatile Express section
  MAINTAINERS: Extend the Stellaris section

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

-- 
2.41.0

[PATCH 1/4] MAINTAINERS: Add include/hw/input/pl050.h to the PrimeCell/CMSDK section

[Thomas Huth]

The corresponding pl050.c file is already listed here, so we should
mention the header here, too.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 00642de2d7..14e8f7702d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -669,6 +669,7 @@ F: include/hw/dma/pl080.h
 F: hw/dma/pl330.c
 F: hw/gpio/pl061.c
 F: hw/input/pl050.c
+F: include/hw/input/pl050.h
 F: hw/intc/pl190.c
 F: hw/sd/pl181.c
 F: hw/ssi/pl022.c
-- 
2.41.0

Re: [PATCH 1/4] MAINTAINERS: Add include/hw/input/pl050.h to the PrimeCell/CMSDK section

[Philippe Mathieu-Daudé]

On 20/10/23 08:09, Thomas Huth wrote:
> The corresponding pl050.c file is already listed here, so we should
> mention the header here, too.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   MAINTAINERS | 1 +
>   1 file changed, 1 insertion(+)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

[PATCH 2/4] MAINTAINERS: Add hw/input/ads7846.c to the PXA2XX section

[Thomas Huth]

The code from hw/input/ads7846.c is only used by hw/arm/spitz.c,
so add this file to the same section where hw/arm/spitz.c is
listed.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 14e8f7702d..34f9036df8 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -938,6 +938,7 @@ F: hw/*/pxa2xx*
 F: hw/display/tc6393xb.c
 F: hw/gpio/max7310.c
 F: hw/gpio/zaurus.c
+F: hw/input/ads7846.c
 F: hw/misc/mst_fpga.c
 F: hw/adc/max111x.c
 F: include/hw/adc/max111x.h
-- 
2.41.0

Re: [PATCH 2/4] MAINTAINERS: Add hw/input/ads7846.c to the PXA2XX section

[Peter Maydell]

On Fri, 20 Oct 2023 at 07:09, Thomas Huth <thuth@redhat.com> wrote:
>
> The code from hw/input/ads7846.c is only used by hw/arm/spitz.c,
> so add this file to the same section where hw/arm/spitz.c is
> listed.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 14e8f7702d..34f9036df8 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -938,6 +938,7 @@ F: hw/*/pxa2xx*
>  F: hw/display/tc6393xb.c
>  F: hw/gpio/max7310.c
>  F: hw/gpio/zaurus.c
> +F: hw/input/ads7846.c
>  F: hw/misc/mst_fpga.c
>  F: hw/adc/max111x.c
>  F: include/hw/adc/max111x.h
> --
> 2.41.0

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

thanks
-- PMM

[PATCH 3/4] MAINTAINERS: Add hw/display/sii9022.c to the Versatile Express section

[Thomas Huth]

This graphics adapter is only used by the Versatile Express machine,
so add it to the corresonding section in MAINTAINERS.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 34f9036df8..4a7339130e 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1006,6 +1006,7 @@ M: Peter Maydell <peter.maydell@linaro.org>
 L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/arm/vexpress.c
+F: hw/display/sii9022.c
 F: docs/system/arm/vexpress.rst
 
 Versatile PB
-- 
2.41.0

Re: [PATCH 3/4] MAINTAINERS: Add hw/display/sii9022.c to the Versatile Express section

[Philippe Mathieu-Daudé]

On 20/10/23 08:09, Thomas Huth wrote:
> This graphics adapter is only used by the Versatile Express machine,
> so add it to the corresonding section in MAINTAINERS.

Typo "corresponding".

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   MAINTAINERS | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 34f9036df8..4a7339130e 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1006,6 +1006,7 @@ M: Peter Maydell <peter.maydell@linaro.org>
>   L: qemu-arm@nongnu.org
>   S: Maintained
>   F: hw/arm/vexpress.c
> +F: hw/display/sii9022.c
>   F: docs/system/arm/vexpress.rst
>   
>   Versatile PB

[PATCH 4/4] MAINTAINERS: Extend the Stellaris section

[Thomas Huth]

This header include/hw/timer/stellaris-gptm.h obviously belongs to the
Stellaris machines, so let's add it to the corresponding section.

And hw/display/ssd0303.c and hw/display/ssd0323.c are only used
by hw/arm/stellaris.c, so add them to the corresponding section
in the MAINTAINERS file, too.

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

diff --git a/MAINTAINERS b/MAINTAINERS
index 4a7339130e..a110a0c7a4 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -991,7 +991,9 @@ M: Peter Maydell <peter.maydell@linaro.org>
 L: qemu-arm@nongnu.org
 S: Maintained
 F: hw/*/stellaris*
+F: hw/display/ssd03*
 F: include/hw/input/gamepad.h
+F: include/hw/timer/stellaris-gptm.h
 F: docs/system/arm/stellaris.rst
 
 STM32VLDISCOVERY
-- 
2.41.0

Re: [PATCH 4/4] MAINTAINERS: Extend the Stellaris section

[Philippe Mathieu-Daudé]

On 20/10/23 08:09, Thomas Huth wrote:
> This header include/hw/timer/stellaris-gptm.h obviously belongs to the
> Stellaris machines, so let's add it to the corresponding section.
> 
> And hw/display/ssd0303.c and hw/display/ssd0323.c are only used
> by hw/arm/stellaris.c, so add them to the corresponding section
> in the MAINTAINERS file, too.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   MAINTAINERS | 2 ++
>   1 file changed, 2 insertions(+)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Re: [PATCH 0/4] MAINTAINERS: Update arm machines with missing files

[Thomas Huth]

On 20/10/2023 08.09, Thomas Huth wrote:
> Some currently unvalued files can be associated with certain arm machines.
> Let's list them in the corresponding sections in the MAINTAINERS file
> so that the get_maintainers.pl script can suggest a maintainer for these
> files.
> 
> Thomas Huth (4):
>    MAINTAINERS: Add include/hw/input/pl050.h to the PrimeCell/CMSDK
>      section
>    MAINTAINERS: Add hw/input/ads7846.c to the PXA2XX section
>    MAINTAINERS: Add hw/display/sii9022.c to the Versatile Express section
>    MAINTAINERS: Extend the Stellaris section
> 
>   MAINTAINERS | 5 +++++
>   1 file changed, 5 insertions(+)

Ping!

Any comments on these patches?

  Thomas