diff options
author | Jisheng Zhang <jszhang@marvell.com> | 2013-08-27 06:41:15 +0200 |
---|---|---|
committer | Jason Cooper <jason@lakedaemon.net> | 2013-09-18 18:41:10 +0200 |
commit | 4ec7fc4abb554c6c9b0eda81ad1c9cb82c29aa79 (patch) | |
tree | a133de800c571e9b28f68f3966d7de800ff1a43e /drivers/bus | |
parent | ARM: mvebu: add missing of_node_put() to fix reference leak (diff) | |
download | linux-4ec7fc4abb554c6c9b0eda81ad1c9cb82c29aa79.tar.xz linux-4ec7fc4abb554c6c9b0eda81ad1c9cb82c29aa79.zip |
bus: mvebu: add missing of_node_put() to fix reference leak
Add of_node_put to properly decrement the refcount when we are
done using a given node.
Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
Reviewed-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Conflicts:
drivers/bus/mvebu-mbus.c
Diffstat (limited to 'drivers/bus')
-rw-r--r-- | drivers/bus/mvebu-mbus.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/bus/mvebu-mbus.c b/drivers/bus/mvebu-mbus.c index 19ab6ff53d59..553185318963 100644 --- a/drivers/bus/mvebu-mbus.c +++ b/drivers/bus/mvebu-mbus.c @@ -700,6 +700,7 @@ static int __init mvebu_mbus_common_init(struct mvebu_mbus_state *mbus, phys_addr_t sdramwins_phys_base, size_t sdramwins_size) { + struct device_node *np; int win; mbus->mbuswins_base = ioremap(mbuswins_phys_base, mbuswins_size); @@ -712,8 +713,11 @@ static int __init mvebu_mbus_common_init(struct mvebu_mbus_state *mbus, return -ENOMEM; } - if (of_find_compatible_node(NULL, NULL, "marvell,coherency-fabric")) + np = of_find_compatible_node(NULL, NULL, "marvell,coherency-fabric"); + if (np) { mbus->hw_io_coherency = 1; + of_node_put(np); + } for (win = 0; win < mbus->soc->num_wins; win++) mvebu_mbus_disable_window(mbus, win); |