ARM: OMAP2+: Use kcalloc() in sr_set_nvalues()
authorMarkus Elfring <elfring@users.sourceforge.net>
Sat, 3 Jun 2017 17:09:07 +0000 (19:09 +0200)
committerTony Lindgren <tony@atomide.com>
Thu, 8 Jun 2017 10:02:31 +0000 (03:02 -0700)
* A multiplication for the size determination of a memory allocation
  indicated that an array data structure should be processed.
  Thus use the corresponding function "kcalloc".

  This issue was detected by using the Coccinelle software.

* Replace the specification of a data structure by a pointer dereference
  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>
Signed-off-by: Tony Lindgren <tony@atomide.com>
arch/arm/mach-omap2/sr_device.c

index 01df4907c0e305a9ba0fecebba1af5c3607d15d9..65775c6e8c272f82630c0b939c9f76c15e1ec4af 100644 (file)
@@ -44,9 +44,7 @@ static void __init sr_set_nvalues(struct omap_volt_data *volt_data,
        while (volt_data[count].volt_nominal)
                count++;
 
-       nvalue_table = kzalloc(sizeof(struct omap_sr_nvalue_table)*count,
-                       GFP_KERNEL);
-
+       nvalue_table = kcalloc(count, sizeof(*nvalue_table), GFP_KERNEL);
        if (!nvalue_table) {
                pr_err("OMAP: SmartReflex: cannot allocate memory for n-value table\n");
                return;