mfd: intel_quark_i2c_gpio: Remove clock tree on error path
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Fri, 19 Feb 2016 08:42:10 +0000 (10:42 +0200)
committerLee Jones <lee.jones@linaro.org>
Mon, 11 Apr 2016 12:31:39 +0000 (13:31 +0100)
commit7f0c5ae18d649ed2f4978cbf07c02a0ff732f23e
treed2ab98dbce17b3209fc465c919ba8b6658985245
parent65433fd561f089f232758efc2c6566b6b6febb47
mfd: intel_quark_i2c_gpio: Remove clock tree on error path

There is a potential resource leak in case when ->probe() fails. We have to
unregister and remove clock tree which is done here.

This is a follow up to previously pushed commit c4726abce63b ("mfd:
intel_quark_i2c_gpio: Use clkdev_create()") that prevents double free() when
clkdev_drop() followed by kfree() in devm_kcalloc() release stage.

I leave Fixes tag here, but the backporting will require to backport the commit
c4726abce63b ("mfd: intel_quark_i2c_gpio: Use clkdev_create()") first.

Cc: stable@vger.kernel.org
Fixes: 60ae5b9f5cdd (mfd: intel_quark_i2c_gpio: Add Intel Quark X1000 I2C-GPIO MFD Driver)
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Stephen Boyd <sboyd@codeaurora.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
drivers/mfd/intel_quark_i2c_gpio.c