| Message ID | 1512724105-1778-6-git-send-email-smitha.t@samsung.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Le vendredi 08 décembre 2017 à 14:38 +0530, Smitha T Murthy a écrit : > Add V4L2 definition for HEVC compressed format > > Signed-off-by: Smitha T Murthy <smitha.t@samsung.com> > Reviewed-by: Andrzej Hajda <a.hajda@samsung.com> > Reviewed-by: Stanimir Varbanov <stanimir.varbanov@linaro.org> > Acked-by: Hans Verkuil <hans.verkuil@cisco.com> > --- > include/uapi/linux/videodev2.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h > index 185d6a0..bd9b5d5 100644 > --- a/include/uapi/linux/videodev2.h > +++ b/include/uapi/linux/videodev2.h > @@ -634,6 +634,7 @@ struct v4l2_pix_format { > #define V4L2_PIX_FMT_VC1_ANNEX_L v4l2_fourcc('V', 'C', '1', 'L') /* SMPTE 421M Annex L compliant stream */ > #define V4L2_PIX_FMT_VP8 v4l2_fourcc('V', 'P', '8', '0') /* VP8 */ > #define V4L2_PIX_FMT_VP9 v4l2_fourcc('V', 'P', '9', '0') /* VP9 */ > +#define V4L2_PIX_FMT_HEVC v4l2_fourcc('H', 'E', 'V', 'C') /* HEVC aka H.265 */ Wouldn't it be more consistent to call it V4L2_PIX_FMT_H265 as we have used H264 for the previous generation, or is there a formal rationale ? Also, this is byte-stream right ? With start codes ? > > /* Vendor-specific formats */ > #define V4L2_PIX_FMT_CPIA1 v4l2_fourcc('C', 'P', 'I', 'A') /* cpia1 YUV */
Em Fri, 08 Dec 2017 10:47:24 -0500 Nicolas Dufresne <nicolas@ndufresne.ca> escreveu: > Le vendredi 08 décembre 2017 à 14:38 +0530, Smitha T Murthy a écrit : > > Add V4L2 definition for HEVC compressed format > > > > Signed-off-by: Smitha T Murthy <smitha.t@samsung.com> > > Reviewed-by: Andrzej Hajda <a.hajda@samsung.com> > > Reviewed-by: Stanimir Varbanov <stanimir.varbanov@linaro.org> > > Acked-by: Hans Verkuil <hans.verkuil@cisco.com> > > --- > > include/uapi/linux/videodev2.h | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h > > index 185d6a0..bd9b5d5 100644 > > --- a/include/uapi/linux/videodev2.h > > +++ b/include/uapi/linux/videodev2.h > > @@ -634,6 +634,7 @@ struct v4l2_pix_format { > > #define V4L2_PIX_FMT_VC1_ANNEX_L v4l2_fourcc('V', 'C', '1', 'L') /* SMPTE 421M Annex L compliant stream */ > > #define V4L2_PIX_FMT_VP8 v4l2_fourcc('V', 'P', '8', '0') /* VP8 */ > > #define V4L2_PIX_FMT_VP9 v4l2_fourcc('V', 'P', '9', '0') /* VP9 */ > > +#define V4L2_PIX_FMT_HEVC v4l2_fourcc('H', 'E', 'V', 'C') /* HEVC aka H.265 */ > > Wouldn't it be more consistent to call it V4L2_PIX_FMT_H265 as we have > used H264 for the previous generation, or is there a formal rationale ? It would, but what I'm afraid on using specs name here, due to what happened with MPEG-4 specs. Originally, it was referred to one codec (ASP). Then, part 10 came with a different compression codec (AVC). So, now, when referring to MPEG-4, one need to explicitly mention if the code is AVC or not. The ITU-T didn't make the same mistake (yet?), but nobody can predict what will happen in the future. So, now that an spec can be modified to include additional parts with different codecs, it sounds less risky to use the codec name instead of the spec number, as this shouldn't change :-) > Also, this is byte-stream right ? With start codes ? > > > > > /* Vendor-specific formats */ > > #define V4L2_PIX_FMT_CPIA1 v4l2_fourcc('C', 'P', 'I', 'A') /* cpia1 YUV */ Thanks, Mauro
diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h index 185d6a0..bd9b5d5 100644 --- a/include/uapi/linux/videodev2.h +++ b/include/uapi/linux/videodev2.h @@ -634,6 +634,7 @@ struct v4l2_pix_format { #define V4L2_PIX_FMT_VC1_ANNEX_L v4l2_fourcc('V', 'C', '1', 'L') /* SMPTE 421M Annex L compliant stream */ #define V4L2_PIX_FMT_VP8 v4l2_fourcc('V', 'P', '8', '0') /* VP8 */ #define V4L2_PIX_FMT_VP9 v4l2_fourcc('V', 'P', '9', '0') /* VP9 */ +#define V4L2_PIX_FMT_HEVC v4l2_fourcc('H', 'E', 'V', 'C') /* HEVC aka H.265 */ /* Vendor-specific formats */ #define V4L2_PIX_FMT_CPIA1 v4l2_fourcc('C', 'P', 'I', 'A') /* cpia1 YUV */