> BTW, I've looked at the clock turnaround code both in drivers/ide/hpt366.c > and drivers/ata/pata_hpt3x2n.c and compared it to the HighPoint driver and I'm > dazed and confused (again?): contrary to what HighPoint does (I'm not sure Thats how I usually feel after looking at highpoints driver code. > ATA_TFLAG_WRITE indicates a write transfer (host-to-device) while > hpt3x2n_use_dpll() contrarywise expects the read transfer indication. Comments? That looks like a bug. The code seems to get the DPLL choice right but then the caller as you say is incorrect. Will fix - 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
