[PATCH] mtd: fsl-quadspi: Print the error value

Han Xu xhnjupt at gmail.com
Tue Feb 10 20:07:23 PST 2015


On Mon, Feb 9, 2015 at 6:07 AM, Fabio Estevam
<fabio.estevam at freescale.com> wrote:
> When displaying dev_err() messages it is useful to print the error value.
>
> Signed-off-by: Fabio Estevam <fabio.estevam at freescale.com>
> ---
>  drivers/mtd/spi-nor/fsl-quadspi.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/mtd/spi-nor/fsl-quadspi.c b/drivers/mtd/spi-nor/fsl-quadspi.c
> index 1c7308c..78b1b33 100644
> --- a/drivers/mtd/spi-nor/fsl-quadspi.c
> +++ b/drivers/mtd/spi-nor/fsl-quadspi.c
> @@ -830,27 +830,27 @@ static int fsl_qspi_probe(struct platform_device *pdev)
>
>         ret = clk_prepare_enable(q->clk_en);
>         if (ret) {
> -               dev_err(dev, "can not enable the qspi_en clock\n");
> +               dev_err(dev, "can not enable the qspi_en clock: %d\n", ret);
>                 return ret;
>         }
>
>         ret = clk_prepare_enable(q->clk);
>         if (ret) {
> -               dev_err(dev, "can not enable the qspi clock\n");
> +               dev_err(dev, "can not enable the qspi clock: %d\n", ret);
>                 goto clk_failed;
>         }
>
>         /* find the irq */
>         ret = platform_get_irq(pdev, 0);
>         if (ret < 0) {
> -               dev_err(dev, "failed to get the irq\n");
> +               dev_err(dev, "failed to get the irq: %d\n", ret);
>                 goto irq_failed;
>         }
>
>         ret = devm_request_irq(dev, ret,
>                         fsl_qspi_irq_handler, 0, pdev->name, q);
>         if (ret) {
> -               dev_err(dev, "failed to request irq.\n");
> +               dev_err(dev, "failed to request irq: %d\n", ret);
>                 goto irq_failed;
>         }
>
> --
> 1.9.1
>

Acked-by: Han Xu <han.xu at freescale.com>

>
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/



More information about the linux-mtd mailing list