Commit a6a3a17b authored by Harvey Harrison's avatar Harvey Harrison Committed by Linus Torvalds

media: fix integer as NULL pointer warnings

drivers/media/video/v4l2-common.c:719:16: warning: Using plain integer as NULL pointer
drivers/media/video/au0828/au0828-dvb.c:122:19: warning: Using plain integer as NULL pointer
drivers/media/video/ivtv/ivtv-yuv.c:1101:22: warning: Using plain integer as NULL pointer
drivers/media/video/ivtv/ivtv-yuv.c:1102:23: warning: Using plain integer as NULL pointer
drivers/media/video/pvrusb2/pvrusb2-audio.c:78:39: warning: Using plain integer as NULL pointer
drivers/media/video/pvrusb2/pvrusb2-video-v4l.c:84:39: warning: Using plain integer as NULL pointer
drivers/media/video/pvrusb2/pvrusb2-v4l2.c:1264:9: warning: Using plain integer as NULL pointer
drivers/media/video/pvrusb2/pvrusb2-context.c:197:28: warning: Using plain integer as NULL pointer
drivers/media/video/pvrusb2/pvrusb2-cx2584x-v4l.c:126:39: warning: Using plain integer as NULL pointer
drivers/media/video/pvrusb2/pvrusb2-dvb.c:133:32: warning: Using plain integer as NULL pointer
drivers/media/video/pvrusb2/pvrusb2-dvb.c:145:31: warning: Using plain integer as NULL pointer
drivers/media/video/pvrusb2/pvrusb2-dvb.c:177:55: warning: Using plain integer as NULL pointer
drivers/media/video/videobuf-core.c:100:9: warning: Using plain integer as NULL pointer
Signed-off-by: default avatarHarvey Harrison <harvey.harrison@gmail.com>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 172c122d
...@@ -119,7 +119,7 @@ static int start_urb_transfer(struct au0828_dev *dev) ...@@ -119,7 +119,7 @@ static int start_urb_transfer(struct au0828_dev *dev)
purb->transfer_buffer = kzalloc(URB_BUFSIZE, GFP_KERNEL); purb->transfer_buffer = kzalloc(URB_BUFSIZE, GFP_KERNEL);
if (!purb->transfer_buffer) { if (!purb->transfer_buffer) {
usb_free_urb(purb); usb_free_urb(purb);
dev->urbs[i] = 0; dev->urbs[i] = NULL;
goto err; goto err;
} }
......
...@@ -1098,8 +1098,8 @@ void ivtv_yuv_setup_stream_frame(struct ivtv *itv) ...@@ -1098,8 +1098,8 @@ void ivtv_yuv_setup_stream_frame(struct ivtv *itv)
ivtv_yuv_next_free(itv); ivtv_yuv_next_free(itv);
/* Copy V4L2 parameters to an ivtv_dma_frame struct... */ /* Copy V4L2 parameters to an ivtv_dma_frame struct... */
dma_args.y_source = 0L; dma_args.y_source = NULL;
dma_args.uv_source = 0L; dma_args.uv_source = NULL;
dma_args.src.left = 0; dma_args.src.left = 0;
dma_args.src.top = 0; dma_args.src.top = 0;
dma_args.src.width = yi->v4l2_src_w; dma_args.src.width = yi->v4l2_src_w;
......
...@@ -75,7 +75,7 @@ static void set_stereo(struct pvr2_msp3400_handler *ctxt) ...@@ -75,7 +75,7 @@ static void set_stereo(struct pvr2_msp3400_handler *ctxt)
pvr2_trace(PVR2_TRACE_CHIPS,"i2c msp3400 v4l2 set_stereo"); pvr2_trace(PVR2_TRACE_CHIPS,"i2c msp3400 v4l2 set_stereo");
if ((sid < ARRAY_SIZE(routing_schemes)) && if ((sid < ARRAY_SIZE(routing_schemes)) &&
((sp = routing_schemes + sid) != 0) && ((sp = routing_schemes + sid) != NULL) &&
(hdw->input_val >= 0) && (hdw->input_val >= 0) &&
(hdw->input_val < sp->cnt)) { (hdw->input_val < sp->cnt)) {
route.input = sp->def[hdw->input_val]; route.input = sp->def[hdw->input_val];
......
...@@ -195,7 +195,7 @@ static int pvr2_context_thread_func(void *foo) ...@@ -195,7 +195,7 @@ static int pvr2_context_thread_func(void *foo)
int pvr2_context_global_init(void) int pvr2_context_global_init(void)
{ {
pvr2_context_thread_ptr = kthread_run(pvr2_context_thread_func, pvr2_context_thread_ptr = kthread_run(pvr2_context_thread_func,
0, NULL,
"pvrusb2-context"); "pvrusb2-context");
return (pvr2_context_thread_ptr ? 0 : -ENOMEM); return (pvr2_context_thread_ptr ? 0 : -ENOMEM);
} }
......
...@@ -123,7 +123,7 @@ static void set_input(struct pvr2_v4l_cx2584x *ctxt) ...@@ -123,7 +123,7 @@ static void set_input(struct pvr2_v4l_cx2584x *ctxt)
memset(&route,0,sizeof(route)); memset(&route,0,sizeof(route));
if ((sid < ARRAY_SIZE(routing_schemes)) && if ((sid < ARRAY_SIZE(routing_schemes)) &&
((sp = routing_schemes + sid) != 0) && ((sp = routing_schemes + sid) != NULL) &&
(hdw->input_val >= 0) && (hdw->input_val >= 0) &&
(hdw->input_val < sp->cnt)) { (hdw->input_val < sp->cnt)) {
vid_input = sp->def[hdw->input_val].vid; vid_input = sp->def[hdw->input_val].vid;
......
...@@ -130,7 +130,7 @@ static void pvr2_dvb_stream_end(struct pvr2_dvb_adapter *adap) ...@@ -130,7 +130,7 @@ static void pvr2_dvb_stream_end(struct pvr2_dvb_adapter *adap)
for (idx = 0; idx < PVR2_DVB_BUFFER_COUNT; idx++) { for (idx = 0; idx < PVR2_DVB_BUFFER_COUNT; idx++) {
if (!(adap->buffer_storage[idx])) continue; if (!(adap->buffer_storage[idx])) continue;
kfree(adap->buffer_storage[idx]); kfree(adap->buffer_storage[idx]);
adap->buffer_storage[idx] = 0; adap->buffer_storage[idx] = NULL;
} }
adap->stream_run = 0; adap->stream_run = 0;
} }
...@@ -142,7 +142,7 @@ static int pvr2_dvb_stream_do_start(struct pvr2_dvb_adapter *adap) ...@@ -142,7 +142,7 @@ static int pvr2_dvb_stream_do_start(struct pvr2_dvb_adapter *adap)
unsigned int idx; unsigned int idx;
int ret; int ret;
struct pvr2_buffer *bp; struct pvr2_buffer *bp;
struct pvr2_stream *stream = 0; struct pvr2_stream *stream = NULL;
if (adap->stream_run) return -EIO; if (adap->stream_run) return -EIO;
...@@ -174,7 +174,7 @@ static int pvr2_dvb_stream_do_start(struct pvr2_dvb_adapter *adap) ...@@ -174,7 +174,7 @@ static int pvr2_dvb_stream_do_start(struct pvr2_dvb_adapter *adap)
ret = pvr2_hdw_set_streaming(adap->channel.hdw, 1); ret = pvr2_hdw_set_streaming(adap->channel.hdw, 1);
if (ret < 0) return ret; if (ret < 0) return ret;
while ((bp = pvr2_stream_get_idle_buffer(stream)) != 0) { while ((bp = pvr2_stream_get_idle_buffer(stream)) != NULL) {
ret = pvr2_buffer_queue(bp); ret = pvr2_buffer_queue(bp);
if (ret < 0) return ret; if (ret < 0) return ret;
} }
......
...@@ -1261,7 +1261,7 @@ struct pvr2_v4l2 *pvr2_v4l2_create(struct pvr2_context *mnp) ...@@ -1261,7 +1261,7 @@ struct pvr2_v4l2 *pvr2_v4l2_create(struct pvr2_context *mnp)
fail: fail:
pvr2_trace(PVR2_TRACE_STRUCT,"Failure creating pvr2_v4l2 id=%p",vp); pvr2_trace(PVR2_TRACE_STRUCT,"Failure creating pvr2_v4l2 id=%p",vp);
pvr2_v4l2_destroy_no_lock(vp); pvr2_v4l2_destroy_no_lock(vp);
return 0; return NULL;
} }
/* /*
......
...@@ -81,7 +81,7 @@ static void set_input(struct pvr2_v4l_decoder *ctxt) ...@@ -81,7 +81,7 @@ static void set_input(struct pvr2_v4l_decoder *ctxt)
pvr2_trace(PVR2_TRACE_CHIPS,"i2c v4l2 set_input(%d)",hdw->input_val); pvr2_trace(PVR2_TRACE_CHIPS,"i2c v4l2 set_input(%d)",hdw->input_val);
if ((sid < ARRAY_SIZE(routing_schemes)) && if ((sid < ARRAY_SIZE(routing_schemes)) &&
((sp = routing_schemes + sid) != 0) && ((sp = routing_schemes + sid) != NULL) &&
(hdw->input_val >= 0) && (hdw->input_val >= 0) &&
(hdw->input_val < sp->cnt)) { (hdw->input_val < sp->cnt)) {
route.input = sp->def[hdw->input_val]; route.input = sp->def[hdw->input_val];
......
...@@ -716,7 +716,7 @@ int v4l2_i2c_attach(struct i2c_adapter *adapter, int address, struct i2c_driver ...@@ -716,7 +716,7 @@ int v4l2_i2c_attach(struct i2c_adapter *adapter, int address, struct i2c_driver
int err; int err;
client = kzalloc(sizeof(struct i2c_client), GFP_KERNEL); client = kzalloc(sizeof(struct i2c_client), GFP_KERNEL);
if (client == 0) if (!client)
return -ENOMEM; return -ENOMEM;
client->addr = address; client->addr = address;
......
...@@ -97,7 +97,10 @@ int videobuf_iolock(struct videobuf_queue *q, struct videobuf_buffer *vb, ...@@ -97,7 +97,10 @@ int videobuf_iolock(struct videobuf_queue *q, struct videobuf_buffer *vb,
void *videobuf_queue_to_vmalloc (struct videobuf_queue *q, void *videobuf_queue_to_vmalloc (struct videobuf_queue *q,
struct videobuf_buffer *buf) struct videobuf_buffer *buf)
{ {
return CALL(q, vmalloc, buf); if (q->int_ops->vmalloc)
return q->int_ops->vmalloc(buf);
else
return NULL;
} }
EXPORT_SYMBOL_GPL(videobuf_queue_to_vmalloc); EXPORT_SYMBOL_GPL(videobuf_queue_to_vmalloc);
......
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