On Monday 22 June 2009, you wrote: > From: Bartlomiej Zolnierkiewicz <bzolnier@xxxxxxxxx> > Subject: [PATCH] ide: fix handling of unexpected IRQs vs request_irq() > > Add ide_host_enable_irqs() helper and use it in ide_host_register() > before registering ports. Then remove no longer needed IRQ unmasking > from in init_irq(). > > This should fix the problem with "screaming" shared IRQ on the first > port (after request_irq() call while we have the unexpected IRQ pending > on the second port) which was uncovered by my rework of the serialized > interfaces support. Thanks Bart. This does solve the "nobody cared" problem. Tested-by: Frans Pop <elendil@xxxxxxxxx> I also tested it without David's initial patch (i.e. *with* IDE_HFLAG_SERIALIZE in host-flags) and that seems to work fine too: ide0 at 0x1fe02c00000-0x1fe02c00007,0x1fe02c0000a on irq 14 (serialized) ide1 at 0x1fe02c00010-0x1fe02c00017,0x1fe02c0001a on irq 14 (serialized) No idea whether serialized is to be preferred or not. Guess that's David's call now. I do still get the "bad DMA info in identify block" error for the CD drive, so that's still a regression relative to 2.6.26: hdd: host max PIO5 wanted PIO255(auto-tune) selected PIO4 -hdd: MWDMA2 mode selected +hdd: bad DMA info in identify block +hdd: host max PIO5 wanted PIO255(auto-tune) selected PIO4 Cheers, FJP -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
