On 03/11/14 09:33, Archit Taneja wrote: > The vpe output and capture queues are initially configured to default values in > vpe_open(). A G_FMT before any S_FMTs will result in these values being > populated. > > The colorspace and bytesperline parameter of this initial configuration are > incorrect. This breaks compliance when as we get 'TRY_FMT(G_FMT) != G_FMT'. > > Fix the initial queue configuration such that it wouldn't need to be fixed by > try_fmt. > > Signed-off-by: Archit Taneja <archit@xxxxxx> Reviewed-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> > --- > drivers/media/platform/ti-vpe/vpe.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/media/platform/ti-vpe/vpe.c b/drivers/media/platform/ti-vpe/vpe.c > index 5591d04..85d1122 100644 > --- a/drivers/media/platform/ti-vpe/vpe.c > +++ b/drivers/media/platform/ti-vpe/vpe.c > @@ -2012,9 +2012,11 @@ static int vpe_open(struct file *file) > s_q_data->fmt = &vpe_formats[2]; > s_q_data->width = 1920; > s_q_data->height = 1080; > - s_q_data->sizeimage[VPE_LUMA] = (s_q_data->width * s_q_data->height * > + s_q_data->bytesperline[VPE_LUMA] = (s_q_data->width * > s_q_data->fmt->vpdma_fmt[VPE_LUMA]->depth) >> 3; > - s_q_data->colorspace = V4L2_COLORSPACE_SMPTE170M; > + s_q_data->sizeimage[VPE_LUMA] = (s_q_data->bytesperline[VPE_LUMA] * > + s_q_data->height); > + s_q_data->colorspace = V4L2_COLORSPACE_REC709; > s_q_data->field = V4L2_FIELD_NONE; > s_q_data->c_rect.left = 0; > s_q_data->c_rect.top = 0; > -- 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
