[PATCH v2] mtd: Avoid printing error messages on probe deferrals

Michael Walle michael at walle.cc
Tue Mar 7 12:35:05 PST 2023


Am 2023-03-07 20:25, schrieb Miquel Raynal:
> There is no reason to complain about probe errors in case of deferrals.
> 
> Signed-off-by: Miquel Raynal <miquel.raynal at bootlin.com>
> ---
> Changes in v2:
> * Use of dev_err_probe().
> 
>  drivers/mtd/mtdcore.c | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/mtd/mtdcore.c b/drivers/mtd/mtdcore.c
> index 0feacb9fbdac..e4e102585021 100644
> --- a/drivers/mtd/mtdcore.c
> +++ b/drivers/mtd/mtdcore.c
> @@ -536,12 +536,11 @@ static int mtd_nvmem_add(struct mtd_info *mtd)
>  	mtd->nvmem = nvmem_register(&config);
>  	if (IS_ERR(mtd->nvmem)) {
>  		/* Just ignore if there is no NVMEM support in the kernel */
> -		if (PTR_ERR(mtd->nvmem) == -EOPNOTSUPP) {
> +		if (PTR_ERR(mtd->nvmem) == -EOPNOTSUPP)
>  			mtd->nvmem = NULL;
> -		} else {
> -			dev_err(&mtd->dev, "Failed to register NVMEM device\n");
> -			return PTR_ERR(mtd->nvmem);
> -		}
> +		else
> +			return dev_err_probe(&mtd->dev,	PTR_ERR(mtd->nvmem),

There is a '\t' after the ",".

With that fixed,
Reviewed-by: Michael Walle <michael at walle.cc>

-michael



More information about the linux-mtd mailing list