From a9d7900fa0bbd93ca42ee6f6f1c5661c68f2fbd3 Mon Sep 17 00:00:00 2001 From: Greg Kroah-Hartman Date: Fri, 27 Feb 2009 20:49:46 -0800 Subject: Staging: line6: fix up NULL assignment mistakes Should use NULL for a pointer, not 0, otherwise sparse complains. Cc: Markus Grabner Cc: Mariusz Kozlowski Signed-off-by: Greg Kroah-Hartman --- drivers/staging/line6/audio.c | 4 ++-- drivers/staging/line6/capture.c | 2 +- drivers/staging/line6/driver.c | 14 +++++++------- drivers/staging/line6/midibuf.c | 8 +++----- drivers/staging/line6/playback.c | 2 +- drivers/staging/line6/toneport.c | 2 +- 6 files changed, 15 insertions(+), 17 deletions(-) --- a/drivers/staging/line6/audio.c +++ b/drivers/staging/line6/audio.c @@ -62,10 +62,10 @@ void line6_cleanup_audio(struct usb_line { struct snd_card *card = line6->card; - if(card == 0) + if (card == NULL) return; snd_card_disconnect(card); snd_card_free(card); - line6->card = 0; + line6->card = NULL; } --- a/drivers/staging/line6/capture.c +++ b/drivers/staging/line6/capture.c @@ -279,7 +279,7 @@ static int snd_line6_capture_hw_free(str if(line6pcm->buffer_in) { kfree(line6pcm->buffer_in); - line6pcm->buffer_in = 0; + line6pcm->buffer_in = NULL; } return snd_pcm_lib_free_pages(substream); --- a/drivers/staging/line6/driver.c +++ b/drivers/staging/line6/driver.c @@ -301,7 +301,7 @@ char *line6_alloc_sysex_buffer(struct us if(!buffer) { dev_err(line6->ifcdev, "out of memory\n"); - return 0; + return NULL; } buffer[0] = LINE6_SYSEX_BEGIN; @@ -469,7 +469,7 @@ int line6_read_data(struct usb_line6 *li /* query the serial number: */ ret = usb_control_msg(usbdev, usb_sndctrlpipe(usbdev, 0), 0x67, USB_TYPE_VENDOR | USB_RECIP_DEVICE | USB_DIR_OUT, - (datalen << 8) | 0x21, address, 0, 0, LINE6_TIMEOUT * HZ); + (datalen << 8) | 0x21, address, NULL, 0, LINE6_TIMEOUT * HZ); if(ret < 0) { dev_err(line6->ifcdev, "read request failed (error %d)\n", ret); @@ -630,8 +630,8 @@ static void line6_list_devices(void) static int line6_probe(struct usb_interface *interface, const struct usb_device_id *id) { int devtype; - struct usb_device *usbdev = 0; - struct usb_line6 *line6 = 0; + struct usb_device *usbdev = NULL; + struct usb_line6 *line6 = NULL; const struct line6_properties *properties; int devnum; int interface_number, alternate = 0; @@ -987,7 +987,7 @@ static void line6_disconnect(struct usb_ for(i = LINE6_MAX_DEVICES; i--;) if(line6_devices[i] == line6) - line6_devices[i] = 0; + line6_devices[i] = NULL; } line6_destruct(interface); @@ -1016,13 +1016,13 @@ static int __init line6_init(void) printk("%s driver version %s%s\n", DRIVER_NAME, DRIVER_VERSION, DRIVER_REVISION); line6_workqueue = create_workqueue(DRIVER_NAME); - if(line6_workqueue == 0) { + if (line6_workqueue == NULL) { err("couldn't create workqueue"); return -EINVAL; } for(i = LINE6_MAX_DEVICES; i--;) - line6_devices[i] = 0; + line6_devices[i] = NULL; retval = usb_register(&line6_driver); --- a/drivers/staging/line6/midibuf.c +++ b/drivers/staging/line6/midibuf.c @@ -44,7 +44,7 @@ int midibuf_init(struct MidiBuffer *this { this->buf = (unsigned char *)kmalloc(size, GFP_KERNEL); - if(this->buf == 0) + if (this->buf == NULL) return -ENOMEM; this->size = size; @@ -261,8 +261,6 @@ int midibuf_skip_message(struct MidiBuff void midibuf_destroy(struct MidiBuffer *this) { - if(this->buf != 0) { - kfree(this->buf); - this->buf = 0; - } + kfree(this->buf); + this->buf = NULL; } --- a/drivers/staging/line6/playback.c +++ b/drivers/staging/line6/playback.c @@ -329,7 +329,7 @@ static int snd_line6_playback_hw_free(st if(line6pcm->wrap_out) { kfree(line6pcm->wrap_out); - line6pcm->wrap_out = 0; + line6pcm->wrap_out = NULL; } return snd_pcm_lib_free_pages(substream); --- a/drivers/staging/line6/toneport.c +++ b/drivers/staging/line6/toneport.c @@ -117,7 +117,7 @@ static int toneport_send_cmd(struct usb_ int ret; ret = usb_control_msg(usbdev, usb_sndctrlpipe(usbdev,0), 0x67, USB_TYPE_VENDOR | USB_RECIP_DEVICE | USB_DIR_OUT, - cmd1, cmd2, 0, 0, LINE6_TIMEOUT * HZ); + cmd1, cmd2, NULL, 0, LINE6_TIMEOUT * HZ); if(ret < 0) { err("send failed (error %d)\n", ret);