Re: [PATCH v2 0/9] media: renesas: vsp1: Add colorspace support

[Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>]

Hi Laurent,
Thanks for this series.

On Wed, Apr 30, 2025 at 02:28:55AM +0300, Laurent Pinchart wrote:
> Hello,
> 
> This patch series extends the VSP1 driver with colorspace support. It
> turns out that the VSP RPF and WPF entities can convert between RGB and
> YUV, a feature that we have failed to test so far. The hardware support
> BT.601 and BT.709, in both limited and full range. Proper configuration
> of colorspace is crucial for accurate image rendering.
> 
> Patch 1/9 starts by implementing pixel format enumeration in the driver,
> a feature that was surprisingly missing. Patch 2/9 then continues with
> another fix, restricting the creation of the HSI and HST entities to VSP
> instances that include them. Following with another fix, patch 4/9
> addresses format setting on the RPF and WPF source pad to disable HSV
> <-> { RGB, YUV } conversion, a feature *not* supported by the hardware.
> 
> After those initial fixes, patch 5/9 starts implementing colorspace
> support by reporting the related information to userspace. The driver
> currently hardcodes limited range BT.601 when programming the hardware,
> so that is the value that the patch reports to userspace for YUV
> formats. Patch 6/9 makes the YCbCr encoding and quantization
> configurable, finalizing colorspace support in the VSP driver.
> 
> The next three patches are new in this version. They proceed with
> exposing colorspace configurability in the API exposed to the DU DRM
> driver in patch 8/9 (with 7/9 performing a small refactoring first).
> Patch 9/9 then updates the DU driver accordingly, to create plane
> colorspace properties and wire them up to the VSP.
> 
> The series has been tested with the vsp-tests suite and the kms-tests
> suite. Patches that add CSC support to the vsp-tests suite have been
> posted in [1], and can be found in the vsp-tests git tree in [2].

I've tested this series with out of Tree patches with RZ/G3E.
Basically I've added the equivalent of:

 - drm: rcar-du: Create plane color properties

On drivers/gpu/drm/renesas/rz-du/rzg2l_du_vsp.c

Tested using modetest:

modetest -M rzg2l-du -s 48@46:800x600-60.32@YU16 -d -P 39@46:512x300+200+200@YU16

modetest -M rzg2l-du -w 32:COLOR_ENCODING:0
modetest -M rzg2l-du -w 32:COLOR_ENCODING:1

modetest -M rzg2l-du -w 32:COLOR_RANGE:0
modetest -M rzg2l-du -w 32:COLOR_RANGE:1

Where 32 is the id of the primary plane.

Tested-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>

Thanks & Regards,
Tommaso

> Updates to media-ctl and yavta to support setting colorspace on video
> capture devices and on subdev source pads have been merged in the
> respective projects, make sure to use the latest master branch if you
> want to run the tests.
> 
> The series has also been tested with the latest v4l2-compliance. The bad
> news is that the test flags 56 errors when run on a Renesas Salvator-X
> 2nd version board based on r8a77965:
> 
> Grand Total for vsp1 device /dev/media1: 757, Succeeded: 701, Failed: 56, Warnings: 0
> 
> The good news is that none of those are regressions, quite the contrary:
> without this series applied, the total number of failures is 95, and the
> diff shows no new error. I will therefore address those issues
> separately.
> 
> I would like to get patch 1/9 to 8/9 merged in v6.16, to then merge 9/9
> in v6.17.
> 
> [1] https://lore.kernel.org/linux-renesas-soc/20250409004758.11014-1-laurent.pinchart@ideasonboard.com
> [2] https://git.ideasonboard.com/renesas/vsp-tests.git/log/?h=csc
> 
> Laurent Pinchart (9):
>   media: renesas: vsp1: Implement pixel format enumeration
>   media: renesas: vsp1: Make HSI and HST modules optional
>   media: renesas: vsp1: Fix HSV format enumeration
>   media: renesas: vsp1: Fix media bus code setup on RWPF source pad
>   media: renesas: vsp1: Report colour space information to userspace
>   media: renesas: vsp1: Allow setting encoding and quantization
>   media: renesas: vsp1: Name nested structure in vsp1_drm
>   media: renesas: vsp1: Expose color space through the DRM API
>   drm: rcar-du: Create plane color properties
> 
>  drivers/gpu/drm/renesas/rcar-du/rcar_du_vsp.c |  15 ++
>  drivers/media/platform/renesas/vsp1/vsp1.h    |   1 +
>  .../media/platform/renesas/vsp1/vsp1_brx.c    |   9 +-
>  .../media/platform/renesas/vsp1/vsp1_drm.c    |  22 +-
>  .../media/platform/renesas/vsp1/vsp1_drm.h    |   8 +-
>  .../media/platform/renesas/vsp1/vsp1_drv.c    |  59 +++---
>  .../media/platform/renesas/vsp1/vsp1_entity.c |  22 +-
>  .../media/platform/renesas/vsp1/vsp1_entity.h |   2 +
>  .../media/platform/renesas/vsp1/vsp1_hsit.c   |  11 +-
>  .../media/platform/renesas/vsp1/vsp1_pipe.c   | 188 ++++++++++++++++--
>  .../media/platform/renesas/vsp1/vsp1_pipe.h   |   5 +
>  .../media/platform/renesas/vsp1/vsp1_rpf.c    |  29 ++-
>  .../media/platform/renesas/vsp1/vsp1_rwpf.c   |  51 ++++-
>  .../media/platform/renesas/vsp1/vsp1_sru.c    |   9 +-
>  .../media/platform/renesas/vsp1/vsp1_uds.c    |   9 +-
>  .../media/platform/renesas/vsp1/vsp1_video.c  |  50 ++++-
>  .../media/platform/renesas/vsp1/vsp1_wpf.c    |  29 ++-
>  include/media/vsp1.h                          |   4 +
>  18 files changed, 442 insertions(+), 81 deletions(-)
> 
> 
> base-commit: 398a1b33f1479af35ca915c5efc9b00d6204f8fa
> prerequisite-patch-id: e3d9847d1da4398f1bf0b57f5ef4a612392c7255
> -- 
> Regards,
> 
> Laurent Pinchart
>