Messages in this thread | | | Subject | Re: [PATCH] Input: synaptics-rmi4 - use %ph to form F34 configuration ID | From | Andy Shevchenko <> | Date | Tue, 30 May 2017 22:04:19 +0300 |
| |
On Tue, 2017-05-30 at 10:23 -0700, Joe Perches wrote: > On Tue, 2017-05-30 at 09:59 -0700, Dmitry Torokhov wrote: > > On Tue, May 30, 2017 at 10:23:58AM +0200, Benjamin Tissoires wrote: > > > On May 29 2017 or thereabouts, Dmitry Torokhov wrote: > > > >
> > Or we'd need to introduce %pH I guess. > > As it's debugging output, I'd hazard to guess it doesn't matter. > > If someone _really_ needs upper-case output in *ph,
Starting from the same...
> > A pity that %ph[cdnCDN] isn't possible as that's the style used > by %pU[blBL] for lower/upper case output. > > There are < 80 uses of %[\*\d]+ph[CDN], maybe it'd be reasonable > to do a treewide sed to lowercase for them. Dunno.
And I think less intrusive would be %pH[CDN] since the latter proposal misses plain %*ph use.
But for now I would rather avoid it at all, I'm pretty sure there very rare cases when one needs 1) to print a dump, *and* 2) it should be upper cased.
-- Andy Shevchenko <andriy.shevchenko@linux.intel.com> Intel Finland Oy
| |