On Mon, Sep 16, 2019 at 12:04:32PM +0200, Hans Verkuil wrote: > From: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > > Add two macros that facilitate printing V4L fourcc values with printf > family of functions. v4l2_fourcc_conv provides the printf conversion > specifier for printing formats and v4l2_fourcc_args provides the actual > arguments for that conversion specifier. > > These macros are useful in both user and kernel code whenever you want > to report a pixelformat, therefore put them into videodev2.h. > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > [hverkuil-cisco@xxxxxxxxx: rename v4l2_fourcc_to_conv to v4l2_fourcc_args] > [hverkuil-cisco@xxxxxxxxx: add () around ? : expression] > [hverkuil-cisco@xxxxxxxxx: add comment about fourcc reuse] > [hverkuil-cisco@xxxxxxxxx: update Documentation/media/videodev2.h.rst.exceptions] Were you going to add your SoB when merging the patch to your tree? Or is it just missing here? Feel free to add: Acked-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> -- Sakari Ailus
