Hi Luca, On Fri, 7 Mar 2025 at 12:19, Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx> wrote: > %pC and %pCn print the same string, and commit 900cca294425 ("lib/vsprintf: > add %pC{,n,r} format specifiers for clocks") introducing them does not > clarify any intended difference. It can be assumed %pC is a default for > %pCn as some other specifiers do, but not all are consistent with this > policy. Moreover there is now no other suffix other than 'n', which makes a > default not really useful. The original intention was indeed to serve as a default, like some other formats also have ("%pa[p]", %pOF[f]"). > All users in the kernel were using %pC except for one which has been > converted. So now remove %pCn and all the unnecessary extra code and > documentation. > > Signed-off-by: Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx> I guess this is a worthwhile cleanup. Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds