fsi: master: Remove link enable read-back
authorEddie James <eajames@linux.ibm.com>
Mon, 6 Apr 2020 19:19:36 +0000 (14:19 -0500)
committerJoel Stanley <joel@jms.id.au>
Thu, 10 Sep 2020 02:52:49 +0000 (12:22 +0930)
Both the Aspeed and hub masters read back the link enable register
after enabling the link, but this is unnecessary, so remove it.

Signed-off-by: Eddie James <eajames@linux.ibm.com>
Signed-off-by: Joel Stanley <joel@jms.id.au>
drivers/fsi/fsi-master-aspeed.c
drivers/fsi/fsi-master-hub.c

index 6152cfe6d18e4dbecc8bc6c7ba8deb45e180c23d..2b8ca72baeb559a47cfde556d70a2619a2792180 100644 (file)
@@ -306,7 +306,7 @@ static int aspeed_master_link_enable(struct fsi_master *master, int link,
 {
        struct fsi_master_aspeed *aspeed = to_fsi_master_aspeed(master);
        int idx, bit, ret;
-       __be32 reg, result;
+       __be32 reg;
 
        idx = link / 32;
        bit = link % 32;
@@ -323,15 +323,6 @@ static int aspeed_master_link_enable(struct fsi_master *master, int link,
 
        mdelay(FSI_LINK_ENABLE_SETUP_TIME);
 
-       ret = opb_readl(aspeed, ctrl_base + FSI_MENP0 + (4 * idx), &result);
-       if (ret)
-               return ret;
-
-       if (result != reg) {
-               dev_err(aspeed->dev, "%s failed: %08x\n", __func__, result);
-               return -EIO;
-       }
-
        return 0;
 }
 
index 1d3cf2da6a1624fc715e982f83fc17deabbad4d8..3caa2da7838ca7aab417a91a077d50593840ffd4 100644 (file)
@@ -95,12 +95,12 @@ static int hub_master_link_enable(struct fsi_master *master, int link,
                                        &reg, 4);
 
        rc = fsi_device_write(hub->upstream, FSI_MSENP0 + (4 * idx), &reg, 4);
+       if (rc)
+               return rc;
 
        mdelay(FSI_LINK_ENABLE_SETUP_TIME);
 
-       fsi_device_read(hub->upstream, FSI_MENP0 + (4 * idx), &reg, 4);
-
-       return rc;
+       return 0;
 }
 
 static void hub_master_release(struct device *dev)