> > pci_read_config_byte(pdev, 0x5A, &ata66); > > - if (ata66 & (1 << ap->port_no)) > > + if (ata66 & (2 >> ap->port_no)) > > > > HPT36x are single channel per function, so the shift can be removed. In the case of two channels are you sure the bits appear as bit 2 in both cases ? 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