mmc: use empty initializer list to zero-clear structures

In the MMC subsystem, we see such initializers that only clears the
first member explicitly.

For example,

  struct mmc_request mrq = {NULL};

sets the first member (.sbc) to NULL explicitly.  However, this is
an unstable form because we may insert a non-pointer member at the
top of the struct mmc_request in the future. (if we do so, the
compiler will spit warnings.)

So, using a designated initializer is preferred coding style.  The
expression above is equivalent to:

  struct mmc_request mrq = { .sbc = NULL };

Of course, this does not express our intention.  We want to fill
all struct members with zeros.  Please note struct members are
implicitly zero-cleared unless otherwise specified in the initializer.

After all, the most reasonable (and stable) form is:

  struct mmc_request mrq = {};

Do likewise for mmc_command, mmc_data as well.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>

1 files changed, 13 insertions, 13 deletions

diff --git a/drivers/mmc/core/mmc_test.c b/drivers/mmc/core/mmc_test.c
index 3ab6e52d106c..bd8eb9bfc153 100644
--- a/drivers/mmc/core/mmc_test.c
+++ b/drivers/mmc/core/mmc_test.c
@@ -260,7 +260,7 @@ static int mmc_test_busy(struct mmc_command *cmd)
 static int mmc_test_wait_busy(struct mmc_test_card *test)
 {
         int ret, busy;
-        struct mmc_command cmd = {0};
+        struct mmc_command cmd = {};
 
         busy = 0;
         do {
@@ -292,10 +292,10 @@ static int mmc_test_wait_busy(struct mmc_test_card *test)
 static int mmc_test_buffer_transfer(struct mmc_test_card *test,
         u8 *buffer, unsigned addr, unsigned blksz, int write)
 {
-        struct mmc_request mrq = {0};
-        struct mmc_command cmd = {0};
-        struct mmc_command stop = {0};
-        struct mmc_data data = {0};
+        struct mmc_request mrq = {};
+        struct mmc_command cmd = {};
+        struct mmc_command stop = {};
+        struct mmc_data data = {};
 
         struct scatterlist sg;
 
@@ -885,10 +885,10 @@ static int mmc_test_simple_transfer(struct mmc_test_card *test,
         struct scatterlist *sg, unsigned sg_len, unsigned dev_addr,
         unsigned blocks, unsigned blksz, int write)
 {
-        struct mmc_request mrq = {0};
-        struct mmc_command cmd = {0};
-        struct mmc_command stop = {0};
-        struct mmc_data data = {0};
+        struct mmc_request mrq = {};
+        struct mmc_command cmd = {};
+        struct mmc_command stop = {};
+        struct mmc_data data = {};
 
         mrq.cmd = &cmd;
         mrq.data = &data;
@@ -910,10 +910,10 @@ static int mmc_test_simple_transfer(struct mmc_test_card *test,
 static int mmc_test_broken_transfer(struct mmc_test_card *test,
         unsigned blocks, unsigned blksz, int write)
 {
-        struct mmc_request mrq = {0};
-        struct mmc_command cmd = {0};
-        struct mmc_command stop = {0};
-        struct mmc_data data = {0};
+        struct mmc_request mrq = {};
+        struct mmc_command cmd = {};
+        struct mmc_command stop = {};
+        struct mmc_data data = {};
 
         struct scatterlist sg;