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

From: Joe Perches
Date: Fri Aug 27 2021 - 12:09:45 EST


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:
> > On Fri, Aug 27, 2021 at 01:10:41AM -0700, Joe Perches wrote:
> > > On Fri, 2021-08-27 at 10:51 +0300, Andy Shevchenko wrote:
> > > > On Thu, Aug 26, 2021 at 11:43:00AM -0700, Joe Perches wrote:
> > > > > 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@xxxxxxxxxxx/T/#u
> >
> > 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.