On Thu, 2009-01-08 at 14:29 +0530, Kashyap, Desai wrote: > - > - if (reset_alt_ioc_active && ioc->alt_ioc) { > - /* (re)Enable alt-IOC! (reply interrupt) */ > - dinitprintk(ioc, printk(MYIOC_s_INFO_FMT "alt_ioc reply irq re-enabled\n", > - ioc->alt_ioc->name)); > - CHIPREG_WRITE32(&ioc->alt_ioc->chip->IntMask, MPI_HIM_DIM); > - ioc->alt_ioc->active = 1; > + if (rc == 0) { /* alt ioc */ > + if (reset_alt_ioc_active && ioc->alt_ioc) { > + /* (re)Enable alt-IOC! (reply interrupt) */ > + dinitprintk(ioc, printk(MYIOC_s_DEBUG_FMT "alt-ioc > + reply irq re-enabled\n", > + ioc->alt_ioc->name)); This isn't legal C: you can't split a string constant without a backslash: drivers/message/fusion/mptbase.c:2295:46: warning: missing terminating " character drivers/message/fusion/mptbase.c:2296:27: warning: missing terminating " character drivers/message/fusion/mptbase.c: In function 'mpt_do_ioc_recovery': drivers/message/fusion/mptbase.c:2295: error: missing terminating " character drivers/message/fusion/mptbase.c:2295: error: expected ')' before 'reply' drivers/message/fusion/mptbase.c:2295: error: stray '\' in program drivers/message/fusion/mptbase.c:2295: error: missing terminating " character drivers/message/fusion/mptbase.c:2295: warning: too few arguments for format James -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html