staging/rdma/hfi1: Turning off LED without checking if stepping is Ax

It prevents the LED from staying on when the QSFP module is
not present.

Reviewed-by: Easwar Hariharan <easwar.hariharan@intel.com>
Signed-off-by: Sebastian Sanchez <sebastian.sanchez@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>

diff --git a/drivers/staging/rdma/hfi1/chip.c b/drivers/staging/rdma/hfi1/chip.c
index 4750e3c2db3e6d7728cacdd4c713b622b22fec1b..1294617701860a429c8d5a4eb25ba003155df251 100644 (file)
--- a/drivers/staging/rdma/hfi1/chip.c
+++ b/drivers/staging/rdma/hfi1/chip.c
@@ -13418,8 +13418,8 @@ static void init_chip(struct hfi1_devdata *dd)
        write_csr(dd, CCE_DC_CTRL, 0);
 
        /* Set the LED off */
-       if (is_ax(dd))
-               setextled(dd, 0);
+       setextled(dd, 0);
+
        /*
         * Clear the QSFP reset.
         * An FLR enforces a 0 on all out pins. The driver does not touch

diff --git a/drivers/staging/rdma/hfi1/pcie.c b/drivers/staging/rdma/hfi1/pcie.c
index 5642d859fc7c228f9640a73a907784b12e747b59..04f2d8a37f363957f517d08ec7f6e6463726630f 100644 (file)
--- a/drivers/staging/rdma/hfi1/pcie.c
+++ b/drivers/staging/rdma/hfi1/pcie.c
@@ -1261,8 +1261,7 @@ retry:
        write_csr(dd, CCE_DC_CTRL, 0);
 
        /* Set the LED off */
-       if (is_ax(dd))
-               setextled(dd, 0);
+       setextled(dd, 0);
 
        /* check for any per-lane errors */
        pci_read_config_dword(dd->pcidev, PCIE_CFG_SPCIE2, &reg32);