Aren/NetBSD
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

In icu_intr_establish(), don't panic if the specified irq

Browse Source 
is already in use and just return NULL instead.

Such situation could happen with devices which have
unconfigured PCI interrupt pins (because firmware
doesn't configure multiple function devices) and
each driver could handle the return value properly.
This commit is contained in:
tsutsui 2007-12-22 14:42:21 +00:00
parent 6698b79073
commit 9e4e6d48cb
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
sys/arch/cobalt/cobalt/machdep.c
View File

@ -1,4 +1,4 @@
/* $NetBSD: machdep.c,v 1.90 2007/12/03 15:33:27 ad Exp $ */
/* $NetBSD: machdep.c,v 1.91 2007/12/22 14:42:21 tsutsui Exp $ */
/*
* Copyright (c) 2006 Izumi Tsutsui.
@ -53,7 +53,7 @@
*/
#include <sys/cdefs.h>
__KERNEL_RCSID(0, "$NetBSD: machdep.c,v 1.90 2007/12/03 15:33:27 ad Exp $");
__KERNEL_RCSID(0, "$NetBSD: machdep.c,v 1.91 2007/12/22 14:42:21 tsutsui Exp $");
#include "opt_ddb.h"
#include "opt_kgdb.h"
@ -531,8 +531,10 @@ icu_intr_establish(int irq, int type, int ipl, int (*func)(void *),
struct cobalt_intrhand *ih;
ih = &icu_intrtab[irq];
if (ih->ih_func != NULL)
panic("icu_intr_establish(): irq %d is already in use", irq);
if (ih->ih_func != NULL) {
printf("%s: irq %d is already in use\n", __func__, irq);
return NULL;
}
ih->ih_cookie_type = COBALT_COOKIE_TYPE_ICU;
ih->ih_func = func;