clk: qcom: cleanup some dev_err_probe() calls
authorDan Carpenter <dan.carpenter@oracle.com>
Tue, 11 May 2021 07:09:08 +0000 (10:09 +0300)
committerStephen Boyd <sboyd@kernel.org>
Tue, 1 Jun 2021 23:34:02 +0000 (16:34 -0700)
The dev_err_probe() function prints an error message if the error
code is not -EPROBE_DEFER.  If we know the error code in is -ENODEV
then there is no reason to check.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/YJotlJBJ1CVAgvMT@mwanda
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/qcom/apcs-sdx55.c

index 6810637..e599f86 100644 (file)
@@ -56,8 +56,10 @@ static int qcom_apcs_sdx55_clk_probe(struct platform_device *pdev)
        int ret;
 
        regmap = dev_get_regmap(parent, NULL);
-       if (!regmap)
-               return dev_err_probe(dev, -ENODEV, "Failed to get parent regmap\n");
+       if (!regmap) {
+               dev_err(dev, "Failed to get parent regmap\n");
+               return -ENODEV;
+       }
 
        a7cc = devm_kzalloc(dev, sizeof(*a7cc), GFP_KERNEL);
        if (!a7cc)