lkml.org 
[lkml]   [2017]   [Jun]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] platform/x86: wmi: Fix printing info about WDG structure
From
Date
On Thu, 2017-06-08 at 08:16 -0700, Darren Hart wrote:
> On Sat, May 27, 2017 at 11:49:22PM +0300, Andy Shevchenko wrote:
> > On Sat, May 27, 2017 at 11:48 PM, Pali Rohár <pali.rohar@gmail.com> wrote:
> > > On Saturday 27 May 2017 15:33:14 Andy Shevchenko wrote:
> > > > On Sat, May 27, 2017 at 4:17 PM, Pali Rohár <pali.rohar@gmail.com>
> > > > wrote:
> > > > Okay, got it. But on your opinion does it make sense to do
> > > >
> > > > pr_info("\tobject_id: %2pE\n", g->object_id);
> > > >
> > > > instead?
> > > >
> > > > For ASCII it will go as is previously, otherwise escaping would be
> > > > done.
> > >
> > > Both is OK for me. Do you want to send a new patch with %2pE?
> >
> > To me it looks slightly cleaner.
>
> I don't want to let this one fall through the cracks. Pali, is a new one coming?

Perhaps
pr_info("\t%*pE\n", (int)sizeof(g->object_id), g->object_id);

\
 
 \ /
  Last update: 2017-06-12 01:32    [W:1.852 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site