Thread (11 messages) flat view 11 messages, 3 authors, 2021-08-27

Re: [PATCH 0/5] vsprintf and uses: Add upper case output to %*ph extension

From: Greg KH <gregkh@linuxfoundation.org>
Date: 2021-08-27 10:23:57
Also in: linux-doc, linux-scsi, lkml

On Fri, Aug 27, 2021 at 11:46:20AM +0300, Andy Shevchenko wrote:
On Fri, Aug 27, 2021 at 01:10:41AM -0700, Joe Perches wrote:
quoted
On Fri, 2021-08-27 at 10:51 +0300, Andy Shevchenko wrote:
quoted
On Thu, Aug 26, 2021 at 11:43:00AM -0700, Joe Perches wrote:
quoted
Several sysfs uses that could use %*ph are upper case hex output.
Add a flag to the short hex formatting routine in vsprintf to support them.
Add documentation too.
Thanks!

Unfortunately I have got only first patch and this cover letter. Can you,
please, Cc entire series?
It's on lore.

https://lore.kernel.org/lkml/cover.1630003183.git.joe@perches.com/T/#u (local)
Thanks. So, you won't me to review them in a regular way :-)

TBH, I think those examples may pretty much be safe to use small
letters always.
I agree, let's just fix the users here to use small letters instead of
adding another modifier to the kernel.

thanks,

greg k-h
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help