On 09.12.19 17:54, Andy Shevchenko wrote: > %pa takes into consideration the special types such as resource_size_t. > Use this specifier %instead of explicit casting. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > --- > mm/early_ioremap.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/mm/early_ioremap.c b/mm/early_ioremap.c > index 1826f191e72c..a0018ad1a1f6 100644 > --- a/mm/early_ioremap.c > +++ b/mm/early_ioremap.c > @@ -121,8 +121,8 @@ __early_ioremap(resource_size_t phys_addr, unsigned long size, pgprot_t prot) > } > } > > - if (WARN(slot < 0, "%s(%08llx, %08lx) not found slot\n", > - __func__, (u64)phys_addr, size)) > + if (WARN(slot < 0, "%s(%pa, %08lx) not found slot\n", > + __func__, &phys_addr, size)) > return NULL; > > /* Don't allow wraparound or zero size */ > @@ -158,8 +158,8 @@ __early_ioremap(resource_size_t phys_addr, unsigned long size, pgprot_t prot) > --idx; > --nrpages; > } > - WARN(early_ioremap_debug, "%s(%08llx, %08lx) [%d] => %08lx + %08lx\n", > - __func__, (u64)phys_addr, size, slot, offset, slot_virt[slot]); > + WARN(early_ioremap_debug, "%s(%pa, %08lx) [%d] => %08lx + %08lx\n", > + __func__, &phys_addr, size, slot, offset, slot_virt[slot]); > > prev_map[slot] = (void __iomem *)(offset + slot_virt[slot]); > return prev_map[slot]; > Yes, that's what Documentation/core-api/printk-formats.rst wants us to do :) Reviewed-by: David Hildenbrand <david@xxxxxxxxxx> -- Thanks, David / dhildenb
