> + switch (udma_type) { > + case ATA_UDMA4: > + UT = T / 2; break; > + case ATA_UDMA5: > + UT = T / 3; break; > + case ATA_UDMA6: > + UT = T / 4; break; > + } UT = T (udma_type - ATA_UDMA3) ? Looks good -- 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