Hello, Timur Tabi wrote: >> ahci: drop intx manipulation on msi enable >> >> There's no need to turn off intx explicitly on msi enable. This is >> automatically handled by pci. Drop it. >> >> This might be needed on machines if the BIOS turns intx off during >> boot. However, there's no evidence of such behavior for ahci and >> the only such case seems to be ICH5 PATA according to ata_piix. >> Also, given the way ahci operates, it's highly unlikely BIOS ever >> disables IRQ for the controller. However, as this change has slight >> possibility of introducing failure, please schedule it for #upstream. > > breaks SATA support on a Freescale MPC8610 HPCD (PowerPC) reference > board. When I apply the patch, I get this result during boot: ... > This board uses a ULI M1575 for SATA support. > > Unfortunately, I don't know enough about SATA or PCI interrupts to > understand why you made the change or why it breaks on my board. cc'ing Jeff. Hmmm... that means intx isn't set by default. I'm not sure what is the right thing to do here. I think it's something which should be handled by the PCI layer. Oh well, maybe we should just revert the change and keep setting intx? Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
