IB/hfi1: Change EPROM handling to use resource reservation
authorDean Luick <dean.luick@intel.com>
Sat, 5 Mar 2016 16:49:55 +0000 (08:49 -0800)
committerDoug Ledford <dledford@redhat.com>
Thu, 17 Mar 2016 19:55:14 +0000 (15:55 -0400)
Change EPROM handling to use the new ASIC resource reservation system.

Reviewed-by: Mitko Haralanov <mitko.haralanov@intel.com>
Reviewed-by: Easwar Hariharan <easwar.hariharan@intel.com>
Signed-off-by: Dean Luick <dean.luick@intel.com>
Signed-off-by: Jubin John <jubin.john@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/staging/rdma/hfi1/eprom.c

index 87114af3a38e4644b1151bf7f267b072f8f883fc..bd8771570f81ac569b54be9496247720255fb59d 100644 (file)
 #define EPROM_WP_N BIT_ULL(14) /* EPROM write line */
 
 /*
- * Use the EP mutex to guard against other callers from within the driver.
+ * How long to wait for the EPROM to become available, in ms.
+ * The spec 32 Mb EPROM takes around 40s to erase then write.
+ * Double it for safety.
  */
-static DEFINE_MUTEX(eprom_mutex);
+#define EPROM_TIMEOUT 80000 /* ms */
 
 /*
  * Turn on external enable line that allows writing on the flash.
@@ -371,14 +373,9 @@ int handle_eprom_command(struct file *fp, const struct hfi1_cmd *cmd)
        if (!dd->eprom_available)
                return -EOPNOTSUPP;
 
-       /* lock against other callers touching the ASIC block */
-       mutex_lock(&eprom_mutex);
-
-       /* lock against the other HFI on another OS */
-       ret = acquire_hw_mutex(dd);
+       ret = acquire_chip_resource(dd, CR_EPROM, EPROM_TIMEOUT);
        if (ret) {
-               dd_dev_err(dd,
-                          "%s: unable to acquire hw mutex, no EPROM support\n",
+               dd_dev_err(dd, "%s: unable to acquire EPROM resource\n",
                           __func__);
                goto done_asic;
        }
@@ -428,9 +425,8 @@ int handle_eprom_command(struct file *fp, const struct hfi1_cmd *cmd)
                break;
        }
 
-       release_hw_mutex(dd);
+       release_chip_resource(dd, CR_EPROM);
 done_asic:
-       mutex_unlock(&eprom_mutex);
        return ret;
 }
 
@@ -441,23 +437,18 @@ int eprom_init(struct hfi1_devdata *dd)
 {
        int ret = 0;
 
-       /* only the discrete chip has an EPROM, nothing to do */
+       /* only the discrete chip has an EPROM */
        if (dd->pcidev->device != PCI_DEVICE_ID_INTEL0)
                return 0;
 
-       /* lock against other callers */
-       mutex_lock(&eprom_mutex);
-
        /*
-        * Lock against the other HFI on another OS - the mutex above
-        * would have caught anything in this driver.  It is OK if
-        * both OSes reset the EPROM - as long as they don't do it at
-        * the same time.
+        * It is OK if both HFIs reset the EPROM as long as they don't
+        * do it at the same time.
         */
-       ret = acquire_hw_mutex(dd);
+       ret = acquire_chip_resource(dd, CR_EPROM, EPROM_TIMEOUT);
        if (ret) {
                dd_dev_err(dd,
-                          "%s: unable to acquire hw mutex, no EPROM support\n",
+                          "%s: unable to acquire EPROM resource, no EPROM support\n",
                           __func__);
                goto done_asic;
        }
@@ -474,8 +465,7 @@ int eprom_init(struct hfi1_devdata *dd)
        write_csr(dd, ASIC_EEP_ADDR_CMD, CMD_RELEASE_POWERDOWN_NOID);
 
        dd->eprom_available = true;
-       release_hw_mutex(dd);
+       release_chip_resource(dd, CR_EPROM);
 done_asic:
-       mutex_unlock(&eprom_mutex);
        return ret;
 }