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. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html