diff options
author | Ezequiel Garcia <ezequiel.garcia@free-electrons.com> | 2013-01-25 07:19:47 -0500 |
---|---|---|
committer | Tony Lindgren <tony@atomide.com> | 2013-02-01 16:55:45 -0500 |
commit | a16723709cff6d69567024c28d32d1344f08cecf (patch) | |
tree | 0a6f7d10b58a34fc6f1ad253057299f8bb3e8486 /arch/arm/mach-omap2 | |
parent | bc6b1e7b86f5d8e4a6fc1c0189e64bba4077efe0 (diff) |
ARM: OMAP2+: gpmc: Remove unneeded of_node_put()
for_each_node_by_name() automatically calls of_node_put() on each
node passed; so don't do it explicitly unless there's an error.
Reported-by: Mark Rutland <mark.rutland@arm.com>
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Acked-by: Jon Hunter <jon-hunter@ti.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
Diffstat (limited to 'arch/arm/mach-omap2')
-rw-r--r-- | arch/arm/mach-omap2/gpmc.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/arch/arm/mach-omap2/gpmc.c b/arch/arm/mach-omap2/gpmc.c index 01ce462e265d..c6255f788ab2 100644 --- a/arch/arm/mach-omap2/gpmc.c +++ b/arch/arm/mach-omap2/gpmc.c | |||
@@ -1271,9 +1271,10 @@ static int gpmc_probe_dt(struct platform_device *pdev) | |||
1271 | 1271 | ||
1272 | for_each_node_by_name(child, "nand") { | 1272 | for_each_node_by_name(child, "nand") { |
1273 | ret = gpmc_probe_nand_child(pdev, child); | 1273 | ret = gpmc_probe_nand_child(pdev, child); |
1274 | of_node_put(child); | 1274 | if (ret < 0) { |
1275 | if (ret < 0) | 1275 | of_node_put(child); |
1276 | return ret; | 1276 | return ret; |
1277 | } | ||
1277 | } | 1278 | } |
1278 | 1279 | ||
1279 | return 0; | 1280 | return 0; |