Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754253AbdLHPra (ORCPT ); Fri, 8 Dec 2017 10:47:30 -0500 Received: from mail-qt0-f195.google.com ([209.85.216.195]:42442 "EHLO mail-qt0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754184AbdLHPr2 (ORCPT ); Fri, 8 Dec 2017 10:47:28 -0500 X-Google-Smtp-Source: AGs4zMaJe9AkciuiaDqy5HD44ZgSuxVZsIrnB0urfOxekI9xrgRTlpNnB/fgIorSCHiONsPufTByRA== Message-ID: <1512748044.24635.1.camel@ndufresne.ca> Subject: Re: [Patch v6 05/12] [media] videodev2.h: Add v4l2 definition for HEVC From: Nicolas Dufresne To: Smitha T Murthy , linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Cc: kyungmin.park@samsung.com, kamil@wypas.org, jtp.park@samsung.com, a.hajda@samsung.com, mchehab@kernel.org, pankaj.dubey@samsung.com, krzk@kernel.org, m.szyprowski@samsung.com, s.nawrocki@samsung.com Date: Fri, 08 Dec 2017 10:47:24 -0500 In-Reply-To: <1512724105-1778-6-git-send-email-smitha.t@samsung.com> References: <1512724105-1778-1-git-send-email-smitha.t@samsung.com> <1512724105-1778-6-git-send-email-smitha.t@samsung.com> Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature"; boundary="=-zBflTywZusYwpgjvxYQ5" X-Mailer: Evolution 3.26.2 (3.26.2-1.fc27) Mime-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1875 Lines: 51 --=-zBflTywZusYwpgjvxYQ5 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Le vendredi 08 d=C3=A9cembre 2017 =C3=A0 14:38 +0530, Smitha T Murthy a =C3= =A9crit : > Add V4L2 definition for HEVC compressed format >=20 > Signed-off-by: Smitha T Murthy > Reviewed-by: Andrzej Hajda > Reviewed-by: Stanimir Varbanov > Acked-by: Hans Verkuil > --- > include/uapi/linux/videodev2.h | 1 + > 1 file changed, 1 insertion(+) >=20 > diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev= 2.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') /* SMPT= E 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 ak= a 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 ? > =20 > /* Vendor-specific formats */ > #define V4L2_PIX_FMT_CPIA1 v4l2_fourcc('C', 'P', 'I', 'A') /* cpia1 Y= UV */ --=-zBflTywZusYwpgjvxYQ5 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- iF0EABECAB0WIQSScpfJiL+hb5vvd45xUwItrAaoHAUCWiq0DAAKCRBxUwItrAao HLNBAKDXtuL/XqlyvgodZ7lwK6E6esKRqACeKTMpiOEDhZ9IswDMc/PtewU6ioc= =FLnp -----END PGP SIGNATURE----- --=-zBflTywZusYwpgjvxYQ5--