On Wed, Feb 04, 2026 at 11:25:01AM +0000, Isaac Scott wrote:
> To be able to use YUV bypass, YUV formats need to be available on the
> source pad of the rkisp1 resizer. Add them.
>
> Signed-off-by: Isaac Scott <isaac.scott@ideasonboard.com>
> ---
> .../platform/rockchip/rkisp1/rkisp1-resizer.c | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-resizer.c b/drivers/media/platform/rockchip/rkisp1/rkisp1-resizer.c
> index 8e6b753d3081..e72b76ab078c 100644
> --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-resizer.c
> +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-resizer.c
> @@ -23,11 +23,26 @@ struct rkisp1_rsz_yuv_mbus_info {
> };
>
> static const struct rkisp1_rsz_yuv_mbus_info rkisp1_rsz_yuv_src_formats[] = {
> + {
> + .mbus_code = MEDIA_BUS_FMT_UYVY8_2X8, /* YUV422 */
> + .hdiv = 1,
> + .vdiv = 1,
> + },
> + {
> + .mbus_code = MEDIA_BUS_FMT_VYUY8_2X8, /* YUV422 */
> + .hdiv = 1,
> + .vdiv = 1,
> + },
> {
> .mbus_code = MEDIA_BUS_FMT_YUYV8_2X8, /* YUV422 */
> .hdiv = 2,
> .vdiv = 1,
> },
> + {
> + .mbus_code = MEDIA_BUS_FMT_YVYU8_2X8, /* YUV422 */
> + .hdiv = 1,
Shouldn't hdiv be 2 for all those new formats ?
But overall, I don't think those formats are needed. I would be very
surprised if the resizer used different YUV orders on its output bus.
Can't you use MEDIA_BUS_FMT_YUYV8_2X8 unconditionally ?
> + .vdiv = 1,
> + },
> {
> .mbus_code = MEDIA_BUS_FMT_YUYV8_1_5X8, /* YUV420 */
> .hdiv = 2,
--
Regards,
Laurent Pinchart