[media] solo6x10: fix various format-related compliancy issues
authorHans Verkuil <hans.verkuil@cisco.com>
Tue, 12 Mar 2013 21:43:21 +0000 (18:43 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 25 Mar 2013 11:43:34 +0000 (08:43 -0300)
- try_fmt should never return -EBUSY.
- invalid pix->field values were not mapped to a valid value.
- the priv field of struct v4l2_pix_format wasn't zeroed.
- the try_fmt error code was not checked in set_fmt.
- enum_framesizes/intervals is valid for both MJPEG and MPEG pixel formats.
- enum_frameintervals didn't check width and height and reported the
  wrong range.
- s_parm didn't set readbuffers.
- don't fail on invalid colorspace, just replace with the valid colorspace.
- bytesperline should be 0 for compressed formats.

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
drivers/staging/media/solo6x10/v4l2.c

index 8296d8d9dfae990c0190a661bce2ea60c7ab603e..5b2fdfbaef65951b8da7c5e11681c199659ae14d 100644 (file)
@@ -1039,13 +1039,6 @@ static int solo_enc_try_fmt_cap(struct file *file, void *priv,
            pix->pixelformat != V4L2_PIX_FMT_MJPEG)
                return -EINVAL;
 
-       /* We cannot change width/height in mid mpeg */
-       if (atomic_read(&solo_enc->mpeg_readers) > 0) {
-               if (pix->width != solo_enc->width ||
-                   pix->height != solo_enc->height)
-                       return -EBUSY;
-       }
-
        if (pix->width < solo_dev->video_hsize ||
            pix->height < solo_dev->video_vsize << 1) {
                /* Default to CIF 1/2 size */
@@ -1057,14 +1050,20 @@ static int solo_enc_try_fmt_cap(struct file *file, void *priv,
                pix->height = solo_dev->video_vsize << 1;
        }
 
-       if (pix->field == V4L2_FIELD_ANY)
-               pix->field = V4L2_FIELD_INTERLACED;
-       else if (pix->field != V4L2_FIELD_INTERLACED)
+       switch (pix->field) {
+       case V4L2_FIELD_NONE:
+       case V4L2_FIELD_INTERLACED:
+               break;
+       case V4L2_FIELD_ANY:
+       default:
                pix->field = V4L2_FIELD_INTERLACED;
+               break;
+       }
 
        /* Just set these */
        pix->colorspace = V4L2_COLORSPACE_SMPTE170M;
        pix->sizeimage = FRAME_BUF_SIZE;
+       pix->priv = 0;
 
        return 0;
 }
@@ -1081,6 +1080,15 @@ static int solo_enc_set_fmt_cap(struct file *file, void *priv,
        mutex_lock(&solo_enc->enable_lock);
 
        ret = solo_enc_try_fmt_cap(file, priv, f);
+       if (ret)
+               return ret;
+
+       /* We cannot change width/height in mid read */
+       if (!ret && atomic_read(&solo_enc->readers) > 0) {
+               if (pix->width != solo_enc->width ||
+                   pix->height != solo_enc->height)
+                       ret = -EBUSY;
+       }
        if (ret) {
                mutex_unlock(&solo_enc->enable_lock);
                return ret;
@@ -1116,6 +1124,7 @@ static int solo_enc_get_fmt_cap(struct file *file, void *priv,
                     V4L2_FIELD_NONE;
        pix->sizeimage = FRAME_BUF_SIZE;
        pix->colorspace = V4L2_COLORSPACE_SMPTE170M;
+       pix->priv = 0;
 
        return 0;
 }
@@ -1205,7 +1214,8 @@ static int solo_enum_framesizes(struct file *file, void *priv,
        struct solo_enc_fh *fh = priv;
        struct solo_dev *solo_dev = fh->enc->solo_dev;
 
-       if (fsize->pixel_format != V4L2_PIX_FMT_MPEG)
+       if (fsize->pixel_format != V4L2_PIX_FMT_MPEG &&
+           fsize->pixel_format != V4L2_PIX_FMT_MJPEG)
                return -EINVAL;
 
        switch (fsize->index) {
@@ -1232,16 +1242,24 @@ static int solo_enum_frameintervals(struct file *file, void *priv,
        struct solo_enc_fh *fh = priv;
        struct solo_dev *solo_dev = fh->enc->solo_dev;
 
-       if (fintv->pixel_format != V4L2_PIX_FMT_MPEG || fintv->index)
+       if (fintv->pixel_format != V4L2_PIX_FMT_MPEG &&
+           fintv->pixel_format != V4L2_PIX_FMT_MJPEG)
+               return -EINVAL;
+       if (fintv->index)
+               return -EINVAL;
+       if ((fintv->width != solo_dev->video_hsize >> 1 ||
+            fintv->height != solo_dev->video_vsize) &&
+           (fintv->width != solo_dev->video_hsize ||
+            fintv->height != solo_dev->video_vsize << 1))
                return -EINVAL;
 
        fintv->type = V4L2_FRMIVAL_TYPE_STEPWISE;
 
-       fintv->stepwise.min.numerator = solo_dev->fps;
-       fintv->stepwise.min.denominator = 1;
+       fintv->stepwise.min.denominator = solo_dev->fps;
+       fintv->stepwise.min.numerator = 15;
 
-       fintv->stepwise.max.numerator = solo_dev->fps;
-       fintv->stepwise.max.denominator = 15;
+       fintv->stepwise.max.denominator = solo_dev->fps;
+       fintv->stepwise.max.numerator = 1;
 
        fintv->stepwise.step.numerator = 1;
        fintv->stepwise.step.denominator = 1;
@@ -1298,6 +1316,7 @@ static int solo_s_parm(struct file *file, void *priv,
        solo_enc->interval = cp->timeperframe.numerator;
 
        cp->capability = V4L2_CAP_TIMEPERFRAME;
+       cp->readbuffers = 2;
 
        solo_update_mode(solo_enc);
 
index 4243bbe8a3edc6f3a9ae13597f2c8d8548b1516e..668dc48458a9cfe3a5b7ec3972ad186ae5135238 100644 (file)
@@ -34,8 +34,6 @@
 #include "solo6x10.h"
 #include "tw28.h"
 
-#define SOLO_DISP_PIX_FIELD    V4L2_FIELD_INTERLACED
-
 /* Image size is two fields, SOLO_HW_BPL is one horizontal line in hardware */
 #define SOLO_HW_BPL            2048
 #define solo_vlines(__solo)    (__solo->video_vsize * 2)
@@ -439,7 +437,7 @@ static int solo_v4l2_open(struct file *file)
        videobuf_queue_dma_contig_init(&fh->vidq, &solo_video_qops,
                                       &solo_dev->pdev->dev, &fh->slock,
                                       V4L2_BUF_TYPE_VIDEO_CAPTURE,
-                                      SOLO_DISP_PIX_FIELD,
+                                      V4L2_FIELD_INTERLACED,
                                       sizeof(struct videobuf_buffer),
                                       fh, NULL);
        return 0;
@@ -581,23 +579,16 @@ static int solo_try_fmt_cap(struct file *file, void *priv,
        struct v4l2_pix_format *pix = &f->fmt.pix;
        int image_size = solo_image_size(solo_dev);
 
-       /* Check supported sizes */
-       if (pix->width != solo_dev->video_hsize)
-               pix->width = solo_dev->video_hsize;
-       if (pix->height != solo_vlines(solo_dev))
-               pix->height = solo_vlines(solo_dev);
-       if (pix->sizeimage != image_size)
-               pix->sizeimage = image_size;
-
-       /* Check formats */
-       if (pix->field == V4L2_FIELD_ANY)
-               pix->field = SOLO_DISP_PIX_FIELD;
-
-       if (pix->pixelformat != V4L2_PIX_FMT_UYVY ||
-           pix->field       != SOLO_DISP_PIX_FIELD ||
-           pix->colorspace  != V4L2_COLORSPACE_SMPTE170M)
+       if (pix->pixelformat != V4L2_PIX_FMT_UYVY)
                return -EINVAL;
 
+       pix->width = solo_dev->video_hsize;
+       pix->height = solo_vlines(solo_dev);
+       pix->sizeimage = image_size;
+       pix->field = V4L2_FIELD_INTERLACED;
+       pix->pixelformat = V4L2_PIX_FMT_UYVY;
+       pix->colorspace = V4L2_COLORSPACE_SMPTE170M;
+       pix->priv = 0;
        return 0;
 }
 
@@ -624,10 +615,11 @@ static int solo_get_fmt_cap(struct file *file, void *priv,
        pix->width = solo_dev->video_hsize;
        pix->height = solo_vlines(solo_dev);
        pix->pixelformat = V4L2_PIX_FMT_UYVY;
-       pix->field = SOLO_DISP_PIX_FIELD;
+       pix->field = V4L2_FIELD_INTERLACED;
        pix->sizeimage = solo_image_size(solo_dev);
        pix->colorspace = V4L2_COLORSPACE_SMPTE170M;
        pix->bytesperline = solo_bytesperline(solo_dev);
+       pix->priv = 0;
 
        return 0;
 }