counter: ti-ecap-capture: Add check for clk_enable()
authorJiasheng Jiang <jiashengjiangcool@gmail.com>
Mon, 4 Nov 2024 19:40:59 +0000 (19:40 +0000)
committerWilliam Breathitt Gray <wbg@kernel.org>
Tue, 5 Nov 2024 00:20:11 +0000 (09:20 +0900)
Add check for the return value of clk_enable() in order to catch the
potential exception.

Fixes: 4e2f42aa00b6 ("counter: ti-ecap-capture: capture driver support for ECAP")
Reviewed-by: Julien Panis <jpanis@baylibre.com>
Signed-off-by: Jiasheng Jiang <jiashengjiangcool@gmail.com>
Link: https://lore.kernel.org/r/20241104194059.47924-1-jiashengjiangcool@gmail.com
Signed-off-by: William Breathitt Gray <wbg@kernel.org>
drivers/counter/ti-ecap-capture.c

index 6754473..b119aee 100644 (file)
@@ -574,8 +574,13 @@ static int ecap_cnt_resume(struct device *dev)
 {
        struct counter_device *counter_dev = dev_get_drvdata(dev);
        struct ecap_cnt_dev *ecap_dev = counter_priv(counter_dev);
+       int ret;
 
-       clk_enable(ecap_dev->clk);
+       ret = clk_enable(ecap_dev->clk);
+       if (ret) {
+               dev_err(dev, "Cannot enable clock %d\n", ret);
+               return ret;
+       }
 
        ecap_cnt_capture_set_evmode(counter_dev, ecap_dev->pm_ctx.ev_mode);