2017-01-16 0:12 GMT+01:00 Tejun Heo [off-list ref]:

On Fri, Jan 13, 2017 at 01:38:00PM +0100, Bartosz Golaszewski wrote:

quoted

+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