Commit 50743768 authored by Christophe JAILLET's avatar Christophe JAILLET Committed by Heiko Stuebner
Browse files

drm/rockchip: Fix an error handling path rockchip_dp_probe() 



Should component_add() fail, we should call analogix_dp_remove() in the
error handling path, as already done in the remove function.

Fixes: 152cce00 ("drm/bridge: analogix_dp: Split bind() into probe() and real bind()")
Signed-off-by: default avatarChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: default avatarHeiko Stuebner <heiko@sntech.de>
Link: https://patchwork.freedesktop.org/patch/msgid/b719d9061bb97eb85145fbd3c5e63f4549f2e13e.1655572071.git.christophe.jaillet@wanadoo.fr
parent 1449110b

drivers/gpu/drm/rockchip/analogix_dp-rockchip.c

+9 −1
Original line number Diff line number Diff line
@@ -408,7 +408,15 @@ static int rockchip_dp_probe(struct platform_device *pdev) 
	if (IS_ERR(dp->adp))

		return PTR_ERR(dp->adp);



	return component_add(dev, &rockchip_dp_component_ops);

	ret = component_add(dev, &rockchip_dp_component_ops);

	if (ret)

		goto err_dp_remove;



	return 0;



err_dp_remove:

	analogix_dp_remove(dp->adp);

	return ret;

}



static int rockchip_dp_remove(struct platform_device *pdev)