linux-kernel - Re: [PATCH] mmc: sdhci-of-arasan: Do now show error message in case of deffered probe

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Wed, 22 Aug 2018 11:48:08 +0200
From:   Ulf Hansson <ulf.hansson@...aro.org>
To:     Michal Simek <michal.simek@...inx.com>
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Michal Simek <monstr@...str.eu>,
        Adrian Hunter <adrian.hunter@...el.com>,
        "linux-mmc@...r.kernel.org" <linux-mmc@...r.kernel.org>,
        Jaehoon Chung <jh80.chung@...sung.com>, u-boot@...ts.denx.de,
        Linux ARM <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH] mmc: sdhci-of-arasan: Do now show error message in case
 of deffered probe

On 6 August 2018 at 10:43, Michal Simek <michal.simek@...inx.com> wrote:
> When mmc-pwrseq property is passed mmc_pwrseq_alloc() can return
> -EPROBE_DEFER because driver for power sequence provider is not probed
> yet. Do not show error message when this situation happens.
>
> Signed-off-by: Michal Simek <michal.simek@...inx.com>

Thanks, queued for v4.20!

Kind regards
Uffe

> ---
>
>  drivers/mmc/host/sdhci-of-arasan.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/host/sdhci-of-arasan.c b/drivers/mmc/host/sdhci-of-arasan.c
> index e3332a522a5d..e65f3bce3643 100644
> --- a/drivers/mmc/host/sdhci-of-arasan.c
> +++ b/drivers/mmc/host/sdhci-of-arasan.c
> @@ -772,7 +772,8 @@ static int sdhci_arasan_probe(struct platform_device *pdev)
>
>         ret = mmc_of_parse(host->mmc);
>         if (ret) {
> -               dev_err(&pdev->dev, "parsing dt failed (%d)\n", ret);
> +               if (ret != -EPROBE_DEFER)
> +                       dev_err(&pdev->dev, "parsing dt failed (%d)\n", ret);
>                 goto unreg_clk;
>         }
>
> --
> 1.9.1
>