[media] solo6x10: disable the 'priv' abuse
authorHans Verkuil <hans.verkuil@cisco.com>
Fri, 15 Mar 2013 19:35:28 +0000 (16:35 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 25 Mar 2013 11:50:22 +0000 (08:50 -0300)
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/staging/media/solo6x10/v4l2-enc.c

index 4fbfb351d373be443a9cb9501a40e8675a130570..0efb6237f6858e27605d75ae85cd597faa9341af 100644 (file)
@@ -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;
 }