> ata_port_desc(host->ports[i], "irq %d", pdev->irq);
> + }
> } else if (legacy_mode) {
> if (!ata_port_is_dummy(host->ports[0])) {
> rc = devm_request_irq(dev, ATA_PRIMARY_IRQ(pdev),
This bit looks fine - in fact it's a nice clean up anyway
> + }
> + if (!(reg & ENPORT_SECONDARY)) {
> + dev_printk(KERN_NOTICE, &pdev->dev, "Secondary port is disabled\n");
> + ppi[1] = &ata_dummy_port_info;
> + }
And you just broke split bridge setups. Also need to check if the bits
are valid for this chipset in native mode officialy - Sergei probably
knows the answer to that.
We can detect the Mobility electronics split bridges at least (and I
suspect they are the only 'common' CMD64x hot plug device indeed possibly
the only one) because the parent bridge of the CMD64x will have a PCI
vendor id of 0x14f2 and a device id 0x0001, 0x0002, or 0x0120.
Alan
--
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
