Re: [PATCH] mediabus: add MIPI CSI-2 pixel format codes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Guennadi,

On Friday 23 July 2010 10:13:37 Guennadi Liakhovetski wrote:
> Add pixel format codes, defined in the MIPI CSI-2 specification.
> 
> Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@xxxxxx>
> ---
> 
> Even though it affects the same enum as my patch from yesterday, they are
> independent, Hans and Laurent CCed just to avoid possible conflicts, when
> further patching this file.
> 
>  include/media/v4l2-mediabus.h |   26 ++++++++++++++++++++++++++
>  1 files changed, 26 insertions(+), 0 deletions(-)
> 
> diff --git a/include/media/v4l2-mediabus.h b/include/media/v4l2-mediabus.h
> index a870965..b0dcace 100644
> --- a/include/media/v4l2-mediabus.h
> +++ b/include/media/v4l2-mediabus.h
> @@ -41,6 +41,32 @@ enum v4l2_mbus_pixelcode {
>  	V4L2_MBUS_FMT_SBGGR10_2X8_PADHI_BE,
>  	V4L2_MBUS_FMT_SBGGR10_2X8_PADLO_BE,
>  	V4L2_MBUS_FMT_SGRBG8_1X8,
> +	/* MIPI CSI-2 codes */
> +	V4L2_MBUS_FMT_MIPI_CSI2_YUV420_8_L,
> +	V4L2_MBUS_FMT_MIPI_CSI2_YUV420_8,
> +	V4L2_MBUS_FMT_MIPI_CSI2_YUV420_10,
> +	V4L2_MBUS_FMT_MIPI_CSI2_YUV420_8_CSPS,
> +	V4L2_MBUS_FMT_MIPI_CSI2_YUV420_10_CSPS,
> +	V4L2_MBUS_FMT_MIPI_CSI2_YUV422_8,
> +	V4L2_MBUS_FMT_MIPI_CSI2_YUV422_10,
> +	V4L2_MBUS_FMT_MIPI_CSI2_RGB888,
> +	V4L2_MBUS_FMT_MIPI_CSI2_RGB666,
> +	V4L2_MBUS_FMT_MIPI_CSI2_RGB565,
> +	V4L2_MBUS_FMT_MIPI_CSI2_RGB555,
> +	V4L2_MBUS_FMT_MIPI_CSI2_RGB444,
> +	V4L2_MBUS_FMT_MIPI_CSI2_RAW6,
> +	V4L2_MBUS_FMT_MIPI_CSI2_RAW7,
> +	V4L2_MBUS_FMT_MIPI_CSI2_RAW8,
> +	V4L2_MBUS_FMT_MIPI_CSI2_RAW10,
> +	V4L2_MBUS_FMT_MIPI_CSI2_RAW12,
> +	V4L2_MBUS_FMT_MIPI_CSI2_RAW14,
> +	V4L2_MBUS_FMT_MIPI_CSI2_GEN_NULL,
> +	V4L2_MBUS_FMT_MIPI_CSI2_GEN_BLANKING,
> +	V4L2_MBUS_FMT_MIPI_CSI2_GEN_EMBEDDED8,
> +	V4L2_MBUS_FMT_MIPI_CSI2_USER_1,
> +	V4L2_MBUS_FMT_MIPI_CSI2_USER_2,
> +	V4L2_MBUS_FMT_MIPI_CSI2_USER_3,
> +	V4L2_MBUS_FMT_MIPI_CSI2_USER_4,

I don't think I like this. Take the raw formats for instance, they're used for 
Bayer RGB. V4L2_MBUS_FMT_MIPI_CSI2_RAW8 could map to any Bayer format (GRBG, 
RGGB, ...). Why don't we just use "standard" pixel codes ?

>  };
> 
>  /**

-- 
Regards,

Laurent Pinchart
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux