serial: 8250_dw: Use devm_add_action_or_reset()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Mon, 9 May 2022 17:21:28 +0000 (20:21 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 19 May 2022 16:29:02 +0000 (18:29 +0200)
Slightly simplify ->probe() and drop a few goto labels by using
devm_add_action_or_reset() for clock and reset cleanup.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20220509172129.37770-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/8250/8250_dw.c

index 7934e46..e7ef618 100644 (file)
@@ -484,6 +484,16 @@ static void dw8250_quirks(struct uart_port *p, struct dw8250_data *data)
        }
 }
 
+static void dw8250_clk_disable_unprepare(void *data)
+{
+       clk_disable_unprepare(data);
+}
+
+static void dw8250_reset_control_assert(void *data)
+{
+       reset_control_assert(data);
+}
+
 static int dw8250_probe(struct platform_device *pdev)
 {
        struct uart_8250_port uart = {}, *up = &uart;
@@ -585,35 +595,43 @@ static int dw8250_probe(struct platform_device *pdev)
        if (err)
                dev_warn(dev, "could not enable optional baudclk: %d\n", err);
 
+       err = devm_add_action_or_reset(dev, dw8250_clk_disable_unprepare, data->clk);
+       if (err)
+               return err;
+
        if (data->clk)
                p->uartclk = clk_get_rate(data->clk);
 
        /* If no clock rate is defined, fail. */
        if (!p->uartclk) {
                dev_err(dev, "clock rate not defined\n");
-               err = -EINVAL;
-               goto err_clk;
+               return -EINVAL;
        }
 
        data->pclk = devm_clk_get_optional(dev, "apb_pclk");
-       if (IS_ERR(data->pclk)) {
-               err = PTR_ERR(data->pclk);
-               goto err_clk;
-       }
+       if (IS_ERR(data->pclk))
+               return PTR_ERR(data->pclk);
 
        err = clk_prepare_enable(data->pclk);
        if (err) {
                dev_err(dev, "could not enable apb_pclk\n");
-               goto err_clk;
+               return err;
        }
 
+       err = devm_add_action_or_reset(dev, dw8250_clk_disable_unprepare, data->pclk);
+       if (err)
+               return err;
+
        data->rst = devm_reset_control_get_optional_exclusive(dev, NULL);
-       if (IS_ERR(data->rst)) {
-               err = PTR_ERR(data->rst);
-               goto err_pclk;
-       }
+       if (IS_ERR(data->rst))
+               return PTR_ERR(data->rst);
+
        reset_control_deassert(data->rst);
 
+       err = devm_add_action_or_reset(dev, dw8250_reset_control_assert, data->rst);
+       if (err)
+               return err;
+
        dw8250_quirks(p, data);
 
        /* If the Busy Functionality is not implemented, don't handle it */
@@ -631,10 +649,8 @@ static int dw8250_probe(struct platform_device *pdev)
        }
 
        data->data.line = serial8250_register_8250_port(up);
-       if (data->data.line < 0) {
-               err = data->data.line;
-               goto err_reset;
-       }
+       if (data->data.line < 0)
+               return data->data.line;
 
        /*
         * Some platforms may provide a reference clock shared between several
@@ -655,17 +671,6 @@ static int dw8250_probe(struct platform_device *pdev)
        pm_runtime_enable(dev);
 
        return 0;
-
-err_reset:
-       reset_control_assert(data->rst);
-
-err_pclk:
-       clk_disable_unprepare(data->pclk);
-
-err_clk:
-       clk_disable_unprepare(data->clk);
-
-       return err;
 }
 
 static int dw8250_remove(struct platform_device *pdev)
@@ -683,12 +688,6 @@ static int dw8250_remove(struct platform_device *pdev)
 
        serial8250_unregister_port(data->data.line);
 
-       reset_control_assert(data->rst);
-
-       clk_disable_unprepare(data->pclk);
-
-       clk_disable_unprepare(data->clk);
-
        pm_runtime_disable(dev);
        pm_runtime_put_noidle(dev);