diff options
| author | Artem Bityutskiy <artem.bityutskiy@linux.intel.com> | 2012-03-09 12:24:26 -0500 |
|---|---|---|
| committer | David Woodhouse <David.Woodhouse@intel.com> | 2012-03-26 19:53:20 -0400 |
| commit | 42d7fbe223ab878b23de9e3b0166f8cd665a2aa5 (patch) | |
| tree | 844f3b407e7cc7b335899909b81811e1369dcdef /drivers/mtd/nand/pxa3xx_nand.c | |
| parent | ee478af8b675908b217198a75cf759d422a81ccb (diff) | |
mtd: do not use plain 0 as NULL
The first 3 arguments of 'mtd_device_parse_register()' are pointers,
but many callers pass '0' instead of 'NULL'. Fix this globally. Thanks
to coccinelle for making it easy to do with the following semantic patch:
@@
expression mtd, types, parser_data, parts, nr_parts;
@@
(
-mtd_device_parse_register(mtd, 0, parser_data, parts, nr_parts)
+mtd_device_parse_register(mtd, NULL, parser_data, parts, nr_parts)
|
-mtd_device_parse_register(mtd, types, 0, parts, nr_parts)
+mtd_device_parse_register(mtd, types, NULL, parts, nr_parts)
|
-mtd_device_parse_register(mtd, types, parser_data, 0, nr_parts)
+mtd_device_parse_register(mtd, types, parser_data, NULL, nr_parts)
)
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Diffstat (limited to 'drivers/mtd/nand/pxa3xx_nand.c')
| -rw-r--r-- | drivers/mtd/nand/pxa3xx_nand.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c index 5c3d719c37e6..d3bdc909c939 100644 --- a/drivers/mtd/nand/pxa3xx_nand.c +++ b/drivers/mtd/nand/pxa3xx_nand.c | |||
| @@ -1228,8 +1228,9 @@ static int pxa3xx_nand_probe(struct platform_device *pdev) | |||
| 1228 | continue; | 1228 | continue; |
| 1229 | } | 1229 | } |
| 1230 | 1230 | ||
| 1231 | ret = mtd_device_parse_register(info->host[cs]->mtd, NULL, 0, | 1231 | ret = mtd_device_parse_register(info->host[cs]->mtd, NULL, |
| 1232 | pdata->parts[cs], pdata->nr_parts[cs]); | 1232 | NULL, pdata->parts[cs], |
| 1233 | pdata->nr_parts[cs]); | ||
| 1233 | if (!ret) | 1234 | if (!ret) |
| 1234 | probe_success = 1; | 1235 | probe_success = 1; |
| 1235 | } | 1236 | } |