All 4 of the "colorspace" components were not originally handled.
Causing issue related to xfer_func not being initialized properly.

This was found with v4l2-compliance test.

Signed-off-by: Benoit Parrot <bpar...@ti.com>
---
 drivers/media/platform/ti-vpe/vpe.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/media/platform/ti-vpe/vpe.c 
b/drivers/media/platform/ti-vpe/vpe.c
index 295cbcef3a0e..7c74b21e0164 100644
--- a/drivers/media/platform/ti-vpe/vpe.c
+++ b/drivers/media/platform/ti-vpe/vpe.c
@@ -1582,11 +1582,14 @@ static int vpe_g_fmt(struct file *file, void *priv, 
struct v4l2_format *f)
                struct vpe_q_data *s_q_data;
                struct v4l2_pix_format_mplane *spix;
 
-               /* get colorspace from the source queue */
+               /* get colorimetry from the source queue */
                s_q_data = get_q_data(ctx, V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE);
                spix = &s_q_data->format.fmt.pix_mp;
 
                pix->colorspace = spix->colorspace;
+               pix->xfer_func = spix->xfer_func;
+               pix->ycbcr_enc = spix->ycbcr_enc;
+               pix->quantization = spix->quantization;
        }
 
        return 0;
-- 
2.17.1

Reply via email to