* [PATCH 2/2] memory: tegra186-emc: Simplify with dev_err_probe()
2020-08-28 15:37 [PATCH 1/2] memory: brcmstb_dpfe: Simplify with dev_err_probe() Krzysztof Kozlowski
@ 2020-08-28 15:37 ` Krzysztof Kozlowski
2020-09-02 15:23 ` Krzysztof Kozlowski
2020-08-28 16:12 ` [PATCH 1/2] memory: brcmstb_dpfe: " Markus Mayer
` (2 subsequent siblings)
3 siblings, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-28 15:37 UTC (permalink / raw)
To: Markus Mayer, bcm-kernel-feedback-list, Krzysztof Kozlowski,
Florian Fainelli, Thierry Reding, Jonathan Hunter,
linux-arm-kernel, linux-kernel, linux-tegra
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>
---
drivers/memory/tegra/tegra186-emc.c | 10 ++--------
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/memory/tegra/tegra186-emc.c b/drivers/memory/tegra/tegra186-emc.c
index 8478f59db432..fa8af17b0e2d 100644
--- a/drivers/memory/tegra/tegra186-emc.c
+++ b/drivers/memory/tegra/tegra186-emc.c
@@ -172,14 +172,8 @@ static int tegra186_emc_probe(struct platform_device *pdev)
return -ENOMEM;
emc->bpmp = tegra_bpmp_get(&pdev->dev);
- if (IS_ERR(emc->bpmp)) {
- err = PTR_ERR(emc->bpmp);
-
- if (err != -EPROBE_DEFER)
- dev_err(&pdev->dev, "failed to get BPMP: %d\n", err);
-
- return err;
- }
+ if (IS_ERR(emc->bpmp))
+ return dev_err_probe(&pdev->dev, PTR_ERR(emc->bpmp), "failed to get BPMP\n");
emc->clk = devm_clk_get(&pdev->dev, "emc");
if (IS_ERR(emc->clk)) {
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 2/2] memory: tegra186-emc: Simplify with dev_err_probe()
2020-08-28 15:37 ` [PATCH 2/2] memory: tegra186-emc: " Krzysztof Kozlowski
@ 2020-09-02 15:23 ` Krzysztof Kozlowski
0 siblings, 0 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2020-09-02 15:23 UTC (permalink / raw)
To: Markus Mayer, bcm-kernel-feedback-list, Florian Fainelli,
Thierry Reding, Jonathan Hunter, linux-arm-kernel, linux-kernel,
linux-tegra
On Fri, Aug 28, 2020 at 05:37:47PM +0200, Krzysztof Kozlowski wrote:
> 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>
> ---
> drivers/memory/tegra/tegra186-emc.c | 10 ++--------
> 1 file changed, 2 insertions(+), 8 deletions(-)
Applied.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] memory: brcmstb_dpfe: Simplify with dev_err_probe()
2020-08-28 15:37 [PATCH 1/2] memory: brcmstb_dpfe: Simplify with dev_err_probe() Krzysztof Kozlowski
2020-08-28 15:37 ` [PATCH 2/2] memory: tegra186-emc: " Krzysztof Kozlowski
@ 2020-08-28 16:12 ` Markus Mayer
2020-08-28 17:11 ` Florian Fainelli
2020-09-02 15:22 ` Krzysztof Kozlowski
3 siblings, 0 replies; 6+ messages in thread
From: Markus Mayer @ 2020-08-28 16:12 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: BCM Kernel Feedback, Florian Fainelli, Thierry Reding,
Jonathan Hunter, Linux ARM Kernel, Linux Kernel, linux-tegra
On Fri, 28 Aug 2020 at 08:37, Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> 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>
Acked-by: Markus Mayer <mmayer@broadcom.com>
> ---
> drivers/memory/brcmstb_dpfe.c | 7 ++-----
> 1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/memory/brcmstb_dpfe.c b/drivers/memory/brcmstb_dpfe.c
> index dcf50bb8dd69..f43ba69fbb3e 100644
> --- a/drivers/memory/brcmstb_dpfe.c
> +++ b/drivers/memory/brcmstb_dpfe.c
> @@ -901,11 +901,8 @@ static int brcmstb_dpfe_probe(struct platform_device *pdev)
> }
>
> ret = brcmstb_dpfe_download_firmware(priv);
> - if (ret) {
> - if (ret != -EPROBE_DEFER)
> - dev_err(dev, "Couldn't download firmware -- %d\n", ret);
> - return ret;
> - }
> + if (ret)
> + return dev_err_probe(dev, ret, "Couldn't download firmware\n");
>
> ret = sysfs_create_groups(&pdev->dev.kobj, priv->dpfe_api->sysfs_attrs);
> if (!ret)
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 1/2] memory: brcmstb_dpfe: Simplify with dev_err_probe()
2020-08-28 15:37 [PATCH 1/2] memory: brcmstb_dpfe: Simplify with dev_err_probe() Krzysztof Kozlowski
2020-08-28 15:37 ` [PATCH 2/2] memory: tegra186-emc: " Krzysztof Kozlowski
2020-08-28 16:12 ` [PATCH 1/2] memory: brcmstb_dpfe: " Markus Mayer
@ 2020-08-28 17:11 ` Florian Fainelli
2020-09-02 15:22 ` Krzysztof Kozlowski
3 siblings, 0 replies; 6+ messages in thread
From: Florian Fainelli @ 2020-08-28 17:11 UTC (permalink / raw)
To: Krzysztof Kozlowski, Markus Mayer, bcm-kernel-feedback-list,
Florian Fainelli, Thierry Reding, Jonathan Hunter,
linux-arm-kernel, linux-kernel, linux-tegra
On 8/28/20 8:37 AM, Krzysztof Kozlowski wrote:
> 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>
Acked-by: Florian Fainelli <f.fainelli@gmail.com>
--
Florian
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] memory: brcmstb_dpfe: Simplify with dev_err_probe()
2020-08-28 15:37 [PATCH 1/2] memory: brcmstb_dpfe: Simplify with dev_err_probe() Krzysztof Kozlowski
` (2 preceding siblings ...)
2020-08-28 17:11 ` Florian Fainelli
@ 2020-09-02 15:22 ` Krzysztof Kozlowski
3 siblings, 0 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2020-09-02 15:22 UTC (permalink / raw)
To: Markus Mayer, bcm-kernel-feedback-list, Florian Fainelli,
Thierry Reding, Jonathan Hunter, linux-arm-kernel, linux-kernel,
linux-tegra
On Fri, Aug 28, 2020 at 05:37:46PM +0200, Krzysztof Kozlowski wrote:
> 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>
> ---
> drivers/memory/brcmstb_dpfe.c | 7 ++-----
Applied.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread