Hello, This patch series fixes and improves color space support in the rkisp1 driver. The first two patches initialize the color space fields to default values on the ISP subdev video sink and source pads, and allow setting the color space on the ISP sink pad (this has no influence on the ISP configuration, and only serves to propagate the correct color space on the pipeline). Patch 3/7 fixes a bug in the ISP source pad configuration, which allowed setting a Bayer output format with a YUV input format, a clearly invalid configuration. Patch then 4/7 mimicks patch 2/7 by allowing setting of color space fields on the source pad. The last three patches configure the RGB to YUV matrix (in the CSM module) using the ISP source pad YCbCr encoding. Patch 5/7 fixes a small bug in color space handling that resulted in the sink pad quantization controlling the ISP output, instead of using the source pad quantization. Patch 6/7 is a small internal API refactoring, and finally patch 7/7 handles the CSM configuration. If anyone is curious about how the matrix coefficients of patch 7/7 have been computed, see the script posted in [1] that will be merged in libcamera. [1] https://lists.libcamera.org/pipermail/libcamera-devel/2022-August/033183.html Laurent Pinchart (7): media: rkisp1: Initialize color space on sink and source pads media: rkisp1: Allow setting color space on ISP sink pad media: rkisp1: Fix source pad format configuration media: rkisp1: Allow setting all color space fields on ISP source pad media: rkisp1: Configure quantization using ISP source pad media: rkisp1: Don't pass the quantization to rkisp1_csm_config() media: rkisp1: Configure CSM based on YCbCr encoding .../platform/rockchip/rkisp1/rkisp1-common.h | 5 +- .../platform/rockchip/rkisp1/rkisp1-isp.c | 139 ++++++++++++++++-- .../platform/rockchip/rkisp1/rkisp1-params.c | 105 +++++++++---- 3 files changed, 207 insertions(+), 42 deletions(-) -- Regards, Laurent Pinchart
