linux-kernel - Re: [PATCH v1 1/2] media: uapi: videodev2: Fix comment for 12-bit packed Bayer formats

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20250623135348.GB300@pendragon.ideasonboard.com>
Date: Mon, 23 Jun 2025 16:53:48 +0300
From: Laurent Pinchart <laurent.pinchart@...asonboard.com>
To: Mehdi Djait <mehdi.djait@...ux.intel.com>
Cc: sakari.ailus@...ux.intel.com, hverkuil@...all.nl,
	tomi.valkeinen@...asonboard.com, jacopo.mondi@...asonboard.com,
	kieran.bingham@...asonboard.com, mchehab@...nel.org,
	michael.riesch@...labora.com, nicolas.dufresne@...labora.com,
	linux-media@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 1/2] media: uapi: videodev2: Fix comment for 12-bit
 packed Bayer formats

On Mon, Jun 23, 2025 at 03:51:15PM +0200, Mehdi Djait wrote:
> For 12-bit packed Bayer formats: every two consecutive samples are
> packed into three bytes. Fix the corresponding comment.
> 
> Signed-off-by: Mehdi Djait <mehdi.djait@...ux.intel.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart@...asonboard.com>

> ---
>  include/uapi/linux/videodev2.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h
> index 9e3b366d5fc7..421cc0d22ed7 100644
> --- a/include/uapi/linux/videodev2.h
> +++ b/include/uapi/linux/videodev2.h
> @@ -726,7 +726,7 @@ struct v4l2_pix_format {
>  #define V4L2_PIX_FMT_SGBRG12 v4l2_fourcc('G', 'B', '1', '2') /* 12  GBGB.. RGRG.. */
>  #define V4L2_PIX_FMT_SGRBG12 v4l2_fourcc('B', 'A', '1', '2') /* 12  GRGR.. BGBG.. */
>  #define V4L2_PIX_FMT_SRGGB12 v4l2_fourcc('R', 'G', '1', '2') /* 12  RGRG.. GBGB.. */
> -	/* 12bit raw bayer packed, 6 bytes for every 4 pixels */
> +	/* 12bit raw bayer packed, 3 bytes for every 2 pixels */
>  #define V4L2_PIX_FMT_SBGGR12P v4l2_fourcc('p', 'B', 'C', 'C')
>  #define V4L2_PIX_FMT_SGBRG12P v4l2_fourcc('p', 'G', 'C', 'C')
>  #define V4L2_PIX_FMT_SGRBG12P v4l2_fourcc('p', 'g', 'C', 'C')

-- 
Regards,

Laurent Pinchart