clk: bcm2835: add missing release if devm_clk_hw_register fails
[ Upstream commitf6c992ca7d
] In the implementation of bcm2835_register_pll(), the allocated pll is leaked if devm_clk_hw_register() fails to register hw. Release pll if devm_clk_hw_register() fails. Signed-off-by: Navid Emamdoost <navid.emamdoost@gmail.com> Link: https://lore.kernel.org/r/20200809231202.15811-1-navid.emamdoost@gmail.com Fixes:41691b8862
("clk: bcm2835: Add support for programming the audio domain clocks") Signed-off-by: Stephen Boyd <sboyd@kernel.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
c10e3c919a
commit
9a3ee7177f
|
@ -1336,8 +1336,10 @@ static struct clk_hw *bcm2835_register_pll(struct bcm2835_cprman *cprman,
|
||||||
pll->hw.init = &init;
|
pll->hw.init = &init;
|
||||||
|
|
||||||
ret = devm_clk_hw_register(cprman->dev, &pll->hw);
|
ret = devm_clk_hw_register(cprman->dev, &pll->hw);
|
||||||
if (ret)
|
if (ret) {
|
||||||
|
kfree(pll);
|
||||||
return NULL;
|
return NULL;
|
||||||
|
}
|
||||||
return &pll->hw;
|
return &pll->hw;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue