Patchwork [4/4] PPC: Change PPC maintainer

login
register
mail settings
Submitter Alexander Graf
Date Sept. 7, 2010, 11:53 a.m.
Message ID <1283860398-11637-5-git-send-email-agraf@suse.de>
Download mbox | patch
Permalink /patch/64000/
State New
Headers show

Comments

Alexander Graf - Sept. 7, 2010, 11:53 a.m.
Since nobody else seems interested in maintaining PPC, let's change the
maintainer to myself. I keep a staging tree anyways and am probably the
person touching most of that code these days.

This changes the maintainer entry for working ppc targets to myself.

Signed-off-by: Alexander Graf <agraf@suse.de>
---
 MAINTAINERS |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)
Andreas Färber - Sept. 7, 2010, 9:17 p.m.
Am 07.09.2010 um 13:53 schrieb Alexander Graf:

> Since nobody else seems interested in maintaining PPC, let's change  
> the
> maintainer to myself. I keep a staging tree anyways and am probably  
> the
> person touching most of that code these days.
>
> This changes the maintainer entry for working ppc targets to myself.
>
> Signed-off-by: Alexander Graf <agraf@suse.de>

Fwiw,

Acked-by: Andreas Färber <andreas.faerber@web.de>

Alex seems like the most active in the ppc machines code and its  
OpenBIOS counterpart lately.

Malc has been the maintainer of TCG/ppc[64] though.

Cheers,
Andreas
Alexander Graf - Sept. 7, 2010, 9:36 p.m.
On 07.09.2010, at 23:17, Andreas Färber wrote:

> Am 07.09.2010 um 13:53 schrieb Alexander Graf:
> 
>> Since nobody else seems interested in maintaining PPC, let's change the
>> maintainer to myself. I keep a staging tree anyways and am probably the
>> person touching most of that code these days.
>> 
>> This changes the maintainer entry for working ppc targets to myself.
>> 
>> Signed-off-by: Alexander Graf <agraf@suse.de>
> 
> Fwiw,
> 
> Acked-by: Andreas Färber <andreas.faerber@web.de>
> 
> Alex seems like the most active in the ppc machines code and its OpenBIOS counterpart lately.

Thanks for the support :). May I recommend you as the Mac OS X host support maintainer?

> Malc has been the maintainer of TCG/ppc[64] though.

Yup. TCG backends are missing completely from the MAINTAINERS file, no?


Alex
Andreas Färber - Sept. 7, 2010, 10:21 p.m.
Am 07.09.2010 um 23:36 schrieb Alexander Graf:

> On 07.09.2010, at 23:17, Andreas Färber wrote:
>
>> Am 07.09.2010 um 13:53 schrieb Alexander Graf:
>>
>>> Since nobody else seems interested in maintaining PPC, let's  
>>> change the
>>> maintainer to myself. I keep a staging tree anyways and am  
>>> probably the
>>> person touching most of that code these days.
>>>
>>> This changes the maintainer entry for working ppc targets to myself.
>>>
>>> Signed-off-by: Alexander Graf <agraf@suse.de>
>>
>> Fwiw,
>>
>> Acked-by: Andreas Färber <andreas.faerber@web.de>
>>
>> Alex seems like the most active in the ppc machines code and its  
>> OpenBIOS counterpart lately.
>
> Thanks for the support :). May I recommend you as the Mac OS X host  
> support maintainer?

Hm, I would volunteer as Cocoa maintainer. Virtually all Cocoa patches  
than were committed, except for the DisplayState conversion, went  
through my queue for some time now.
Some topics still on my list include: 'daemon' deprecation warning in  
nbd code (same on Solaris), the qemu_main ugliness and a Cocoa warning  
on Snow Leopard spotted by the Nokia guys. What I don't intend to  
investigate any time soon is the chatty CoreAudio code.

>> Malc has been the maintainer of TCG/ppc[64] though.
>
> Yup. TCG backends are missing completely from the MAINTAINERS file,  
> no?

Right. Elsewhere I suggested to add entries for them, doesn't need to  
be in this patch of course.

I'd consider it handy to put the maintainer's preferred email address  
alongside there, but that's yet another potential source of spam then.

Andreas
malc - Sept. 7, 2010, 10:48 p.m.
On Wed, 8 Sep 2010, Andreas F?rber wrote:

> Am 07.09.2010 um 23:36 schrieb Alexander Graf:
> 
> > On 07.09.2010, at 23:17, Andreas F?rber wrote:
> > 
> > > Am 07.09.2010 um 13:53 schrieb Alexander Graf:
> > > 
> > > > Since nobody else seems interested in maintaining PPC, let's change the
> > > > maintainer to myself. I keep a staging tree anyways and am probably the
> > > > person touching most of that code these days.
> > > > 
> > > > This changes the maintainer entry for working ppc targets to myself.
> > > > 
> > > > Signed-off-by: Alexander Graf <agraf@suse.de>
> > > 
> > > Fwiw,
> > > 
> > > Acked-by: Andreas F?rber <andreas.faerber@web.de>
> > > 
> > > Alex seems like the most active in the ppc machines code and its OpenBIOS
> > > counterpart lately.
> > 
> > Thanks for the support :). May I recommend you as the Mac OS X host support
> > maintainer?
> 
> Hm, I would volunteer as Cocoa maintainer. Virtually all Cocoa patches than
> were committed, except for the DisplayState conversion, went through my queue
> for some time now.
> Some topics still on my list include: 'daemon' deprecation warning in nbd code
> (same on Solaris), the qemu_main ugliness and a Cocoa warning on Snow Leopard
> spotted by the Nokia guys. What I don't intend to investigate any time soon is
> the chatty CoreAudio code.

What about it?

> 
> > > Malc has been the maintainer of TCG/ppc[64] though.
> > 
> > Yup. TCG backends are missing completely from the MAINTAINERS file, no?
> 
> Right. Elsewhere I suggested to add entries for them, doesn't need to be in
> this patch of course.
> 
> I'd consider it handy to put the maintainer's preferred email address
> alongside there, but that's yet another potential source of spam then.
> 
> Andreas
Alexander Graf - Sept. 7, 2010, 11 p.m.
On 08.09.2010, at 00:48, malc wrote:

> On Wed, 8 Sep 2010, Andreas F?rber wrote:
> 
>> Am 07.09.2010 um 23:36 schrieb Alexander Graf:
>> 
>>> On 07.09.2010, at 23:17, Andreas F?rber wrote:
>>> 
>>>> Am 07.09.2010 um 13:53 schrieb Alexander Graf:
>>>> 
>>>>> Since nobody else seems interested in maintaining PPC, let's change the
>>>>> maintainer to myself. I keep a staging tree anyways and am probably the
>>>>> person touching most of that code these days.
>>>>> 
>>>>> This changes the maintainer entry for working ppc targets to myself.
>>>>> 
>>>>> Signed-off-by: Alexander Graf <agraf@suse.de>
>>>> 
>>>> Fwiw,
>>>> 
>>>> Acked-by: Andreas F?rber <andreas.faerber@web.de>
>>>> 
>>>> Alex seems like the most active in the ppc machines code and its OpenBIOS
>>>> counterpart lately.
>>> 
>>> Thanks for the support :). May I recommend you as the Mac OS X host support
>>> maintainer?
>> 
>> Hm, I would volunteer as Cocoa maintainer. Virtually all Cocoa patches than
>> were committed, except for the DisplayState conversion, went through my queue
>> for some time now.
>> Some topics still on my list include: 'daemon' deprecation warning in nbd code
>> (same on Solaris), the qemu_main ugliness and a Cocoa warning on Snow Leopard
>> spotted by the Nokia guys. What I don't intend to investigate any time soon is
>> the chatty CoreAudio code.
> 
> What about it?

It throws about 100 warnings when compiled on 10.6 :).


Alex
malc - Sept. 8, 2010, 1:19 a.m.
On Wed, 8 Sep 2010, Alexander Graf wrote:

> 
> On 08.09.2010, at 00:48, malc wrote:
> 
> > On Wed, 8 Sep 2010, Andreas F?rber wrote:
> > 
> >> Am 07.09.2010 um 23:36 schrieb Alexander Graf:
> >> 
> >>> On 07.09.2010, at 23:17, Andreas F?rber wrote:
> >>> 
> >>>> Am 07.09.2010 um 13:53 schrieb Alexander Graf:
> >>>> 

[..snip..]

> >> Some topics still on my list include: 'daemon' deprecation warning in nbd code
> >> (same on Solaris), the qemu_main ugliness and a Cocoa warning on Snow Leopard
> >> spotted by the Nokia guys. What I don't intend to investigate any time soon is
> >> the chatty CoreAudio code.
> > 
> > What about it?
> 
> It throws about 100 warnings when compiled on 10.6 :).
> 

Oh, i thought it was a runtime issue, aslo and given that i don't have
10.6...
Andreas Färber - Sept. 9, 2010, 8:23 p.m.
Am 08.09.2010 um 03:19 schrieb malc:

> On Wed, 8 Sep 2010, Alexander Graf wrote:
>
>> On 08.09.2010, at 00:48, malc wrote:
>>
>>> On Wed, 8 Sep 2010, Andreas F?rber wrote:
>>>> What I don't intend to investigate any time soon is
>>>> the chatty CoreAudio code.
>>>
>>> What about it?
>>
>> It throws about 100 warnings when compiled on 10.6 :).
>
> Oh, i thought it was a runtime issue, aslo and given that i don't have
> 10.6...

For reference here's v10.5/ppc64 audio output (ppc below):

   CC    audio/audio.o
   CC    audio/noaudio.o
   CC    audio/wavaudio.o
   CC    audio/mixeng.o
In file included from /Users/andreas/QEMU/qemu/audio/mixeng.c:112:
/Users/andreas/QEMU/qemu/audio/mixeng_template.h: In function  
‘conv_natural_int32_t’:
/Users/andreas/QEMU/qemu/audio/mixeng_template.h:59: warning: integer  
overflow in expression
/Users/andreas/QEMU/qemu/audio/mixeng_template.h: In function  
‘clip_natural_int32_t’:
/Users/andreas/QEMU/qemu/audio/mixeng_template.h:76: warning: integer  
overflow in expression
In file included from /Users/andreas/QEMU/qemu/audio/mixeng.c:117:
/Users/andreas/QEMU/qemu/audio/mixeng_template.h: In function  
‘conv_swap_int32_t’:
/Users/andreas/QEMU/qemu/audio/mixeng_template.h:59: warning: integer  
overflow in expression
/Users/andreas/QEMU/qemu/audio/mixeng_template.h: In function  
‘clip_swap_int32_t’:
/Users/andreas/QEMU/qemu/audio/mixeng_template.h:76: warning: integer  
overflow in expression
   CC    audio/coreaudio.o
/Users/andreas/QEMU/qemu/audio/coreaudio.c: In function  
‘coreaudio_logstatus’:
/Users/andreas/QEMU/qemu/audio/coreaudio.c:59: warning: initialization  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:63: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:67: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:71: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:75: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:79: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:83: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:87: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:91: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:95: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:99: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:103: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:107: warning: format ‘%ld’  
expects type ‘long int’, but argument 3 has type ‘OSStatus’
/Users/andreas/QEMU/qemu/audio/coreaudio.c: In function  
‘coreaudio_init_out’:
/Users/andreas/QEMU/qemu/audio/coreaudio.c:364: warning: format ‘%ld’  
expects type ‘long int’, but argument 4 has type ‘UInt32’
/Users/andreas/QEMU/qemu/audio/coreaudio.c:419: warning:  
‘AudioDeviceAddIOProc’ is deprecated (declared at /System/Library/ 
Frameworks/CoreAudio.framework/Headers/AudioHardware.h:2067)
/Users/andreas/QEMU/qemu/audio/coreaudio.c:431: warning:  
‘AudioDeviceRemoveIOProc’ is deprecated (declared at /System/Library/ 
Frameworks/CoreAudio.framework/Headers/AudioHardware.h:2081)
/Users/andreas/QEMU/qemu/audio/coreaudio.c: In function  
‘coreaudio_fini_out’:
/Users/andreas/QEMU/qemu/audio/coreaudio.c:456: warning:  
‘AudioDeviceRemoveIOProc’ is deprecated (declared at /System/Library/ 
Frameworks/CoreAudio.framework/Headers/AudioHardware.h:2081)
   CC    audio/wavcapture.o

v10.5/ppc:

   CC    audio/audio.o
   CC    audio/noaudio.o
   CC    audio/wavaudio.o
   CC    audio/mixeng.o
