Re: [PATCH 1/1] Remove unneeded "0x" when %pa formatting is used.

From: Dmitry Krivenok
Date: Mon Nov 30 2015 - 15:33:16 EST


> First of all, please, split per subsystem / driver. Here seems one
> file â one patch.

Sure, will do.

>> 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.

Thanks,
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/