ARM: imx: add missing of_node_put()
[ Upstream commit 87b30c4b0efb6a194a7b8eac2568a3da520d905f ] Calling of_find_compatible_node() returns a node pointer with refcount incremented. Use of_node_put() on it when done. The patch fixes the same problem on different i.MX platforms. Fixes:8b88f7ef31
("ARM: mx25: Retrieve IIM base from dt") Fixes: 94b2bec1b0e05 ("ARM: imx27: Retrieve the SYSCTRL base address from devicetree") Fixes: 3172225d45bd9 ("ARM: imx31: Retrieve the IIM base address from devicetree") Fixes: f68ea682d1da7 ("ARM: imx35: Retrieve the IIM base address from devicetree") Fixes:ee18a7154e
("ARM: imx5: retrieve iim base from device tree") Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com> Reviewed-by: Fabio Estevam <festevam@gmail.com> Reviewed-by: Martin Kaiser <martin@kaiser.cx> Signed-off-by: Shawn Guo <shawnguo@kernel.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
12a763015b
commit
4be51ea91a
@ -23,6 +23,7 @@ static int mx25_read_cpu_rev(void)
|
|||||||
|
|
||||||
np = of_find_compatible_node(NULL, NULL, "fsl,imx25-iim");
|
np = of_find_compatible_node(NULL, NULL, "fsl,imx25-iim");
|
||||||
iim_base = of_iomap(np, 0);
|
iim_base = of_iomap(np, 0);
|
||||||
|
of_node_put(np);
|
||||||
BUG_ON(!iim_base);
|
BUG_ON(!iim_base);
|
||||||
rev = readl(iim_base + MXC_IIMSREV);
|
rev = readl(iim_base + MXC_IIMSREV);
|
||||||
iounmap(iim_base);
|
iounmap(iim_base);
|
||||||
|
@ -28,6 +28,7 @@ static int mx27_read_cpu_rev(void)
|
|||||||
|
|
||||||
np = of_find_compatible_node(NULL, NULL, "fsl,imx27-ccm");
|
np = of_find_compatible_node(NULL, NULL, "fsl,imx27-ccm");
|
||||||
ccm_base = of_iomap(np, 0);
|
ccm_base = of_iomap(np, 0);
|
||||||
|
of_node_put(np);
|
||||||
BUG_ON(!ccm_base);
|
BUG_ON(!ccm_base);
|
||||||
/*
|
/*
|
||||||
* now we have access to the IO registers. As we need
|
* now we have access to the IO registers. As we need
|
||||||
|
@ -39,6 +39,7 @@ static int mx31_read_cpu_rev(void)
|
|||||||
|
|
||||||
np = of_find_compatible_node(NULL, NULL, "fsl,imx31-iim");
|
np = of_find_compatible_node(NULL, NULL, "fsl,imx31-iim");
|
||||||
iim_base = of_iomap(np, 0);
|
iim_base = of_iomap(np, 0);
|
||||||
|
of_node_put(np);
|
||||||
BUG_ON(!iim_base);
|
BUG_ON(!iim_base);
|
||||||
|
|
||||||
/* read SREV register from IIM module */
|
/* read SREV register from IIM module */
|
||||||
|
@ -21,6 +21,7 @@ static int mx35_read_cpu_rev(void)
|
|||||||
|
|
||||||
np = of_find_compatible_node(NULL, NULL, "fsl,imx35-iim");
|
np = of_find_compatible_node(NULL, NULL, "fsl,imx35-iim");
|
||||||
iim_base = of_iomap(np, 0);
|
iim_base = of_iomap(np, 0);
|
||||||
|
of_node_put(np);
|
||||||
BUG_ON(!iim_base);
|
BUG_ON(!iim_base);
|
||||||
|
|
||||||
rev = imx_readl(iim_base + MXC_IIMSREV);
|
rev = imx_readl(iim_base + MXC_IIMSREV);
|
||||||
|
@ -28,6 +28,7 @@ static u32 imx5_read_srev_reg(const char *compat)
|
|||||||
|
|
||||||
np = of_find_compatible_node(NULL, NULL, compat);
|
np = of_find_compatible_node(NULL, NULL, compat);
|
||||||
iim_base = of_iomap(np, 0);
|
iim_base = of_iomap(np, 0);
|
||||||
|
of_node_put(np);
|
||||||
WARN_ON(!iim_base);
|
WARN_ON(!iim_base);
|
||||||
|
|
||||||
srev = readl(iim_base + IIM_SREV) & 0xff;
|
srev = readl(iim_base + IIM_SREV) & 0xff;
|
||||||
|
Loading…
Reference in New Issue
Block a user