diff options
author | Julia Lawall <julia@diku.dk> | 2008-12-16 10:13:09 -0500 |
---|---|---|
committer | Pierre Ossman <drzeus@drzeus.cx> | 2008-12-31 12:18:14 -0500 |
commit | a0d045cac9bcb3e9a9796d596415f7ffb64852e2 (patch) | |
tree | 961152a30a5b71d54c4a495f516284214f44ae00 | |
parent | f9134319c81c6c56e0ddf38e7adac2492b243d9b (diff) |
drivers/mmc: Move a dereference below a NULL test
In each case, if the NULL test is necessary, then the dereference should be
moved below the NULL test.
The semantic patch that makes this change is as follows:
(http://www.emn.fr/x-info/coccinelle/)
// <smpl>
@@
type T;
expression E;
identifier i,fld;
statement S;
@@
- T i = E->fld;
+ T i;
... when != E
when != i
if (E == NULL) S
+ i = E->fld;
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Pierre Ossman <drzeus@drzeus.cx>
-rw-r--r-- | drivers/mmc/host/tmio_mmc.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/mmc/host/tmio_mmc.c b/drivers/mmc/host/tmio_mmc.c index 95430b81ec1..6a7a6190483 100644 --- a/drivers/mmc/host/tmio_mmc.c +++ b/drivers/mmc/host/tmio_mmc.c | |||
@@ -224,7 +224,7 @@ static inline void tmio_mmc_data_irq(struct tmio_mmc_host *host) | |||
224 | { | 224 | { |
225 | void __iomem *ctl = host->ctl; | 225 | void __iomem *ctl = host->ctl; |
226 | struct mmc_data *data = host->data; | 226 | struct mmc_data *data = host->data; |
227 | struct mmc_command *stop = data->stop; | 227 | struct mmc_command *stop; |
228 | 228 | ||
229 | host->data = NULL; | 229 | host->data = NULL; |
230 | 230 | ||
@@ -232,6 +232,7 @@ static inline void tmio_mmc_data_irq(struct tmio_mmc_host *host) | |||
232 | pr_debug("Spurious data end IRQ\n"); | 232 | pr_debug("Spurious data end IRQ\n"); |
233 | return; | 233 | return; |
234 | } | 234 | } |
235 | stop = data->stop; | ||
235 | 236 | ||
236 | /* FIXME - return correct transfer count on errors */ | 237 | /* FIXME - return correct transfer count on errors */ |
237 | if (!data->error) | 238 | if (!data->error) |