Commit e9ad78bc authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

media: cxusb-analog: Fix some coding style issues

This is a new file, so the best moment to make it to follow
Kernel coding style is now.

This patch was partially generated with:

	./scripts/checkpatch.pl --fix-inplace --strict -f drivers/media/usb/dvb-usb/cxusb-analog.c

And manually checked and adjusted to avoid any warnings.
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
parent 5a433fcc
...@@ -204,7 +204,7 @@ static bool cxusb_medion_cf_refc_fld_chg(struct dvb_usb_device *dvbdev, ...@@ -204,7 +204,7 @@ static bool cxusb_medion_cf_refc_fld_chg(struct dvb_usb_device *dvbdev,
"field %c after line %u field change\n", "field %c after line %u field change\n",
firstfield ? '1' : '2', bt656->line); firstfield ? '1' : '2', bt656->line);
if (bt656->buf != NULL && remsamples > 0) { if (bt656->buf && remsamples > 0) {
memset(bt656->buf, 0, remsamples); memset(bt656->buf, 0, remsamples);
bt656->buf += remsamples; bt656->buf += remsamples;
...@@ -219,7 +219,7 @@ static bool cxusb_medion_cf_refc_fld_chg(struct dvb_usb_device *dvbdev, ...@@ -219,7 +219,7 @@ static bool cxusb_medion_cf_refc_fld_chg(struct dvb_usb_device *dvbdev,
} }
remlines = maxlines - bt656->line; remlines = maxlines - bt656->line;
if (bt656->buf != NULL && remlines > 0) { if (bt656->buf && remlines > 0) {
memset(bt656->buf, 0, remlines * maxlinesamples); memset(bt656->buf, 0, remlines * maxlinesamples);
bt656->buf += remlines * maxlinesamples; bt656->buf += remlines * maxlinesamples;
...@@ -277,7 +277,7 @@ static void cxusb_medion_cf_refc_line_smpl(struct dvb_usb_device *dvbdev, ...@@ -277,7 +277,7 @@ static void cxusb_medion_cf_refc_line_smpl(struct dvb_usb_device *dvbdev,
bt656->line, bt656->pos); bt656->line, bt656->pos);
remsamples = maxlinesamples - bt656->linesamples; remsamples = maxlinesamples - bt656->linesamples;
if (bt656->buf != NULL && remsamples > 0) { if (bt656->buf && remsamples > 0) {
memset(bt656->buf, 0, remsamples); memset(bt656->buf, 0, remsamples);
bt656->buf += remsamples; bt656->buf += remsamples;
...@@ -313,9 +313,9 @@ static bool cxusb_medion_cf_ref_code(struct dvb_usb_device *dvbdev, ...@@ -313,9 +313,9 @@ static bool cxusb_medion_cf_ref_code(struct dvb_usb_device *dvbdev,
unsigned int maxlinesamples, unsigned int maxlinesamples,
unsigned char buf[4]) unsigned char buf[4])
{ {
if (bt656->fmode == START_SEARCH) if (bt656->fmode == START_SEARCH) {
cxusb_medion_cf_refc_start_sch(dvbdev, bt656, firstfield, buf); cxusb_medion_cf_refc_start_sch(dvbdev, bt656, firstfield, buf);
else if (bt656->fmode == LINE_SAMPLES) { } else if (bt656->fmode == LINE_SAMPLES) {
cxusb_medion_cf_refc_line_smpl(dvbdev, bt656, firstfield, cxusb_medion_cf_refc_line_smpl(dvbdev, bt656, firstfield,
maxlinesamples, buf); maxlinesamples, buf);
return false; return false;
...@@ -359,7 +359,7 @@ static void cxusb_medion_cs_line_smpl(struct cxusb_bt656_params *bt656, ...@@ -359,7 +359,7 @@ static void cxusb_medion_cs_line_smpl(struct cxusb_bt656_params *bt656,
unsigned int maxlinesamples, unsigned int maxlinesamples,
unsigned char val) unsigned char val)
{ {
if (bt656->buf != NULL) if (bt656->buf)
*(bt656->buf++) = val; *(bt656->buf++) = val;
bt656->linesamples++; bt656->linesamples++;
...@@ -505,9 +505,10 @@ static bool cxusb_medion_v_process_auxbuf(struct cxusb_medion_dev *cxdev, ...@@ -505,9 +505,10 @@ static bool cxusb_medion_v_process_auxbuf(struct cxusb_medion_dev *cxdev,
struct cxusb_medion_vbuffer, struct cxusb_medion_vbuffer,
list); list);
list_del(&cxdev->vbuf->list); list_del(&cxdev->vbuf->list);
} else } else {
dev_warn(&dvbdev->udev->dev, "no free buffers\n"); dev_warn(&dvbdev->udev->dev, "no free buffers\n");
} }
}
if (bt656->mode == NEW_FRAME || reset) { if (bt656->mode == NEW_FRAME || reset) {
cxusb_vprintk(dvbdev, URB, "will copy field 1\n"); cxusb_vprintk(dvbdev, URB, "will copy field 1\n");
...@@ -516,7 +517,7 @@ static bool cxusb_medion_v_process_auxbuf(struct cxusb_medion_dev *cxdev, ...@@ -516,7 +517,7 @@ static bool cxusb_medion_v_process_auxbuf(struct cxusb_medion_dev *cxdev,
bt656->fmode = START_SEARCH; bt656->fmode = START_SEARCH;
bt656->line = 0; bt656->line = 0;
if (cxdev->vbuf != NULL) { if (cxdev->vbuf) {
cxdev->vbuf->vb2.vb2_buf.timestamp = ktime_get_ns(); cxdev->vbuf->vb2.vb2_buf.timestamp = ktime_get_ns();
bt656->buf = vb2_plane_vaddr(&cxdev->vbuf->vb2.vb2_buf, bt656->buf = vb2_plane_vaddr(&cxdev->vbuf->vb2.vb2_buf,
0); 0);
...@@ -550,7 +551,7 @@ static bool cxusb_medion_v_process_auxbuf(struct cxusb_medion_dev *cxdev, ...@@ -550,7 +551,7 @@ static bool cxusb_medion_v_process_auxbuf(struct cxusb_medion_dev *cxdev,
bt656->mode = NEW_FRAME; bt656->mode = NEW_FRAME;
if (cxdev->vbuf != NULL) { if (cxdev->vbuf) {
vb2_set_plane_payload(&cxdev->vbuf->vb2.vb2_buf, 0, vb2_set_plane_payload(&cxdev->vbuf->vb2.vb2_buf, 0,
cxdev->width * cxdev->height * 2); cxdev->width * cxdev->height * 2);
...@@ -593,7 +594,7 @@ static bool cxusb_medion_v_complete_handle_urb(struct cxusb_medion_dev *cxdev, ...@@ -593,7 +594,7 @@ static bool cxusb_medion_v_complete_handle_urb(struct cxusb_medion_dev *cxdev,
cxdev->nexturb++; cxdev->nexturb++;
cxdev->nexturb %= CXUSB_VIDEO_URBS; cxdev->nexturb %= CXUSB_VIDEO_URBS;
urb = cxdev->streamurbs[cxdev->nexturb]; urb = cxdev->streamurbs[cxdev->nexturb];
} while (urb == NULL); } while (!urb);
urb = cxdev->streamurbs[urbn]; urb = cxdev->streamurbs[urbn];
cxusb_vprintk(dvbdev, URB, "URB %u status = %d\n", urbn, urb->status); cxusb_vprintk(dvbdev, URB, "URB %u status = %d\n", urbn, urb->status);
...@@ -609,9 +610,9 @@ static bool cxusb_medion_v_complete_handle_urb(struct cxusb_medion_dev *cxdev, ...@@ -609,9 +610,9 @@ static bool cxusb_medion_v_complete_handle_urb(struct cxusb_medion_dev *cxdev,
len); len);
if (len > 0) { if (len > 0) {
if (cxdev->raw_mode) if (cxdev->raw_mode) {
cxusb_medion_v_process_urb_raw(cxdev, urb); cxusb_medion_v_process_urb_raw(cxdev, urb);
else { } else {
cxusb_vprintk(dvbdev, URB, "appending URB\n"); cxusb_vprintk(dvbdev, URB, "appending URB\n");
/* /*
...@@ -704,7 +705,7 @@ static void cxusb_medion_urbs_free(struct cxusb_medion_dev *cxdev) ...@@ -704,7 +705,7 @@ static void cxusb_medion_urbs_free(struct cxusb_medion_dev *cxdev)
unsigned int i; unsigned int i;
for (i = 0; i < CXUSB_VIDEO_URBS; i++) for (i = 0; i < CXUSB_VIDEO_URBS; i++)
if (cxdev->streamurbs[i] != NULL) { if (cxdev->streamurbs[i]) {
kfree(cxdev->streamurbs[i]->transfer_buffer); kfree(cxdev->streamurbs[i]->transfer_buffer);
usb_free_urb(cxdev->streamurbs[i]); usb_free_urb(cxdev->streamurbs[i]);
cxdev->streamurbs[i] = NULL; cxdev->streamurbs[i] = NULL;
...@@ -724,7 +725,7 @@ static void cxusb_medion_return_buffers(struct cxusb_medion_dev *cxdev, ...@@ -724,7 +725,7 @@ static void cxusb_medion_return_buffers(struct cxusb_medion_dev *cxdev,
VB2_BUF_STATE_ERROR); VB2_BUF_STATE_ERROR);
} }
if (cxdev->vbuf != NULL) { if (cxdev->vbuf) {
vb2_buffer_done(&cxdev->vbuf->vb2.vb2_buf, vb2_buffer_done(&cxdev->vbuf->vb2.vb2_buf,
requeue ? VB2_BUF_STATE_QUEUED : requeue ? VB2_BUF_STATE_QUEUED :
VB2_BUF_STATE_ERROR); VB2_BUF_STATE_ERROR);
...@@ -763,7 +764,7 @@ static int cxusb_medion_v_ss_auxbuf_alloc(struct cxusb_medion_dev *cxdev, ...@@ -763,7 +764,7 @@ static int cxusb_medion_v_ss_auxbuf_alloc(struct cxusb_medion_dev *cxdev,
auxbuflen = framelen + urblen; auxbuflen = framelen + urblen;
buf = vmalloc(auxbuflen); buf = vmalloc(auxbuflen);
if (buf == NULL) if (!buf)
return -ENOMEM; return -ENOMEM;
cxusb_auxbuf_init(dvbdev, &cxdev->auxbuf, buf, auxbuflen); cxusb_auxbuf_init(dvbdev, &cxdev->auxbuf, buf, auxbuflen);
...@@ -804,11 +805,11 @@ static u32 cxusb_medion_field_order(struct cxusb_medion_dev *cxdev) ...@@ -804,11 +805,11 @@ static u32 cxusb_medion_field_order(struct cxusb_medion_dev *cxdev)
return field; return field;
ret = v4l2_subdev_call(cxdev->cx25840, video, g_std, &norm); ret = v4l2_subdev_call(cxdev->cx25840, video, g_std, &norm);
if (ret != 0) if (ret != 0) {
cxusb_vprintk(dvbdev, OPS, cxusb_vprintk(dvbdev, OPS,
"cannot get current standard for input %u\n", "cannot get current standard for input %u\n",
(unsigned int)cxdev->input); (unsigned int)cxdev->input);
else { } else {
field = cxusb_medion_norm2field_order(norm); field = cxusb_medion_norm2field_order(norm);
if (field != V4L2_FIELD_NONE) if (field != V4L2_FIELD_NONE)
return field; return field;
...@@ -853,9 +854,9 @@ static int cxusb_medion_v_start_streaming(struct vb2_queue *q, ...@@ -853,9 +854,9 @@ static int cxusb_medion_v_start_streaming(struct vb2_queue *q,
goto ret_unstream_cx; goto ret_unstream_cx;
} }
if (cxdev->raw_mode) if (cxdev->raw_mode) {
npackets = CXUSB_VIDEO_MAX_FRAME_PKTS; npackets = CXUSB_VIDEO_MAX_FRAME_PKTS;
else { } else {
ret = cxusb_medion_v_ss_auxbuf_alloc(cxdev, &npackets); ret = cxusb_medion_v_ss_auxbuf_alloc(cxdev, &npackets);
if (ret != 0) if (ret != 0)
goto ret_unstream_md; goto ret_unstream_md;
...@@ -873,16 +874,16 @@ static int cxusb_medion_v_start_streaming(struct vb2_queue *q, ...@@ -873,16 +874,16 @@ static int cxusb_medion_v_start_streaming(struct vb2_queue *q,
*/ */
streambuf = kmalloc(npackets * CXUSB_VIDEO_PKT_SIZE, streambuf = kmalloc(npackets * CXUSB_VIDEO_PKT_SIZE,
GFP_KERNEL); GFP_KERNEL);
if (streambuf == NULL) { if (!streambuf) {
if (i < 2) { if (i < 2) {
ret = -ENOMEM; ret = -ENOMEM;
goto ret_freeab; goto ret_freeab;
} else }
break; break;
} }
surb = usb_alloc_urb(npackets, GFP_KERNEL); surb = usb_alloc_urb(npackets, GFP_KERNEL);
if (surb == NULL) { if (!surb) {
kfree(streambuf); kfree(streambuf);
ret = -ENOMEM; ret = -ENOMEM;
goto ret_freeu; goto ret_freeu;
...@@ -922,7 +923,7 @@ static int cxusb_medion_v_start_streaming(struct vb2_queue *q, ...@@ -922,7 +923,7 @@ static int cxusb_medion_v_start_streaming(struct vb2_queue *q,
} }
for (i = 0; i < CXUSB_VIDEO_URBS; i++) for (i = 0; i < CXUSB_VIDEO_URBS; i++)
if (cxdev->streamurbs[i] != NULL) { if (cxdev->streamurbs[i]) {
ret = usb_submit_urb(cxdev->streamurbs[i], ret = usb_submit_urb(cxdev->streamurbs[i],
GFP_KERNEL); GFP_KERNEL);
if (ret != 0) if (ret != 0)
...@@ -977,7 +978,7 @@ static void cxusb_medion_v_stop_streaming(struct vb2_queue *q) ...@@ -977,7 +978,7 @@ static void cxusb_medion_v_stop_streaming(struct vb2_queue *q)
mutex_unlock(cxdev->videodev->lock); mutex_unlock(cxdev->videodev->lock);
for (i = 0; i < CXUSB_VIDEO_URBS; i++) for (i = 0; i < CXUSB_VIDEO_URBS; i++)
if (cxdev->streamurbs[i] != NULL) if (cxdev->streamurbs[i])
usb_kill_urb(cxdev->streamurbs[i]); usb_kill_urb(cxdev->streamurbs[i]);
flush_work(&cxdev->urbwork); flush_work(&cxdev->urbwork);
...@@ -1771,7 +1772,7 @@ static int cxusb_medion_register_analog_video(struct dvb_usb_device *dvbdev) ...@@ -1771,7 +1772,7 @@ static int cxusb_medion_register_analog_video(struct dvb_usb_device *dvbdev)
} }
cxdev->videodev = video_device_alloc(); cxdev->videodev = video_device_alloc();
if (cxdev->videodev == NULL) { if (!cxdev->videodev) {
dev_err(&dvbdev->udev->dev, "video device alloc failed\n"); dev_err(&dvbdev->udev->dev, "video device alloc failed\n");
ret = -ENOMEM; ret = -ENOMEM;
goto ret_qrelease; goto ret_qrelease;
...@@ -1813,7 +1814,7 @@ static int cxusb_medion_register_analog_radio(struct dvb_usb_device *dvbdev) ...@@ -1813,7 +1814,7 @@ static int cxusb_medion_register_analog_radio(struct dvb_usb_device *dvbdev)
int ret; int ret;
cxdev->radiodev = video_device_alloc(); cxdev->radiodev = video_device_alloc();
if (cxdev->radiodev == NULL) { if (!cxdev->radiodev) {
dev_err(&dvbdev->udev->dev, "radio device alloc failed\n"); dev_err(&dvbdev->udev->dev, "radio device alloc failed\n");
return -ENOMEM; return -ENOMEM;
} }
...@@ -1849,7 +1850,7 @@ static int cxusb_medion_register_analog_subdevs(struct dvb_usb_device *dvbdev) ...@@ -1849,7 +1850,7 @@ static int cxusb_medion_register_analog_subdevs(struct dvb_usb_device *dvbdev)
cxdev->cx25840 = v4l2_i2c_new_subdev(&cxdev->v4l2dev, cxdev->cx25840 = v4l2_i2c_new_subdev(&cxdev->v4l2dev,
&dvbdev->i2c_adap, &dvbdev->i2c_adap,
"cx25840", 0x44, NULL); "cx25840", 0x44, NULL);
if (cxdev->cx25840 == NULL) { if (!cxdev->cx25840) {
dev_err(&dvbdev->udev->dev, "cx25840 not found\n"); dev_err(&dvbdev->udev->dev, "cx25840 not found\n");
return -ENODEV; return -ENODEV;
} }
...@@ -1882,7 +1883,7 @@ static int cxusb_medion_register_analog_subdevs(struct dvb_usb_device *dvbdev) ...@@ -1882,7 +1883,7 @@ static int cxusb_medion_register_analog_subdevs(struct dvb_usb_device *dvbdev)
cxdev->tuner = v4l2_i2c_new_subdev(&cxdev->v4l2dev, cxdev->tuner = v4l2_i2c_new_subdev(&cxdev->v4l2dev,
&dvbdev->i2c_adap, &dvbdev->i2c_adap,
"tuner", 0x61, NULL); "tuner", 0x61, NULL);
if (cxdev->tuner == NULL) { if (!cxdev->tuner) {
dev_err(&dvbdev->udev->dev, "tuner not found\n"); dev_err(&dvbdev->udev->dev, "tuner not found\n");
return -ENODEV; return -ENODEV;
} }
...@@ -1898,7 +1899,7 @@ static int cxusb_medion_register_analog_subdevs(struct dvb_usb_device *dvbdev) ...@@ -1898,7 +1899,7 @@ static int cxusb_medion_register_analog_subdevs(struct dvb_usb_device *dvbdev)
cxdev->tda9887 = v4l2_i2c_new_subdev(&cxdev->v4l2dev, cxdev->tda9887 = v4l2_i2c_new_subdev(&cxdev->v4l2dev,
&dvbdev->i2c_adap, &dvbdev->i2c_adap,
"tuner", 0x43, NULL); "tuner", 0x43, NULL);
if (cxdev->tda9887 == NULL) { if (!cxdev->tda9887) {
dev_err(&dvbdev->udev->dev, "tda9887 not found\n"); dev_err(&dvbdev->udev->dev, "tda9887 not found\n");
return -ENODEV; return -ENODEV;
} }
......
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