zaurus patches -- arm/pxa -- is Eric alive

zaurus patches -- arm/pxa -- is Eric alive

[Pavel Machek]

Hi!

There are some rather critical patches for zaurus C-3000 (spitz)
available... with no response. What is going on?
								Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

zaurus patches -- arm/pxa -- is Eric alive

[Dmitry Eremin-Solenikov]

Hi,

On 7/11/11, Pavel Machek <pavel@ucw.cz> wrote:
> Hi!
>
> There are some rather critical patches for zaurus C-3000 (spitz)
> available... with no response. What is going on?

Eric seems alive. At least he've sent pull requests few days ago.
Maybe you can point to those patches, that you mention as critical?

-- 
With best wishes
Dmitry

zaurus patches -- arm/pxa -- is Eric alive

[Cyril Hrubis]

Hi!
> > There are some rather critical patches for zaurus C-3000 (spitz)
> > available... with no response. What is going on?
> 
> Eric seems alive. At least he've sent pull requests few days ago.
> Maybe you can point to those patches, that you mention as critical?

Perhaps the most critical one is:

[PATCH] MAX1111: Fix race condition causing NULL pointer exception

http://ns1.yosemitephotos.net/lists/arm-kernel/msg125356.html

Which fixes memory corruption that caused random oops.

And then there is a bunch of fixes for charging and power management from
Pavel Machek:

Disable CF during suspend

https://lkml.org/lkml/2011/5/21/46

and spitz pm

https://lkml.org/lkml/2011/5/21/72
https://lkml.org/lkml/2011/5/21/73
https://lkml.org/lkml/2011/5/21/76

-- 
metan

zaurus patches -- arm/pxa -- is Eric alive

[Wolfram Sang]

On Mon, Jul 11, 2011 at 04:03:03PM +0200, Cyril Hrubis wrote:
> Hi!
> > > There are some rather critical patches for zaurus C-3000 (spitz)
> > > available... with no response. What is going on?
> > 
> > Eric seems alive. At least he've sent pull requests few days ago.
> > Maybe you can point to those patches, that you mention as critical?
> 
> Perhaps the most critical one is:
> 
> [PATCH] MAX1111: Fix race condition causing NULL pointer exception
> 
> http://ns1.yosemitephotos.net/lists/arm-kernel/msg125356.html

Ahem, this one was opposed to. The right one seems to be:

http://ns1.yosemitephotos.net/lists/arm-kernel/msg125592.html

Note that it is a hwmon-driver and neither the apropriate list nor
maintainers have been CCed. Eric cannot take this patch. Check
MAINTAINERAS (or use scripts/get_maintainer.pl) to find the right people
and this will increase chances to get your patch upstream a lot.

Regards,

   Wolfram

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20110711/36212a6c/attachment.sig>

zaurus patches -- arm/pxa -- is Eric alive

[Cyril Hrubis]

Hi!
> > > > There are some rather critical patches for zaurus C-3000 (spitz)
> > > > available... with no response. What is going on?
> > > 
> > > Eric seems alive. At least he've sent pull requests few days ago.
> > > Maybe you can point to those patches, that you mention as critical?
> > 
> > Perhaps the most critical one is:
> > 
> > [PATCH] MAX1111: Fix race condition causing NULL pointer exception
> > 
> > http://ns1.yosemitephotos.net/lists/arm-kernel/msg125356.html
> 
> Ahem, this one was opposed to. The right one seems to be:
> 
> http://ns1.yosemitephotos.net/lists/arm-kernel/msg125592.html
> 
> Note that it is a hwmon-driver and neither the apropriate list nor
> maintainers have been CCed. Eric cannot take this patch. Check
> MAINTAINERAS (or use scripts/get_maintainer.pl) to find the right people
> and this will increase chances to get your patch upstream a lot.

Right, CCing patch author.

-- 
metan

zaurus patches -- arm/pxa -- is Eric alive

[Pavel Herrmann]

