Re: [PATCH] drm: Use vsnprintf extension %ph

From: Andy Shevchenko
Date: Sun Jun 11 2017 - 09:52:10 EST


On Sun, Jun 11, 2017 at 3:20 AM, Joe Perches <joe@xxxxxxxxxxx> wrote:
> On Sat, 2017-06-10 at 19:12 +0300, Andy Shevchenko wrote:
>> On Wed, May 31, 2017 at 2:35 AM, Joe Perches <joe@xxxxxxxxxxx> wrote:
>> > Using the extension saves a bit of code.
>> > + seq_printf(m, "faux/mst: %*ph\n", 2, buf);
>> > + seq_printf(m, "mst ctrl: %*ph\n", 1, buf);
>> > + seq_printf(m, "branch oui: %*phN devid: ", 3, buf);
>>
>> All above may use shorter form, i.e.
>> "..., "%Xph", buf);", where X is a constant written explicitly inplace.
>
> I know. Consistency is better though.
> Note the use of DP_RECEIVER_CAP_SIZE, etc...

Fair enough.

--
With Best Regards,
Andy Shevchenko