Re: [PATCH] atkbd: restore autorepeat rate after resume

[Vojtech Pavlik <vojtech@suse.cz>]

On Mon, Jun 26, 2006 at 10:58:46AM -0400, Dmitry Torokhov wrote:
> On 6/26/06, Vojtech Pavlik <vojtech@suse.cz> wrote:
> >On Mon, Jun 26, 2006 at 10:35:44AM -0400, Dmitry Torokhov wrote:
> >> On 6/26/06, Alan Stern <stern@rowland.harvard.edu> wrote:
> >> >From: Linus Torvalds <torvalds@osdl.org>
> >> >
> >> >This patch (as728) makes the AT keyboard driver store the current
> >> >autorepeat rate so that it can be restored properly following a
> >> >suspend/resume cycle.
> >> >
> >>
> >> Alan,
> >>
> >> I think it should be a per-device, not global parameter. Anyway, I'll
> >> adjust adn apply, thank you.
> >
> >You can't make it per-device when there is no device when the keyboard
> >isn't plugged in. ;)
> 
> It there is no keyboard then you could not change repeat rate before
> suspending and we don't have anyhting to restore ;)
 
What the patch is trying to achieve is that you have the keyboard, set
the rate, unplug the keyboard, replug the keyboard, get the original
setting.

In the middle of the process, you have no device to attach the
information to. That's why the patch uses a global variable.

-- 
Vojtech Pavlik
Director SuSE Labs