V4L/DVB (3344a): Conversions from kmalloc+memset to k(z|c)alloc

Conversions from kmalloc+memset to k(z|c)alloc.

Signed-off-by: Panagiotis Issaris <takis@issaris.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>

2 files changed, 5 insertions, 10 deletions

diff --git a/drivers/media/common/saa7146_core.c b/drivers/media/common/saa7146_core.c
index 2899d34e5f7e..04c1938b9c91 100644
--- a/drivers/media/common/saa7146_core.c
+++ b/drivers/media/common/saa7146_core.c
@@ -109,10 +109,9 @@ static struct scatterlist* vmalloc_to_sg(unsigned char *virt, int nr_pages)
         struct page *pg;
         int i;
 
-        sglist = kmalloc(sizeof(struct scatterlist)*nr_pages, GFP_KERNEL);
+        sglist = kcalloc(nr_pages, sizeof(struct scatterlist), GFP_KERNEL);
         if (NULL == sglist)
                 return NULL;
-        memset(sglist,0,sizeof(struct scatterlist)*nr_pages);
         for (i = 0; i < nr_pages; i++, virt += PAGE_SIZE) {
                 pg = vmalloc_to_page(virt);
                 if (NULL == pg)
@@ -306,15 +305,13 @@ static int saa7146_init_one(struct pci_dev *pci, const struct pci_device_id *ent
         struct saa7146_dev *dev;
         int err = -ENOMEM;
 
-        dev = kmalloc(sizeof(struct saa7146_dev), GFP_KERNEL);
+        /* clear out mem for sure */
+        dev = kzalloc(sizeof(struct saa7146_dev), GFP_KERNEL);
         if (!dev) {
                 ERR(("out of memory.\n"));
                 goto out;
         }
 
-        /* clear out mem for sure */
-        memset(dev, 0x0, sizeof(struct saa7146_dev));
-
         DEB_EE(("pci:%p\n",pci));
 
         err = pci_enable_device(pci);
diff --git a/drivers/media/common/saa7146_fops.c b/drivers/media/common/saa7146_fops.c
index b614612be7b4..f8cf73ed49ad 100644
--- a/drivers/media/common/saa7146_fops.c
+++ b/drivers/media/common/saa7146_fops.c
@@ -239,13 +239,12 @@ static int fops_open(struct inode *inode, struct file *file)
         }
 
         /* allocate per open data */
-        fh = kmalloc(sizeof(*fh),GFP_KERNEL);
+        fh = kzalloc(sizeof(*fh),GFP_KERNEL);
         if (NULL == fh) {
                 DEB_S(("cannot allocate memory for per open data.\n"));
                 result = -ENOMEM;
                 goto out;
         }
-        memset(fh,0,sizeof(*fh));
 
         file->private_data = fh;
         fh->dev = dev;
@@ -464,12 +463,11 @@ static struct video_device device_template =
 
 int saa7146_vv_init(struct saa7146_dev* dev, struct saa7146_ext_vv *ext_vv)
 {
-        struct saa7146_vv *vv = kmalloc (sizeof(struct saa7146_vv),GFP_KERNEL);
+        struct saa7146_vv *vv = kzalloc (sizeof(struct saa7146_vv),GFP_KERNEL);
         if( NULL == vv ) {
                 ERR(("out of memory. aborting.\n"));
                 return -1;
         }
-        memset(vv, 0x0, sizeof(*vv));
 
         DEB_EE(("dev:%p\n",dev));