Hi Hugues,
On Tue, Jan 23, 2018 at 02:23:14PM +0100, Hugues Fruchet wrote:
> +static int ov5640_get_frame_desc(struct v4l2_subdev *sd, unsigned int pad,
> + struct v4l2_mbus_frame_desc *fd)
> +{
> + struct ov5640_dev *sensor = to_ov5640_dev(sd);
> +
> + if (pad != 0 || !fd)
> + return -EINVAL;
> +
> + mutex_lock(&sensor->lock);
> + fd->entry[0].length = sensor->jpeg_size;
> + mutex_unlock(&sensor->lock);
> + fd->entry[0].pixelcode = MEDIA_BUS_FMT_JPEG_1X8;
> + fd->entry[0].flags = V4L2_MBUS_FRAME_DESC_FL_LEN_MAX;
> + fd->num_entries = 1;
Missed this on the previous time --- the frame descriptor now describes the
JPEG frame _only_. This needs to work for non-JPEG formats, too.
--
Sakari Ailus
e-mail: sakari.ailus@xxxxxx
