[PATCH V2 6/8] memory: emif: Fix the incorrect 'size' parameter in memcpy

Lokesh Vutla lokeshvutla at ti.com
Sat Mar 16 02:16:46 EDT 2013


From: Oleksandr Dmytryshyn <oleksandr.dmytryshyn at ti.com>

The issue was that only the first timings table was added to the
emif platform data at the emif driver registration. All other
timings tables was filled with zeros. Now all emif timings table
are added to the platform data.

Signed-off-by: Oleksandr Dmytryshyn <oleksandr.dmytryshyn at ti.com>
Signed-off-by: Lokesh Vutla <lokeshvutla at ti.com>
Acked-by: Santosh Shilimkar <santosh.shilimkar at ti.com>
---
Changes since V1: Updated $subject

 drivers/memory/emif.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/memory/emif.c b/drivers/memory/emif.c
index 8db74e4..d27111e 100644
--- a/drivers/memory/emif.c
+++ b/drivers/memory/emif.c
@@ -1468,7 +1468,7 @@ static struct emif_data *__init_or_module get_device_details(
 	if (pd->timings) {
 		temp = devm_kzalloc(dev, size, GFP_KERNEL);
 		if (temp) {
-			memcpy(temp, pd->timings, sizeof(*pd->timings));
+			memcpy(temp, pd->timings, size);
 			pd->timings = temp;
 		} else {
 			dev_warn(dev, "%s:%d: allocation error\n", __func__,
-- 
1.7.9.5




More information about the linux-arm-kernel mailing list