spi: bcm63xx: Don't check clk_free
authorSean Anderson <seanga2@gmail.com>
Sat, 15 Jan 2022 22:25:02 +0000 (17:25 -0500)
committerSean Anderson <seanga2@gmail.com>
Wed, 30 Mar 2022 17:02:55 +0000 (13:02 -0400)
This function always succeeds, so don't check its return value.

Signed-off-by: Sean Anderson <seanga2@gmail.com>
Link: https://lore.kernel.org/r/20220115222504.617013-6-seanga2@gmail.com
drivers/spi/bcm63xx_hsspi.c
drivers/spi/bcm63xx_spi.c

index 85108df..47002f8 100644 (file)
@@ -355,9 +355,7 @@ static int bcm63xx_hsspi_probe(struct udevice *dev)
        if (ret < 0 && ret != -ENOSYS)
                return ret;
 
-       ret = clk_free(&clk);
-       if (ret < 0 && ret != -ENOSYS)
-               return ret;
+       clk_free(&clk);
 
        /* get clock rate */
        ret = clk_get_by_name(dev, "pll", &clk);
@@ -366,9 +364,7 @@ static int bcm63xx_hsspi_probe(struct udevice *dev)
 
        priv->clk_rate = clk_get_rate(&clk);
 
-       ret = clk_free(&clk);
-       if (ret < 0 && ret != -ENOSYS)
-               return ret;
+       clk_free(&clk);
 
        /* perform reset */
        ret = reset_get_by_index(dev, 0, &rst_ctl);
index dd5e62b..0600d56 100644 (file)
@@ -391,9 +391,7 @@ static int bcm63xx_spi_probe(struct udevice *dev)
        if (ret < 0)
                return ret;
 
-       ret = clk_free(&clk);
-       if (ret < 0)
-               return ret;
+       clk_free(&clk);
 
        /* perform reset */
        ret = reset_get_by_index(dev, 0, &rst_ctl);