On 11/1/07, Jeff Garzik <jeff@xxxxxxxxxx> wrote: > + lo = tf->lbam; > + hi = tf->lbam; > + ibyte = (hi << 8) | lo; > + > + lo = result_tf->lbam; > + hi = result_tf->lbam; That doesn't look right. I suspect this was intended: lo = tf->lbam; hi = tf->lbah; Torsten - 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