2017-01-16 0:12 GMT+01:00 Tejun Heo <tj@xxxxxxxxxx>: > On Fri, Jan 13, 2017 at 01:38:00PM +0100, Bartosz Golaszewski wrote: >> +static int ahci_da850_softreset(struct ata_link *link, >> + unsigned int *class, unsigned long deadline) >> +{ >> + int pmp, ret; >> + >> + pmp = sata_srst_pmp(link); >> + >> + ret = ahci_do_softreset(link, class, pmp, deadline, ahci_check_ready); >> + if (pmp && ret == -EBUSY) >> + return ahci_do_softreset(link, class, 0, >> + deadline, ahci_check_ready); >> + >> + return ret; >> +} > > Please add some comments explaining what's going on. Sure, I'll add some explanation in v2. Thanks, Bartosz Golaszewski -- 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