2020-11-06 07:40:57

by Jing Xiangfeng

Subject: [PATCH v2] mfd: htc-i2cpld: Add the missed i2c_put_adapter() in htcpld_register_chip_i2c()

htcpld_register_chip_i2c() misses to call i2c_put_adapter() in an error
path. Add the missed function call to fix it.

Fixes: 6048a3dd2371 ("mfd: Add HTCPLD driver")
Signed-off-by: Jing Xiangfeng <[email protected]>
---
v2:
- add the tag Fixes

---
drivers/mfd/htc-i2cpld.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/drivers/mfd/htc-i2cpld.c b/drivers/mfd/htc-i2cpld.c
index 247f9849e54a..417b0355d904 100644
--- a/drivers/mfd/htc-i2cpld.c
+++ b/drivers/mfd/htc-i2cpld.c
@@ -346,6 +346,7 @@ static int htcpld_register_chip_i2c(
if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_READ_BYTE_DATA)) {
dev_warn(dev, "i2c adapter %d non-functional\n",
pdata->i2c_adapter_id);
+ i2c_put_adapter(adapter);
return -EINVAL;
}

@@ -360,6 +361,7 @@ static int htcpld_register_chip_i2c(
/* I2C device registration failed, contineu with the next */
dev_warn(dev, "Unable to add I2C device for 0x%x\n",
plat_chip_data->addr);
+ i2c_put_adapter(adapter);
return PTR_ERR(client);
}

--
2.17.1

2020-11-11 16:31:17

by Lee Jones

Subject: Re: [PATCH v2] mfd: htc-i2cpld: Add the missed i2c_put_adapter() in htcpld_register_chip_i2c()

On Fri, 06 Nov 2020, Jing Xiangfeng wrote:

> htcpld_register_chip_i2c() misses to call i2c_put_adapter() in an error
> path. Add the missed function call to fix it.
>
> Fixes: 6048a3dd2371 ("mfd: Add HTCPLD driver")
> Signed-off-by: Jing Xiangfeng <[email protected]>
> ---
> v2:
> - add the tag Fixes
>
> ---
> drivers/mfd/htc-i2cpld.c | 2 ++
> 1 file changed, 2 insertions(+)

Applied, thanks.

--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog