Re: [Qemu-trivial] [Qemu-devel] MAINTAINERS: Add some SPARC machine related files

[Thomas Huth <thuth@redhat.com>]

On 23.09.2016 16:05, Mark Cave-Ayland wrote:
> On 19/09/16 20:58, Thomas Huth wrote:
> 
>> And while we're at it, remove Blue Swirl from the list
>> of maintainers. Blue has apparently been inactive for
>> quite a while now, so I assume he's unfortunately
>> not available as maintainer anymore.
>>
>> Signed-off-by: Thomas Huth <huth@tuxfamily.org>
>> ---
>>  MAINTAINERS | 9 +++++++--
>>  1 file changed, 7 insertions(+), 2 deletions(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index a9fab46..c6297c2 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -650,22 +650,27 @@ F: hw/sh4/shix.c
>>  SPARC Machines
>>  --------------
>>  Sun4m
>> -M: Blue Swirl <blauwirbel@gmail.com>
>>  M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>>  S: Maintained
>>  F: hw/sparc/sun4m.c
>> +F: hw/dma/sparc32_dma.c
>> +F: hw/dma/sun4m_iommu.c
>> +F: include/hw/sparc/sparc32_dma.h
>> +F: include/hw/sparc/sun4m.h
>> +F: pc-bios/openbios-sparc32
>>  
>>  Sun4u
>> -M: Blue Swirl <blauwirbel@gmail.com>
>>  M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>>  S: Maintained
>>  F: hw/sparc64/sun4u.c
>> +F: pc-bios/openbios-sparc64
>>  
>>  Leon3
>>  M: Fabien Chouteau <chouteau@adacore.com>
>>  S: Maintained
>>  F: hw/sparc/leon3.c
>>  F: hw/*/grlib*
>> +F: include/hw/sparc/grlib.h
>>  
>>  S390 Machines
>>  -------------
> 
> No issue with the sun4m/sun4u changes, however shouldn't the OpenBIOS
> binaries be handled separately by the OpenBIOS project rather than by
> the SPARC maintainers? (I guess technically that is currently also me,
> but generally I do a pull request that updates all SPARC/PPC binaries in
> one go)

Unless you explicitely want to add an "OPENBIOS" section to MAINTAINERS,
I think it's currently OK to assign these files to the SPARC machines,
since both is handled by you (SPARC machines and OpenBIOS binaries).
(openbios-ppc might get more problematic, though... but that's not so
easy to assign it to a specific machine anyway since it is shared
between Old World and New World machines, so we might need a separate
solution for openbios-ppc later anyway)

 Thomas