Am Donnerstag, 8. Januar 2009 schrieb Milton Miller:
> Only call free_irq if we marked the request_irq has having succeeded
> instead of whenever the the sub-driver identified the interrupt to use.
>
> Signed-off-by: Milton Miller <milt...@bga.com>
> ---
> Appears to be a bugfix but might use a bit of time in -next.
> This code was created in 2.6.28 to allow s390 to build without adding
ifdefs.
> Many hvc-console sub-drivers have a single channel and are not modules.
>
> Index: work.git/drivers/char/hvc_irq.c
> ===================================================================
> --- work.git.orig/drivers/char/hvc_irq.c 2009-01-08 04:07:28.000000000
> -0600
> +++ work.git/drivers/char/hvc_irq.c 2009-01-08 04:07:47.000000000 -0600
> @@ -37,7 +37,7 @@ int notifier_add_irq(struct hvc_struct *
>
> void notifier_del_irq(struct hvc_struct *hp, int irq)
> {
> - if (!irq)
> + if (!hp->irq_requested)
> return;
> free_irq(irq, hp);
> hp->irq_requested = 0;
>
Looks sane.
Acked-by: Christian Borntraeger <borntrae...@de.ibm.com>
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
