On Mon, 10 May 2021, Dan Carpenter wrote:
> 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);
Oops! Thanks. That is easy to fix.
julia
>
> 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
>
>
