diff options
| author | Julia Lawall <julia@diku.dk> | 2009-01-06 14:41:45 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-01-06 15:59:20 -0800 |
| commit | 51e911e27690aecea01d4fe9cb22d38bdd0748ac (patch) | |
| tree | 0da60b105c3a74ac10a5e34ed9aa0a18af5673ef /drivers/spi | |
| parent | ee9c1fbfe130a20e0f23d1693d6427dac97239bc (diff) | |
| download | kernel_samsung_crespo-51e911e27690aecea01d4fe9cb22d38bdd0748ac.zip kernel_samsung_crespo-51e911e27690aecea01d4fe9cb22d38bdd0748ac.tar.gz kernel_samsung_crespo-51e911e27690aecea01d4fe9cb22d38bdd0748ac.tar.bz2 | |
drivers/spi: 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>
Acked-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/spi')
| -rw-r--r-- | drivers/spi/pxa2xx_spi.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/spi/pxa2xx_spi.c b/drivers/spi/pxa2xx_spi.c index 6104f46..d0fc4ca 100644 --- a/drivers/spi/pxa2xx_spi.c +++ b/drivers/spi/pxa2xx_spi.c @@ -1561,11 +1561,12 @@ out_error_master_alloc: static int pxa2xx_spi_remove(struct platform_device *pdev) { struct driver_data *drv_data = platform_get_drvdata(pdev); - struct ssp_device *ssp = drv_data->ssp; + struct ssp_device *ssp; int status = 0; if (!drv_data) return 0; + ssp = drv_data->ssp; /* Remove the queue */ status = destroy_queue(drv_data); |