In file included from /Users/andreas/QEMU/qemu/audio/mixeng.c:112:
/Users/andreas/QEMU/qemu/audio/mixeng_template.h: In function  
‘conv_natural_int32_t’:
/Users/andreas/QEMU/qemu/audio/mixeng_template.h:59: warning: integer  
overflow in expression
/Users/andreas/QEMU/qemu/audio/mixeng_template.h: In function  
‘clip_natural_int32_t’:
/Users/andreas/QEMU/qemu/audio/mixeng_template.h:76: warning: integer  
overflow in expression
In file included from /Users/andreas/QEMU/qemu/audio/mixeng.c:117:
/Users/andreas/QEMU/qemu/audio/mixeng_template.h: In function  
‘conv_swap_int32_t’:
/Users/andreas/QEMU/qemu/audio/mixeng_template.h:59: warning: integer  
overflow in expression
/Users/andreas/QEMU/qemu/audio/mixeng_template.h: In function  
‘clip_swap_int32_t’:
/Users/andreas/QEMU/qemu/audio/mixeng_template.h:76: warning: integer  
overflow in expression
   CC    audio/coreaudio.o
/Users/andreas/QEMU/qemu/audio/coreaudio.c: In function  
‘coreaudio_logstatus’:
/Users/andreas/QEMU/qemu/audio/coreaudio.c:59: warning: initialization  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:63: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:67: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:71: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:75: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:79: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:83: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:87: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:91: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:95: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:99: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c:103: warning: assignment  
discards qualifiers from pointer target type
/Users/andreas/QEMU/qemu/audio/coreaudio.c: In function  
‘coreaudio_init_out’:
/Users/andreas/QEMU/qemu/audio/coreaudio.c:419: warning:  
‘AudioDeviceAddIOProc’ is deprecated (declared at /System/Library/ 
Frameworks/CoreAudio.framework/Headers/AudioHardware.h:2067)
/Users/andreas/QEMU/qemu/audio/coreaudio.c:431: warning:  
‘AudioDeviceRemoveIOProc’ is deprecated (declared at /System/Library/ 
Frameworks/CoreAudio.framework/Headers/AudioHardware.h:2081)
/Users/andreas/QEMU/qemu/audio/coreaudio.c: In function  
‘coreaudio_fini_out’:
/Users/andreas/QEMU/qemu/audio/coreaudio.c:456: warning:  
‘AudioDeviceRemoveIOProc’ is deprecated (declared at /System/Library/ 
Frameworks/CoreAudio.framework/Headers/AudioHardware.h:2081)
   CC    audio/wavcapture.o

Haven't tested whether there are any audio runtime issues.

Andreas
malc - Sept. 9, 2010, 9:31 p.m.
On Thu, 9 Sep 2010, Andreas F?rber wrote:

> Am 08.09.2010 um 03:19 schrieb malc:
> 
> > On Wed, 8 Sep 2010, Alexander Graf wrote:
> > 
> > > On 08.09.2010, at 00:48, malc wrote:
> > > 
> > > > On Wed, 8 Sep 2010, Andreas F?rber wrote:
> > > > > What I don't intend to investigate any time soon is
> > > > > the chatty CoreAudio code.
> > > > 
> > > > What about it?
> > > 
> > > It throws about 100 warnings when compiled on 10.6 :).
> > 
> > Oh, i thought it was a runtime issue, aslo and given that i don't have
> > 10.6...
> 
> For reference here's v10.5/ppc64 audio output (ppc below):
> 

[..snip..]

Thanks. Though most of those warnings are either bogus or require access
to the 10.5/6 system to fix.

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 1dca65e..e5165fb 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -14,7 +14,7 @@  x86                Fabrice Bellard
 ARM                Paul Brook
 SPARC              Blue Swirl
 MIPS               ?
-PowerPC            ?
+PowerPC            Alexander Graf
 M68K               Paul Brook
 SH4                ?
 CRIS               Edgar E. Iglesias
@@ -48,9 +48,9 @@  MIPS
   mips_mipssim.c          ?
 PowerPC
   ppc_prep.c              ?
-  ppc_oldworld.c          Fabrice Bellard
-  ppc_chrp.c              Fabrice Bellard
-  ppc405_boards.c         ?
+  ppc_oldworld.c          Alexander Graf
+  ppc_newworld.c          Alexander Graf
+  ppc405_boards.c         Alexander Graf
 M86K
   mcf5208.c               Paul Brook
   an5206.c                Paul Brook