On Sun, May 09, 2021 at 11:17:42PM +0200, Julia Lawall wrote:
> ---------------------------
>
> Patch of a recent linux next:
>
> diff --git a/arch/arm/mach-pxa/cm-x300.c b/arch/arm/mach-pxa/cm-x300.c
> index 2e35354b61f5..ff75e928772a 100644
> --- a/arch/arm/mach-pxa/cm-x300.c
> +++ b/arch/arm/mach-pxa/cm-x300.c
> @@ -531,7 +531,7 @@ static int cm_x300_u2d_init(struct device *dev)
> pout_clk = clk_get(NULL, "CLK_POUT");
> if (IS_ERR(pout_clk)) {
> err = PTR_ERR(pout_clk);
> - pr_err("failed to get CLK_POUT: %d\n", err);
> + pr_err("failed to get CLK_POUT: %pe\n", err);
It has to print the pointer and not the err. So it should be:
pr_err("failed to get CLK_POUT: %pe\n", pout_clk);
But really someone needs to introduce an %e (and everyone can see the
need for %e but it's just a matter of finding someone who is feeling
motivated).
regards,
dan carpenter
