diff options
| author |   Dan Carpenter <dan.carpenter@oracle.com> | 2021-05-11 10:09:08 +0300 |
|---|---|---|
| committer |   Stephen Boyd <sboyd@kernel.org> | 2021-06-01 16:34:02 -0700 |
| commit | 15abefda7fde690b7091ac5854135da41c7f759c (patch) | |
| tree | 46f38f82e9468ee40a74511dccfaeb0b96fedb4a | |
| parent | clk: qcom: Simplify usage of dev_err_probe() (diff) | |
| download | wireguard-linux-15abefda7fde690b7091ac5854135da41c7f759c.tar.xz wireguard-linux-15abefda7fde690b7091ac5854135da41c7f759c.zip | |
clk: qcom: cleanup some dev_err_probe() calls
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>
| -rw-r--r-- | drivers/clk/qcom/apcs-sdx55.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/clk/qcom/apcs-sdx55.c b/drivers/clk/qcom/apcs-sdx55.c index 6810637f32e3..e599f862ec44 100644 --- a/drivers/clk/qcom/apcs-sdx55.c +++ b/drivers/clk/qcom/apcs-sdx55.c @@ -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) |