Re: blink driver power saving

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Stephen Hemminger <shemminger@...>
Cc: Linus Torvalds <torvalds@...>, Andrew Morton <akpm@...>, <linux-kernel@...>
Date: Sunday, July 1, 2007 - 5:26 pm

On Sunday 01 July 2007 18:50:35 Stephen Hemminger wrote:

It's really a debugging tool where you normally don't care about
details like this. I wrote it to get visual feedback during kdump,
but it is nothing you should ever run during normal operation.

But I don't get how your patch is supposed to work. The blink driver
is not supposed to blink after panic -- panic does that anyways --
but always. So hooking it into the panic notifier chain which
is only called on panic makes about zero sense. 

If it's really needed to fix the wakeup issue the interface to
the keyboard blinking would need to be changed.

-Andi
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
blink driver power saving, Stephen Hemminger, (Sun Jul 1, 12:50 pm)
Re: blink driver power saving, Andi Kleen, (Sun Jul 1, 5:26 pm)
Re: blink driver power saving, Linus Torvalds, (Sun Jul 1, 2:07 pm)
Re: blink driver power saving, Andi Kleen, (Sun Jul 1, 5:29 pm)
Re: blink driver power saving, Linus Torvalds, (Sun Jul 1, 6:14 pm)
Re: blink driver power saving, Andi Kleen, (Sun Jul 1, 7:59 pm)
Re: blink driver power saving, Stephen Hemminger, (Mon Jul 2, 1:50 pm)
Re: blink driver power saving, Andi Kleen, (Mon Jul 2, 3:08 pm)
Re: blink driver power saving, Pavel Machek, (Mon Jul 2, 7:18 pm)
Re: blink driver power saving, Dmitry Torokhov, (Mon Jul 2, 3:03 pm)
Re: blink driver power saving, Linus Torvalds, (Mon Jul 2, 11:51 am)
Re: blink driver power saving, Alan Cox, (Mon Jul 2, 12:59 pm)