[3/4] usb: dwc3: trace: log ep commands in hex

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Andy,

On Thursday, 23 August 2018 12:57:57 EEST Andy Shevchenko wrote:
> On Mon, Aug 20, 2018 at 3:21 PM Laurent Pinchart wrote:
> > On Monday, 20 August 2018 15:06:31 EEST Andy Shevchenko wrote:
> >> On Mon, Aug 20, 2018 at 2:25 PM, Laurent Pinchart wrote:
> >>>> -     TP_printk("%s: cmd '%s' [%d] params %08x %08x %08x --> status:
> >>>> %s",
> >>>> +     TP_printk("%s: cmd '%s' [%x] params %08x %08x %08x --> status:
> >>>> %s",
> >>> 
> >>> How about 0x%x ?
> >> 
> >> Side note: # is one character less for the same.
> > 
> > Doesn't that print 0 instead of 0x0 ? There's no ambiguity with 0, but I
> > find that always printing the 0x is more consistent. I'll leave that up
> > to Felipe, I'm OK with both options.
> 
>        #      The value should be converted to an "alternate form".
> For o conversions, the first character  of
>              the  output  string is made zero (by prefixing a 0 if it
> was not zero already).  For x and X con‐
>              versions, a nonzero result has the string "0x" (or "0X"
> for X conversions) prepended to it.

That's exactly my point, it will only prepend 0x when the value is not zero. 
Small inconsistency, but I don't mind too much.