diff options
| author |   Krzysztof Kozlowski <krzk@kernel.org> | 2020-08-28 17:26:36 +0200 |
|---|---|---|
| committer |   Vinod Koul <vkoul@kernel.org> | 2020-09-03 12:38:59 +0530 |
| commit | 1c966e1d948ba1abf646966ff5f0e57634ae70b7 (patch) | |
| tree | 491f25f146cd5fc012826891cccb8368818fb631 /drivers/dma/stm32-mdma.c | |
| parent | dmaengine: pl330: Simplify with dev_err_probe() (diff) | |
| download | linux-dev-1c966e1d948ba1abf646966ff5f0e57634ae70b7.tar.xz linux-dev-1c966e1d948ba1abf646966ff5f0e57634ae70b7.zip | |
dmaengine: stm32: Simplify with dev_err_probe()
Common pattern of handling deferred probe can be simplified with
dev_err_probe(). Less code and the error value gets printed.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Link: https://lore.kernel.org/r/20200828152637.16903-2-krzk@kernel.org
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'drivers/dma/stm32-mdma.c')
| -rw-r--r-- | drivers/dma/stm32-mdma.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/dma/stm32-mdma.c b/drivers/dma/stm32-mdma.c index 5469563703d1..08cfbfab837b 100644 --- a/drivers/dma/stm32-mdma.c +++ b/drivers/dma/stm32-mdma.c @@ -1580,12 +1580,9 @@ static int stm32_mdma_probe(struct platform_device *pdev) return PTR_ERR(dmadev->base); dmadev->clk = devm_clk_get(&pdev->dev, NULL); - if (IS_ERR(dmadev->clk)) { - ret = PTR_ERR(dmadev->clk); - if (ret != -EPROBE_DEFER) - dev_err(&pdev->dev, "Missing clock controller\n"); - return ret; - } + if (IS_ERR(dmadev->clk)) + return dev_err_probe(&pdev->dev, PTR_ERR(dmadev->clk), + "Missing clock controller\n"); ret = clk_prepare_enable(dmadev->clk); if (ret < 0) { |