Re: [PATCH] drm/bridge: lt9611uxc: Fix an error handling path in lt9611uxc_probe()

[Dmitry Baryshkov]

On Sat, 16 Mar 2024 at 21:51, Christophe JAILLET
 wrote:
>
> If lt9611uxc_audio_init() fails, some resources still need to be released
> before returning the error code.
>
> Use the existing error handling path.
>
> Fixes: 0cbbd5b1a012 ("drm: bridge: add support for lontium LT9611UXC bridge")
> Signed-off-by: Christophe JAILLET 
> ---
> Compile tested only.
> ---
>  drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 6 +-
>  1 file changed, 5 insertions(+), 1 deletion(-)

Reviewed-by: Dmitry Baryshkov 


-- 
With best wishes
Dmitry

[PATCH] drm/bridge: lt9611uxc: Fix an error handling path in lt9611uxc_probe()

[Christophe JAILLET]

If lt9611uxc_audio_init() fails, some resources still need to be released
before returning the error code.

Use the existing error handling path.

Fixes: 0cbbd5b1a012 ("drm: bridge: add support for lontium LT9611UXC bridge")
Signed-off-by: Christophe JAILLET 
---
Compile tested only.
---
 drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 6 +-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c 
b/drivers/gpu/drm/bridge/lontium-lt9611uxc.c
index f4f593ad8f79..d0c77630a2f9 100644
--- a/drivers/gpu/drm/bridge/lontium-lt9611uxc.c
+++ b/drivers/gpu/drm/bridge/lontium-lt9611uxc.c
@@ -965,7 +965,11 @@ static int lt9611uxc_probe(struct i2c_client *client)
}
}
 
-   return lt9611uxc_audio_init(dev, lt9611uxc);
+   ret = lt9611uxc_audio_init(dev, lt9611uxc);
+   if (ret)
+   goto err_remove_bridge;
+
+   return 0;
 
 err_remove_bridge:
free_irq(client->irq, lt9611uxc);
-- 
2.44.0