GPU-DRM: Replace two kzalloc() calls by kcalloc() in drm_legacy_addbufs_pci()
authorMarkus Elfring <elfring@users.sourceforge.net>
Mon, 19 Sep 2016 15:17:34 +0000 (17:17 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 21 Sep 2016 11:24:27 +0000 (13:24 +0200)
The script "checkpatch.pl" can point information out like the following.

WARNING: Prefer kcalloc over kzalloc with multiply

Thus fix the affected source code places.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/51f88f44-130d-48ac-f531-32ca9ae3919f@users.sourceforge.net
drivers/gpu/drm/drm_bufs.c

index ed33f4362dfe7fbd1fecfd32813ceb231f7d7c86..8a31dacc173683c92468710eaaea904743d57763 100644 (file)
@@ -905,14 +905,14 @@ int drm_legacy_addbufs_pci(struct drm_device *dev,
                return -EINVAL;
        }
 
-       entry->buflist = kzalloc(count * sizeof(*entry->buflist), GFP_KERNEL);
+       entry->buflist = kcalloc(count, sizeof(*entry->buflist), GFP_KERNEL);
        if (!entry->buflist) {
                mutex_unlock(&dev->struct_mutex);
                atomic_dec(&dev->buf_alloc);
                return -ENOMEM;
        }
 
-       entry->seglist = kzalloc(count * sizeof(*entry->seglist), GFP_KERNEL);
+       entry->seglist = kcalloc(count, sizeof(*entry->seglist), GFP_KERNEL);
        if (!entry->seglist) {
                kfree(entry->buflist);
                mutex_unlock(&dev->struct_mutex);