Hello, Tejun. On Mon, Jun 20, 2011 at 4:19 PM, Tejun Heo <tj@xxxxxxxxxx> wrote: >> +static int ahci_pmp_check_ready(struct ata_link *link) > > It would be better if the name reflects that it's not for specific > workaround. Do you mean it would be better to name it for specific workaround? How about ahci_bad_pmp_check_ready? Because this function is to avoid checking TFDATA if BAD PMP is found. Thanks. Yuan-Hsin -- 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