This function always succeeds, so don't check its return value.

Signed-off-by: Sean Anderson <sean...@gmail.com>
---

 drivers/net/bcm6348-eth.c | 6 +-----
 drivers/net/bcm6368-eth.c | 6 +-----
 2 files changed, 2 insertions(+), 10 deletions(-)

diff --git a/drivers/net/bcm6348-eth.c b/drivers/net/bcm6348-eth.c
index aad7b61213..06e0dd74a5 100644
--- a/drivers/net/bcm6348-eth.c
+++ b/drivers/net/bcm6348-eth.c
@@ -461,11 +461,7 @@ static int bcm6348_eth_probe(struct udevice *dev)
                        return ret;
                }
 
-               ret = clk_free(&clk);
-               if (ret < 0) {
-                       pr_err("%s: error freeing clock %d\n", __func__, i);
-                       return ret;
-               }
+               clk_free(&clk);
        }
 
        /* try to perform resets */
diff --git a/drivers/net/bcm6368-eth.c b/drivers/net/bcm6368-eth.c
index 29abe7fc96..c2a8b9f057 100644
--- a/drivers/net/bcm6368-eth.c
+++ b/drivers/net/bcm6368-eth.c
@@ -546,11 +546,7 @@ static int bcm6368_eth_probe(struct udevice *dev)
                        return ret;
                }
 
-               ret = clk_free(&clk);
-               if (ret < 0) {
-                       pr_err("%s: error freeing clock %d\n", __func__, i);
-                       return ret;
-               }
+               clk_free(&clk);
        }
 
        /* try to perform resets */
-- 
2.34.1

Reply via email to