diff options
author | Chris Ball <cjb@laptop.org> | 2011-04-13 23:46:05 -0400 |
---|---|---|
committer | Chris Ball <cjb@laptop.org> | 2011-05-24 21:01:56 -0400 |
commit | a61ad2b49bfce94dfddce828cd9222e4b9e7825b (patch) | |
tree | 36f93e61adc7c7d7523c78c78c3df1e75fae5e33 /drivers | |
parent | 1278dba167f01bb3c6626d16450d31129d041087 (diff) |
mmc: initialize struct mmc_data at declaration time
Converts from:
struct mmc_data data;
memset(&data, 0, sizeof(struct mmc_data));
to:
struct mmc_data data = {0};
because it's shorter, as performant, and easier to work out whether
initialization has happened.
Signed-off-by: Chris Ball <cjb@laptop.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/mmc/card/block.c | 4 | ||||
-rw-r--r-- | drivers/mmc/card/mmc_test.c | 9 | ||||
-rw-r--r-- | drivers/mmc/core/mmc_ops.c | 6 | ||||
-rw-r--r-- | drivers/mmc/core/sd_ops.c | 9 | ||||
-rw-r--r-- | drivers/mmc/core/sdio_ops.c | 3 |
5 files changed, 10 insertions, 21 deletions
diff --git a/drivers/mmc/card/block.c b/drivers/mmc/card/block.c index 93a7efcbfde9..fb387c803205 100644 --- a/drivers/mmc/card/block.c +++ b/drivers/mmc/card/block.c | |||
@@ -261,7 +261,7 @@ static u32 mmc_sd_num_wr_blocks(struct mmc_card *card) | |||
261 | 261 | ||
262 | struct mmc_request mrq; | 262 | struct mmc_request mrq; |
263 | struct mmc_command cmd = {0}; | 263 | struct mmc_command cmd = {0}; |
264 | struct mmc_data data; | 264 | struct mmc_data data = {0}; |
265 | unsigned int timeout_us; | 265 | unsigned int timeout_us; |
266 | 266 | ||
267 | struct scatterlist sg; | 267 | struct scatterlist sg; |
@@ -282,8 +282,6 @@ static u32 mmc_sd_num_wr_blocks(struct mmc_card *card) | |||
282 | cmd.arg = 0; | 282 | cmd.arg = 0; |
283 | cmd.flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 | MMC_CMD_ADTC; | 283 | cmd.flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 | MMC_CMD_ADTC; |
284 | 284 | ||
285 | memset(&data, 0, sizeof(struct mmc_data)); | ||
286 | |||
287 | data.timeout_ns = card->csd.tacc_ns * 100; | 285 | data.timeout_ns = card->csd.tacc_ns * 100; |
288 | data.timeout_clks = card->csd.tacc_clks * 100; | 286 | data.timeout_clks = card->csd.tacc_clks * 100; |
289 | 287 | ||
diff --git a/drivers/mmc/card/mmc_test.c b/drivers/mmc/card/mmc_test.c index 95c298faeb48..f44762b6e592 100644 --- a/drivers/mmc/card/mmc_test.c +++ b/drivers/mmc/card/mmc_test.c | |||
@@ -249,12 +249,11 @@ static int mmc_test_buffer_transfer(struct mmc_test_card *test, | |||
249 | struct mmc_request mrq; | 249 | struct mmc_request mrq; |
250 | struct mmc_command cmd = {0}; | 250 | struct mmc_command cmd = {0}; |
251 | struct mmc_command stop = {0}; | 251 | struct mmc_command stop = {0}; |
252 | struct mmc_data data; | 252 | struct mmc_data data = {0}; |
253 | 253 | ||
254 | struct scatterlist sg; | 254 | struct scatterlist sg; |
255 | 255 | ||
256 | memset(&mrq, 0, sizeof(struct mmc_request)); | 256 | memset(&mrq, 0, sizeof(struct mmc_request)); |
257 | memset(&data, 0, sizeof(struct mmc_data)); | ||
258 | 257 | ||
259 | mrq.cmd = &cmd; | 258 | mrq.cmd = &cmd; |
260 | mrq.data = &data; | 259 | mrq.data = &data; |
@@ -732,10 +731,9 @@ static int mmc_test_simple_transfer(struct mmc_test_card *test, | |||
732 | struct mmc_request mrq; | 731 | struct mmc_request mrq; |
733 | struct mmc_command cmd = {0}; | 732 | struct mmc_command cmd = {0}; |
734 | struct mmc_command stop = {0}; | 733 | struct mmc_command stop = {0}; |
735 | struct mmc_data data; | 734 | struct mmc_data data = {0}; |
736 | 735 | ||
737 | memset(&mrq, 0, sizeof(struct mmc_request)); | 736 | memset(&mrq, 0, sizeof(struct mmc_request)); |
738 | memset(&data, 0, sizeof(struct mmc_data)); | ||
739 | 737 | ||
740 | mrq.cmd = &cmd; | 738 | mrq.cmd = &cmd; |
741 | mrq.data = &data; | 739 | mrq.data = &data; |
@@ -760,12 +758,11 @@ static int mmc_test_broken_transfer(struct mmc_test_card *test, | |||
760 | struct mmc_request mrq; | 758 | struct mmc_request mrq; |
761 | struct mmc_command cmd = {0}; | 759 | struct mmc_command cmd = {0}; |
762 | struct mmc_command stop = {0}; | 760 | struct mmc_command stop = {0}; |
763 | struct mmc_data data; | 761 | struct mmc_data data = {0}; |
764 | 762 | ||
765 | struct scatterlist sg; | 763 | struct scatterlist sg; |
766 | 764 | ||
767 | memset(&mrq, 0, sizeof(struct mmc_request)); | 765 | memset(&mrq, 0, sizeof(struct mmc_request)); |
768 | memset(&data, 0, sizeof(struct mmc_data)); | ||
769 | 766 | ||
770 | mrq.cmd = &cmd; | 767 | mrq.cmd = &cmd; |
771 | mrq.data = &data; | 768 | mrq.data = &data; |
diff --git a/drivers/mmc/core/mmc_ops.c b/drivers/mmc/core/mmc_ops.c index de4e1855b8ed..1ed3866e990f 100644 --- a/drivers/mmc/core/mmc_ops.c +++ b/drivers/mmc/core/mmc_ops.c | |||
@@ -235,7 +235,7 @@ mmc_send_cxd_data(struct mmc_card *card, struct mmc_host *host, | |||
235 | { | 235 | { |
236 | struct mmc_request mrq; | 236 | struct mmc_request mrq; |
237 | struct mmc_command cmd = {0}; | 237 | struct mmc_command cmd = {0}; |
238 | struct mmc_data data; | 238 | struct mmc_data data = {0}; |
239 | struct scatterlist sg; | 239 | struct scatterlist sg; |
240 | void *data_buf; | 240 | void *data_buf; |
241 | 241 | ||
@@ -247,7 +247,6 @@ mmc_send_cxd_data(struct mmc_card *card, struct mmc_host *host, | |||
247 | return -ENOMEM; | 247 | return -ENOMEM; |
248 | 248 | ||
249 | memset(&mrq, 0, sizeof(struct mmc_request)); | 249 | memset(&mrq, 0, sizeof(struct mmc_request)); |
250 | memset(&data, 0, sizeof(struct mmc_data)); | ||
251 | 250 | ||
252 | mrq.cmd = &cmd; | 251 | mrq.cmd = &cmd; |
253 | mrq.data = &data; | 252 | mrq.data = &data; |
@@ -459,7 +458,7 @@ mmc_send_bus_test(struct mmc_card *card, struct mmc_host *host, u8 opcode, | |||
459 | { | 458 | { |
460 | struct mmc_request mrq; | 459 | struct mmc_request mrq; |
461 | struct mmc_command cmd = {0}; | 460 | struct mmc_command cmd = {0}; |
462 | struct mmc_data data; | 461 | struct mmc_data data = {0}; |
463 | struct scatterlist sg; | 462 | struct scatterlist sg; |
464 | u8 *data_buf; | 463 | u8 *data_buf; |
465 | u8 *test_buf; | 464 | u8 *test_buf; |
@@ -489,7 +488,6 @@ mmc_send_bus_test(struct mmc_card *card, struct mmc_host *host, u8 opcode, | |||
489 | memcpy(data_buf, test_buf, len); | 488 | memcpy(data_buf, test_buf, len); |
490 | 489 | ||
491 | memset(&mrq, 0, sizeof(struct mmc_request)); | 490 | memset(&mrq, 0, sizeof(struct mmc_request)); |
492 | memset(&data, 0, sizeof(struct mmc_data)); | ||
493 | 491 | ||
494 | mrq.cmd = &cmd; | 492 | mrq.cmd = &cmd; |
495 | mrq.data = &data; | 493 | mrq.data = &data; |
diff --git a/drivers/mmc/core/sd_ops.c b/drivers/mmc/core/sd_ops.c index dfe9a9c3f958..8023f580b901 100644 --- a/drivers/mmc/core/sd_ops.c +++ b/drivers/mmc/core/sd_ops.c | |||
@@ -245,7 +245,7 @@ int mmc_app_send_scr(struct mmc_card *card, u32 *scr) | |||
245 | int err; | 245 | int err; |
246 | struct mmc_request mrq; | 246 | struct mmc_request mrq; |
247 | struct mmc_command cmd = {0}; | 247 | struct mmc_command cmd = {0}; |
248 | struct mmc_data data; | 248 | struct mmc_data data = {0}; |
249 | struct scatterlist sg; | 249 | struct scatterlist sg; |
250 | void *data_buf; | 250 | void *data_buf; |
251 | 251 | ||
@@ -267,7 +267,6 @@ int mmc_app_send_scr(struct mmc_card *card, u32 *scr) | |||
267 | return -ENOMEM; | 267 | return -ENOMEM; |
268 | 268 | ||
269 | memset(&mrq, 0, sizeof(struct mmc_request)); | 269 | memset(&mrq, 0, sizeof(struct mmc_request)); |
270 | memset(&data, 0, sizeof(struct mmc_data)); | ||
271 | 270 | ||
272 | mrq.cmd = &cmd; | 271 | mrq.cmd = &cmd; |
273 | mrq.data = &data; | 272 | mrq.data = &data; |
@@ -307,7 +306,7 @@ int mmc_sd_switch(struct mmc_card *card, int mode, int group, | |||
307 | { | 306 | { |
308 | struct mmc_request mrq; | 307 | struct mmc_request mrq; |
309 | struct mmc_command cmd = {0}; | 308 | struct mmc_command cmd = {0}; |
310 | struct mmc_data data; | 309 | struct mmc_data data = {0}; |
311 | struct scatterlist sg; | 310 | struct scatterlist sg; |
312 | 311 | ||
313 | BUG_ON(!card); | 312 | BUG_ON(!card); |
@@ -319,7 +318,6 @@ int mmc_sd_switch(struct mmc_card *card, int mode, int group, | |||
319 | value &= 0xF; | 318 | value &= 0xF; |
320 | 319 | ||
321 | memset(&mrq, 0, sizeof(struct mmc_request)); | 320 | memset(&mrq, 0, sizeof(struct mmc_request)); |
322 | memset(&data, 0, sizeof(struct mmc_data)); | ||
323 | 321 | ||
324 | mrq.cmd = &cmd; | 322 | mrq.cmd = &cmd; |
325 | mrq.data = &data; | 323 | mrq.data = &data; |
@@ -355,7 +353,7 @@ int mmc_app_sd_status(struct mmc_card *card, void *ssr) | |||
355 | int err; | 353 | int err; |
356 | struct mmc_request mrq; | 354 | struct mmc_request mrq; |
357 | struct mmc_command cmd = {0}; | 355 | struct mmc_command cmd = {0}; |
358 | struct mmc_data data; | 356 | struct mmc_data data = {0}; |
359 | struct scatterlist sg; | 357 | struct scatterlist sg; |
360 | 358 | ||
361 | BUG_ON(!card); | 359 | BUG_ON(!card); |
@@ -369,7 +367,6 @@ int mmc_app_sd_status(struct mmc_card *card, void *ssr) | |||
369 | return err; | 367 | return err; |
370 | 368 | ||
371 | memset(&mrq, 0, sizeof(struct mmc_request)); | 369 | memset(&mrq, 0, sizeof(struct mmc_request)); |
372 | memset(&data, 0, sizeof(struct mmc_data)); | ||
373 | 370 | ||
374 | mrq.cmd = &cmd; | 371 | mrq.cmd = &cmd; |
375 | mrq.data = &data; | 372 | mrq.data = &data; |
diff --git a/drivers/mmc/core/sdio_ops.c b/drivers/mmc/core/sdio_ops.c index f77b1bc318ed..147dfc1879b3 100644 --- a/drivers/mmc/core/sdio_ops.c +++ b/drivers/mmc/core/sdio_ops.c | |||
@@ -123,7 +123,7 @@ int mmc_io_rw_extended(struct mmc_card *card, int write, unsigned fn, | |||
123 | { | 123 | { |
124 | struct mmc_request mrq; | 124 | struct mmc_request mrq; |
125 | struct mmc_command cmd = {0}; | 125 | struct mmc_command cmd = {0}; |
126 | struct mmc_data data; | 126 | struct mmc_data data = {0}; |
127 | struct scatterlist sg; | 127 | struct scatterlist sg; |
128 | 128 | ||
129 | BUG_ON(!card); | 129 | BUG_ON(!card); |
@@ -137,7 +137,6 @@ int mmc_io_rw_extended(struct mmc_card *card, int write, unsigned fn, | |||
137 | return -EINVAL; | 137 | return -EINVAL; |
138 | 138 | ||
139 | memset(&mrq, 0, sizeof(struct mmc_request)); | 139 | memset(&mrq, 0, sizeof(struct mmc_request)); |
140 | memset(&data, 0, sizeof(struct mmc_data)); | ||
141 | 140 | ||
142 | mrq.cmd = &cmd; | 141 | mrq.cmd = &cmd; |
143 | mrq.data = &data; | 142 | mrq.data = &data; |