Commit da983503 authored by Hans Verkuil's avatar Hans Verkuil Committed by Mauro Carvalho Chehab

[media] cx231xx: fix compiler warnings

v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c: In function ‘snd_cx231xx_hw_capture_params’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c:527:6: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c:526:31: warning: variable ‘format’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c:526:25: warning: variable ‘rate’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c:526:15: warning: variable ‘channels’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c: In function ‘snd_cx231xx_capture_trigger’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c:589:6: warning: variable ‘retval’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-video.c: In function ‘cx231xx_isoc_copy’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-video.c:331:17: warning: variable ‘outp’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-video.c: In function ‘cx231xx_bulk_copy’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-video.c:434:17: warning: variable ‘outp’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-video.c: In function ‘cx231xx_reset_video_buffer’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-video.c:704:7: warning: variable ‘outp’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c: In function ‘cx231xx_set_mode’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c:699:6: warning: variable ‘errCode’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c: In function ‘cx231xx_ep5_bulkout’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c:763:6: warning: variable ‘errCode’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c: In function ‘cx231xx_isoc_irq_callback’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c:800:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c: In function ‘cx231xx_bulk_irq_callback’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c:846:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c: In function ‘cx231xx_stop_TS1’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c:1234:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c: In function ‘cx231xx_start_TS1’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c:1254:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_enable656’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:937:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_disable656’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:955:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_dump_HH_reg’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:1323:5: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_dump_SC_reg’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:1358:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_Setup_AFE_for_LowIF’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:1444:5: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_set_Colibri_For_LowIF’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:1504:5: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_set_DIF_bandpass’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:1559:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_gpio_i2c_write’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:3093:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-417.c: In function ‘cx231xx_initialize_codec’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-417.c:1098:9: warning: variable ‘data’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-417.c: In function ‘vidioc_streamon’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-417.c:1795:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-vbi.c: In function ‘cx231xx_isoc_vbi_copy’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-vbi.c:86:25: warning: variable ‘buf’ set but not used [-Wunused-but-set-variable]
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-vbi.c: In function ‘cx231xx_irq_vbi_callback’:
v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-vbi.c:313:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable]
Signed-off-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent d51dbecc
...@@ -1095,7 +1095,7 @@ static int cx231xx_initialize_codec(struct cx231xx *dev) ...@@ -1095,7 +1095,7 @@ static int cx231xx_initialize_codec(struct cx231xx *dev)
{ {
int version; int version;
int retval; int retval;
u32 i, data[7]; u32 i;
u32 val = 0; u32 val = 0;
dprintk(1, "%s()\n", __func__); dprintk(1, "%s()\n", __func__);
...@@ -1154,6 +1154,11 @@ static int cx231xx_initialize_codec(struct cx231xx *dev) ...@@ -1154,6 +1154,11 @@ static int cx231xx_initialize_codec(struct cx231xx *dev)
CX231xx_CUSTOM_EXTENSION_USR_DATA, 0, 0, 0, 0, 0, 0, 0, 0, 0, CX231xx_CUSTOM_EXTENSION_USR_DATA, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0); 0, 0);
*/ */
#if 0
/* TODO */
u32 data[7];
/* Setup to capture VBI */ /* Setup to capture VBI */
data[0] = 0x0001BD00; data[0] = 0x0001BD00;
data[1] = 1; /* frames per interrupt */ data[1] = 1; /* frames per interrupt */
...@@ -1162,7 +1167,7 @@ static int cx231xx_initialize_codec(struct cx231xx *dev) ...@@ -1162,7 +1167,7 @@ static int cx231xx_initialize_codec(struct cx231xx *dev)
data[4] = 0x206080C0; /* stop codes */ data[4] = 0x206080C0; /* stop codes */
data[5] = 6; /* lines */ data[5] = 6; /* lines */
data[6] = 64; /* BPL */ data[6] = 64; /* BPL */
/*
cx231xx_api_cmd(dev, CX2341X_ENC_SET_VBI_CONFIG, 7, 0, data[0], data[1], cx231xx_api_cmd(dev, CX2341X_ENC_SET_VBI_CONFIG, 7, 0, data[0], data[1],
data[2], data[3], data[4], data[5], data[6]); data[2], data[3], data[4], data[5], data[6]);
...@@ -1175,7 +1180,7 @@ static int cx231xx_initialize_codec(struct cx231xx *dev) ...@@ -1175,7 +1180,7 @@ static int cx231xx_initialize_codec(struct cx231xx *dev)
cx231xx_api_cmd(dev, CX2341X_ENC_SET_VBI_LINE, 5, 0, cx231xx_api_cmd(dev, CX2341X_ENC_SET_VBI_LINE, 5, 0,
i | 0x80000000, valid, 0, 0, 0); i | 0x80000000, valid, 0, 0, 0);
} }
*/ #endif
/* cx231xx_api_cmd(dev, CX2341X_ENC_MUTE_AUDIO, 1, 0, CX231xx_UNMUTE); /* cx231xx_api_cmd(dev, CX2341X_ENC_MUTE_AUDIO, 1, 0, CX231xx_UNMUTE);
msleep(60); msleep(60);
*/ */
...@@ -1792,17 +1797,16 @@ static int vidioc_streamon(struct file *file, void *priv, ...@@ -1792,17 +1797,16 @@ static int vidioc_streamon(struct file *file, void *priv,
struct cx231xx_fh *fh = file->private_data; struct cx231xx_fh *fh = file->private_data;
struct cx231xx *dev = fh->dev; struct cx231xx *dev = fh->dev;
int rc = 0;
dprintk(3, "enter vidioc_streamon()\n"); dprintk(3, "enter vidioc_streamon()\n");
cx231xx_set_alt_setting(dev, INDEX_TS1, 0); cx231xx_set_alt_setting(dev, INDEX_TS1, 0);
rc = cx231xx_set_mode(dev, CX231XX_DIGITAL_MODE); cx231xx_set_mode(dev, CX231XX_DIGITAL_MODE);
if (dev->USE_ISO) if (dev->USE_ISO)
rc = cx231xx_init_isoc(dev, CX231XX_NUM_PACKETS, cx231xx_init_isoc(dev, CX231XX_NUM_PACKETS,
CX231XX_NUM_BUFS, CX231XX_NUM_BUFS,
dev->video_mode.max_pkt_size, dev->video_mode.max_pkt_size,
cx231xx_isoc_copy); cx231xx_isoc_copy);
else { else {
rc = cx231xx_init_bulk(dev, 320, cx231xx_init_bulk(dev, 320,
5, 5,
dev->ts1_mode.max_pkt_size, dev->ts1_mode.max_pkt_size,
cx231xx_bulk_copy); cx231xx_bulk_copy);
......
...@@ -523,21 +523,24 @@ static int snd_cx231xx_pcm_close(struct snd_pcm_substream *substream) ...@@ -523,21 +523,24 @@ static int snd_cx231xx_pcm_close(struct snd_pcm_substream *substream)
static int snd_cx231xx_hw_capture_params(struct snd_pcm_substream *substream, static int snd_cx231xx_hw_capture_params(struct snd_pcm_substream *substream,
struct snd_pcm_hw_params *hw_params) struct snd_pcm_hw_params *hw_params)
{ {
unsigned int channels, rate, format;
int ret; int ret;
dprintk("Setting capture parameters\n"); dprintk("Setting capture parameters\n");
ret = snd_pcm_alloc_vmalloc_buffer(substream, ret = snd_pcm_alloc_vmalloc_buffer(substream,
params_buffer_bytes(hw_params)); params_buffer_bytes(hw_params));
#if 0
/* TODO: set up cx231xx audio chip to deliver the correct audio format,
current default is 48000hz multiplexed => 96000hz mono
which shouldn't matter since analogue TV only supports mono */
unsigned int channels, rate, format;
format = params_format(hw_params); format = params_format(hw_params);
rate = params_rate(hw_params); rate = params_rate(hw_params);
channels = params_channels(hw_params); channels = params_channels(hw_params);
#endif
/* TODO: set up cx231xx audio chip to deliver the correct audio format, return ret;
current default is 48000hz multiplexed => 96000hz mono
which shouldn't matter since analogue TV only supports mono */
return 0;
} }
static int snd_cx231xx_hw_capture_free(struct snd_pcm_substream *substream) static int snd_cx231xx_hw_capture_free(struct snd_pcm_substream *substream)
...@@ -586,7 +589,7 @@ static int snd_cx231xx_capture_trigger(struct snd_pcm_substream *substream, ...@@ -586,7 +589,7 @@ static int snd_cx231xx_capture_trigger(struct snd_pcm_substream *substream,
int cmd) int cmd)
{ {
struct cx231xx *dev = snd_pcm_substream_chip(substream); struct cx231xx *dev = snd_pcm_substream_chip(substream);
int retval; int retval = 0;
if (dev->state & DEV_DISCONNECTED) if (dev->state & DEV_DISCONNECTED)
return -ENODEV; return -ENODEV;
...@@ -601,12 +604,13 @@ static int snd_cx231xx_capture_trigger(struct snd_pcm_substream *substream, ...@@ -601,12 +604,13 @@ static int snd_cx231xx_capture_trigger(struct snd_pcm_substream *substream,
break; break;
default: default:
retval = -EINVAL; retval = -EINVAL;
break;
} }
spin_unlock(&dev->adev.slock); spin_unlock(&dev->adev.slock);
schedule_work(&dev->wq_trigger); schedule_work(&dev->wq_trigger);
return 0; return retval;
} }
static snd_pcm_uframes_t snd_cx231xx_capture_pointer(struct snd_pcm_substream static snd_pcm_uframes_t snd_cx231xx_capture_pointer(struct snd_pcm_substream
......
...@@ -754,7 +754,7 @@ int cx231xx_set_mode(struct cx231xx *dev, enum cx231xx_mode set_mode) ...@@ -754,7 +754,7 @@ int cx231xx_set_mode(struct cx231xx *dev, enum cx231xx_mode set_mode)
} }
} }
return 0; return errCode ? -EINVAL : 0;
} }
EXPORT_SYMBOL_GPL(cx231xx_set_mode); EXPORT_SYMBOL_GPL(cx231xx_set_mode);
...@@ -764,7 +764,7 @@ int cx231xx_ep5_bulkout(struct cx231xx *dev, u8 *firmware, u16 size) ...@@ -764,7 +764,7 @@ int cx231xx_ep5_bulkout(struct cx231xx *dev, u8 *firmware, u16 size)
int actlen, ret = -ENOMEM; int actlen, ret = -ENOMEM;
u32 *buffer; u32 *buffer;
buffer = kzalloc(4096, GFP_KERNEL); buffer = kzalloc(4096, GFP_KERNEL);
if (buffer == NULL) { if (buffer == NULL) {
cx231xx_info("out of mem\n"); cx231xx_info("out of mem\n");
return -ENOMEM; return -ENOMEM;
...@@ -772,16 +772,16 @@ buffer = kzalloc(4096, GFP_KERNEL); ...@@ -772,16 +772,16 @@ buffer = kzalloc(4096, GFP_KERNEL);
memcpy(&buffer[0], firmware, 4096); memcpy(&buffer[0], firmware, 4096);
ret = usb_bulk_msg(dev->udev, usb_sndbulkpipe(dev->udev, 5), ret = usb_bulk_msg(dev->udev, usb_sndbulkpipe(dev->udev, 5),
buffer, 4096, &actlen, 2000); buffer, 4096, &actlen, 2000);
if (ret) if (ret)
cx231xx_info("bulk message failed: %d (%d/%d)", ret, cx231xx_info("bulk message failed: %d (%d/%d)", ret,
size, actlen); size, actlen);
else { else {
errCode = actlen != size ? -1 : 0; errCode = actlen != size ? -1 : 0;
} }
kfree(buffer); kfree(buffer);
return 0; return errCode;
} }
/***************************************************************** /*****************************************************************
...@@ -797,7 +797,7 @@ static void cx231xx_isoc_irq_callback(struct urb *urb) ...@@ -797,7 +797,7 @@ static void cx231xx_isoc_irq_callback(struct urb *urb)
struct cx231xx_video_mode *vmode = struct cx231xx_video_mode *vmode =
container_of(dma_q, struct cx231xx_video_mode, vidq); container_of(dma_q, struct cx231xx_video_mode, vidq);
struct cx231xx *dev = container_of(vmode, struct cx231xx, video_mode); struct cx231xx *dev = container_of(vmode, struct cx231xx, video_mode);
int rc, i; int i;
switch (urb->status) { switch (urb->status) {
case 0: /* success */ case 0: /* success */
...@@ -814,7 +814,7 @@ static void cx231xx_isoc_irq_callback(struct urb *urb) ...@@ -814,7 +814,7 @@ static void cx231xx_isoc_irq_callback(struct urb *urb)
/* Copy data from URB */ /* Copy data from URB */
spin_lock(&dev->video_mode.slock); spin_lock(&dev->video_mode.slock);
rc = dev->video_mode.isoc_ctl.isoc_copy(dev, urb); dev->video_mode.isoc_ctl.isoc_copy(dev, urb);
spin_unlock(&dev->video_mode.slock); spin_unlock(&dev->video_mode.slock);
/* Reset urb buffers */ /* Reset urb buffers */
...@@ -822,7 +822,6 @@ static void cx231xx_isoc_irq_callback(struct urb *urb) ...@@ -822,7 +822,6 @@ static void cx231xx_isoc_irq_callback(struct urb *urb)
urb->iso_frame_desc[i].status = 0; urb->iso_frame_desc[i].status = 0;
urb->iso_frame_desc[i].actual_length = 0; urb->iso_frame_desc[i].actual_length = 0;
} }
urb->status = 0;
urb->status = usb_submit_urb(urb, GFP_ATOMIC); urb->status = usb_submit_urb(urb, GFP_ATOMIC);
if (urb->status) { if (urb->status) {
...@@ -843,7 +842,6 @@ static void cx231xx_bulk_irq_callback(struct urb *urb) ...@@ -843,7 +842,6 @@ static void cx231xx_bulk_irq_callback(struct urb *urb)
struct cx231xx_video_mode *vmode = struct cx231xx_video_mode *vmode =
container_of(dma_q, struct cx231xx_video_mode, vidq); container_of(dma_q, struct cx231xx_video_mode, vidq);
struct cx231xx *dev = container_of(vmode, struct cx231xx, video_mode); struct cx231xx *dev = container_of(vmode, struct cx231xx, video_mode);
int rc;
switch (urb->status) { switch (urb->status) {
case 0: /* success */ case 0: /* success */
...@@ -860,12 +858,10 @@ static void cx231xx_bulk_irq_callback(struct urb *urb) ...@@ -860,12 +858,10 @@ static void cx231xx_bulk_irq_callback(struct urb *urb)
/* Copy data from URB */ /* Copy data from URB */
spin_lock(&dev->video_mode.slock); spin_lock(&dev->video_mode.slock);
rc = dev->video_mode.bulk_ctl.bulk_copy(dev, urb); dev->video_mode.bulk_ctl.bulk_copy(dev, urb);
spin_unlock(&dev->video_mode.slock); spin_unlock(&dev->video_mode.slock);
/* Reset urb buffers */ /* Reset urb buffers */
urb->status = 0;
urb->status = usb_submit_urb(urb, GFP_ATOMIC); urb->status = usb_submit_urb(urb, GFP_ATOMIC);
if (urb->status) { if (urb->status) {
cx231xx_isocdbg("urb resubmit failed (error=%i)\n", cx231xx_isocdbg("urb resubmit failed (error=%i)\n",
...@@ -1231,42 +1227,40 @@ int cx231xx_init_bulk(struct cx231xx *dev, int max_packets, ...@@ -1231,42 +1227,40 @@ int cx231xx_init_bulk(struct cx231xx *dev, int max_packets,
EXPORT_SYMBOL_GPL(cx231xx_init_bulk); EXPORT_SYMBOL_GPL(cx231xx_init_bulk);
void cx231xx_stop_TS1(struct cx231xx *dev) void cx231xx_stop_TS1(struct cx231xx *dev)
{ {
int status = 0;
u8 val[4] = { 0, 0, 0, 0 }; u8 val[4] = { 0, 0, 0, 0 };
val[0] = 0x00; val[0] = 0x00;
val[1] = 0x03; val[1] = 0x03;
val[2] = 0x00; val[2] = 0x00;
val[3] = 0x00; val[3] = 0x00;
status = cx231xx_write_ctrl_reg(dev, VRT_SET_REGISTER, cx231xx_write_ctrl_reg(dev, VRT_SET_REGISTER,
TS_MODE_REG, val, 4); TS_MODE_REG, val, 4);
val[0] = 0x00; val[0] = 0x00;
val[1] = 0x70; val[1] = 0x70;
val[2] = 0x04; val[2] = 0x04;
val[3] = 0x00; val[3] = 0x00;
status = cx231xx_write_ctrl_reg(dev, VRT_SET_REGISTER, cx231xx_write_ctrl_reg(dev, VRT_SET_REGISTER,
TS1_CFG_REG, val, 4); TS1_CFG_REG, val, 4);
} }
/* EXPORT_SYMBOL_GPL(cx231xx_stop_TS1); */ /* EXPORT_SYMBOL_GPL(cx231xx_stop_TS1); */
void cx231xx_start_TS1(struct cx231xx *dev) void cx231xx_start_TS1(struct cx231xx *dev)
{ {
int status = 0;
u8 val[4] = { 0, 0, 0, 0 }; u8 val[4] = { 0, 0, 0, 0 };
val[0] = 0x03; val[0] = 0x03;
val[1] = 0x03; val[1] = 0x03;
val[2] = 0x00; val[2] = 0x00;
val[3] = 0x00; val[3] = 0x00;
status = cx231xx_write_ctrl_reg(dev, VRT_SET_REGISTER, cx231xx_write_ctrl_reg(dev, VRT_SET_REGISTER,
TS_MODE_REG, val, 4); TS_MODE_REG, val, 4);
val[0] = 0x04; val[0] = 0x04;
val[1] = 0xA3; val[1] = 0xA3;
val[2] = 0x3B; val[2] = 0x3B;
val[3] = 0x00; val[3] = 0x00;
status = cx231xx_write_ctrl_reg(dev, VRT_SET_REGISTER, cx231xx_write_ctrl_reg(dev, VRT_SET_REGISTER,
TS1_CFG_REG, val, 4); TS1_CFG_REG, val, 4);
} }
/* EXPORT_SYMBOL_GPL(cx231xx_start_TS1); */ /* EXPORT_SYMBOL_GPL(cx231xx_start_TS1); */
/***************************************************************** /*****************************************************************
......
...@@ -83,7 +83,6 @@ static inline void print_err_status(struct cx231xx *dev, int packet, int status) ...@@ -83,7 +83,6 @@ static inline void print_err_status(struct cx231xx *dev, int packet, int status)
*/ */
static inline int cx231xx_isoc_vbi_copy(struct cx231xx *dev, struct urb *urb) static inline int cx231xx_isoc_vbi_copy(struct cx231xx *dev, struct urb *urb)
{ {
struct cx231xx_buffer *buf;
struct cx231xx_dmaqueue *dma_q = urb->context; struct cx231xx_dmaqueue *dma_q = urb->context;
int rc = 1; int rc = 1;
unsigned char *p_buffer; unsigned char *p_buffer;
...@@ -102,8 +101,6 @@ static inline int cx231xx_isoc_vbi_copy(struct cx231xx *dev, struct urb *urb) ...@@ -102,8 +101,6 @@ static inline int cx231xx_isoc_vbi_copy(struct cx231xx *dev, struct urb *urb)
return 0; return 0;
} }
buf = dev->vbi_mode.bulk_ctl.buf;
/* get buffer pointer and length */ /* get buffer pointer and length */
p_buffer = urb->transfer_buffer; p_buffer = urb->transfer_buffer;
buffer_size = urb->actual_length; buffer_size = urb->actual_length;
...@@ -310,7 +307,6 @@ static void cx231xx_irq_vbi_callback(struct urb *urb) ...@@ -310,7 +307,6 @@ static void cx231xx_irq_vbi_callback(struct urb *urb)
struct cx231xx_video_mode *vmode = struct cx231xx_video_mode *vmode =
container_of(dma_q, struct cx231xx_video_mode, vidq); container_of(dma_q, struct cx231xx_video_mode, vidq);
struct cx231xx *dev = container_of(vmode, struct cx231xx, vbi_mode); struct cx231xx *dev = container_of(vmode, struct cx231xx, vbi_mode);
int rc;
switch (urb->status) { switch (urb->status) {
case 0: /* success */ case 0: /* success */
...@@ -328,7 +324,7 @@ static void cx231xx_irq_vbi_callback(struct urb *urb) ...@@ -328,7 +324,7 @@ static void cx231xx_irq_vbi_callback(struct urb *urb)
/* Copy data from URB */ /* Copy data from URB */
spin_lock(&dev->vbi_mode.slock); spin_lock(&dev->vbi_mode.slock);
rc = dev->vbi_mode.bulk_ctl.bulk_copy(dev, urb); dev->vbi_mode.bulk_ctl.bulk_copy(dev, urb);
spin_unlock(&dev->vbi_mode.slock); spin_unlock(&dev->vbi_mode.slock);
/* Reset status */ /* Reset status */
......
...@@ -326,9 +326,7 @@ static inline void get_next_buf(struct cx231xx_dmaqueue *dma_q, ...@@ -326,9 +326,7 @@ static inline void get_next_buf(struct cx231xx_dmaqueue *dma_q,
*/ */
static inline int cx231xx_isoc_copy(struct cx231xx *dev, struct urb *urb) static inline int cx231xx_isoc_copy(struct cx231xx *dev, struct urb *urb)
{ {
struct cx231xx_buffer *buf;
struct cx231xx_dmaqueue *dma_q = urb->context; struct cx231xx_dmaqueue *dma_q = urb->context;
unsigned char *outp = NULL;
int i, rc = 1; int i, rc = 1;
unsigned char *p_buffer; unsigned char *p_buffer;
u32 bytes_parsed = 0, buffer_size = 0; u32 bytes_parsed = 0, buffer_size = 0;
...@@ -346,10 +344,6 @@ static inline int cx231xx_isoc_copy(struct cx231xx *dev, struct urb *urb) ...@@ -346,10 +344,6 @@ static inline int cx231xx_isoc_copy(struct cx231xx *dev, struct urb *urb)
return 0; return 0;
} }
buf = dev->video_mode.isoc_ctl.buf;
if (buf != NULL)
outp = videobuf_to_vmalloc(&buf->vb);
for (i = 0; i < urb->number_of_packets; i++) { for (i = 0; i < urb->number_of_packets; i++) {
int status = urb->iso_frame_desc[i].status; int status = urb->iso_frame_desc[i].status;
...@@ -429,9 +423,7 @@ static inline int cx231xx_isoc_copy(struct cx231xx *dev, struct urb *urb) ...@@ -429,9 +423,7 @@ static inline int cx231xx_isoc_copy(struct cx231xx *dev, struct urb *urb)
static inline int cx231xx_bulk_copy(struct cx231xx *dev, struct urb *urb) static inline int cx231xx_bulk_copy(struct cx231xx *dev, struct urb *urb)
{ {
struct cx231xx_buffer *buf;
struct cx231xx_dmaqueue *dma_q = urb->context; struct cx231xx_dmaqueue *dma_q = urb->context;
unsigned char *outp = NULL;
int rc = 1; int rc = 1;
unsigned char *p_buffer; unsigned char *p_buffer;
u32 bytes_parsed = 0, buffer_size = 0; u32 bytes_parsed = 0, buffer_size = 0;
...@@ -449,10 +441,6 @@ static inline int cx231xx_bulk_copy(struct cx231xx *dev, struct urb *urb) ...@@ -449,10 +441,6 @@ static inline int cx231xx_bulk_copy(struct cx231xx *dev, struct urb *urb)
return 0; return 0;
} }
buf = dev->video_mode.bulk_ctl.buf;
if (buf != NULL)
outp = videobuf_to_vmalloc(&buf->vb);
if (1) { if (1) {
/* get buffer pointer and length */ /* get buffer pointer and length */
...@@ -701,13 +689,9 @@ void cx231xx_reset_video_buffer(struct cx231xx *dev, ...@@ -701,13 +689,9 @@ void cx231xx_reset_video_buffer(struct cx231xx *dev,
buf = dev->video_mode.bulk_ctl.buf; buf = dev->video_mode.bulk_ctl.buf;
if (buf == NULL) { if (buf == NULL) {
u8 *outp = NULL;
/* first try to get the buffer */ /* first try to get the buffer */
get_next_buf(dma_q, &buf); get_next_buf(dma_q, &buf);
if (buf)
outp = videobuf_to_vmalloc(&buf->vb);
dma_q->pos = 0; dma_q->pos = 0;
dma_q->field1_done = 0; dma_q->field1_done = 0;
dma_q->current_field = -1; dma_q->current_field = -1;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment