linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [2.4.23: pc_keyb.c] request_irq() without free_irq().
@ 2003-12-10 14:32 Guennadi Liakhovetski
  2003-12-10 16:52 ` Randy.Dunlap
  0 siblings, 1 reply; 3+ messages in thread
From: Guennadi Liakhovetski @ 2003-12-10 14:32 UTC (permalink / raw)
  To: linux-kernel

Hello

Looks like the the counterpart for kbd_request_irq() is missing. Am I
right?

Thanks
Guennadi
---------------------------------
Guennadi Liakhovetski, Ph.D.
DSA Daten- und Systemtechnik GmbH
Pascalstr. 28
D-52076 Aachen
Germany


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [2.4.23: pc_keyb.c] request_irq() without free_irq().
  2003-12-10 14:32 [2.4.23: pc_keyb.c] request_irq() without free_irq() Guennadi Liakhovetski
@ 2003-12-10 16:52 ` Randy.Dunlap
  2003-12-10 17:08   ` Guennadi Liakhovetski
  0 siblings, 1 reply; 3+ messages in thread
From: Randy.Dunlap @ 2003-12-10 16:52 UTC (permalink / raw)
  To: Guennadi Liakhovetski; +Cc: linux-kernel

On Wed, 10 Dec 2003 15:32:22 +0100 (CET) Guennadi Liakhovetski <gl@dsa-ac.de> wrote:

| Hello
| 
| Looks like the the counterpart for kbd_request_irq() is missing. Am I
| right?

Did you find the counterpart for kbd_request_region()?

In 2.4.x, the pc keyboard driver option is Y/N.  It can't be a module.
If it's built into the kernel, it just hangs around until reboot.
It doesn't need to release resources when the kernel is shutting down.

--
~Randy
MOTD:  Always include version info.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [2.4.23: pc_keyb.c] request_irq() without free_irq().
  2003-12-10 16:52 ` Randy.Dunlap
@ 2003-12-10 17:08   ` Guennadi Liakhovetski
  0 siblings, 0 replies; 3+ messages in thread
From: Guennadi Liakhovetski @ 2003-12-10 17:08 UTC (permalink / raw)
  To: Randy.Dunlap; +Cc: linux-kernel

On Wed, 10 Dec 2003, Randy.Dunlap wrote:

> On Wed, 10 Dec 2003 15:32:22 +0100 (CET) Guennadi Liakhovetski <gl@dsa-ac.de> wrote:
>
> | Hello
> |
> | Looks like the the counterpart for kbd_request_irq() is missing. Am I
> | right?
>
> Did you find the counterpart for kbd_request_region()?
>
> In 2.4.x, the pc keyboard driver option is Y/N.  It can't be a module.
> If it's built into the kernel, it just hangs around until reboot.
> It doesn't need to release resources when the kernel is shutting down.

Yep, I noticed already, that there's no release method at all:-)) I'm just
adapting the code to an ARM platform (actually, the patch already exists
and already works), and, no, we don't really need to unload it, but, just
thought, wasn't quite clean... But, well, it's probably not going to
change in 2.4. Not that it was very needed:-)

Thanks
Guennadi
---------------------------------
Guennadi Liakhovetski, Ph.D.
DSA Daten- und Systemtechnik GmbH
Pascalstr. 28
D-52076 Aachen
Germany


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2003-12-10 17:08 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-12-10 14:32 [2.4.23: pc_keyb.c] request_irq() without free_irq() Guennadi Liakhovetski
2003-12-10 16:52 ` Randy.Dunlap
2003-12-10 17:08   ` Guennadi Liakhovetski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).