On Monday 11 of July 2011 16:22:10 Cyril Hrubis wrote:
> > Note that it is a hwmon-driver and neither the apropriate list nor
> > maintainers have been CCed. Eric cannot take this patch. Check
> > MAINTAINERAS (or use scripts/get_maintainer.pl) to find the right people
> > and this will increase chances to get your patch upstream a lot.
> 
> Right, CCing patch author.

Yeah, what was I thinking! just because you're listed as author of something 
doesn't give you any right to accept and push patches on it.
but hey, what is the point of being new to something if you won't screw 
everything up?

on a more serious note, resent to output of get_maintainers

zaurus patches -- arm/pxa -- is Eric alive

[Eric Miao]

On Mon, Jul 11, 2011 at 10:03 PM, Cyril Hrubis <metan@ucw.cz> wrote:
> Hi!
>> > There are some rather critical patches for zaurus C-3000 (spitz)
>> > available... with no response. What is going on?
>>
>> Eric seems alive. At least he've sent pull requests few days ago.
>> Maybe you can point to those patches, that you mention as critical?
>
> Perhaps the most critical one is:
>
> [PATCH] MAX1111: Fix race condition causing NULL pointer exception
>
> http://ns1.yosemitephotos.net/lists/arm-kernel/msg125356.html
>
> Which fixes memory corruption that caused random oops.
>
> And then there is a bunch of fixes for charging and power management from
> Pavel Machek:

Guys, I'm still alive, at least still breathing. Just too many mails
and I was lost.

>
> Disable CF during suspend
>
> https://lkml.org/lkml/2011/5/21/46

The fix is trivial, think I can take it. A better approach will be to have a
generic CF power control function and invoke it in pcmcia driver when
suspending.

>
> and spitz pm
>
> https://lkml.org/lkml/2011/5/21/72
> https://lkml.org/lkml/2011/5/21/73
> https://lkml.org/lkml/2011/5/21/76

Looks good to me, yet I need the last one Acked by power supply people.

>
> --
> metan
>

zaurus patches -- arm/pxa -- is Eric alive

[Pavel Machek]

Hi!

> > Perhaps the most critical one is:
> >
> > [PATCH] MAX1111: Fix race condition causing NULL pointer exception
> >
> > http://ns1.yosemitephotos.net/lists/arm-kernel/msg125356.html

Yep, that one (or equivalent) is most critical.

> > Which fixes memory corruption that caused random oops.
> >
> > And then there is a bunch of fixes for charging and power management from
> > Pavel Machek:
> 
> Guys, I'm still alive, at least still breathing. Just too many mails
> and I was lost.

Good :-).

> > Disable CF during suspend
> >
> > https://lkml.org/lkml/2011/5/21/46
> 
> The fix is trivial, think I can take it. A better approach will be to have a
> generic CF power control function and invoke it in pcmcia driver when
> suspending.

Please take it. It is also regression from 2.6.15-or-so.

> > and spitz pm
> >
> > https://lkml.org/lkml/2011/5/21/72
> > https://lkml.org/lkml/2011/5/21/73
> > https://lkml.org/lkml/2011/5/21/76
> 
> Looks good to me, yet I need the last one Acked by power supply people.

Can you take the first two? They are trivial preparations... And I'll
get the ACKs...
								Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

zaurus patches -- arm/pxa -- is Eric alive

[Pavel Machek]

Hi!

> > > There are some rather critical patches for zaurus C-3000 (spitz)
> > > available... with no response. What is going on?
> > 
> > Eric seems alive. At least he've sent pull requests few days ago.
> > Maybe you can point to those patches, that you mention as critical?
> 
> Perhaps the most critical one is:
> 
> [PATCH] MAX1111: Fix race condition causing NULL pointer exception
> 
> http://ns1.yosemitephotos.net/lists/arm-kernel/msg125356.html

> Which fixes memory corruption that caused random oops.

This one is in 3.1-rc3. Good.

> And then there is a bunch of fixes for charging and power management from
> Pavel Machek:
> 
> Disable CF during suspend
> 
> https://lkml.org/lkml/2011/5/21/46

This one is not. Could we get it pushed?
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html