On Fri, 7 Mar 2025 13:13:19 +0100 Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote: > Hi Luca, > > On Fri, 7 Mar 2025 at 12:19, Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx> wrote: > > There are two printk format specifiers for clocks: %pC and %pCn, and they > > print exactly the same string. The reason for having two is not totally > > clear (see discussion in patch 2), but there seem to be no advantage in > > having two instead of one. > > > > Definitely having two without properly documenting they do the same creates > > misunderstandings [0]. > > > > Since %pCn is used in a single place, replace it with %pC and remove %pCn > > to simplify such format specifiers implementation and avoid > > misunderstandings. > > > > [0] https://lore.kernel.org/dri-devel/71c44221-b18b-4928-8faf-00893ec4a109@xxxxxxx/ > > The link looks unrelated? Wrong link, sorry. The correct one is: https://lore.kernel.org/dri-devel/f8df2b5e-b005-4ada-8108-159b2b94a72e@xxxxxxx/ Luca -- Luca Ceresoli, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
