ste_dma40: Replace four kzalloc() calls by kcalloc() in d40_hw_detect_init()
authorMarkus Elfring <elfring@users.sourceforge.net>
Sat, 17 Sep 2016 07:56:32 +0000 (09:56 +0200)
committerVinod Koul <vinod.koul@intel.com>
Mon, 26 Sep 2016 17:36:17 +0000 (23:06 +0530)
* 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.

* Replace the specification of data types by pointer dereferences
  to make the corresponding size determination a bit safer according to
  the Linux coding style convention.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
drivers/dma/ste_dma40.c

index 4a651b23e577d10401601b2e100f7c7f09029b0c..a426abd55deeb4babb27fc444221832e8399aa3e 100644 (file)
@@ -3283,19 +3283,20 @@ static struct d40_base * __init d40_hw_detect_init(struct platform_device *pdev)
                base->gen_dmac.init_reg_size = ARRAY_SIZE(dma_init_reg_v4a);
        }
 
-       base->phy_res = kzalloc(num_phy_chans * sizeof(struct d40_phy_res),
+       base->phy_res = kcalloc(num_phy_chans,
+                               sizeof(*base->phy_res),
                                GFP_KERNEL);
        if (!base->phy_res)
                goto failure;
 
-       base->lookup_phy_chans = kzalloc(num_phy_chans *
-                                        sizeof(struct d40_chan *),
+       base->lookup_phy_chans = kcalloc(num_phy_chans,
+                                        sizeof(*base->lookup_phy_chans),
                                         GFP_KERNEL);
        if (!base->lookup_phy_chans)
                goto failure;
 
-       base->lookup_log_chans = kzalloc(num_log_chans *
-                                        sizeof(struct d40_chan *),
+       base->lookup_log_chans = kcalloc(num_log_chans,
+                                        sizeof(*base->lookup_log_chans),
                                         GFP_KERNEL);
        if (!base->lookup_log_chans)
                goto failure;
@@ -3306,9 +3307,10 @@ static struct d40_base * __init d40_hw_detect_init(struct platform_device *pdev)
        if (!base->reg_val_backup_chan)
                goto failure;
 
-       base->lcla_pool.alloc_map =
-               kzalloc(num_phy_chans * sizeof(struct d40_desc *)
-                       * D40_LCLA_LINK_PER_EVENT_GRP, GFP_KERNEL);
+       base->lcla_pool.alloc_map = kcalloc(num_phy_chans
+                                           * D40_LCLA_LINK_PER_EVENT_GRP,
+                                           sizeof(*base->lcla_pool.alloc_map),
+                                           GFP_KERNEL);
        if (!base->lcla_pool.alloc_map)
                goto failure;