* zaurus patches -- arm/pxa -- is Eric alive
@ 2011-07-11 14:03 ` Cyril Hrubis
0 siblings, 0 replies; 18+ messages in thread
From: Cyril Hrubis @ 2011-07-11 14:03 UTC (permalink / raw)
To: linux-arm-kernel
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
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: zaurus patches -- arm/pxa -- is Eric alive
2011-07-11 14:03 ` Cyril Hrubis
@ 2011-07-11 14:17 ` Wolfram Sang
-1 siblings, 0 replies; 18+ messages in thread
From: Wolfram Sang @ 2011-07-11 14:17 UTC (permalink / raw)
To: Cyril Hrubis
Cc: Dmitry Eremin-Solenikov, thommycheck, Marek Vasut, Russell King,
arminlitzel, kernel list, utx, lenz, rpurdie, omegamoon,
Pavel Machek, eric.y.miao, Linus Torvalds, zaurus-devel,
linux-arm-kernel
[-- Attachment #1: Type: text/plain, Size: 1117 bytes --]
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/ |
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* zaurus patches -- arm/pxa -- is Eric alive
@ 2011-07-11 14:17 ` Wolfram Sang
0 siblings, 0 replies; 18+ messages in thread
From: Wolfram Sang @ 2011-07-11 14:17 UTC (permalink / raw)
To: linux-arm-kernel
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>
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: zaurus patches -- arm/pxa -- is Eric alive
2011-07-11 14:17 ` Wolfram Sang
@ 2011-07-11 14:22 ` Cyril Hrubis
-1 siblings, 0 replies; 18+ messages in thread
From: Cyril Hrubis @ 2011-07-11 14:22 UTC (permalink / raw)
To: Wolfram Sang
Cc: Dmitry Eremin-Solenikov, thommycheck, Marek Vasut, Russell King,
arminlitzel, kernel list, utx, lenz, rpurdie, omegamoon,
Pavel Machek, eric.y.miao, Linus Torvalds, zaurus-devel,
linux-arm-kernel, Pavel Herrmann
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
^ permalink raw reply [flat|nested] 18+ messages in thread
* zaurus patches -- arm/pxa -- is Eric alive
@ 2011-07-11 14:22 ` Cyril Hrubis
0 siblings, 0 replies; 18+ messages in thread
From: Cyril Hrubis @ 2011-07-11 14:22 UTC (permalink / raw)
To: linux-arm-kernel
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
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: zaurus patches -- arm/pxa -- is Eric alive
2011-07-11 14:22 ` Cyril Hrubis
@ 2011-07-11 18:58 ` Pavel Herrmann
-1 siblings, 0 replies; 18+ messages in thread
From: Pavel Herrmann @ 2011-07-11 18:58 UTC (permalink / raw)
To: Cyril Hrubis
Cc: Wolfram Sang, Dmitry Eremin-Solenikov, thommycheck, Marek Vasut,
Russell King, arminlitzel, kernel list, utx, lenz, rpurdie,
omegamoon, Pavel Machek, eric.y.miao, Linus Torvalds,
zaurus-devel, linux-arm-kernel
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
^ permalink raw reply [flat|nested] 18+ messages in thread
* zaurus patches -- arm/pxa -- is Eric alive
@ 2011-07-11 18:58 ` Pavel Herrmann
0 siblings, 0 replies; 18+ messages in thread
From: Pavel Herrmann @ 2011-07-11 18:58 UTC (permalink / raw)
To: linux-arm-kernel
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
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: zaurus patches -- arm/pxa -- is Eric alive
2011-07-11 14:03 ` Cyril Hrubis
@ 2011-07-11 15:13 ` Eric Miao
-1 siblings, 0 replies; 18+ messages in thread
From: Eric Miao @ 2011-07-11 15:13 UTC (permalink / raw)
To: Cyril Hrubis
Cc: Dmitry Eremin-Solenikov, Pavel Machek, rpurdie, lenz,
kernel list, arminlitzel, thommycheck, linux-arm-kernel,
omegamoon, utx, zaurus-devel, Marek Vasut, Russell King,
Linus Torvalds
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
>
^ permalink raw reply [flat|nested] 18+ messages in thread
* zaurus patches -- arm/pxa -- is Eric alive
@ 2011-07-11 15:13 ` Eric Miao
0 siblings, 0 replies; 18+ messages in thread
From: Eric Miao @ 2011-07-11 15:13 UTC (permalink / raw)
To: linux-arm-kernel
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
>
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: zaurus patches -- arm/pxa -- is Eric alive
2011-07-11 15:13 ` Eric Miao
@ 2011-07-11 21:09 ` Pavel Machek
-1 siblings, 0 replies; 18+ messages in thread
From: Pavel Machek @ 2011-07-11 21:09 UTC (permalink / raw)
To: Eric Miao
Cc: Cyril Hrubis, Dmitry Eremin-Solenikov, rpurdie, lenz,
kernel list, arminlitzel, thommycheck, linux-arm-kernel,
omegamoon, utx, zaurus-devel, Marek Vasut, Russell King,
Linus Torvalds
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
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: zaurus patches -- arm/pxa -- is Eric alive
2011-07-11 14:03 ` Cyril Hrubis
@ 2011-08-28 9:35 ` Pavel Machek
-1 siblings, 0 replies; 18+ messages in thread
From: Pavel Machek @ 2011-08-28 9:35 UTC (permalink / raw)
To: Cyril Hrubis
Cc: Dmitry Eremin-Solenikov, thommycheck, Marek Vasut, Russell King,
arminlitzel, kernel list, utx, lenz, rpurdie, omegamoon,
eric.y.miao, Linus Torvalds, zaurus-devel, linux-arm-kernel
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
^ permalink raw reply [flat|nested] 18+ messages in thread