Messages in this thread | | | From | Andy Shevchenko <> | Date | Thu, 10 Dec 2020 15:05:02 +0200 | Subject | Re: [PATCH v5 1/1] lib/vsprintf: Add support for printing V4L2 and DRM fourccs |
| |
On Thu, Dec 10, 2020 at 2:16 PM Petr Mladek <pmladek@suse.com> wrote: > On Fri 2020-11-13 12:54:41, Sakari Ailus wrote: > > Add a printk modifier %p4cc (for pixel format) for printing V4L2 and DRM > > pixel formats denoted by fourccs. The fourcc encoding is the same for both > > so the same implementation can be used. > > > > Suggested-by: Mauro Carvalho Chehab <mchehab@kernel.org> > > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> > > Andy, Rasmus, > > the last version looks fine to me. I am going to push it. > Please, speak up if you are against it.
My concerns are: - not so standard format of representation (why not to use string_escape_mem() helper?) or is it? - no compatibility with generic 4cc (I would rather have an additional specifier here for v4l2 cases. OTOH generic %p4cc to me sounds like an equivalent to %4pEh (but we have similar cases with MAC where %6ph is the same as %pM).
But I'm not insisting on them, consider it like just my 2 cents to the discussion.
-- With Best Regards, Andy Shevchenko
| |