clk: keystone: sci-clk: use devm_kmemdup_array()

Convert to use devm_kmemdup_array() which is more robust.

Signed-off-by: Raag Jadav <raag.jadav@intel.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>

diff --git a/drivers/clk/keystone/sci-clk.c b/drivers/clk/keystone/sci-clk.c
index c5894fc9395e833c8436a9a5a970884ace5d845f..a4b42811de55deb822f76800ba68acbbdf27f4c5 100644 (file)
--- a/drivers/clk/keystone/sci-clk.c
+++ b/drivers/clk/keystone/sci-clk.c
@@ -480,13 +480,10 @@ static int ti_sci_scan_clocks_from_fw(struct sci_clk_provider *provider)
                num_clks++;
        }
 
-       provider->clocks = devm_kmalloc_array(dev, num_clks, sizeof(sci_clk),
-                                             GFP_KERNEL);
+       provider->clocks = devm_kmemdup_array(dev, clks, num_clks, sizeof(sci_clk), GFP_KERNEL);
        if (!provider->clocks)
                return -ENOMEM;
 
-       memcpy(provider->clocks, clks, num_clks * sizeof(sci_clk));
-
        provider->num_clocks = num_clks;
 
        devm_kfree(dev, clks);