Merge branch 'v4l_for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media
Pull media fixes from Mauro Carvalho Chehab: - gspca: add needed delay for I2C traffic for sonixb/sonixj cameras - gspca: add one missing Kinect USB ID - usbvideo: some regression fixes - omap3isp: fix some build issues - videobuf2: fix video output handling - exynos s5p/m5mols: a few regression fixes. * 'v4l_for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media: [media] uvcvideo: Set error_idx properly for S_EXT_CTRLS failures [media] uvcvideo: Cleanup leftovers of partial revert [media] uvcvideo: Return -EACCES when trying to set a read-only control [media] omap3isp: Don't include <plat/cpu.h> [media] s5p-mfc: Fix interrupt error handling routine [media] s5p-fimc: Fix return value of __fimc_md_create_flite_source_links() [media] m5mols: Fix typo in get_fmt callback [media] v4l: vb2: Set data_offset to 0 for single-plane output buffers [media] [FOR,v3.8] omap3isp: Don't include deleted OMAP plat/ header files [media] gspca_sonixj: Add a small delay after i2c_w1 [media] gspca_sonixb: Properly wait between i2c writes [media] gspca_kinect: add Kinect for Windows USB id
This commit is contained in:
commit
2116e02a96
|
@ -556,7 +556,7 @@ static int m5mols_get_fmt(struct v4l2_subdev *sd, struct v4l2_subdev_fh *fh,
|
|||
mutex_lock(&info->lock);
|
||||
|
||||
format = __find_format(info, fh, fmt->which, info->res_type);
|
||||
if (!format)
|
||||
if (format)
|
||||
fmt->format = *format;
|
||||
else
|
||||
ret = -EINVAL;
|
||||
|
|
|
@ -35,9 +35,6 @@
|
|||
#include <linux/vmalloc.h>
|
||||
#include <media/v4l2-dev.h>
|
||||
#include <media/v4l2-ioctl.h>
|
||||
#include <plat/iommu.h>
|
||||
#include <plat/iovmm.h>
|
||||
#include <plat/omap-pm.h>
|
||||
|
||||
#include "ispvideo.h"
|
||||
#include "isp.h"
|
||||
|
|
|
@ -593,7 +593,7 @@ static int __fimc_md_create_flite_source_links(struct fimc_md *fmd)
|
|||
{
|
||||
struct media_entity *source, *sink;
|
||||
unsigned int flags = MEDIA_LNK_FL_ENABLED;
|
||||
int i, ret;
|
||||
int i, ret = 0;
|
||||
|
||||
for (i = 0; i < FIMC_LITE_MAX_DEVS; i++) {
|
||||
struct fimc_lite *fimc = fmd->fimc_lite[i];
|
||||
|
|
|
@ -412,62 +412,48 @@ leave_handle_frame:
|
|||
}
|
||||
|
||||
/* Error handling for interrupt */
|
||||
static void s5p_mfc_handle_error(struct s5p_mfc_ctx *ctx,
|
||||
unsigned int reason, unsigned int err)
|
||||
static void s5p_mfc_handle_error(struct s5p_mfc_dev *dev,
|
||||
struct s5p_mfc_ctx *ctx, unsigned int reason, unsigned int err)
|
||||
{
|
||||
struct s5p_mfc_dev *dev;
|
||||
unsigned long flags;
|
||||
|
||||
/* If no context is available then all necessary
|
||||
* processing has been done. */
|
||||
if (ctx == NULL)
|
||||
return;
|
||||
|
||||
dev = ctx->dev;
|
||||
mfc_err("Interrupt Error: %08x\n", err);
|
||||
s5p_mfc_hw_call(dev->mfc_ops, clear_int_flags, dev);
|
||||
wake_up_dev(dev, reason, err);
|
||||
|
||||
/* Error recovery is dependent on the state of context */
|
||||
switch (ctx->state) {
|
||||
case MFCINST_INIT:
|
||||
/* This error had to happen while acquireing instance */
|
||||
case MFCINST_GOT_INST:
|
||||
/* This error had to happen while parsing the header */
|
||||
case MFCINST_HEAD_PARSED:
|
||||
/* This error had to happen while setting dst buffers */
|
||||
case MFCINST_RETURN_INST:
|
||||
/* This error had to happen while releasing instance */
|
||||
clear_work_bit(ctx);
|
||||
wake_up_ctx(ctx, reason, err);
|
||||
if (test_and_clear_bit(0, &dev->hw_lock) == 0)
|
||||
BUG();
|
||||
s5p_mfc_clock_off();
|
||||
ctx->state = MFCINST_ERROR;
|
||||
break;
|
||||
case MFCINST_FINISHING:
|
||||
case MFCINST_FINISHED:
|
||||
case MFCINST_RUNNING:
|
||||
/* It is higly probable that an error occured
|
||||
* while decoding a frame */
|
||||
clear_work_bit(ctx);
|
||||
ctx->state = MFCINST_ERROR;
|
||||
/* Mark all dst buffers as having an error */
|
||||
spin_lock_irqsave(&dev->irqlock, flags);
|
||||
s5p_mfc_hw_call(dev->mfc_ops, cleanup_queue, &ctx->dst_queue,
|
||||
&ctx->vq_dst);
|
||||
/* Mark all src buffers as having an error */
|
||||
s5p_mfc_hw_call(dev->mfc_ops, cleanup_queue, &ctx->src_queue,
|
||||
&ctx->vq_src);
|
||||
spin_unlock_irqrestore(&dev->irqlock, flags);
|
||||
if (test_and_clear_bit(0, &dev->hw_lock) == 0)
|
||||
BUG();
|
||||
s5p_mfc_clock_off();
|
||||
break;
|
||||
default:
|
||||
mfc_err("Encountered an error interrupt which had not been handled\n");
|
||||
break;
|
||||
if (ctx != NULL) {
|
||||
/* Error recovery is dependent on the state of context */
|
||||
switch (ctx->state) {
|
||||
case MFCINST_RES_CHANGE_INIT:
|
||||
case MFCINST_RES_CHANGE_FLUSH:
|
||||
case MFCINST_RES_CHANGE_END:
|
||||
case MFCINST_FINISHING:
|
||||
case MFCINST_FINISHED:
|
||||
case MFCINST_RUNNING:
|
||||
/* It is higly probable that an error occured
|
||||
* while decoding a frame */
|
||||
clear_work_bit(ctx);
|
||||
ctx->state = MFCINST_ERROR;
|
||||
/* Mark all dst buffers as having an error */
|
||||
spin_lock_irqsave(&dev->irqlock, flags);
|
||||
s5p_mfc_hw_call(dev->mfc_ops, cleanup_queue,
|
||||
&ctx->dst_queue, &ctx->vq_dst);
|
||||
/* Mark all src buffers as having an error */
|
||||
s5p_mfc_hw_call(dev->mfc_ops, cleanup_queue,
|
||||
&ctx->src_queue, &ctx->vq_src);
|
||||
spin_unlock_irqrestore(&dev->irqlock, flags);
|
||||
wake_up_ctx(ctx, reason, err);
|
||||
break;
|
||||
default:
|
||||
clear_work_bit(ctx);
|
||||
ctx->state = MFCINST_ERROR;
|
||||
wake_up_ctx(ctx, reason, err);
|
||||
break;
|
||||
}
|
||||
}
|
||||
if (test_and_clear_bit(0, &dev->hw_lock) == 0)
|
||||
BUG();
|
||||
s5p_mfc_hw_call(dev->mfc_ops, clear_int_flags, dev);
|
||||
s5p_mfc_clock_off();
|
||||
wake_up_dev(dev, reason, err);
|
||||
return;
|
||||
}
|
||||
|
||||
|
@ -632,7 +618,7 @@ static irqreturn_t s5p_mfc_irq(int irq, void *priv)
|
|||
dev->warn_start)
|
||||
s5p_mfc_handle_frame(ctx, reason, err);
|
||||
else
|
||||
s5p_mfc_handle_error(ctx, reason, err);
|
||||
s5p_mfc_handle_error(dev, ctx, reason, err);
|
||||
clear_bit(0, &dev->enter_suspend);
|
||||
break;
|
||||
|
||||
|
|
|
@ -381,6 +381,7 @@ static const struct sd_desc sd_desc = {
|
|||
/* -- module initialisation -- */
|
||||
static const struct usb_device_id device_table[] = {
|
||||
{USB_DEVICE(0x045e, 0x02ae)},
|
||||
{USB_DEVICE(0x045e, 0x02bf)},
|
||||
{}
|
||||
};
|
||||
|
||||
|
|
|
@ -496,7 +496,7 @@ static void reg_w(struct gspca_dev *gspca_dev,
|
|||
}
|
||||
}
|
||||
|
||||
static void i2c_w(struct gspca_dev *gspca_dev, const __u8 *buffer)
|
||||
static void i2c_w(struct gspca_dev *gspca_dev, const u8 *buf)
|
||||
{
|
||||
int retry = 60;
|
||||
|
||||
|
@ -504,16 +504,19 @@ static void i2c_w(struct gspca_dev *gspca_dev, const __u8 *buffer)
|
|||
return;
|
||||
|
||||
/* is i2c ready */
|
||||
reg_w(gspca_dev, 0x08, buffer, 8);
|
||||
reg_w(gspca_dev, 0x08, buf, 8);
|
||||
while (retry--) {
|
||||
if (gspca_dev->usb_err < 0)
|
||||
return;
|
||||
msleep(10);
|
||||
msleep(1);
|
||||
reg_r(gspca_dev, 0x08);
|
||||
if (gspca_dev->usb_buf[0] & 0x04) {
|
||||
if (gspca_dev->usb_buf[0] & 0x08) {
|
||||
dev_err(gspca_dev->v4l2_dev.dev,
|
||||
"i2c write error\n");
|
||||
"i2c error writing %02x %02x %02x %02x"
|
||||
" %02x %02x %02x %02x\n",
|
||||
buf[0], buf[1], buf[2], buf[3],
|
||||
buf[4], buf[5], buf[6], buf[7]);
|
||||
gspca_dev->usb_err = -EIO;
|
||||
}
|
||||
return;
|
||||
|
@ -530,7 +533,7 @@ static void i2c_w_vector(struct gspca_dev *gspca_dev,
|
|||
for (;;) {
|
||||
if (gspca_dev->usb_err < 0)
|
||||
return;
|
||||
reg_w(gspca_dev, 0x08, *buffer, 8);
|
||||
i2c_w(gspca_dev, *buffer);
|
||||
len -= 8;
|
||||
if (len <= 0)
|
||||
break;
|
||||
|
|
|
@ -1550,6 +1550,7 @@ static void i2c_w1(struct gspca_dev *gspca_dev, u8 reg, u8 val)
|
|||
0,
|
||||
gspca_dev->usb_buf, 8,
|
||||
500);
|
||||
msleep(2);
|
||||
if (ret < 0) {
|
||||
pr_err("i2c_w1 err %d\n", ret);
|
||||
gspca_dev->usb_err = ret;
|
||||
|
|
|
@ -1431,8 +1431,10 @@ int uvc_ctrl_set(struct uvc_video_chain *chain,
|
|||
int ret;
|
||||
|
||||
ctrl = uvc_find_control(chain, xctrl->id, &mapping);
|
||||
if (ctrl == NULL || (ctrl->info.flags & UVC_CTRL_FLAG_SET_CUR) == 0)
|
||||
if (ctrl == NULL)
|
||||
return -EINVAL;
|
||||
if (!(ctrl->info.flags & UVC_CTRL_FLAG_SET_CUR))
|
||||
return -EACCES;
|
||||
|
||||
/* Clamp out of range values. */
|
||||
switch (mapping->v4l2_type) {
|
||||
|
|
|
@ -657,8 +657,7 @@ static long uvc_v4l2_do_ioctl(struct file *file, unsigned int cmd, void *arg)
|
|||
ret = uvc_ctrl_get(chain, ctrl);
|
||||
if (ret < 0) {
|
||||
uvc_ctrl_rollback(handle);
|
||||
ctrls->error_idx = ret == -ENOENT
|
||||
? ctrls->count : i;
|
||||
ctrls->error_idx = i;
|
||||
return ret;
|
||||
}
|
||||
}
|
||||
|
@ -686,8 +685,7 @@ static long uvc_v4l2_do_ioctl(struct file *file, unsigned int cmd, void *arg)
|
|||
ret = uvc_ctrl_set(chain, ctrl);
|
||||
if (ret < 0) {
|
||||
uvc_ctrl_rollback(handle);
|
||||
ctrls->error_idx = (ret == -ENOENT &&
|
||||
cmd == VIDIOC_S_EXT_CTRLS)
|
||||
ctrls->error_idx = cmd == VIDIOC_S_EXT_CTRLS
|
||||
? ctrls->count : i;
|
||||
return ret;
|
||||
}
|
||||
|
|
|
@ -921,8 +921,10 @@ static void __fill_vb2_buffer(struct vb2_buffer *vb, const struct v4l2_buffer *b
|
|||
* In videobuf we use our internal V4l2_planes struct for
|
||||
* single-planar buffers as well, for simplicity.
|
||||
*/
|
||||
if (V4L2_TYPE_IS_OUTPUT(b->type))
|
||||
if (V4L2_TYPE_IS_OUTPUT(b->type)) {
|
||||
v4l2_planes[0].bytesused = b->bytesused;
|
||||
v4l2_planes[0].data_offset = 0;
|
||||
}
|
||||
|
||||
if (b->memory == V4L2_MEMORY_USERPTR) {
|
||||
v4l2_planes[0].m.userptr = b->m.userptr;
|
||||
|
|
Loading…
Reference in New Issue