ASoC: qcom: lpass-sc7180: Delete redundant error log from _resume()
authorMatthias Kaehlcke <mka@chromium.org>
Tue, 29 Nov 2022 00:16:54 +0000 (00:16 +0000)
committerMark Brown <broonie@kernel.org>
Tue, 29 Nov 2022 11:32:07 +0000 (11:32 +0000)
sc7180_lpass_dev_resume() logs an error if clk_bulk_prepare_enable()
fails. The clock framework already generates error logs if anything
goes wrong, so the logging in _resume() is redundant, drop it.

Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
Link: https://lore.kernel.org/r/20221129001633.v2.1.I8d1993f41f0da1eac0ecba321678ac489f9c0b9b@changeid
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/qcom/lpass-sc7180.c

index 9c3365d..b96b85a 100644 (file)
@@ -165,15 +165,9 @@ static int sc7180_lpass_exit(struct platform_device *pdev)
 
 static int sc7180_lpass_dev_resume(struct device *dev)
 {
-       int ret = 0;
        struct lpass_data *drvdata = dev_get_drvdata(dev);
 
-       ret = clk_bulk_prepare_enable(drvdata->num_clks, drvdata->clks);
-       if (ret) {
-               dev_err(dev, "sc7180 clk prepare and enable failed\n");
-               return ret;
-       }
-       return ret;
+       return clk_bulk_prepare_enable(drvdata->num_clks, drvdata->clks);
 }
 
 static int sc7180_lpass_dev_suspend(struct device *dev)