Messages in this thread | ![/](/images/icornerl.gif) | | Date | Sun, 13 Dec 2015 03:13:46 +0200 | Subject | Re: [PATCH 1/1] Remove unneeded "0x" when %pa formatting is used. | From | Andy Shevchenko <> |
| |
On Mon, Nov 30, 2015 at 10:33 PM, Dmitry Krivenok <krivenok.dmitry@gmail.com> wrote: >>> if (size > max_size) { >>> dev_err(pcie->dev, >>> - "res size 0x%pap exceeds max supported size 0x%llx\n", >>> + "res size %pap exceeds max supported size 0x%llx\n", >> >> %pa also works. >> > > Yes, but AFAIK %pap and %pa are identical (see address_val() in lib/vsprintf.c), > so I'm not sure I understand your comment. > > Did you mean that %pa should be used instead of %pap? > I see that %pa is used much more often than %pap, but I didn't find any > evidence that %pap is deprecated.
+Rasmus.
No, it's not, but it explicitly for phys_addr_t (it might be nice to to have %par, for example for explicitly shown that is for resource_size_t, Rasmus?).
For my opinion for resources better to use %pa for now, though %pap will work as well.
-- With Best Regards, Andy Shevchenko
| ![\](/images/icornerr.gif) |