diff options
| author | Michael Ellerman <mpe@ellerman.id.au> | 2021-04-16 21:07:06 +1000 |
|---|---|---|
| committer | Michael Ellerman <mpe@ellerman.id.au> | 2021-04-16 21:12:50 +1000 |
| commit | 7767d9ac89cee29c68f5dd278b3bb411d1c69287 (patch) | |
| tree | 8172295718fd8347f26a8737913f740ca07ccad9 /arch/powerpc/platforms/pseries/papr_scm.c | |
| parent | d6481a7195df4a8c828f9ee0b382f2dd36d3575c (diff) | |
powerpc/papr_scm: Fix build error due to wrong printf specifier
When I changed the rc variable to be long rather than int64_t I
neglected to update the printk(), leading to a build break:
arch/powerpc/platforms/pseries/papr_scm.c: In function 'papr_scm_pmem_flush':
arch/powerpc/platforms/pseries/papr_scm.c:144:26: warning: format
'%lld' expects argument of type 'long long int', but argument 3 has
type 'long int' [-Wformat=]
Fixes: 75b7c05ebf90 ("powerpc/papr_scm: Implement support for H_SCM_FLUSH hcall")
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20210416111209.765444-2-mpe@ellerman.id.au
Diffstat (limited to 'arch/powerpc/platforms/pseries/papr_scm.c')
| -rw-r--r-- | arch/powerpc/platforms/pseries/papr_scm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/powerpc/platforms/pseries/papr_scm.c b/arch/powerpc/platforms/pseries/papr_scm.c index ae6f5d80d5ce..48de21902116 100644 --- a/arch/powerpc/platforms/pseries/papr_scm.c +++ b/arch/powerpc/platforms/pseries/papr_scm.c @@ -141,7 +141,7 @@ static int papr_scm_pmem_flush(struct nd_region *nd_region, } while (rc == H_BUSY); if (rc) { - dev_err(&p->pdev->dev, "flush error: %lld", rc); + dev_err(&p->pdev->dev, "flush error: %ld", rc); rc = -EIO; } else { dev_dbg(&p->pdev->dev, "flush drc 0x%x complete", p->drc_index); |