Re: [Intel-gfx] [PATCH] drm/i915/guc: Replace %phn with %phN

2018-04-10 Thread Chris Wilson
Quoting Michal Wajdeczko (2018-04-10 11:57:03) > On Tue, 10 Apr 2018 11:14:35 +0200, Chris Wilson > wrote: > > > %phn is not a valid specifier, > > Well, implementation allows that as alias for default separator ;) > > switch (fmt[1]) { > case 'C': > separator

Re: [Intel-gfx] [PATCH] drm/i915/guc: Replace %phn with %phN

2018-04-10 Thread Michal Wajdeczko
On Tue, 10 Apr 2018 11:14:35 +0200, Chris Wilson wrote: %phn is not a valid specifier, Well, implementation allows that as alias for default separator ;) switch (fmt[1]) { case 'C': separator = ':'; break; case 'D': se

[Intel-gfx] [PATCH] drm/i915/guc: Replace %phn with %phN

2018-04-10 Thread Chris Wilson
%phn is not a valid specifier, so I presume %phN was meant for an encoded hex string with no separator (and not that the hex string should be followed by the letter 'n'!). drivers/gpu/drm/i915/intel_guc_ct.c:616 ctb_read() warn: '%ph' cannot be followed by 'n' drivers/gpu/drm/i915/intel_guc_ct.c: