* [PATCH] drm/rockchip: Fix an error handling path
@ 2021-05-01 15:13 Christophe JAILLET
2021-05-28 18:55 ` Heiko Stuebner
0 siblings, 1 reply; 2+ messages in thread
From: Christophe JAILLET @ 2021-05-01 15:13 UTC (permalink / raw)
To: hjc, heiko, airlied, daniel, miquel.raynal
Cc: kernel-janitors, linux-kernel, dri-devel, linux-rockchip,
Christophe JAILLET, linux-arm-kernel
'ret' is know to be 0 a this point. Checking the return value of
'phy_init()' and 'phy_set_mode()' was intended instead.
So add the missing assignments.
Fixes: cca1705c3d89 ("drm/rockchip: lvds: Add PX30 support")
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
---
drivers/gpu/drm/rockchip/rockchip_lvds.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/rockchip/rockchip_lvds.c b/drivers/gpu/drm/rockchip/rockchip_lvds.c
index bd5ba10822c2..489d63c05c0d 100644
--- a/drivers/gpu/drm/rockchip/rockchip_lvds.c
+++ b/drivers/gpu/drm/rockchip/rockchip_lvds.c
@@ -499,11 +499,11 @@ static int px30_lvds_probe(struct platform_device *pdev,
if (IS_ERR(lvds->dphy))
return PTR_ERR(lvds->dphy);
- phy_init(lvds->dphy);
+ ret = phy_init(lvds->dphy);
if (ret)
return ret;
- phy_set_mode(lvds->dphy, PHY_MODE_LVDS);
+ ret = phy_set_mode(lvds->dphy, PHY_MODE_LVDS);
if (ret)
return ret;
--
2.30.2
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] drm/rockchip: Fix an error handling path
2021-05-01 15:13 [PATCH] drm/rockchip: Fix an error handling path Christophe JAILLET
@ 2021-05-28 18:55 ` Heiko Stuebner
0 siblings, 0 replies; 2+ messages in thread
From: Heiko Stuebner @ 2021-05-28 18:55 UTC (permalink / raw)
To: daniel, Christophe JAILLET, miquel.raynal, airlied, hjc
Cc: kernel-janitors, linux-kernel, dri-devel, linux-rockchip,
linux-arm-kernel
On Sat, 1 May 2021 17:13:16 +0200, Christophe JAILLET wrote:
> 'ret' is know to be 0 a this point. Checking the return value of
> 'phy_init()' and 'phy_set_mode()' was intended instead.
>
> So add the missing assignments.
Applied, thanks!
[1/1] drm/rockchip: Fix an error handling path
commit: 3dfa159f6b0c054eb63673fbf643a5f2cc862e63
Best regards,
--
Heiko Stuebner <heiko@sntech.de>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-05-28 18:56 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-01 15:13 [PATCH] drm/rockchip: Fix an error handling path Christophe JAILLET
2021-05-28 18:55 ` Heiko Stuebner
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).