Re: BUG in irq.c?

Andrew E. Mileski (aem@netcom.ca)
Sun, 16 Feb 1997 00:01:56 -0500 (EST)


> linux-2.1.26/arch/i386/kernel/irq.c
> 221: static struct irqaction *irq_action[16] = {
> 480: void free_irq(unsigned int irq, void *dev_id)
> 497: if (!irq[irq_action]) {
> ^^^^^^^^^^^^^^^
> Playing dumb - Shouldn't this be "irq_action[irq]"?
>
> It is the same thing. The joys C pointer calculations...

Okay, I can see it now...but IMHO it really should be changed
because it is not intuitive (at least for me).

--
Andrew E. Mileski   mailto:aem@netcom.ca
Linux Plug-and-Play Kernel Project http://www.redhat.com/linux-info/pnp/
XFree86 Matrox Team http://www.bf.rmit.edu.au/~ajv/xf86-matrox.html