[media] video/saa7164: Fix sparse warning: Using plain integer as NULL pointer
authorPeter Huewe <PeterHuewe@gmx.de>
Sun, 30 Jan 2011 19:33:01 +0000 (16:33 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 21 Mar 2011 23:32:19 +0000 (20:32 -0300)
This patch fixes the warning "Using plain integer as NULL pointer",
generated by sparse, by replacing
if (var == 0)
with
if (!var)
after an allocation
and all other offending 0s with NULL.

KernelVersion: linus' tree-1f0324c

Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/saa7164/saa7164-api.c
drivers/media/video/saa7164/saa7164-buffer.c
drivers/media/video/saa7164/saa7164-bus.c
drivers/media/video/saa7164/saa7164-cmd.c
drivers/media/video/saa7164/saa7164-core.c
drivers/media/video/saa7164/saa7164-dvb.c
drivers/media/video/saa7164/saa7164-encoder.c
drivers/media/video/saa7164/saa7164-fw.c
drivers/media/video/saa7164/saa7164-vbi.c

index bd86d970f4c205de528ea39ec3d0778a248f0553..8a98ab68239e10ee5f1155f4fa31e678b58f7e48 100644 (file)
@@ -743,7 +743,7 @@ int saa7164_api_configure_dif(struct saa7164_port *port, u32 std)
 int saa7164_api_initialize_dif(struct saa7164_port *port)
 {
        struct saa7164_dev *dev = port->dev;
-       struct saa7164_port *p = 0;
+       struct saa7164_port *p = NULL;
        int ret = -EINVAL;
        u32 std = 0;
 
@@ -926,9 +926,9 @@ int saa7164_api_configure_port_mpeg2ps(struct saa7164_dev *dev,
 
 int saa7164_api_dump_subdevs(struct saa7164_dev *dev, u8 *buf, int len)
 {
-       struct saa7164_port *tsport = 0;
-       struct saa7164_port *encport = 0;
-       struct saa7164_port *vbiport = 0;
+       struct saa7164_port *tsport = NULL;
+       struct saa7164_port *encport = NULL;
+       struct saa7164_port *vbiport = NULL;
        u32 idx, next_offset;
        int i;
        struct tmComResDescrHeader *hdr, *t;
@@ -1340,7 +1340,7 @@ int saa7164_api_enum_subdevs(struct saa7164_dev *dev)
 
        /* Allocate enough storage for all of the descs */
        buf = kzalloc(buflen, GFP_KERNEL);
-       if (buf == NULL)
+       if (!buf)
                return SAA_ERR_NO_RESOURCES;
 
        /* Retrieve them */
index ddd25211c9e81b3a44ccb208d51412c062e75782..66696fa8341d1d1b646565d6802f99b1d33e0898 100644 (file)
@@ -93,7 +93,7 @@ struct saa7164_buffer *saa7164_buffer_alloc(struct saa7164_port *port,
        u32 len)
 {
        struct tmHWStreamParameters *params = &port->hw_streamingparams;
-       struct saa7164_buffer *buf = 0;
+       struct saa7164_buffer *buf = NULL;
        struct saa7164_dev *dev = port->dev;
        int i;
 
@@ -103,7 +103,7 @@ struct saa7164_buffer *saa7164_buffer_alloc(struct saa7164_port *port,
        }
 
        buf = kzalloc(sizeof(struct saa7164_buffer), GFP_KERNEL);
-       if (buf == NULL) {
+       if (!buf) {
                log_warn("%s() SAA_ERR_NO_RESOURCES\n", __func__);
                goto ret;
        }
@@ -157,7 +157,7 @@ fail2:
 fail1:
        kfree(buf);
 
-       buf = 0;
+       buf = NULL;
 ret:
        return buf;
 }
@@ -289,14 +289,14 @@ struct saa7164_user_buffer *saa7164_buffer_alloc_user(struct saa7164_dev *dev,
        struct saa7164_user_buffer *buf;
 
        buf = kzalloc(sizeof(struct saa7164_user_buffer), GFP_KERNEL);
-       if (buf == 0)
-               return 0;
+       if (!buf)
+               return NULL;
 
        buf->data = kzalloc(len, GFP_KERNEL);
 
-       if (buf->data == 0) {
+       if (!buf->data) {
                kfree(buf);
-               return 0;
+               return NULL;
        }
 
        buf->actual_size = len;
@@ -315,7 +315,7 @@ void saa7164_buffer_dealloc_user(struct saa7164_user_buffer *buf)
                return;
 
        kfree(buf->data);
-       buf->data = 0;
+       buf->data = NULL;
 
        kfree(buf);
 }
index b2b0d97101d091d306926206ea9fcf17c2ae0bfa..466e1b02f91f42c38c9a8af77d0eb1f9794c55f5 100644 (file)
@@ -158,7 +158,7 @@ int saa7164_bus_set(struct saa7164_dev *dev, struct tmComResInfo* msg,
                return SAA_ERR_BAD_PARAMETER;
        }
 
-       if ((msg->size > 0) && (buf == 0)) {
+       if ((msg->size > 0) && (buf == NULL)) {
                printk(KERN_ERR "%s() Missing message buffer\n", __func__);
                return SAA_ERR_BAD_PARAMETER;
        }
@@ -315,7 +315,7 @@ int saa7164_bus_get(struct saa7164_dev *dev, struct tmComResInfo* msg,
 
        saa7164_bus_verify(dev);
 
-       if (msg == 0)
+       if (msg == NULL)
                return ret;
 
        if (msg->size > dev->bus.m_wMaxReqSize) {
@@ -324,7 +324,7 @@ int saa7164_bus_get(struct saa7164_dev *dev, struct tmComResInfo* msg,
                return ret;
        }
 
-       if ((peekonly == 0) && (msg->size > 0) && (buf == 0)) {
+       if ((peekonly == 0) && (msg->size > 0) && (buf == NULL)) {
                printk(KERN_ERR
                        "%s() Missing msg buf, size should be %d bytes\n",
                        __func__, msg->size);
@@ -392,7 +392,7 @@ int saa7164_bus_get(struct saa7164_dev *dev, struct tmComResInfo* msg,
 
                printk(KERN_ERR "%s() Unexpected msg miss-match\n", __func__);
                saa7164_bus_dumpmsg(dev, msg, buf);
-               saa7164_bus_dumpmsg(dev, &msg_tmp, 0);
+               saa7164_bus_dumpmsg(dev, &msg_tmp, NULL);
                ret = SAA_ERR_INVALID_COMMAND;
                goto out;
        }
index a97ae17b36c2ca3f52a422f9a8e647ece56e4900..6a4c217ed3a7632bf332026cf557a9041cff9759 100644 (file)
@@ -84,7 +84,7 @@ int saa7164_irq_dequeue(struct saa7164_dev *dev)
 {
        int ret = SAA_OK, i = 0;
        u32 timeout;
-       wait_queue_head_t *q = 0;
+       wait_queue_head_t *q = NULL;
        u8 tmp[512];
        dprintk(DBGLVL_CMD, "%s()\n", __func__);
 
@@ -137,7 +137,7 @@ int saa7164_cmd_dequeue(struct saa7164_dev *dev)
        int loop = 1;
        int ret;
        u32 timeout;
-       wait_queue_head_t *q = 0;
+       wait_queue_head_t *q = NULL;
        u8 tmp[512];
        dprintk(DBGLVL_CMD, "%s()\n", __func__);
 
@@ -261,7 +261,7 @@ out:
  */
 int saa7164_cmd_wait(struct saa7164_dev *dev, u8 seqno)
 {
-       wait_queue_head_t *q = 0;
+       wait_queue_head_t *q = NULL;
        int ret = SAA_BUS_TIMEOUT;
        unsigned long stamp;
        int r;
@@ -357,7 +357,7 @@ int saa7164_cmd_send(struct saa7164_dev *dev, u8 id, enum tmComResCmd command,
                "sel = 0x%x)\n", __func__, saa7164_unitid_name(dev, id), id,
                command, controlselector);
 
-       if ((size == 0) || (buf == 0)) {
+       if ((size == 0) || (buf == NULL)) {
                printk(KERN_ERR "%s() Invalid param\n", __func__);
                return SAA_ERR_BAD_PARAMETER;
        }
@@ -538,7 +538,7 @@ int saa7164_cmd_send(struct saa7164_dev *dev, u8 id, enum tmComResCmd command,
 
                        /* Invalid */
                        dprintk(DBGLVL_CMD, "%s() Invalid\n", __func__);
-                       ret = saa7164_bus_get(dev, presponse_t, 0, 0);
+                       ret = saa7164_bus_get(dev, presponse_t, NULL, 0);
                        if (ret != SAA_OK) {
                                printk(KERN_ERR "get failed\n");
                                return ret;
index 58af67f2278bc5b2cf9b045752fdf08e7b3838b1..b813aec1e456cd46a807c5e86790cf8d2e2e0a0f 100644 (file)
@@ -277,8 +277,8 @@ static void saa7164_histogram_print(struct saa7164_port *port,
 static void saa7164_work_enchandler_helper(struct saa7164_port *port, int bufnr)
 {
        struct saa7164_dev *dev = port->dev;
-       struct saa7164_buffer *buf = 0;
-       struct saa7164_user_buffer *ubuf = 0;
+       struct saa7164_buffer *buf = NULL;
+       struct saa7164_user_buffer *ubuf = NULL;
        struct list_head *c, *n;
        int i = 0;
        u8 __iomem *p;
@@ -649,7 +649,7 @@ static irqreturn_t saa7164_irq(int irq, void *dev_id)
        u32 intid, intstat[INT_SIZE/4];
        int i, handled = 0, bit;
 
-       if (dev == 0) {
+       if (dev == NULL) {
                printk(KERN_ERR "%s() No device specified\n", __func__);
                handled = 0;
                goto out;
@@ -945,7 +945,7 @@ static int get_resources(struct saa7164_dev *dev)
 
 static int saa7164_port_init(struct saa7164_dev *dev, int portnr)
 {
-       struct saa7164_port *port = 0;
+       struct saa7164_port *port = NULL;
 
        if ((portnr < 0) || (portnr >= SAA7164_MAX_PORTS))
                BUG();
index b305a01b3bdec6c7875b6d4aad7d6a513c9b9dcf..f65eab63ca871fb634be16d0f17a0457ee1d933c 100644 (file)
@@ -309,8 +309,8 @@ static int dvb_register(struct saa7164_port *port)
 
        port->hw_streamingparams.pitch = 188;
        port->hw_streamingparams.linethreshold = 0;
-       port->hw_streamingparams.pagetablelistvirt = 0;
-       port->hw_streamingparams.pagetablelistphys = 0;
+       port->hw_streamingparams.pagetablelistvirt = NULL;
+       port->hw_streamingparams.pagetablelistphys = NULL;
        port->hw_streamingparams.numpagetables = 2 +
                ((SAA7164_TS_NUMBER_OF_LINES * 188) / PAGE_SIZE);
 
index 1838408cd5cb2635c64bf2ccee7e766cb90bd3aa..f9d5946988323c5e125c7d1a8bb70de9a34e8a8e 100644 (file)
@@ -152,8 +152,8 @@ static int saa7164_encoder_buffers_alloc(struct saa7164_port *port)
        /* Init and establish defaults */
        params->bitspersample = 8;
        params->linethreshold = 0;
-       params->pagetablelistvirt = 0;
-       params->pagetablelistphys = 0;
+       params->pagetablelistvirt = NULL;
+       params->pagetablelistphys = NULL;
        params->numpagetableentries = port->hwcfg.buffercount;
 
        /* Allocate the PCI resources, buffers (hard) */
@@ -1108,7 +1108,7 @@ static int fops_release(struct file *file)
 
 struct saa7164_user_buffer *saa7164_enc_next_buf(struct saa7164_port *port)
 {
-       struct saa7164_user_buffer *ubuf = 0;
+       struct saa7164_user_buffer *ubuf = NULL;
        struct saa7164_dev *dev = port->dev;
        u32 crc;
 
@@ -1443,7 +1443,7 @@ int saa7164_encoder_register(struct saa7164_port *port)
        port->v4l_device = saa7164_encoder_alloc(port,
                dev->pci, &saa7164_mpeg_template, "mpeg");
 
-       if (port->v4l_device == NULL) {
+       if (!port->v4l_device) {
                printk(KERN_INFO "%s: can't allocate mpeg device\n",
                        dev->name);
                result = -ENOMEM;
index ebed6f786a23750a69d17df841924563b66a9123..b369300cce0695eae159acd664d31de0e6dd6d70 100644 (file)
@@ -88,7 +88,7 @@ int saa7164_downloadimage(struct saa7164_dev *dev, u8 *src, u32 srcsize,
                "%s(image=%p, size=%d, flags=0x%x, dst=%p, dstsize=0x%x)\n",
                __func__, src, srcsize, dlflags, dst, dstsize);
 
-       if ((src == 0) || (dst == 0)) {
+       if ((src == NULL) || (dst == NULL)) {
                ret = -EIO;
                goto out;
        }
index 8abbe6d661e452149ae9e73cf517d3e85659adcd..9e5b01c29cf5fa7509c9affa0b8b7f5b7280d4b3 100644 (file)
@@ -123,8 +123,8 @@ static int saa7164_vbi_buffers_alloc(struct saa7164_port *port)
                ((params->numberoflines * params->pitch) / PAGE_SIZE);
        params->bitspersample = 8;
        params->linethreshold = 0;
-       params->pagetablelistvirt = 0;
-       params->pagetablelistphys = 0;
+       params->pagetablelistvirt = NULL;
+       params->pagetablelistphys = NULL;
        params->numpagetableentries = port->hwcfg.buffercount;
 
        /* Allocate the PCI resources, buffers (hard) */
@@ -1054,7 +1054,7 @@ static int fops_release(struct file *file)
 
 struct saa7164_user_buffer *saa7164_vbi_next_buf(struct saa7164_port *port)
 {
-       struct saa7164_user_buffer *ubuf = 0;
+       struct saa7164_user_buffer *ubuf = NULL;
        struct saa7164_dev *dev = port->dev;
        u32 crc;
 
@@ -1334,7 +1334,7 @@ int saa7164_vbi_register(struct saa7164_port *port)
        port->v4l_device = saa7164_vbi_alloc(port,
                dev->pci, &saa7164_vbi_template, "vbi");
 
-       if (port->v4l_device == NULL) {
+       if (!port->v4l_device) {
                printk(KERN_INFO "%s: can't allocate vbi device\n",
                        dev->name);
                result = -ENOMEM;