Commit 554d7c54 authored by Markus Elfring's avatar Markus Elfring Committed by Ulf Hansson

mmc: mmc_test: Improve a size determination in five functions

Replace the specification of a data structure by a pointer dereference
as the parameter for the operator "sizeof" to make the corresponding size
determination a bit safer according to the Linux coding style convention.
Signed-off-by: default avatarMarkus Elfring <elfring@users.sourceforge.net>
Reviewed-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Reviewed-by: default avatarShawn Lin <shawn.lin@rock-chips.com>
Reviewed-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
parent 7200449d
...@@ -356,7 +356,7 @@ static struct mmc_test_mem *mmc_test_alloc_mem(unsigned long min_sz, ...@@ -356,7 +356,7 @@ static struct mmc_test_mem *mmc_test_alloc_mem(unsigned long min_sz,
if (max_segs > max_page_cnt) if (max_segs > max_page_cnt)
max_segs = max_page_cnt; max_segs = max_page_cnt;
mem = kzalloc(sizeof(struct mmc_test_mem), GFP_KERNEL); mem = kzalloc(sizeof(*mem), GFP_KERNEL);
if (!mem) if (!mem)
return NULL; return NULL;
...@@ -545,7 +545,7 @@ static void mmc_test_save_transfer_result(struct mmc_test_card *test, ...@@ -545,7 +545,7 @@ static void mmc_test_save_transfer_result(struct mmc_test_card *test,
if (!test->gr) if (!test->gr)
return; return;
tr = kmalloc(sizeof(struct mmc_test_transfer_result), GFP_KERNEL); tr = kmalloc(sizeof(*tr), GFP_KERNEL);
if (!tr) if (!tr)
return; return;
...@@ -2973,8 +2973,7 @@ static void mmc_test_run(struct mmc_test_card *test, int testcase) ...@@ -2973,8 +2973,7 @@ static void mmc_test_run(struct mmc_test_card *test, int testcase)
} }
} }
gr = kzalloc(sizeof(struct mmc_test_general_result), gr = kzalloc(sizeof(*gr), GFP_KERNEL);
GFP_KERNEL);
if (gr) { if (gr) {
INIT_LIST_HEAD(&gr->tr_lst); INIT_LIST_HEAD(&gr->tr_lst);
...@@ -3108,7 +3107,7 @@ static ssize_t mtf_test_write(struct file *file, const char __user *buf, ...@@ -3108,7 +3107,7 @@ static ssize_t mtf_test_write(struct file *file, const char __user *buf,
if (ret) if (ret)
return ret; return ret;
test = kzalloc(sizeof(struct mmc_test_card), GFP_KERNEL); test = kzalloc(sizeof(*test), GFP_KERNEL);
if (!test) if (!test)
return -ENOMEM; return -ENOMEM;
...@@ -3213,7 +3212,7 @@ static int __mmc_test_register_dbgfs_file(struct mmc_card *card, ...@@ -3213,7 +3212,7 @@ static int __mmc_test_register_dbgfs_file(struct mmc_card *card,
return -ENODEV; return -ENODEV;
} }
df = kmalloc(sizeof(struct mmc_test_dbgfs_file), GFP_KERNEL); df = kmalloc(sizeof(*df), GFP_KERNEL);
if (!df) { if (!df) {
debugfs_remove(file); debugfs_remove(file);
dev_err(&card->dev, dev_err(&card->dev,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment