From: Hans Verkuil Date: Fri, 15 Mar 2013 19:35:28 +0000 (-0300) Subject: [media] solo6x10: disable the 'priv' abuse X-Git-Url: http://git.mmlx.us/?a=commitdiff_plain;h=69996873478d75381d14abbc9e768cc38d82e94b;p=linux-edison.git [media] solo6x10: disable the 'priv' abuse Signed-off-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/staging/media/solo6x10/v4l2-enc.c b/drivers/staging/media/solo6x10/v4l2-enc.c index 4fbfb351d37..0efb6237f68 100644 --- a/drivers/staging/media/solo6x10/v4l2-enc.c +++ b/drivers/staging/media/solo6x10/v4l2-enc.c @@ -860,8 +860,16 @@ static int solo_enc_set_fmt_cap(struct file *file, void *priv, /* This does not change the encoder at all */ solo_enc->fmt = pix->pixelformat; + /* + * More information is needed about these 'extended' types. As far + * as I can tell these are basically additional video streams with + * different MPEG encoding attributes that can run in parallel with + * the main stream. If so, then this should be implemented as a + * second video node. Abusing priv like this is certainly not the + * right approach. if (pix->priv) solo_enc->type = SOLO_ENC_TYPE_EXT; + */ return 0; }