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

From: Greg KH
Date: Fri Aug 27 2021 - 06:23:57 EST


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.

thanks,

greg k-h