Re: [Qemu-devel] [PATCH] MAINTAINERS: XIVE is an interrupt controller, not a machine

[Philippe Mathieu-Daudé]

On 1/30/19 4:45 PM, Thomas Huth wrote:
> The "XIVE" section is currently listed in the "PowerPC Machines"
> section, which is weird, since this is an interrupt controller
> device. Move it to the "Devices" section instead.
> 
> Signed-off-by: Thomas Huth <address@hidden>
> ---
>  MAINTAINERS | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b334b53..eac36b6 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1038,14 +1038,6 @@ F: tests/libqos/*spapr*
>  F: tests/rtas*
>  F: tests/libqos/rtas*
>  
> -XIVE
> -M: David Gibson <address@hidden>
> -M: Cédric Le Goater <address@hidden>
> -L: address@hidden
> -S: Supported
> -F: hw/*/*xive*
> -F: include/hw/*/*xive*
> -
>  virtex_ml507
>  M: Edgar E. Iglesias <address@hidden>
>  L: address@hidden
> @@ -1645,6 +1637,14 @@ F: tests/libqos/fw_cfg.c
>  F: tests/fw_cfg-test.c
>  T: git https://github.com/philmd/qemu.git fw_cfg-next
>  
> +XIVE
> +M: David Gibson <address@hidden>
> +M: Cédric Le Goater <address@hidden>
> +L: address@hidden
> +S: Supported
> +F: hw/*/*xive*

   F: hw/intc/*xive*

> +F: include/hw/*/*xive*

I'd expect them in /intc/ like the source, but they are in:

   F: include/hw/ppc/*xive*

> +
>  Subsystems
>  ----------
>  Audio
> 

With or without the paths updated:
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>