Thread (12 messages) flat view 12 messages, 4 authors, 2021-08-28

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

From: Joe Perches <joe@perches.com>
Date: 2021-08-27 16:09:45
Also in: linux-scsi, linux-staging, lkml

On Fri, 2021-08-27 at 12:23 +0200, Greg KH wrote:
On Fri, Aug 27, 2021 at 11:46:20AM +0300, Andy Shevchenko wrote:
quoted
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.
ABI _should_ mean stability for random parsers.

I don't use these so I don't care that much.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help